| Message ID | 20191218180708.120337-1-sgarzare@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | VSOCK: add vsock_test test suite | expand |
On Wed, Dec 18, 2019 at 07:06:57PM +0100, Stefano Garzarella wrote: > The vsock_diag.ko module already has a test suite but the core AF_VSOCK > functionality has no tests. This patch series adds several test cases that > exercise AF_VSOCK SOCK_STREAM socket semantics (send/recv, connect/accept, > half-closed connections, simultaneous connections). > > The v1 of this series was originally sent by Stefan. > > v3: > - Patch 6: > * check the byte received in the recv_byte() > * use send(2)/recv(2) instead of write(2)/read(2) to test also flags > (e.g. MSG_PEEK) > - Patch 8: > * removed unnecessary control_expectln("CLOSED") [Stefan]. > - removed patches 9,10,11 added in the v2 > - new Patch 9 add parameters to list and skip tests (e.g. useful for vmci > that doesn't support half-closed socket in the host) > - new Patch 10 prints a list of options in the help > - new Patch 11 tests MSG_PEEK flags of recv(2) > > v2: https://patchwork.ozlabs.org/cover/1140538/ > v1: https://patchwork.ozlabs.org/cover/847998/ > > Stefan Hajnoczi (7): > VSOCK: fix header include in vsock_diag_test > VSOCK: add SPDX identifiers to vsock tests > VSOCK: extract utility functions from vsock_diag_test.c > VSOCK: extract connect/accept functions from vsock_diag_test.c > VSOCK: add full barrier between test cases > VSOCK: add send_byte()/recv_byte() test utilities > VSOCK: add AF_VSOCK test cases > > Stefano Garzarella (4): > vsock_test: wait for the remote to close the connection > testing/vsock: add parameters to list and skip tests > testing/vsock: print list of options and description > vsock_test: add SOCK_STREAM MSG_PEEK test > > tools/testing/vsock/.gitignore | 1 + > tools/testing/vsock/Makefile | 9 +- > tools/testing/vsock/README | 3 +- > tools/testing/vsock/control.c | 15 +- > tools/testing/vsock/control.h | 2 + > tools/testing/vsock/timeout.h | 1 + > tools/testing/vsock/util.c | 376 +++++++++++++++++++++++++ > tools/testing/vsock/util.h | 49 ++++ > tools/testing/vsock/vsock_diag_test.c | 202 ++++---------- > tools/testing/vsock/vsock_test.c | 379 ++++++++++++++++++++++++++ > 10 files changed, 883 insertions(+), 154 deletions(-) > create mode 100644 tools/testing/vsock/util.c > create mode 100644 tools/testing/vsock/util.h > create mode 100644 tools/testing/vsock/vsock_test.c > > -- > 2.24.1 > > _______________________________________________ > Virtualization mailing list > Virtualization@lists.linux-foundation.org > https://lists.linuxfoundation.org/mailman/listinfo/virtualization Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
From: Stefano Garzarella <sgarzare@redhat.com> Date: Wed, 18 Dec 2019 19:06:57 +0100 > The vsock_diag.ko module already has a test suite but the core AF_VSOCK > functionality has no tests. This patch series adds several test cases that > exercise AF_VSOCK SOCK_STREAM socket semantics (send/recv, connect/accept, > half-closed connections, simultaneous connections). > > The v1 of this series was originally sent by Stefan. > > v3: > - Patch 6: > * check the byte received in the recv_byte() > * use send(2)/recv(2) instead of write(2)/read(2) to test also flags > (e.g. MSG_PEEK) > - Patch 8: > * removed unnecessary control_expectln("CLOSED") [Stefan]. > - removed patches 9,10,11 added in the v2 > - new Patch 9 add parameters to list and skip tests (e.g. useful for vmci > that doesn't support half-closed socket in the host) > - new Patch 10 prints a list of options in the help > - new Patch 11 tests MSG_PEEK flags of recv(2) > > v2: https://patchwork.ozlabs.org/cover/1140538/ > v1: https://patchwork.ozlabs.org/cover/847998/ Series applied, thanks.
The vsock_diag.ko module already has a test suite but the core AF_VSOCK functionality has no tests. This patch series adds several test cases that exercise AF_VSOCK SOCK_STREAM socket semantics (send/recv, connect/accept, half-closed connections, simultaneous connections). The v1 of this series was originally sent by Stefan. v3: - Patch 6: * check the byte received in the recv_byte() * use send(2)/recv(2) instead of write(2)/read(2) to test also flags (e.g. MSG_PEEK) - Patch 8: * removed unnecessary control_expectln("CLOSED") [Stefan]. - removed patches 9,10,11 added in the v2 - new Patch 9 add parameters to list and skip tests (e.g. useful for vmci that doesn't support half-closed socket in the host) - new Patch 10 prints a list of options in the help - new Patch 11 tests MSG_PEEK flags of recv(2) v2: https://patchwork.ozlabs.org/cover/1140538/ v1: https://patchwork.ozlabs.org/cover/847998/ Stefan Hajnoczi (7): VSOCK: fix header include in vsock_diag_test VSOCK: add SPDX identifiers to vsock tests VSOCK: extract utility functions from vsock_diag_test.c VSOCK: extract connect/accept functions from vsock_diag_test.c VSOCK: add full barrier between test cases VSOCK: add send_byte()/recv_byte() test utilities VSOCK: add AF_VSOCK test cases Stefano Garzarella (4): vsock_test: wait for the remote to close the connection testing/vsock: add parameters to list and skip tests testing/vsock: print list of options and description vsock_test: add SOCK_STREAM MSG_PEEK test tools/testing/vsock/.gitignore | 1 + tools/testing/vsock/Makefile | 9 +- tools/testing/vsock/README | 3 +- tools/testing/vsock/control.c | 15 +- tools/testing/vsock/control.h | 2 + tools/testing/vsock/timeout.h | 1 + tools/testing/vsock/util.c | 376 +++++++++++++++++++++++++ tools/testing/vsock/util.h | 49 ++++ tools/testing/vsock/vsock_diag_test.c | 202 ++++---------- tools/testing/vsock/vsock_test.c | 379 ++++++++++++++++++++++++++ 10 files changed, 883 insertions(+), 154 deletions(-) create mode 100644 tools/testing/vsock/util.c create mode 100644 tools/testing/vsock/util.h create mode 100644 tools/testing/vsock/vsock_test.c