[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/10] tests/vhost-user-bridge: add client mode
From: |
Victor Kaplansky |
Subject: |
Re: [Qemu-devel] [PATCH 02/10] tests/vhost-user-bridge: add client mode |
Date: |
Thu, 9 Jun 2016 12:53:24 +0300 |
On Mon, Jun 06, 2016 at 06:45:00PM +0200, address@hidden wrote:
> From: Marc-André Lureau <address@hidden>
>
> If -c is specified, vubr will try to connect to the socket instead of
> listening for connections.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> Tested-by: Yuanhan Liu <address@hidden>
> Reviewed-by: Yuanhan Liu <address@hidden>
> ---
> tests/vhost-user-bridge.c | 38 ++++++++++++++++++++++++++------------
> 1 file changed, 26 insertions(+), 12 deletions(-)
>
> diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
> index 0779ba2..f907ce7 100644
> --- a/tests/vhost-user-bridge.c
> +++ b/tests/vhost-user-bridge.c
> @@ -1204,12 +1204,13 @@ vubr_accept_cb(int sock, void *ctx)
> }
>
> static VubrDev *
> -vubr_new(const char *path)
> +vubr_new(const char *path, bool client)
> {
> VubrDev *dev = (VubrDev *) calloc(1, sizeof(VubrDev));
> dev->nregions = 0;
> int i;
> struct sockaddr_un un;
> + CallbackFunc cb;
> size_t len;
>
> for (i = 0; i < MAX_NR_VIRTQUEUE; i++) {
> @@ -1238,19 +1239,27 @@ vubr_new(const char *path)
> un.sun_family = AF_UNIX;
> strcpy(un.sun_path, path);
> len = sizeof(un.sun_family) + strlen(path);
> - unlink(path);
>
> - if (bind(dev->sock, (struct sockaddr *) &un, len) == -1) {
> - vubr_die("bind");
> - }
> + if (!client) {
> + unlink(path);
> +
> + if (bind(dev->sock, (struct sockaddr *) &un, len) == -1) {
> + vubr_die("bind");
> + }
>
> - if (listen(dev->sock, 1) == -1) {
> - vubr_die("listen");
> + if (listen(dev->sock, 1) == -1) {
> + vubr_die("listen");
> + }
> + cb = vubr_accept_cb;
> + } else {
> + if (connect(dev->sock, (struct sockaddr *)&un, len) == -1) {
> + vubr_die("connect");
> + }
> + cb = vubr_receive_cb;
> }
>
> dispatcher_init(&dev->dispatcher);
> - dispatcher_add(&dev->dispatcher, dev->sock, (void *)dev,
> - vubr_accept_cb);
> + dispatcher_add(&dev->dispatcher, dev->sock, (void *)dev, cb);
>
> DPRINT("Waiting for connections on UNIX socket %s ...\n", path);
I think this message should be issued only for server mode.
> return dev;
> @@ -1369,8 +1378,9 @@ main(int argc, char *argv[])
> {
> VubrDev *dev;
> int opt;
> + bool client = false;
>
> - while ((opt = getopt(argc, argv, "l:r:u:")) != -1) {
> + while ((opt = getopt(argc, argv, "l:r:u:c")) != -1) {
>
> switch (opt) {
> case 'l':
> @@ -1386,16 +1396,20 @@ main(int argc, char *argv[])
> case 'u':
> ud_socket_path = strdup(optarg);
> break;
> + case 'c':
> + client = true;
> + break;
> default:
> goto out;
> }
> }
>
> - DPRINT("ud socket: %s\n", ud_socket_path);
> + DPRINT("ud socket: %s (%s)\n", ud_socket_path,
> + client ? "client" : "server");
> DPRINT("local: %s:%s\n", lhost, lport);
> DPRINT("remote: %s:%s\n", rhost, rport);
>
> - dev = vubr_new(ud_socket_path);
> + dev = vubr_new(ud_socket_path, client);
> if (!dev) {
> return 1;
> }
> --
> 2.7.4
>
- [Qemu-devel] [PATCH 00/10] vhost-user: simple reconnection support, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 01/10] vhost-user: add ability to know vhost-user backend disconnection, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 03/10] tests/vhost-user-bridge: workaround stale vring base, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 02/10] tests/vhost-user-bridge: add client mode, marcandre . lureau, 2016/06/06
- Re: [Qemu-devel] [PATCH 02/10] tests/vhost-user-bridge: add client mode,
Victor Kaplansky <=
- [Qemu-devel] [PATCH 04/10] qemu-char: add qemu_chr_disconnect to close a fd accepted by listen fd, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 05/10] vhost-user: disconnect on start failure, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 07/10] vhost-net: save & restore vhost-user acked features, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 06/10] vhost-net: do not crash if backend is not present, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 08/10] vhost-net: save & restore vring enable state, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 09/10] tests: append i386 tests, marcandre . lureau, 2016/06/06
- [Qemu-devel] [PATCH 10/10] test: start vhost-user reconnect test, marcandre . lureau, 2016/06/06
- Re: [Qemu-devel] [PATCH 00/10] vhost-user: simple reconnection support, Victor Kaplansky, 2016/06/09