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 9523DA0536; Tue, 4 Feb 2020 23:19:25 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id DC84F1C1FC; Tue, 4 Feb 2020 23:19:24 +0100 (CET) Received: from mail.valinux.co.jp (mail.valinux.co.jp [210.128.90.3]) by dpdk.org (Postfix) with ESMTP id 769731C11D for ; Tue, 4 Feb 2020 23:19:23 +0100 (CET) Received: from localhost (localhost [127.0.0.1]) by mail.valinux.co.jp (Postfix) with ESMTP id 109F3B5C73; Wed, 5 Feb 2020 07:19:22 +0900 (JST) X-Virus-Scanned: Debian amavisd-new at valinux.co.jp Received: from mail.valinux.co.jp ([127.0.0.1]) by localhost (mail.valinux.co.jp [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id sflCHjobawjv; Wed, 5 Feb 2020 07:19:22 +0900 (JST) Received: from [127.0.0.1] (vagw.valinux.co.jp [210.128.90.14]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.valinux.co.jp (Postfix) with ESMTPS id EC860B5BA7; Wed, 5 Feb 2020 07:19:21 +0900 (JST) Date: Wed, 05 Feb 2020 07:19:22 +0900 From: Itsuro ODA To: Maxime Coquelin Cc: tiwei.bie@intel.com, zhihong.wang@intel.com, anatoly.burakov@intel.com, dev@dpdk.org, thomas@monjalon.net In-Reply-To: <966a1a8b-1d67-a822-64b6-9cc3c55a7780@redhat.com> References: <20200120111750.5DEB.277DD91C@valinux.co.jp> <966a1a8b-1d67-a822-64b6-9cc3c55a7780@redhat.com> Message-Id: <20200205071922.5E3F.277DD91C@valinux.co.jp> MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Mailer: Becky! ver. 2.74.02 [ja] Subject: Re: [dpdk-dev] [PATCH 0/4] make vhost PMD available for secondary processes 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" Hi Maxime, Thank you for your reply and review. I will make fixes of the patches according to your indication and post them. Thanks. Itsuro Oda On Tue, 4 Feb 2020 18:54:23 +0100 Maxime Coquelin wrote: > Hi Itsuro, > > On 1/20/20 3:17 AM, Itsuro ODA wrote: > > Hi vhost PMD maitainers, > > > > I have not got any feedback yet. > > Since this is the first time I submit a patch, something > > may be wrong, would you tell me what should I do ? > > Sorry for the delay, and thanks for the contribution. > > You series does not apply properly on dpdk-next-virtio master branch: > https://git.dpdk.org/next/dpdk-next-virtio > > I will review it, so when doing v3, please rebase it. > > More generally, you series comprises fixes (patch 2 to 4), and > cleanup (patch 1). > > Cleanup patch should be the last, in order to ease the backporting > of the fixes to LTSes (we avoid backporting cleanup patches). > > Regarding fixes patches, it should tag the faulty commit in > master branch, and stable@dpdk.org should be Cc'ed. > > Example of commit message with fixes tag: > http://patches.dpdk.org/patch/63305/ > > Finally, when you'll post the v3, please prefix the patches subject > with the revision number: > > git format-patch --subject-prefix"PATCH v3" ... > > Thanks, > Maxime > > > Thanks. > > > > On Thu, 9 Jan 2020 08:22:05 +0900 > > Itsuro Oda wrote: > > > >> vhost PMD has not been available for secondary processes since > >> DPDK v18.11. (https://bugs.dpdk.org/show_bug.cgi?id=194) > >> (for a long term !) > >> This series of patches intend to make vhost PMD available for > >> secondary processes. > >> Because now setting vhost driver to communicate with a vhost-user > >> master (ex. Qemu) is accomplished by the probe function of the > >> primary process, only the primary process can be a vhost-user > >> slave. > >> With this patch, setting vhost driver is delayed at eth_dev > >> configuration in order to be able to set it from a secondary > >> process. Because (in the first place,) setting vhost driver is not > >> necessary to be done at probe (it is enough to be done up to eth_dev > >> start), this fix is no problem for the primary process. > >> There is a precondition that the same process has to operate > >> a vhost interface from beginning to end (from eth_dev configuration > >> to eth_dev close). (This patch leaves it to user's responsibility.) > >> This precondition will not be a problem in most use cases > >> (including SPP). > >> > >> v2: > >> - add signed-off-by > >> - fix spelling error > >> > >> Itsuro Oda (4): > >> net/vhost: remove an unused member > >> net/vhost: allocate iface_name from heap > >> net/vhost: delay vhost driver setup > >> net/vhost: make secondary probe complete > >> > >> drivers/net/vhost/rte_eth_vhost.c | 152 +++++++++++++++++------------- > >> 1 file changed, 88 insertions(+), 64 deletions(-) > >> > >> -- > >> 2.17.0 > > -- Itsuro ODA