From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Coquelin Subject: Re: [PATCH v2 0/5] vhost_user.c code cleanup Date: Tue, 11 Sep 2018 11:38:24 +0200 Message-ID: <952f7763-205c-12d8-10c9-c24ddd57dfbf@redhat.com> References: <153202755842.21481.1772155561595981441.stgit@T460> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Nikolay Nikolaev , tiwei.bie@intel.com, zhihong.wang@intel.com Return-path: Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by dpdk.org (Postfix) with ESMTP id E394E4C74 for ; Tue, 11 Sep 2018 11:38:27 +0200 (CEST) In-Reply-To: <153202755842.21481.1772155561595981441.stgit@T460> Content-Language: en-US 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 Nikolay, On 07/19/2018 09:13 PM, Nikolay Nikolaev wrote: > vhost: vhost_user.c code cleanup > > This patchesries introduce a set of code redesigns in vhost_user.c. > > The goal is to unify and simplify vhost-user message handling. The > patches do not intend to introduce any functional changes. > > v2 changes: > - Fix the comments by Tiwei Bie > - Keep the old behavior > - Fall through when the callback returns VH_RESULT_ERR > - Fall through if the request is out of range > > --- > > Nikolay Nikolaev (5): > vhost: unify VhostUserMsg usage > vhost: make message handling functions prepare the reply > vhost: handle unsupported message types in functions > vhost: unify message handling function signature > vhost: message handling implemented as a callback array > > > lib/librte_vhost/vhost_user.c | 392 ++++++++++++++++++++++------------------- > 1 file changed, 208 insertions(+), 184 deletions(-) > > -- > Signature > Could you please rebase and fix the series (see Iliya comments) on top of git://dpdk.org/next/dpdk-next-virtio master branch? Thanks, Maxime