From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 7C25EA00E6 for ; Fri, 22 Mar 2019 18:43:32 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 477FD1B72C; Fri, 22 Mar 2019 18:43:31 +0100 (CET) Received: from mail-vk1-f194.google.com (mail-vk1-f194.google.com [209.85.221.194]) by dpdk.org (Postfix) with ESMTP id CE1261B726 for ; Fri, 22 Mar 2019 18:43:25 +0100 (CET) Received: by mail-vk1-f194.google.com with SMTP id k64so657370vke.1 for ; Fri, 22 Mar 2019 10:43:25 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=p5VKJkab4dGNG+s9bQtNKzXcs+atu44/B99YwUEJZhY=; b=UVkfvRagg41Utla+7GudA3H4RhaB2GF1LEMyhMv8orRLu06uXOULToDaIMpMqkPGn/ YH+sS1h+x5vA2UEO/DB3CH2eNPzH9cnAJpFK/AyVXJSvFTTfE33MxTLCrjNf9B8Pt8sE N6mx3rICFmQMATNEU1V1W52uKdV9UzliL/YAwl6rhH6VWrGBSKeuXdi4J4X0kQwAFkbI g+84DMgQvtFXqy+q3Pn0j01UfuRSdPusAO3EF4kJBY2iJph0iu2h12EooMtdJuWA3PqR n+mh/zDfb/fSsIYCIBQi3zTVhwzqvoBpjvRgPS3xMMmWXjid1m8Th/2EEvIWKCY2Yo8q 9O+g== X-Gm-Message-State: APjAAAWpSNCxHgq6sjCtPK4di6yL8SaRmrhHSKfr48bjxp2Eh2UQhqjT fXsIU07AQPTGUN01KXJOXSdPJCbEsdp+qgydltnydw== X-Google-Smtp-Source: APXvYqzO5WNyshft0ZWaGJpQHj1tEWDU3ktBeAUd5tVn3HWZEdLfsNmIwr0ibSAmQ/U28IvaEzakeEJha7qsGTUXmko= X-Received: by 2002:a1f:c06:: with SMTP id 6mr6811612vkm.52.1553276605209; Fri, 22 Mar 2019 10:43:25 -0700 (PDT) MIME-Version: 1.0 References: <1553076154-3907-1-git-send-email-david.marchand@redhat.com> <1553261824-1881-1-git-send-email-david.marchand@redhat.com> <1553261824-1881-2-git-send-email-david.marchand@redhat.com> <8568275c-04a1-9d1a-6fa7-ecf7487cce3e@solarflare.com> In-Reply-To: <8568275c-04a1-9d1a-6fa7-ecf7487cce3e@solarflare.com> From: David Marchand Date: Fri, 22 Mar 2019 18:43:14 +0100 Message-ID: To: Andrew Rybchenko Cc: dev , Wenzhuo Lu , Jingjing Wu , "Iremonger, Bernard" , Rami Rosen , "Yigit, Ferruh" , dpdk stable Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-stable] [PATCH v4 1/4] app/testpmd: add missing newline when showing statistics X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On Fri, Mar 22, 2019 at 6:35 PM Andrew Rybchenko wrote: > On 22.03.2019 16:37, David Marchand wrote: > > Having the standard stats and the rx burst stats on the same line gives a > > really long line and is not consistent with the rest. > > > > Before: > > RX-packets: 3542977 TX-packets: 3542971 TX-dropped: 6 > RX-bursts : 499440 [24% of 2 pkts + 15% of 1 pkts + 61% of > others] > > TX-bursts : 499440 [24% of 2 pkts + 15% of 1 pkts + 61% of others] > > > > After: > > RX-packets: 4629969 TX-packets: 4629969 TX-dropped: 0 > > RX-bursts : 663328 [19% of 2 pkts + 17% of 3 pkts + 64% of others] > > TX-bursts : 663328 [19% of 2 pkts + 17% of 3 pkts + 64% of others] > > > > Fixes: af75078fece3 ("first public release") > > Cc: stable@dpdk.org > > > > Signed-off-by: David Marchand > > Reviewed-by: Rami Rosen > > Reviewed-by: Andrew Rybchenko > > --- > > Changelog since v2: > > - Cc'd stable > > > > --- > > app/test-pmd/testpmd.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c > > index 216be47..40199c1 100644 > > --- a/app/test-pmd/testpmd.c > > +++ b/app/test-pmd/testpmd.c > > @@ -1459,7 +1459,7 @@ struct extmem_param { > > "TX Port=%2d/Queue=%2d %s\n", > > fwd_top_stats_border, fs->rx_port, fs->rx_queue, > > fs->tx_port, fs->tx_queue, fwd_top_stats_border); > > - printf(" RX-packets: %-14u TX-packets: %-14u TX-dropped: %-14u", > > + printf(" RX-packets: %-14u TX-packets: %-14u TX-dropped: %-14u\n", > > fs->rx_packets, fs->tx_packets, fs->fwd_dropped); > > > > /* if checksum mode */ > > > In fact bad Rx checksum counters follow and I think they should be in > this line. > > That's why there is no \n here from the very beginning. > > My fix for the bug (local) just add \n in else branch below. > Mm, ok, I will look at this monday. I might add the \n in pkt_burst_stats_display() and update other callers. -- David Marchand