netdev - Re: [PATCH net-next 1/8] net: implement lockless SO

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]

Message-ID: <f7b690e1-2afb-1126-8ca7-210dfa77ba85@linux.ibm.com>
Date: Fri, 22 Sep 2023 01:37:42 +0200
From: Wenjia Zhang <wenjia@...ux.ibm.com>
To: Eric Dumazet <edumazet@...gle.com>,
        "David S . Miller"
 <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>
Cc: netdev@...r.kernel.org, eric.dumazet@...il.com
Subject: Re: [PATCH net-next 1/8] net: implement lockless SO_PRIORITY



On 21.09.23 22:28, Eric Dumazet wrote:
> This is a followup of 8bf43be799d4 ("net: annotate data-races
> around sk->sk_priority").
> 
> sk->sk_priority can be read and written without holding the socket lock.
> 
> Signed-off-by: Eric Dumazet <edumazet@...gle.com>
> ---
>   drivers/net/ppp/pppoe.c           |  2 +-
>   include/net/bluetooth/bluetooth.h |  2 +-
>   net/appletalk/aarp.c              |  2 +-
>   net/ax25/af_ax25.c                |  2 +-
>   net/bluetooth/l2cap_sock.c        |  2 +-
>   net/can/j1939/socket.c            |  2 +-
>   net/can/raw.c                     |  2 +-
>   net/core/sock.c                   | 23 ++++++++++++-----------
>   net/dccp/ipv6.c                   |  2 +-
>   net/ipv4/inet_diag.c              |  2 +-
>   net/ipv4/ip_output.c              |  2 +-
>   net/ipv4/tcp_ipv4.c               |  2 +-
>   net/ipv4/tcp_minisocks.c          |  2 +-
>   net/ipv6/inet6_connection_sock.c  |  2 +-
>   net/ipv6/ip6_output.c             |  2 +-
>   net/ipv6/tcp_ipv6.c               |  4 ++--
>   net/mptcp/sockopt.c               |  2 +-
>   net/netrom/af_netrom.c            |  2 +-
>   net/rose/af_rose.c                |  2 +-
>   net/sched/em_meta.c               |  2 +-
>   net/sctp/ipv6.c                   |  2 +-
>   net/smc/af_smc.c                  |  2 +-
>   net/x25/af_x25.c                  |  2 +-
>   net/xdp/xsk.c                     |  2 +-
>   24 files changed, 36 insertions(+), 35 deletions(-)
> 

Thank you, Eric, for the fix!

Reviewed-by: Wenjia Zhang <wenjia@...ux.ibm.com>