From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 7F78F3238 for ; Fri, 23 Nov 2018 11:29:16 +0100 (CET) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id E2F823002C85; Fri, 23 Nov 2018 10:29:15 +0000 (UTC) Received: from ktraynor.remote.csb (unknown [10.36.118.7]) by smtp.corp.redhat.com (Postfix) with ESMTP id CD1A117DA1; Fri, 23 Nov 2018 10:29:13 +0000 (UTC) From: Kevin Traynor To: Cristian Dumitrescu Cc: Jasvinder Singh , Hongjun Ni , dpdk stable Date: Fri, 23 Nov 2018 10:26:25 +0000 Message-Id: <20181123102713.17309-21-ktraynor@redhat.com> In-Reply-To: <20181123102713.17309-1-ktraynor@redhat.com> References: <20181123102713.17309-1-ktraynor@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.43]); Fri, 23 Nov 2018 10:29:15 +0000 (UTC) Subject: [dpdk-stable] patch 'examples/ip_pipeline: fix port and table stats read' has been queued to stable release 18.08.1 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: , X-List-Received-Date: Fri, 23 Nov 2018 10:29:16 -0000 Hi, FYI, your patch has been queued to stable release 18.08.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/29/18. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. If the code is different (ie: not only metadata diffs), due for example to a change in context or macro names, please double check it. Thanks. Kevin Traynor --- >>From 9f42c99f9fa195ce4afa24072d6453a53ee99802 Mon Sep 17 00:00:00 2001 From: Cristian Dumitrescu Date: Tue, 30 Oct 2018 18:44:09 +0000 Subject: [PATCH] examples/ip_pipeline: fix port and table stats read [ upstream commit c4160d303026ed11707e4a3a7a2e12b9be863d5a ] Fix the pipeline port and table stats read operation. Fixes: 50e73d051806 ("examples/ip_pipeline: add stats read commands") Signed-off-by: Cristian Dumitrescu Signed-off-by: Jasvinder Singh Signed-off-by: Hongjun Ni --- examples/ip_pipeline/thread.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/examples/ip_pipeline/thread.c b/examples/ip_pipeline/thread.c index b00ea0653..a6d665984 100644 --- a/examples/ip_pipeline/thread.c +++ b/examples/ip_pipeline/thread.c @@ -813,5 +813,5 @@ pipeline_port_in_stats_read(const char *pipeline_name, /* Read response */ status = rsp->status; - if (status) + if (status == 0) memcpy(stats, &rsp->port_in_stats_read.stats, sizeof(*stats)); @@ -961,5 +961,5 @@ pipeline_port_out_stats_read(const char *pipeline_name, /* Read response */ status = rsp->status; - if (status) + if (status == 0) memcpy(stats, &rsp->port_out_stats_read.stats, sizeof(*stats)); @@ -1017,5 +1017,5 @@ pipeline_table_stats_read(const char *pipeline_name, /* Read response */ status = rsp->status; - if (status) + if (status == 0) memcpy(stats, &rsp->table_stats_read.stats, sizeof(*stats)); -- 2.19.0 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2018-11-23 10:22:54.801790701 +0000 +++ 0021-examples-ip_pipeline-fix-port-and-table-stats-read.patch 2018-11-23 10:22:54.000000000 +0000 @@ -1,12 +1,13 @@ -From c4160d303026ed11707e4a3a7a2e12b9be863d5a Mon Sep 17 00:00:00 2001 +From 9f42c99f9fa195ce4afa24072d6453a53ee99802 Mon Sep 17 00:00:00 2001 From: Cristian Dumitrescu Date: Tue, 30 Oct 2018 18:44:09 +0000 Subject: [PATCH] examples/ip_pipeline: fix port and table stats read +[ upstream commit c4160d303026ed11707e4a3a7a2e12b9be863d5a ] + Fix the pipeline port and table stats read operation. Fixes: 50e73d051806 ("examples/ip_pipeline: add stats read commands") -Cc: stable@dpdk.org Signed-off-by: Cristian Dumitrescu Signed-off-by: Jasvinder Singh @@ -16,24 +17,24 @@ 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/examples/ip_pipeline/thread.c b/examples/ip_pipeline/thread.c -index e1b6fb684..272fbbeed 100644 +index b00ea0653..a6d665984 100644 --- a/examples/ip_pipeline/thread.c +++ b/examples/ip_pipeline/thread.c -@@ -821,5 +821,5 @@ pipeline_port_in_stats_read(const char *pipeline_name, +@@ -813,5 +813,5 @@ pipeline_port_in_stats_read(const char *pipeline_name, /* Read response */ status = rsp->status; - if (status) + if (status == 0) memcpy(stats, &rsp->port_in_stats_read.stats, sizeof(*stats)); -@@ -969,5 +969,5 @@ pipeline_port_out_stats_read(const char *pipeline_name, +@@ -961,5 +961,5 @@ pipeline_port_out_stats_read(const char *pipeline_name, /* Read response */ status = rsp->status; - if (status) + if (status == 0) memcpy(stats, &rsp->port_out_stats_read.stats, sizeof(*stats)); -@@ -1025,5 +1025,5 @@ pipeline_table_stats_read(const char *pipeline_name, +@@ -1017,5 +1017,5 @@ pipeline_table_stats_read(const char *pipeline_name, /* Read response */ status = rsp->status; - if (status)