mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
@ 2025-11-03  7:26 kernel test robot
  0 siblings, 0 replies; 4+ messages in thread
From: kernel test robot @ 2025-11-03  7:26 UTC (permalink / raw)
  To: Eric Dumazet
  Cc: oe-kbuild-all, linux-kernel, Jakub Kicinski, Maciej Żenczykowski

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   6146a0f1dfae5d37442a9ddcba012add260bceb0
commit: c51da3f7a161c6822232be832abdffe47eb55b4c net: remove sock_i_uid()
date:   4 months ago
config: alpha-randconfig-r131-20251103 (https://download.01.org/0day-ci/archive/20251103/202511031542.oqD6ABwP-lkp@intel.com/config)
compiler: alpha-linux-gcc (GCC) 15.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251103/202511031542.oqD6ABwP-lkp@intel.com/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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202511031542.oqD6ABwP-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
   net/bluetooth/af_bluetooth.c:248:25: sparse: sparse: context imbalance in 'bt_accept_enqueue' - different lock contexts for basic block
   net/bluetooth/af_bluetooth.c: note: in included file (through include/net/bluetooth/bluetooth.h):
>> include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
   include/net/sock.h:2098:16: sparse: sparse: cast from non-scalar

vim +2098 include/net/sock.h

^1da177e4c3f41 Linus Torvalds 2005-04-16  2094  
e84a4927a404f3 Eric Dumazet   2025-06-20  2095  static inline kuid_t sk_uid(const struct sock *sk)
e84a4927a404f3 Eric Dumazet   2025-06-20  2096  {
e84a4927a404f3 Eric Dumazet   2025-06-20  2097  	/* Paired with WRITE_ONCE() in sockfs_setattr() */
e84a4927a404f3 Eric Dumazet   2025-06-20 @2098  	return READ_ONCE(sk->sk_uid);
e84a4927a404f3 Eric Dumazet   2025-06-20  2099  }
e84a4927a404f3 Eric Dumazet   2025-06-20  2100  

:::::: The code at line 2098 was first introduced by commit
:::::: e84a4927a404f369c842c19de93b216627fcc690 net: annotate races around sk->sk_uid

:::::: TO: Eric Dumazet <edumazet@google.com>
:::::: CC: Jakub Kicinski <kuba@kernel.org>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
  2025-09-29  9:49 ` Eric Dumazet
@ 2025-11-03  8:06   ` Eric Dumazet
  0 siblings, 0 replies; 4+ messages in thread
From: Eric Dumazet @ 2025-11-03  8:06 UTC (permalink / raw)
  To: kernel test robot
  Cc: oe-kbuild-all, linux-kernel, Jakub Kicinski,
	Maciej Żenczykowski, Will Deacon, Paul E. McKenney,
	linux-alpha

On Mon, Sep 29, 2025 at 2:49 AM Eric Dumazet <edumazet@google.com> wrote:
>
> On Mon, Sep 29, 2025 at 1:58 AM kernel test robot <lkp@intel.com> wrote:
> >
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head:   e5f0a698b34ed76002dc5cff3804a61c80233a7a
> > commit: c51da3f7a161c6822232be832abdffe47eb55b4c net: remove sock_i_uid()
> > date:   3 months ago
> > config: alpha-randconfig-r122-20250929 (https://download.01.org/0day-ci/archive/20250929/202509291645.fcBIaXMb-lkp@intel.com/config)
> > compiler: alpha-linux-gcc (GCC) 15.1.0
> > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250929/202509291645.fcBIaXMb-lkp@intel.com/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@intel.com>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202509291645.fcBIaXMb-lkp@intel.com/
> >
> > sparse warnings: (new ones prefixed by >>)
> >    net/packet/diag.c: note: in included file (through include/linux/sock_diag.h):
> > >> include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
> >    include/net/sock.h:2098:16: sparse: sparse: cast from non-scalar
> > --
> >    net/packet/af_packet.c:1099:13: sparse: sparse: context imbalance in '__packet_lookup_frame_in_block' - different lock contexts for basic block
> >    net/packet/af_packet.c:2541:17: sparse: sparse: context imbalance in 'tpacket_rcv' - unexpected unlock
> >    net/packet/af_packet.c: note: in included file (through include/net/inet_sock.h, include/net/ip.h):
> > >> include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
> >    include/net/sock.h:2098:16: sparse: sparse: cast from non-scalar
> >
> > vim +2098 include/net/sock.h
> >
> > ^1da177e4c3f415 Linus Torvalds 2005-04-16  2094
> > e84a4927a404f36 Eric Dumazet   2025-06-20  2095  static inline kuid_t sk_uid(const struct sock *sk)
> > e84a4927a404f36 Eric Dumazet   2025-06-20  2096  {
> > e84a4927a404f36 Eric Dumazet   2025-06-20  2097         /* Paired with WRITE_ONCE() in sockfs_setattr() */
> > e84a4927a404f36 Eric Dumazet   2025-06-20 @2098         return READ_ONCE(sk->sk_uid);
> > e84a4927a404f36 Eric Dumazet   2025-06-20  2099  }
> > e84a4927a404f36 Eric Dumazet   2025-06-20  2100
> >
> > :::::: The code at line 2098 was first introduced by commit
> > :::::: e84a4927a404f369c842c19de93b216627fcc690 net: annotate races around sk->sk_uid
> >
> > :::::: TO: Eric Dumazet <edumazet@google.com>
> > :::::: CC: Jakub Kicinski <kuba@kernel.org>
> >
> > --
> > 0-DAY CI Kernel Test Service
> > https://github.com/intel/lkp-tests/wiki
>
> This seems an Alpha issue to me ?
>
> I can submit the following fix if there is an agreement.
>
> diff --git a/arch/alpha/include/asm/rwonce.h b/arch/alpha/include/asm/rwonce.h
> index 35542bcf92b3a883df353784bcb2d243475ccd91..b6801cd2ace962e11624737ed334a5aeb30478b7
> 100644
> --- a/arch/alpha/include/asm/rwonce.h
> +++ b/arch/alpha/include/asm/rwonce.h
> @@ -22,10 +22,10 @@
>   */
>  #define __READ_ONCE(x)                                                 \
>  ({                                                                     \
> -       __unqual_scalar_typeof(x) __x =                                 \
> -               (*(volatile typeof(__x) *)(&(x)));                      \
> +       __auto_type __x =                                               \
> +               (*(const volatile __unqual_scalar_typeof(x) *)&(x));    \
>         mb();                                                           \
> -       (typeof(x))__x;                                                 \
> +       __x;                                                            \
>  })
>
>  #endif /* CONFIG_SMP */

Gentle ping.

Is alpha arch still alive ?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
  2025-09-29  8:57 kernel test robot
@ 2025-09-29  9:49 ` Eric Dumazet
  2025-11-03  8:06   ` Eric Dumazet
  0 siblings, 1 reply; 4+ messages in thread
From: Eric Dumazet @ 2025-09-29  9:49 UTC (permalink / raw)
  To: kernel test robot
  Cc: oe-kbuild-all, linux-kernel, Jakub Kicinski,
	Maciej Żenczykowski, Will Deacon, Paul E. McKenney,
	linux-alpha

On Mon, Sep 29, 2025 at 1:58 AM kernel test robot <lkp@intel.com> wrote:
>
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   e5f0a698b34ed76002dc5cff3804a61c80233a7a
> commit: c51da3f7a161c6822232be832abdffe47eb55b4c net: remove sock_i_uid()
> date:   3 months ago
> config: alpha-randconfig-r122-20250929 (https://download.01.org/0day-ci/archive/20250929/202509291645.fcBIaXMb-lkp@intel.com/config)
> compiler: alpha-linux-gcc (GCC) 15.1.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250929/202509291645.fcBIaXMb-lkp@intel.com/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@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202509291645.fcBIaXMb-lkp@intel.com/
>
> sparse warnings: (new ones prefixed by >>)
>    net/packet/diag.c: note: in included file (through include/linux/sock_diag.h):
> >> include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
>    include/net/sock.h:2098:16: sparse: sparse: cast from non-scalar
> --
>    net/packet/af_packet.c:1099:13: sparse: sparse: context imbalance in '__packet_lookup_frame_in_block' - different lock contexts for basic block
>    net/packet/af_packet.c:2541:17: sparse: sparse: context imbalance in 'tpacket_rcv' - unexpected unlock
>    net/packet/af_packet.c: note: in included file (through include/net/inet_sock.h, include/net/ip.h):
> >> include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
>    include/net/sock.h:2098:16: sparse: sparse: cast from non-scalar
>
> vim +2098 include/net/sock.h
>
> ^1da177e4c3f415 Linus Torvalds 2005-04-16  2094
> e84a4927a404f36 Eric Dumazet   2025-06-20  2095  static inline kuid_t sk_uid(const struct sock *sk)
> e84a4927a404f36 Eric Dumazet   2025-06-20  2096  {
> e84a4927a404f36 Eric Dumazet   2025-06-20  2097         /* Paired with WRITE_ONCE() in sockfs_setattr() */
> e84a4927a404f36 Eric Dumazet   2025-06-20 @2098         return READ_ONCE(sk->sk_uid);
> e84a4927a404f36 Eric Dumazet   2025-06-20  2099  }
> e84a4927a404f36 Eric Dumazet   2025-06-20  2100
>
> :::::: The code at line 2098 was first introduced by commit
> :::::: e84a4927a404f369c842c19de93b216627fcc690 net: annotate races around sk->sk_uid
>
> :::::: TO: Eric Dumazet <edumazet@google.com>
> :::::: CC: Jakub Kicinski <kuba@kernel.org>
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki

This seems an Alpha issue to me ?

I can submit the following fix if there is an agreement.

diff --git a/arch/alpha/include/asm/rwonce.h b/arch/alpha/include/asm/rwonce.h
index 35542bcf92b3a883df353784bcb2d243475ccd91..b6801cd2ace962e11624737ed334a5aeb30478b7
100644
--- a/arch/alpha/include/asm/rwonce.h
+++ b/arch/alpha/include/asm/rwonce.h
@@ -22,10 +22,10 @@
  */
 #define __READ_ONCE(x)                                                 \
 ({                                                                     \
-       __unqual_scalar_typeof(x) __x =                                 \
-               (*(volatile typeof(__x) *)(&(x)));                      \
+       __auto_type __x =                                               \
+               (*(const volatile __unqual_scalar_typeof(x) *)&(x));    \
        mb();                                                           \
-       (typeof(x))__x;                                                 \
+       __x;                                                            \
 })

 #endif /* CONFIG_SMP */

^ permalink raw reply	[flat|nested] 4+ messages in thread

* include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
@ 2025-09-29  8:57 kernel test robot
  2025-09-29  9:49 ` Eric Dumazet
  0 siblings, 1 reply; 4+ messages in thread
From: kernel test robot @ 2025-09-29  8:57 UTC (permalink / raw)
  To: Eric Dumazet
  Cc: oe-kbuild-all, linux-kernel, Jakub Kicinski, Maciej Żenczykowski

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   e5f0a698b34ed76002dc5cff3804a61c80233a7a
commit: c51da3f7a161c6822232be832abdffe47eb55b4c net: remove sock_i_uid()
date:   3 months ago
config: alpha-randconfig-r122-20250929 (https://download.01.org/0day-ci/archive/20250929/202509291645.fcBIaXMb-lkp@intel.com/config)
compiler: alpha-linux-gcc (GCC) 15.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250929/202509291645.fcBIaXMb-lkp@intel.com/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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202509291645.fcBIaXMb-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
   net/packet/diag.c: note: in included file (through include/linux/sock_diag.h):
>> include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
   include/net/sock.h:2098:16: sparse: sparse: cast from non-scalar
--
   net/packet/af_packet.c:1099:13: sparse: sparse: context imbalance in '__packet_lookup_frame_in_block' - different lock contexts for basic block
   net/packet/af_packet.c:2541:17: sparse: sparse: context imbalance in 'tpacket_rcv' - unexpected unlock
   net/packet/af_packet.c: note: in included file (through include/net/inet_sock.h, include/net/ip.h):
>> include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar
   include/net/sock.h:2098:16: sparse: sparse: cast from non-scalar

vim +2098 include/net/sock.h

^1da177e4c3f415 Linus Torvalds 2005-04-16  2094  
e84a4927a404f36 Eric Dumazet   2025-06-20  2095  static inline kuid_t sk_uid(const struct sock *sk)
e84a4927a404f36 Eric Dumazet   2025-06-20  2096  {
e84a4927a404f36 Eric Dumazet   2025-06-20  2097  	/* Paired with WRITE_ONCE() in sockfs_setattr() */
e84a4927a404f36 Eric Dumazet   2025-06-20 @2098  	return READ_ONCE(sk->sk_uid);
e84a4927a404f36 Eric Dumazet   2025-06-20  2099  }
e84a4927a404f36 Eric Dumazet   2025-06-20  2100  

:::::: The code at line 2098 was first introduced by commit
:::::: e84a4927a404f369c842c19de93b216627fcc690 net: annotate races around sk->sk_uid

:::::: TO: Eric Dumazet <edumazet@google.com>
:::::: CC: Jakub Kicinski <kuba@kernel.org>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-11-03  8:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-11-03  7:26 include/net/sock.h:2098:16: sparse: sparse: cast to non-scalar kernel test robot
  -- strict thread matches above, loose matches on Subject: below --
2025-09-29  8:57 kernel test robot
2025-09-29  9:49 ` Eric Dumazet
2025-11-03  8:06   ` Eric Dumazet

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®