Re: drivers/net/ethernet/freescale/dpaa/dpaa_eth.c:2745:24: sparse: sparse: cast to restricted __be32
From: Vladimir Oltean
Date: Mon Oct 28 2024 - 10:12:51 EST
Hi Breno,
On Mon, Oct 28, 2024 at 03:56:50AM -0700, Breno Leitao wrote:
> On Sun, Oct 27, 2024 at 05:36:56PM +0800, kernel test robot wrote:
> > Hi Breno,
> >
> > First bad commit (maybe != root cause):
> >
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head: 850925a8133c73c4a2453c360b2c3beb3bab67c9
> > commit: 782fe08e9861d00ce8aca370453dd9ceb9a23d50 soc: fsl: qbman: FSL_DPAA depends on COMPILE_TEST
> > date: 3 months ago
> > config: arm-randconfig-r123-20241027 (https://download.01.org/0day-ci/archive/20241027/202410271721.jiVrriXD-lkp@xxxxxxxxx/config)
> > compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 5886454669c3c9026f7f27eab13509dd0241f2d6)
> > reproduce: (https://download.01.org/0day-ci/archive/20241027/202410271721.jiVrriXD-lkp@xxxxxxxxx/reproduce)
> >
> > If you fix the issue in a separate patch/commit (i.e. not just a new version of
> > the same patch/commit), kindly add following tags
> > | Reported-by: kernel test robot <lkp@xxxxxxxxx>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202410271721.jiVrriXD-lkp@xxxxxxxxx/
> >
> > sparse warnings: (new ones prefixed by >>)
> > >> drivers/net/ethernet/freescale/dpaa/dpaa_eth.c:2745:24: sparse: sparse: cast to restricted __be32
> > drivers/net/ethernet/freescale/dpaa/dpaa_eth.c: note: in included file (through include/linux/module.h):
> > include/linux/list.h:83:21: sparse: sparse: self-comparison always evaluates to true
> > drivers/net/ethernet/freescale/dpaa/dpaa_eth.c: note: in included file:
> > >> include/soc/fsl/qman.h:245:16: sparse: sparse: cast to restricted __be32
> > >> include/soc/fsl/qman.h:245:16: sparse: sparse: cast from restricted __be16
>
> Thanks. The commit above "782fe08e98 ("soc: fsl: qbman: FSL_DPAA depends
> on COMPILE_TEST") was created to detect problems like this, since it is
> not possible to compile-test dpaa driver.
>
> Copying Madalin Bucur <madalin.bucur@xxxxxxx>, who currently maintains
> this driver.
I will handle the sparse warnings exposed by your COMPILE_TEST change,
there is nothing actionable for you, except review if you wish.