From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH 1/2] rte_kni: Fix documentation. Date: Wed, 18 May 2016 11:33:12 +0100 Message-ID: <573C44E8.2030908@intel.com> References: <1463250175-3024-1-git-send-email-ee07b291@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Cc: Alex Wang To: Alex Wang , dev@dpdk.org 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) In-Reply-To: <1463250175-3024-1-git-send-email-ee07b291@gmail.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 5/14/2016 7:22 PM, Alex Wang wrote: > From: Alex Wang >=20 > The 'mbufs' alloc/free descriptions for 'rte_kni_tx_burst()' > and 'rte_kni_rx_burst()' should be inverted. >=20 > Signed-off-by: Alex Wang > --- > lib/librte_kni/rte_kni.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) >=20 > 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 *k= ni); > /** > * Retrieve a burst of packets from a KNI interface. The retrieved pac= kets are > * stored in rte_mbuf structures whose pointers are supplied in the ar= ray of > - * mbufs, and the maximum number is indicated by num. It handles the f= reeing of > - * the mbufs in the free queue of KNI interface. > + * mbufs, and the maximum number is indicated by num. It handles alloc= ating > + * 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 *kn= i, > /** > * 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 ar= ray of > - * mbufs, and the maximum number is indicated by num. It handles alloc= ating > - * the mbufs for KNI interface alloc queue. > + * mbufs, and the maximum number is indicated by num. It handles the f= reeing of > + * the mbufs in the free queue of KNI interface. > * > * @param kni > * The KNI interface context. >=20 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-subj= ect-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