Re: [PATCH net 1/4] sockmap, vsock: For connectible sockets allow only connected

2025-02-18 Thread Stefano Garzarella
On Fri, Feb 14, 2025 at 02:11:48PM +0100, Michal Luczaj wrote: ... Another design detail is that listening vsocks are not supposed to have any transport assigned at all. Which implies they are not supported by the sockmap. But this is complicated by the fact that a socket, before switching to TCP

Re: [PATCH net 1/4] sockmap, vsock: For connectible sockets allow only connected

2025-02-14 Thread Michal Luczaj
> ... > Another design detail is that listening vsocks are not supposed to have any > transport assigned at all. Which implies they are not supported by the > sockmap. But this is complicated by the fact that a socket, before > switching to TCP_LISTEN, may have had some transport assigned during a

[PATCH net 1/4] sockmap, vsock: For connectible sockets allow only connected

2025-02-13 Thread Michal Luczaj
sockmap expects all vsocks to have a transport assigned, which is expressed in vsock_proto::psock_update_sk_prot(). However, there is an edge case where an unconnected (connectible) socket may lose its previously assigned transport. This is handled with a NULL check in the vsock/BPF recv path. Ano