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=8e...
> 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(a)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