From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.droids-corp.org (zoll.droids-corp.org [94.23.50.67]) by dpdk.org (Postfix) with ESMTP id BB45F56AB for ; Fri, 7 Oct 2016 17:39:15 +0200 (CEST) Received: from lfbn-1-5996-232.w90-110.abo.wanadoo.fr ([90.110.195.232] helo=[192.168.1.13]) by mail.droids-corp.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.84_2) (envelope-from ) id 1bsXI2-0005jT-CU; Fri, 07 Oct 2016 17:42:26 +0200 To: "De Lara Guarch, Pablo" , "dev@dpdk.org" References: <1473407734-11253-2-git-send-email-olivier.matz@6wind.com> <1475743376-10360-1-git-send-email-olivier.matz@6wind.com> From: Olivier Matz Message-ID: Date: Fri, 7 Oct 2016 17:39:09 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v3] mbuf: add function to dump ol flag list 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: Fri, 07 Oct 2016 15:39:15 -0000 Hi Pablo, On 10/07/2016 05:51 AM, De Lara Guarch, Pablo wrote: > > >> -----Original Message----- >> From: Olivier Matz [mailto:olivier.matz@6wind.com] >> Sent: Thursday, October 06, 2016 1:43 AM >> To: dev@dpdk.org; De Lara Guarch, Pablo >> Subject: [PATCH v3] mbuf: add function to dump ol flag list >> >> The functions rte_get_rx_ol_flag_name() and rte_get_tx_ol_flag_name() >> can dump one flag, or set of flag that are part of the same mask (ex: >> PKT_TX_UDP_CKSUM, part of PKT_TX_L4_MASK). But they are not designed >> to >> dump the list of flags contained in mbuf->ol_flags. >> >> This commit introduce new functions to do that. Similarly to the packet >> type dump functions, the goal is to factorize the code that could be >> used in several applications and reduce the risk of desynchronization >> between the flags and the dump functions. >> >> Signed-off-by: Olivier Matz > > Hi Olivier, > > Sorry, I missed a typo in this patch: "ouput" -> "output". > > Also, check-git-log.sh is complaining about two patches: > > Wrong headline lowercase: > app/testpmd: dump rx flags in csum engine > app/testpmd: display rx port in csum engine > > Lastly, could you send another version of the patchset (including patches without any modifications). > In my opinion, it is a bit difficult to apply the patchset, because Patchwork doesn't tell me > that this patch is the first patch of the patchset. Thank you for the review. I'll fix all these issue and send the whole patchset again (will be called v5 to avoid confusion with separate patches). I didn't want to spam the list with the full patchset for these minor fixes, but you're right it's not that clear. Regards, Olivier