On Wed, Nov 20, 2024 at 10:48:25PM -0800, John Fastabend wrote:
Michal Luczaj wrote:
Two small fixes for vsock: poll() missing a queue check, and close() not invoking sockmap cleanup.
Signed-off-by: Michal Luczaj mhal@rbox.co
Michal Luczaj (4): bpf, vsock: Fix poll() missing a queue selftest/bpf: Add test for af_vsock poll() bpf, vsock: Invoke proto::close on close() selftest/bpf: Add test for vsock removal from sockmap on close()
net/vmw_vsock/af_vsock.c | 70 ++++++++++++-------- .../selftests/bpf/prog_tests/sockmap_basic.c | 77 ++++++++++++++++++++++ 2 files changed, 120 insertions(+), 27 deletions(-)
base-commit: 6c4139b0f19b7397286897caee379f8321e78272 change-id: 20241118-vsock-bpf-poll-close-64f432e682ec
Best regards,
Michal Luczaj mhal@rbox.co
LGTM, would be nice to get an ack from someone on the vsock side though.
Sorry, is at the top of my list but other urgent things have come up.
I will review it by today.
Stefano
Acked-by: John Fastabend john.fastabend@gmail.com