diff mbox series

[12/12] net/tap: fix incorrect rxq errors stat

Message ID	1551698315-2611-13-git-send-email-david.marchand@redhat.com (mailing list archive)
State	Accepted, archived
Delegated to:	Ferruh Yigit
Headers	From: David Marchand <david.marchand@redhat.com> To: dev@dpdk.org Cc: stable@dpdk.org, Keith Wiles <keith.wiles@intel.com> Date: Mon, 4 Mar 2019 12:18:35 +0100 Message-Id: <1551698315-2611-13-git-send-email-david.marchand@redhat.com> In-Reply-To: <1551698315-2611-1-git-send-email-david.marchand@redhat.com> References: <1551698315-2611-1-git-send-email-david.marchand@redhat.com> Subject: [dpdk-dev] [PATCH 12/12] net/tap: fix incorrect rxq errors stat Precedence: list Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org>
Series	rxq q_errors[] statistics fixes \| [00/12] rxq q_errors[] statistics fixes [01/12] net/af_packet: fix incorrect rxq errors stat [02/12] net/avp: fix incorrect rxq errors stat [03/12] net/bnxt: fix incorrect rxq errors stat [04/12] net/cxgbe: fix incorrect rxq errors stat [05/12] net/kni: fix incorrect rxq errors stat [06/12] net/mlx4: fix incorrect rxq errors stat [07/12] net/mlx5: fix incorrect rxq errors stat [08/12] net/null: fix incorrect rxq errors stat [09/12] net/pcap: fix incorrect rxq errors stat [10/12] net/ring: fix incorrect rxq errors stat [11/12] net/szedata2: fix incorrect rxq errors stat [12/12] net/tap: fix incorrect rxq errors stat

Checks

Context	Check	Description
ci/Intel-compilation	success	Compilation OK

Commit Message

David Marchand March 4, 2019, 11:18 a.m. UTC

  Transmit errors must not be reported in q_errors[] which is for
reception.

Fixes: 02f96a0a82d1 ("net/tap: add TUN/TAP device PMD")
Cc: stable@dpdk.org
Cc: Keith Wiles <keith.wiles@intel.com>
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 drivers/net/tap/rte_eth_tap.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Wiles, Keith March 4, 2019, 1:58 p.m. UTC | #1

> On Mar 4, 2019, at 5:18 AM, David Marchand <david.marchand@redhat.com> wrote:
> 
> Transmit errors must not be reported in q_errors[] which is for
> reception.
> 
> Fixes: 02f96a0a82d1 ("net/tap: add TUN/TAP device PMD")
> Cc: stable@dpdk.org
> Cc: Keith Wiles <keith.wiles@intel.com>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
> drivers/net/tap/rte_eth_tap.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/net/tap/rte_eth_tap.c b/drivers/net/tap/rte_eth_tap.c
> index 6f5109f..94c728f 100644
> --- a/drivers/net/tap/rte_eth_tap.c
> +++ b/drivers/net/tap/rte_eth_tap.c
> @@ -968,10 +968,9 @@ struct ipc_queues {
> 
> 	for (i = 0; i < imax; i++) {
> 		tap_stats->q_opackets[i] = pmd->txq[i].stats.opackets;
> -		tap_stats->q_errors[i] = pmd->txq[i].stats.errs;
> 		tap_stats->q_obytes[i] = pmd->txq[i].stats.obytes;
> 		tx_total += tap_stats->q_opackets[i];
> -		tx_err_total += tap_stats->q_errors[i];
> +		tx_err_total += pmd->txq[i].stats.errs;
> 		tx_bytes_total += tap_stats->q_obytes[i];
> 	}
> 
> -- 
> 1.8.3.1
> 
Acked-by: Keith Wiles <keith.wiles@intel.com>

Regards,
Keith

diff mbox series

Patch

diff --git a/drivers/net/tap/rte_eth_tap.c b/drivers/net/tap/rte_eth_tap.c
index 6f5109f..94c728f 100644
--- a/drivers/net/tap/rte_eth_tap.c
+++ b/drivers/net/tap/rte_eth_tap.c
@@ -968,10 +968,9 @@  struct ipc_queues {
 
 	for (i = 0; i < imax; i++) {
 		tap_stats->q_opackets[i] = pmd->txq[i].stats.opackets;
-		tap_stats->q_errors[i] = pmd->txq[i].stats.errs;
 		tap_stats->q_obytes[i] = pmd->txq[i].stats.obytes;
 		tx_total += tap_stats->q_opackets[i];
-		tx_err_total += tap_stats->q_errors[i];
+		tx_err_total += pmd->txq[i].stats.errs;
 		tx_bytes_total += tap_stats->q_obytes[i];
 	}