[soc:tegra/memory 19/20] aarch64-linux-gnu-ld: Unexpected GOT/PLT entries detected!

[Thierry Reding]

On Thu, Jun 17, 2021 at 04:22:56PM +0200, Krzysztof Kozlowski wrote:
> On 17/06/2021 13:54, kernel test robot wrote:
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git tegra/memory
> > head:   2c1bc371268862a991a6498e1dddc8971b9076b8
> > commit: 8eb68595475ac5fcaaa3718a173283df48cb4ef1 [19/20] iommu/arm-smmu: tegra: Implement SID override programming
> > config: arm64-randconfig-r016-20210617 (attached as .config)
> > compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
> > reproduce (this is a W=1 build):
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         # install arm64 cross compiling tool for clang build
> >         # apt-get install binutils-aarch64-linux-gnu
> >         # https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git/commit/?id=8eb68595475ac5fcaaa3718a173283df48cb4ef1
> >         git remote add soc https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git
> >         git fetch --no-tags soc tegra/memory
> >         git checkout 8eb68595475ac5fcaaa3718a173283df48cb4ef1
> >         # save the attached .config to linux build tree
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64 
> > 
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp at intel.com>
> > 
> > All errors (new ones prefixed by >>):
> > 
> >>> aarch64-linux-gnu-ld: Unexpected GOT/PLT entries detected!
> >>> aarch64-linux-gnu-ld: Unexpected run-time procedure linkages detected!
> >    aarch64-linux-gnu-ld: drivers/iommu/arm/arm-smmu/arm-smmu-nvidia.o: in function `nvidia_smmu_probe_finalize':
> >    arm-smmu-nvidia.c:(.text+0xe0): undefined reference to `tegra_mc_probe_device'
> > 
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
> 
> 
> Hi Thierry,
> 
> Will you take a look at this failure and prepare a follow-up patch? This
> came already from soc tree.

I thought I had sent out a reply earlier, but that doesn't seem to have
made it through.

I sent out a patch earlier that fixes this by adding a stub for
tegra_mc_probe_device() if TEGRA_MC is not set. I think that's a bit
better than adding a dependency on TEGRA_MC from the ARM_SMMU driver
because somebody may want to build the ARM_SMMU driver on non-Tegra
configurations.

Another alternative would've been to add another Kconfig symbol for the
Tegra ARM SMMU implementation, but that would've been more intrusive and
would've prevented the Tegra code from getting broader build coverage.

Let me know what you think about the fix and if you want a different
solution than the stub.

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20210618/3696fd12/attachment.sig>