From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtprelay04.ispgateway.de (smtprelay04.ispgateway.de [80.67.31.42]) by dpdk.org (Postfix) with ESMTP id 550CA5A6B for ; Thu, 9 Jul 2015 17:14:43 +0200 (CEST) Received: from [87.172.158.174] (helo=nb-klaus.allegro) by smtprelay04.ispgateway.de with esmtpsa (TLSv1.2:DHE-RSA-AES128-SHA:128) (Exim 4.84) (envelope-from ) id 1ZDDX9-0004mY-KD; Thu, 09 Jul 2015 17:14:43 +0200 To: "Mcnamara, John" , "dev@dpdk.org" References: <1435664375-25648-1-git-send-email-kd@allegro-packets.com> From: Klaus Degner Message-ID: <559E8FE1.7020309@allegro-packets.com> Date: Thu, 9 Jul 2015 17:14:41 +0200 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.0.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Df-Sender: a2RAYWxsZWdyby1wYWNrZXRzLmNvbQ== Subject: Re: [dpdk-dev] [PATCH] add rx and tx byte counter statistics for PCAP PMD 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: Thu, 09 Jul 2015 15:14:43 -0000 Hi John, Thank you for your review. I will submit a v2 of the patch. Klaus Am 08.07.15 um 18:11 schrieb Mcnamara, John: >> -----Original Message----- >> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Klaus Degner >> Sent: Tuesday, June 30, 2015 12:40 PM >> To: dev@dpdk.org >> Subject: [dpdk-dev] [PATCH] add rx and tx byte counter statistics for >> PCAP PMD >> >> PCAP PMD vdev accounts only rx and tx packet statistics. >> This patch adds support for rx and tx bytes statistics. > Hi, > > Thanks for that. > > Just a few minor comments. > > The subject line should contain the library/module being updated and the body is repeats, more or less, the same comment twice. Do a git log on the file to see the standard format. > >> unsigned i; >> - unsigned long rx_total = 0, tx_total = 0, tx_err_total = 0; >> + unsigned long rx_total = 0, tx_total = 0, tx_err_total = 0, >> +rx_b_total = 0, tx_b_total = 0; > This line exceeds the 80 character limit. Maybe separate the rx and tx initialisations onto separate lines. Or else put them all on separate lines in line with the Coding Guidelines: > > http://dpdk.readthedocs.org/en/latest/guidelines/coding_style.html#local-variables > > Run checkpatch on the patch before submission to pick up any issues like this. > > If you make those changes and resubmit as a V2 I'll ack it. > > John. > -- > > >