From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 9C9FB6910 for ; Wed, 18 May 2016 12:33:24 +0200 (CEST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP; 18 May 2016 03:33:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,328,1459839600"; d="scan'208";a="106063925" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.123]) ([10.237.220.123]) by fmsmga004.fm.intel.com with ESMTP; 18 May 2016 03:33:13 -0700 To: Alex Wang , dev@dpdk.org References: <1463250175-3024-1-git-send-email-ee07b291@gmail.com> Cc: Alex Wang From: Ferruh Yigit Message-ID: <573C44E8.2030908@intel.com> Date: Wed, 18 May 2016 11:33:12 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2 MIME-Version: 1.0 In-Reply-To: <1463250175-3024-1-git-send-email-ee07b291@gmail.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH 1/2] rte_kni: Fix documentation. X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 May 2016 10:33:25 -0000 On 5/14/2016 7:22 PM, Alex Wang wrote: > From: Alex Wang > > The 'mbufs' alloc/free descriptions for 'rte_kni_tx_burst()' > and 'rte_kni_rx_burst()' should be inverted. > > Signed-off-by: Alex Wang > --- > lib/librte_kni/rte_kni.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h > index ef9faa9..25fa45e 100644 > --- a/lib/librte_kni/rte_kni.h > +++ b/lib/librte_kni/rte_kni.h > @@ -161,8 +161,8 @@ extern int rte_kni_handle_request(struct rte_kni *kni); > /** > * Retrieve a burst of packets from a KNI interface. The retrieved packets are > * stored in rte_mbuf structures whose pointers are supplied in the array of > - * mbufs, and the maximum number is indicated by num. It handles the freeing of > - * the mbufs in the free queue of KNI interface. > + * mbufs, and the maximum number is indicated by num. It handles allocating > + * the mbufs for KNI interface alloc queue. > * > * @param kni > * The KNI interface context. > @@ -180,8 +180,8 @@ extern unsigned rte_kni_rx_burst(struct rte_kni *kni, > /** > * Send a burst of packets to a KNI interface. The packets to be sent out are > * stored in rte_mbuf structures whose pointers are supplied in the array of > - * mbufs, and the maximum number is indicated by num. It handles allocating > - * the mbufs for KNI interface alloc queue. > + * mbufs, and the maximum number is indicated by num. It handles the freeing of > + * the mbufs in the free queue of KNI interface. > * > * @param kni > * The KNI interface context. > Hi Alex, Can you please update the patch subject, - replace "rte_kni" tag with a "kni", - after space start with lowercase, - remove the "." at the end of the sentences, like: "kni: fix documentation" (these are defined in http://dpdk.org/doc/guides/contributing/patches.html#commit-messages-subject-line) Also can you please add a "Fixes" line, more details on: http://dpdk.org/doc/guides/contributing/patches.html#commit-messages-body Although this information converted into documentation, this is not really the documentation, and the patch title gives little information, if possible can you please add more information while keeping it around 50 chars limit. finally, patch content is OK. Thanks, ferruh