linux-kernel - Re: [linux-stable-rc:linux-4.19.y 1539/3265] drivers/net/ethernet/xilinx/xilinx_axienet_main.c:283:15: error: implicit declaration of function 'read_poll

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 7 Jul 2022 22:24:20 +0000
From:   Robert Hancock <robert.hancock@...ian.com>
To:     "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
CC:     "kbuild-all@...ts.01.org" <kbuild-all@...ts.01.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "lkp@...el.com" <lkp@...el.com>, "andrew@...n.ch" <andrew@...n.ch>
Subject: Re: [linux-stable-rc:linux-4.19.y 1539/3265]
 drivers/net/ethernet/xilinx/xilinx_axienet_main.c:283:15: error: implicit
 declaration of function 'read_poll_timeout'

On Thu, 2022-07-07 at 20:02 +0200, gregkh@...uxfoundation.org wrote:
> On Wed, Jul 06, 2022 at 10:42:48PM +0000, Robert Hancock wrote:
> > On Wed, 2022-07-06 at 18:43 +0800, kernel test robot wrote:
> > > Hi Robert,
> > > 
> > > FYI, the error/warning still remains.
> > > 
> > > tree:  
> > > https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git__;!!IOGos0k!nK_4HG5DiIjhDaFvtB2c4HTsoQMg8UHEBJDv6stlGIMButbUtJc42wGH5qXaVC5JIkAwZas4Ne1iIw$
> > >   linux-4.19.y
> > > head:   b9f174a70c6f609b6132c3a65883b16d552984b8
> > > commit: 8b8ad8a7ac24905937c3975b54c4594b93b66ebb [1539/3265] net:
> > > axienet: Wait for PhyRstCmplt after core reset
> > > config: microblaze-randconfig-r003-20220704
> > > (
> > > https://urldefense.com/v3/__https://download.01.org/0day-ci/archiv
> > > e/
> > > 20220706/202207061830.78pJNuwN-lkp@...el.com/config__;!!IOGos0k!n
> > > K_4H
> > > G5DiIjhDaFvtB2c4HTsoQMg8UHEBJDv6stlGIMButbUtJc42wGH5qXaVC5JIkAwZa
> > > tUAT
> > > tDaQ$ )
> > > compiler: microblaze-linux-gcc (GCC) 11.3.0
> > > reproduce (this is a W=1 build):
> > >         wget
> > > https://urldefense.com/v3/__https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross__;!!IOGos0k!nK_4HG5DiIjhDaFvtB2c4HTsoQMg8UHEBJDv6stlGIMButbUtJc42wGH5qXaVC5JIkAwZautG4jkgg$
> > >   -O ~/bin/make.cross
> > >         chmod +x ~/bin/make.cross
> > >         #
> > > https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=8b8ad8a7ac24905937c3975b54c4594b93b66ebb__;!!IOGos0k!nK_4HG5DiIjhDaFvtB2c4HTsoQMg8UHEBJDv6stlGIMButbUtJc42wGH5qXaVC5JIkAwZas8R9M_sA$
> > >  
> > >         git remote add linux-stable-rc
> > > https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git__;!!IOGos0k!nK_4HG5DiIjhDaFvtB2c4HTsoQMg8UHEBJDv6stlGIMButbUtJc42wGH5qXaVC5JIkAwZas4Ne1iIw$
> > >  
> > >         git fetch --no-tags linux-stable-rc linux-4.19.y
> > >         git checkout 8b8ad8a7ac24905937c3975b54c4594b93b66ebb
> > >         # save the config file
> > >         mkdir build_dir && cp config build_dir/.config
> > >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0
> > > make.cross W=1 O=build_dir ARCH=microblaze SHELL=/bin/bash
> > > drivers/net/ethernet/xilinx/
> > > 
> > > If you fix the issue, kindly add following tag where applicable
> > > Reported-by: kernel test robot <lkp@...el.com>
> > > 
> > > All errors (new ones prefixed by >>):
> > > 
> > >    drivers/net/ethernet/xilinx/xilinx_axienet_main.c: In function
> > > 'axienet_dma_bd_init':
> > >    drivers/net/ethernet/xilinx/xilinx_axienet_main.c:283:9:
> > > error:
> > > 'ret' undeclared (first use in this function); did you mean
> > > 'net'?
> > >      283 |         ret = read_poll_timeout(axienet_ior, value,
> > >          |         ^~~
> > >          |         net
> > >    drivers/net/ethernet/xilinx/xilinx_axienet_main.c:283:9: note:
> > > each undeclared identifier is reported only once for each
> > > function it
> > > appears in
> > > > > drivers/net/ethernet/xilinx/xilinx_axienet_main.c:283:15:
> > > > > error:
> > > > > implicit declaration of function 'read_poll_timeout' [-
> > > > > Werror=implicit-function-declaration]
> > >      283 |         ret = read_poll_timeout(axienet_ior, value,
> > >          |               ^~~~~~~~~~~~~~~~~
> > >    drivers/net/ethernet/xilinx/xilinx_axienet_main.c:283:46:
> > > error:
> > > 'value' undeclared (first use in this function)
> > >      283 |         ret = read_poll_timeout(axienet_ior, value,
> > >          |                                              ^~~~~
> > >    cc1: some warnings being treated as errors
> > > 
> > 
> > This seems like maybe a bad backport of the listed patch to the
> > 4.19
> > stable release?
> 
> If so, can you provide a fix for this?

It looks like read_poll_timeout was not in 4.19 so it's not just a
missing include, the code would need to be modified to use a different
API. I don't have a test environment for 4.19 so the most I could do is
compile test it (and in 4.19 this driver could only be enabled on a
limited set of platforms, Microblaze being one, which I assume is why
there weren't more complaints).

Unless someone had a specific reason for requesting this patch
("axienet: Wait for PhyRstCmplt after core reset") be backported to
4.19, it might be easier to just drop it from the 4.19 stable tree -
the problem this is fixing only shows up in certain hardware
configurations, but the driver not compiling is worse..