From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Coquelin Subject: Re: [PATCH 7/7] vhost: simplify features set/get Date: Wed, 24 Aug 2016 10:11:57 +0200 Message-ID: <33936312-b630-1a4e-e478-b028dc264962@redhat.com> References: <1471510123-4984-1-git-send-email-yuanhan.liu@linux.intel.com> <1471510123-4984-8-git-send-email-yuanhan.liu@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit To: Yuanhan Liu , dev@dpdk.org Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id B42042951 for ; Wed, 24 Aug 2016 10:12:00 +0200 (CEST) In-Reply-To: <1471510123-4984-8-git-send-email-yuanhan.liu@linux.intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 08/18/2016 10:48 AM, Yuanhan Liu wrote: > No need to use a pointer to store/retrieve features. > > Signed-off-by: Yuanhan Liu > --- > lib/librte_vhost/vhost_user.c | 20 ++++++++------------ > 1 file changed, 8 insertions(+), 12 deletions(-) > > diff --git a/lib/librte_vhost/vhost_user.c b/lib/librte_vhost/vhost_user.c > index ef4a0c1..eee99e9 100644 > --- a/lib/librte_vhost/vhost_user.c > +++ b/lib/librte_vhost/vhost_user.c > @@ -155,23 +155,22 @@ vhost_user_reset_owner(struct virtio_net *dev) > /* > * The features that we support are requested. > */ > -static int > -vhost_user_get_features(uint64_t *pu) > +static uint64_t > +vhost_user_get_features(void) > { > - *pu = VHOST_FEATURES; > - return 0; > + return VHOST_FEATURES; > } This is not the topic of this series, but I wonder if it could make sense to be able to override supported features at device init time. It may not match with the orignal purpose of supported features, but could be useful at least for testing without recompilation. For this patch: Reviewed-by: Maxime Coquelin Thanks, Maxime