| Message ID | 20241118-vsock-bpf-poll-close-v1-0-f1b9669cacdc@rbox.co (mailing list archive) |
|---|---|
| Headers | show |
| Series | bpf, vsock: Fix poll() and close() | expand |
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. Acked-by: John Fastabend <john.fastabend@gmail.com>
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> >
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,