Message ID | 20190319105417.16890-1-maxime.coquelin@redhat.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D2A8B25A1; Tue, 19 Mar 2019 11:54:26 +0100 (CET) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 2D07E11A4 for <dev@dpdk.org>; Tue, 19 Mar 2019 11:54:25 +0100 (CET) Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 042E713AAE; Tue, 19 Mar 2019 10:54:24 +0000 (UTC) Received: from localhost.localdomain (ovpn-112-48.ams2.redhat.com [10.36.112.48]) by smtp.corp.redhat.com (Postfix) with ESMTP id 39A6F611D4; Tue, 19 Mar 2019 10:54:20 +0000 (UTC) From: Maxime Coquelin <maxime.coquelin@redhat.com> To: dev@dpdk.org, i.maximets@samsung.com, changpeng.liu@intel.com, tiwei.bie@intel.com, dariusz.stojaczyk@intel.com Cc: Maxime Coquelin <maxime.coquelin@redhat.com> Date: Tue, 19 Mar 2019 11:54:15 +0100 Message-Id: <20190319105417.16890-1-maxime.coquelin@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Tue, 19 Mar 2019 10:54:24 +0000 (UTC) Subject: [dpdk-dev] [PATCH v3 0/2] vhost: Support external backend only vhost-user requests X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Series |
vhost: Support external backend only vhost-user requests
|
|
Message
Maxime Coquelin
March 19, 2019, 10:54 a.m. UTC
The goals of this series is to provide more flexibility to external backends to implement their specific vhost-user request handling without having to patch vhost-user library. First patch implements a new API for external backend to advertize its specific protocol features to vhost-user master. Second patch ensures a request not handled by the vhost-user library but by the external backend only will not be treated as an error or make the vhost lib to crash. Changes in v3: ============== - Fix and improve comments (Ilya) Changes in v2: ============== - Remove 'skip_master' param from documentation (Ilya) - Merge pre and post callback in a single one (Ilya) Changes in v1: ============== - Convert vhost-crypto to the API change (Ilya) - Fix build with recent GCC (fall-through) - Check request > VHOST_USER_NONE (Ilya) - Remove 'Requested invalid message type' check (Ilya) Changes in RFC v2: ================== - Fix build & git message - Add new enum for not handled messages & remove skip_master (Ilya) Maxime Coquelin (2): vhost: add API to set protocol features flags vhost: support requests only handled by external backend lib/librte_vhost/rte_vhost.h | 53 +++++++++-------- lib/librte_vhost/rte_vhost_version.map | 1 + lib/librte_vhost/socket.c | 14 +++++ lib/librte_vhost/vhost_crypto.c | 10 +++- lib/librte_vhost/vhost_user.c | 82 ++++++++++++++++---------- 5 files changed, 102 insertions(+), 58 deletions(-)
Comments
On 3/19/19 11:54 AM, Maxime Coquelin wrote: > The goals of this series is to provide more flexibility to external > backends to implement their specific vhost-user request handling > without having to patch vhost-user library. > > First patch implements a new API for external backend to advertize > its specific protocol features to vhost-user master. > > Second patch ensures a request not handled by the vhost-user library > but by the external backend only will not be treated as an error or > make the vhost lib to crash. > > Changes in v3: > ============== > - Fix and improve comments (Ilya) > > Changes in v2: > ============== > - Remove 'skip_master' param from documentation (Ilya) > - Merge pre and post callback in a single one (Ilya) > > Changes in v1: > ============== > - Convert vhost-crypto to the API change (Ilya) > - Fix build with recent GCC (fall-through) > - Check request > VHOST_USER_NONE (Ilya) > - Remove 'Requested invalid message type' check (Ilya) > > Changes in RFC v2: > ================== > - Fix build & git message > - Add new enum for not handled messages & remove skip_master (Ilya) > > Maxime Coquelin (2): > vhost: add API to set protocol features flags > vhost: support requests only handled by external backend > > lib/librte_vhost/rte_vhost.h | 53 +++++++++-------- > lib/librte_vhost/rte_vhost_version.map | 1 + > lib/librte_vhost/socket.c | 14 +++++ > lib/librte_vhost/vhost_crypto.c | 10 +++- > lib/librte_vhost/vhost_user.c | 82 ++++++++++++++++---------- > 5 files changed, 102 insertions(+), 58 deletions(-) > Applied to dpdk-next-virtio/master branch. Thanks, Maxime
> -----Original Message----- > From: Maxime Coquelin [mailto:maxime.coquelin@redhat.com] > Sent: Wednesday, March 20, 2019 8:35 AM > To: dev@dpdk.org; i.maximets@samsung.com; Liu, Changpeng > <changpeng.liu@intel.com>; Bie, Tiwei <tiwei.bie@intel.com>; Stojaczyk, > Dariusz <dariusz.stojaczyk@intel.com> > Subject: Re: [PATCH v3 0/2] vhost: Support external backend only vhost-user > requests > > > > On 3/19/19 11:54 AM, Maxime Coquelin wrote: > > The goals of this series is to provide more flexibility to external > > backends to implement their specific vhost-user request handling > > without having to patch vhost-user library. > > > > First patch implements a new API for external backend to advertize > > its specific protocol features to vhost-user master. > > > > Second patch ensures a request not handled by the vhost-user library > > but by the external backend only will not be treated as an error or > > make the vhost lib to crash. > > > > Changes in v3: > > ============== > > - Fix and improve comments (Ilya) > > > > Changes in v2: > > ============== > > - Remove 'skip_master' param from documentation (Ilya) > > - Merge pre and post callback in a single one (Ilya) > > > > Changes in v1: > > ============== > > - Convert vhost-crypto to the API change (Ilya) > > - Fix build with recent GCC (fall-through) > > - Check request > VHOST_USER_NONE (Ilya) > > - Remove 'Requested invalid message type' check (Ilya) > > > > Changes in RFC v2: > > ================== > > - Fix build & git message > > - Add new enum for not handled messages & remove skip_master (Ilya) > > > > Maxime Coquelin (2): > > vhost: add API to set protocol features flags > > vhost: support requests only handled by external backend > > > > lib/librte_vhost/rte_vhost.h | 53 +++++++++-------- > > lib/librte_vhost/rte_vhost_version.map | 1 + > > lib/librte_vhost/socket.c | 14 +++++ > > lib/librte_vhost/vhost_crypto.c | 10 +++- > > lib/librte_vhost/vhost_user.c | 82 ++++++++++++++++---------- > > 5 files changed, 102 insertions(+), 58 deletions(-) > > > > Applied to dpdk-next-virtio/master branch. > > Thanks, > Maxime Thanks Maxime, FYI - SPDK vhost in the upstream SPDK master is already capable of running against rte_vhost with those changes. I have just run the full SPDK test suite against today's dpdk-next-virtio and everything's working. Thanks again! D.
On 3/20/19 3:32 PM, Stojaczyk, Dariusz wrote: > >> -----Original Message----- >> From: Maxime Coquelin [mailto:maxime.coquelin@redhat.com] >> Sent: Wednesday, March 20, 2019 8:35 AM >> To: dev@dpdk.org; i.maximets@samsung.com; Liu, Changpeng >> <changpeng.liu@intel.com>; Bie, Tiwei <tiwei.bie@intel.com>; Stojaczyk, >> Dariusz <dariusz.stojaczyk@intel.com> >> Subject: Re: [PATCH v3 0/2] vhost: Support external backend only vhost-user >> requests >> >> >> >> On 3/19/19 11:54 AM, Maxime Coquelin wrote: >>> The goals of this series is to provide more flexibility to external >>> backends to implement their specific vhost-user request handling >>> without having to patch vhost-user library. >>> >>> First patch implements a new API for external backend to advertize >>> its specific protocol features to vhost-user master. >>> >>> Second patch ensures a request not handled by the vhost-user library >>> but by the external backend only will not be treated as an error or >>> make the vhost lib to crash. >>> >>> Changes in v3: >>> ============== >>> - Fix and improve comments (Ilya) >>> >>> Changes in v2: >>> ============== >>> - Remove 'skip_master' param from documentation (Ilya) >>> - Merge pre and post callback in a single one (Ilya) >>> >>> Changes in v1: >>> ============== >>> - Convert vhost-crypto to the API change (Ilya) >>> - Fix build with recent GCC (fall-through) >>> - Check request > VHOST_USER_NONE (Ilya) >>> - Remove 'Requested invalid message type' check (Ilya) >>> >>> Changes in RFC v2: >>> ================== >>> - Fix build & git message >>> - Add new enum for not handled messages & remove skip_master (Ilya) >>> >>> Maxime Coquelin (2): >>> vhost: add API to set protocol features flags >>> vhost: support requests only handled by external backend >>> >>> lib/librte_vhost/rte_vhost.h | 53 +++++++++-------- >>> lib/librte_vhost/rte_vhost_version.map | 1 + >>> lib/librte_vhost/socket.c | 14 +++++ >>> lib/librte_vhost/vhost_crypto.c | 10 +++- >>> lib/librte_vhost/vhost_user.c | 82 ++++++++++++++++---------- >>> 5 files changed, 102 insertions(+), 58 deletions(-) >>> >> >> Applied to dpdk-next-virtio/master branch. >> >> Thanks, >> Maxime > > Thanks Maxime, > > FYI - SPDK vhost in the upstream SPDK master is already capable of running against rte_vhost with those changes. > I have just run the full SPDK test suite against today's dpdk-next-virtio and everything's working. > > Thanks again! > D. > Thanks for your feedback, that's much appreciated. Maxime
Tried with vhost_crypto sample APP and virtio_crypto PMD, unit test and perf test works like a charm! Thanks Maxime! Regards, Fan > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Maxime Coquelin > Sent: Thursday, March 21, 2019 9:21 AM > To: Stojaczyk, Dariusz <dariusz.stojaczyk@intel.com>; dev@dpdk.org; > i.maximets@samsung.com; Liu, Changpeng <changpeng.liu@intel.com>; Bie, > Tiwei <tiwei.bie@intel.com> > Subject: Re: [dpdk-dev] [PATCH v3 0/2] vhost: Support external backend only > vhost-user requests > > > > On 3/20/19 3:32 PM, Stojaczyk, Dariusz wrote: > > > >> -----Original Message----- > >> From: Maxime Coquelin [mailto:maxime.coquelin@redhat.com] > >> Sent: Wednesday, March 20, 2019 8:35 AM > >> To: dev@dpdk.org; i.maximets@samsung.com; Liu, Changpeng > >> <changpeng.liu@intel.com>; Bie, Tiwei <tiwei.bie@intel.com>; > >> Stojaczyk, Dariusz <dariusz.stojaczyk@intel.com> > >> Subject: Re: [PATCH v3 0/2] vhost: Support external backend only > >> vhost-user requests > >> > >> > >> > >> On 3/19/19 11:54 AM, Maxime Coquelin wrote: > >>> The goals of this series is to provide more flexibility to external > >>> backends to implement their specific vhost-user request handling > >>> without having to patch vhost-user library. > >>> > >>> First patch implements a new API for external backend to advertize > >>> its specific protocol features to vhost-user master. > >>> > >>> Second patch ensures a request not handled by the vhost-user library > >>> but by the external backend only will not be treated as an error or > >>> make the vhost lib to crash. > >>> > >>> Changes in v3: > >>> ============== > >>> - Fix and improve comments (Ilya) > >>> > >>> Changes in v2: > >>> ============== > >>> - Remove 'skip_master' param from documentation (Ilya) > >>> - Merge pre and post callback in a single one (Ilya) > >>> > >>> Changes in v1: > >>> ============== > >>> - Convert vhost-crypto to the API change (Ilya) > >>> - Fix build with recent GCC (fall-through) > >>> - Check request > VHOST_USER_NONE (Ilya) > >>> - Remove 'Requested invalid message type' check (Ilya) > >>> > >>> Changes in RFC v2: > >>> ================== > >>> - Fix build & git message > >>> - Add new enum for not handled messages & remove skip_master > >>> (Ilya) > >>> > >>> Maxime Coquelin (2): > >>> vhost: add API to set protocol features flags > >>> vhost: support requests only handled by external backend > >>> > >>> lib/librte_vhost/rte_vhost.h | 53 +++++++++-------- > >>> lib/librte_vhost/rte_vhost_version.map | 1 + > >>> lib/librte_vhost/socket.c | 14 +++++ > >>> lib/librte_vhost/vhost_crypto.c | 10 +++- > >>> lib/librte_vhost/vhost_user.c | 82 ++++++++++++++++---------- > >>> 5 files changed, 102 insertions(+), 58 deletions(-) > >>> > >> > >> Applied to dpdk-next-virtio/master branch. > >> > >> Thanks, > >> Maxime > > > > Thanks Maxime, > > > > FYI - SPDK vhost in the upstream SPDK master is already capable of running > against rte_vhost with those changes. > > I have just run the full SPDK test suite against today's dpdk-next-virtio and > everything's working. > > > > Thanks again! > > D. > > > > Thanks for your feedback, that's much appreciated. > > Maxime