Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-06 Thread Al Viro
On Wed, Jan 06, 2021 at 08:12:27PM +, Ionela Voinescu wrote: > Initially I though it always only makes sense to have a __iomem pointer. > That is, it only makes sense to have a pointer with a cookie attached > specifying that it addresses a device memory space that should only be > accessed

Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-06 Thread Ionela Voinescu
On Wednesday 06 Jan 2021 at 17:47:58 (+), Al Viro wrote: > On Wed, Jan 06, 2021 at 03:07:24PM +, Ionela Voinescu wrote: > > > > > > 367 switch ((u64)reg->address) { > > > > > > That's not a dereference but I guess sparse complains of dropping the > > > __iomem. We could change

Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-06 Thread Al Viro
On Wed, Jan 06, 2021 at 03:07:24PM +, Ionela Voinescu wrote: > > > > 367switch ((u64)reg->address) { > > > > That's not a dereference but I guess sparse complains of dropping the > > __iomem. We could change the cast to (__force u64) to silence sparse. Oh, yes, it is - that of

Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-06 Thread Ionela Voinescu
Hi, On Wednesday 06 Jan 2021 at 16:13:53 (+), Al Viro wrote: > On Wed, Jan 06, 2021 at 03:52:14PM +, Ionela Voinescu wrote: > > > > > > vim +367 arch/arm64/kernel/topology.c > > > > > > > > > > > >362 > > > > > >363 int cpc_read_ffh(int cpu, struct cpc_reg *reg, u64

Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-06 Thread Al Viro
On Wed, Jan 06, 2021 at 03:52:14PM +, Ionela Voinescu wrote: > > > > > vim +367 arch/arm64/kernel/topology.c > > > > > > > > > >362 > > > > >363int cpc_read_ffh(int cpu, struct cpc_reg *reg, u64 *val) > > > > >364{ > > > > >365int ret =

Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-06 Thread Ionela Voinescu
ILER=gcc-9.3.0 make.cross > > > > C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=arm64 > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > Reported-by: kernel test robot > > > > > >

Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-06 Thread Catalin Marinas
linux build tree > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross > > > C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=arm64 > > > > > > If you fix the issue, kindly add following tag as appropriate > > > R

Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-06 Thread Ionela Voinescu
> > If you fix the issue, kindly add following tag as appropriate > > Reported-by: kernel test robot > > > > > > "sparse warnings: (new ones prefixed by >>)" > > >> arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of &g

arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2021-01-05 Thread kernel test robot
"sparse warnings: (new ones prefixed by >>)" >> arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef >> expression vim +367 arch/arm64/kernel/topology.c 362 363 int cpc_read_ffh(int cpu, struct cpc_reg *reg, u64 *val) 36

Re: arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2020-12-18 Thread Catalin Marinas
OME/0day COMPILER=gcc-9.3.0 make.cross C=1 > CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=arm64 > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot > > > "sparse warnings: (new ones prefixed by >>)" > >> ar

arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef expression

2020-12-17 Thread kernel test robot
"sparse warnings: (new ones prefixed by >>)" >> arch/arm64/kernel/topology.c:367:22: sparse: sparse: dereference of noderef >> expression vim +367 arch/arm64/kernel/topology.c 362 363 int cpc_read_ffh(int cpu, struct cpc_reg *reg, u64 *val) 36