Re: [PATCH bpf 3/4] bpf, vsock: Invoke proto::close on close()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: mhal@xxxxxxx
Subject: Re: [PATCH bpf 3/4] bpf, vsock: Invoke proto::close on close()
From: Luigi Leonardi <leonardi@xxxxxxxxxx>
Date: Fri, 22 Nov 2024 17:20:31 +0100
Cc: andrii@xxxxxxxxxx, ast@xxxxxxxxxx, bobby.eshleman@xxxxxxxxxxxxx, bpf@xxxxxxxxxxxxxxx, daniel@xxxxxxxxxxxxx, davem@xxxxxxxxxxxxx, eddyz87@xxxxxxxxx, edumazet@xxxxxxxxxx, haoluo@xxxxxxxxxx, horms@xxxxxxxxxx, john.fastabend@xxxxxxxxx, jolsa@xxxxxxxxxx, kpsingh@xxxxxxxxxx, kuba@xxxxxxxxxx, linux-kselftest@xxxxxxxxxxxxxxx, martin.lau@xxxxxxxxx, mst@xxxxxxxxxx, mykolal@xxxxxx, netdev@xxxxxxxxxxxxxxx, pabeni@xxxxxxxxxx, sdf@xxxxxxxxxxx, sgarzare@xxxxxxxxxx, shuah@xxxxxxxxxx, song@xxxxxxxxxx, yonghong.song@xxxxxxxxx, Luigi Leonardi <leonardi@xxxxxxxxxx>
In-reply-to: <20241118-vsock-bpf-poll-close-v1-3-f1b9669cacdc@rbox.co>

I spent some time checking and nobody but __sock_create (net/socket.c)
and vsock_release can set sock->sk to NULL.

I also ran checkpatch, everything LGTM.
Thanks for the fix!

Reviewed-by: Luigi Leonardi <leonardi@xxxxxxxxxx>

References:
- [PATCH bpf 3/4] bpf, vsock: Invoke proto::close on close()
  - From: Michal Luczaj

Prev by Date: Re: [PATCH net-next v11 15/23] ovpn: implement keepalive mechanism
Next by Date: Re: [PATCH bpf 1/4] bpf, vsock: Fix poll() missing a queue
Previous by thread: Re: [PATCH bpf 3/4] bpf, vsock: Invoke proto::close on close()
Next by thread: [PATCH bpf 1/4] bpf, vsock: Fix poll() missing a queue
Index(es):
- Date
- Thread

[Index of Archives] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Samba] [Device Mapper]

Powered by Linux