From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 481FFA04B5; Tue, 27 Oct 2020 04:09:32 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 22F262B9D; Tue, 27 Oct 2020 04:09:30 +0100 (CET) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id BBF81100C for ; Tue, 27 Oct 2020 04:09:26 +0100 (CET) IronPort-SDR: +5mHswpqverR+j/RFpMi4jrtOsz76bWoDRb9a4ixzKjP2c1eaGtPYaWVAYleR4p9i4+Ld7i5o4 0il0j155hbtg== X-IronPort-AV: E=McAfee;i="6000,8403,9786"; a="229656678" X-IronPort-AV: E=Sophos;i="5.77,422,1596524400"; d="scan'208";a="229656678" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Oct 2020 20:09:25 -0700 IronPort-SDR: V81bV9Sw/hPnwbxF9NSbxF75vPbFx3X6ExcEHZFQK0YT5RqZlaSvXCW7Acv5k9OWBbjmd9K+r4 5IeUg7pM+jKg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,422,1596524400"; d="scan'208";a="394324743" Received: from fmsmsx603.amr.corp.intel.com ([10.18.126.83]) by orsmga001.jf.intel.com with ESMTP; 26 Oct 2020 20:09:25 -0700 Received: from shsmsx606.ccr.corp.intel.com (10.109.6.216) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Mon, 26 Oct 2020 20:09:24 -0700 Received: from shsmsx605.ccr.corp.intel.com (10.109.6.215) by SHSMSX606.ccr.corp.intel.com (10.109.6.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Tue, 27 Oct 2020 11:09:22 +0800 Received: from shsmsx605.ccr.corp.intel.com ([10.109.6.215]) by SHSMSX605.ccr.corp.intel.com ([10.109.6.215]) with mapi id 15.01.1713.004; Tue, 27 Oct 2020 11:09:22 +0800 From: "Jiang, YuX" To: Adrian Moreno , "dev@dpdk.org" CC: "Wang, Yinan" , "Fu, Patrick" , "Xia, Chenbo" , "Wang, Zhihong" , "maxime.coquelin@redhat.com" , "Jiang, YuX" Thread-Topic: [dpdk-dev] [PATCH v3 0/6] net/virtio-user: fix server mode Thread-Index: AQHWq7bX6HSKrYG0ZEuLAQc3VjHeT6mqxJAw Date: Tue, 27 Oct 2020 03:09:22 +0000 Message-ID: <8cc94ae050884ba1adc236a5d1c5f3bd@intel.com> References: <20201026163930.94032-1-amorenoz@redhat.com> In-Reply-To: <20201026163930.94032-1-amorenoz@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.36] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v3 0/6] net/virtio-user: fix server mode X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Tested-by: JiangYuX Best Regards Jiang yu > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Adrian Moreno > Sent: Tuesday, October 27, 2020 12:39 AM > To: dev@dpdk.org > Cc: Wang, Yinan ; Fu, Patrick > ; Xia, Chenbo ; Wang, > Zhihong ; maxime.coquelin@redhat.com; Adrian > Moreno > Subject: [dpdk-dev] [PATCH v3 0/6] net/virtio-user: fix server mode >=20 > A number of issues have been detected that currently break virtio-user > server mode. > This series addresses such issues. > Note that virtio-user server mode is broken by design and many of the > problems that it currently has should be fixed by a bigger rework. >=20 > -- > v2->v3: > Fix potential concurrency problem on get/set state Handle STATUS protocol > feature flag Fix undefined behaviour if STATUS feature is not supported > Ensure packed virtqueues are reset on reconnection >=20 >=20 > v1->v2: > Added patch 2 and 3 addressing additional issues Check errno to select vh= ost- > user backend and log the detected backend type >=20 > Adrian Moreno (6): > net/virtio-user: fix backend selection if stat fails > net/virtio-user: don't set/get_status until FEATURES_OK > net/virtio-user: ignore result if STATUS is unsupported > net/virtio-user: lock-protect status updates > net/virtio-user: don't assume vhost status feature > net/virtio-user: set status on socket reconnect >=20 > drivers/net/virtio/virtio_user/vhost_user.c | 14 ++--- > .../net/virtio/virtio_user/virtio_user_dev.c | 53 +++++++++++++------ > .../net/virtio/virtio_user/virtio_user_dev.h | 5 +- > drivers/net/virtio/virtio_user_ethdev.c | 40 ++++++++++---- > 4 files changed, 78 insertions(+), 34 deletions(-) >=20 > -- > 2.26.2