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

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

[kernel test robot]

[-- Attachment #1: Type: text/plain, Size: 1750 bytes --]

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@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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 36843 bytes --]

[-- Attachment #3: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

[Krzysztof Kozlowski]

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@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.

Best regards,
Krzysztof

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

[Thierry Reding]

[-- Attachment #1.1: Type: text/plain, Size: 2866 bytes --]

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@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

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

[Krzysztof Kozlowski]

On 18/06/2021 16:57, Thierry Reding wrote:
> 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@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.

Stub works for me, thanks!


Best regards,
Krzysztof

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel