Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Message-ID: <a141e090-7049-d693-1dc2-7c0a593b9682@infradead.org>
Date:   Mon, 7 Feb 2022 08:06:59 -0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.5.0
Subject: Re: ld.lld: error: undefined symbol: socfpga_reset_init
Content-Language: en-US
To:     Krzysztof Kozlowski <krzk@kernel.org>,
        Nathan Chancellor <nathan@kernel.org>
Cc:     kernel test robot <lkp@intel.com>, llvm@lists.linux.dev,
        kbuild-all@lists.01.org, linux-kernel@vger.kernel.org,
        Dinh Nguyen <dinguyen@kernel.org>
References: <202202050539.Mtk9I4Ef-lkp@intel.com>
 <67f432e2-1bfd-29dc-f2ed-051b47ef0844@kernel.org>
 <YgBtAdhl8dZVTAV+@dev-arch.archlinux-ax161>
 <8c315714-ea7b-d3ec-f131-6beded67b479@kernel.org>
From:   Randy Dunlap <rdunlap@infradead.org>
In-Reply-To: <8c315714-ea7b-d3ec-f131-6beded67b479@kernel.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 2/6/22 23:55, Krzysztof Kozlowski wrote:
> On 07/02/2022 01:51, Nathan Chancellor wrote:
>> Hi Krzysztof,
>>
>> On Sun, Feb 06, 2022 at 12:55:57PM +0100, Krzysztof Kozlowski wrote:
>>> On 04/02/2022 22:12, kernel test robot wrote:
>>>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>>>> head:   dcb85f85fa6f142aae1fe86f399d4503d49f2b60
>>>> commit: 4483397b03536506535d611b0cb28a81a69e8edf ARM: socfpga: drop ARCH_SOCFPGA
>>>> date:   11 months ago
>>>> config: arm-socfpga_defconfig (https://download.01.org/0day-ci/archive/20220205/202202050539.Mtk9I4Ef-lkp@intel.com/config)
>>>> compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project a73e4ce6a59b01f0e37037761c1e6889d539d233)
>>>> 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 arm cross compiling tool for clang build
>>>>         # apt-get install binutils-arm-linux-gnueabi
>>>>         # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=4483397b03536506535d611b0cb28a81a69e8edf
>>>>         git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>>>>         git fetch --no-tags linus master
>>>>         git checkout 4483397b03536506535d611b0cb28a81a69e8edf
>>>>         # save the config file to linux build tree
>>>>         mkdir build_dir
>>>>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash
>>>>
>>>> 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 >>):
>>>>
>>>>>> ld.lld: error: undefined symbol: socfpga_reset_init
>>>>    >>> referenced by socfpga.c
>>>>    >>>               socfpga.o:(socfpga_init_irq) in archive arch/arm/mach-socfpga/built-in.a
>>>>    >>> referenced by socfpga.c
>>>>    >>>               socfpga.o:(socfpga_arria10_init_irq) in archive arch/arm/mach-socfpga/built-in.a
>>>>
>>>
>>> This looks like a false positive. On the socfpga defconfig arm build,
>>> the socfpga_reset_init() (drivers/reset/reset-socfpga.c) is built.
>>>
>>> I also could not reproduce it with GCC.
>>
>> I can reproduce this error with the configuration that the bot provides
>> at both 4483397b03536506535d611b0cb28a81a69e8edf and v5.17-rc3, even
>> with GCC:
>>
>> $ curl -LSso .config https://download.01.org/0day-ci/archive/20220205/202202050539.Mtk9I4Ef-lkp@intel.com/config
>>
>> $ make -skj"$(nproc)" ARCH=arm CROSS_COMPILE=arm-linux-gnueabi- olddefconfig all
>> ...
>> arm-linux-gnueabi-ld: arch/arm/mach-socfpga/socfpga.o: in function `socfpga_init_irq':
>> socfpga.c:(.init.text+0x104): undefined reference to `socfpga_reset_init'
>> ...
>>
>> Whether or not this reproduces with the in-tree socfpga_defconfig, I am
>> not sure.
> 
> Thanks for checking - it seems I misread (and find it confusing...) the
> report - that it is socfpga defconfig. It clearly says "config:
> arm-socfpga_defconfig"...

Aha, I did the same thing and could not reproduce it. :(

> I can reproduce it with the config from the URL. I'll work on fix.

Thanks.

-- 
~Randy