From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dmarchan@redhat.com>
Received: from mail-vk1-f195.google.com (mail-vk1-f195.google.com
 [209.85.221.195]) by dpdk.org (Postfix) with ESMTP id 804512C17
 for <dev@dpdk.org>; Mon,  4 Mar 2019 13:35:48 +0100 (CET)
Received: by mail-vk1-f195.google.com with SMTP id v187so1077815vkf.12
 for <dev@dpdk.org>; Mon, 04 Mar 2019 04:35:48 -0800 (PST)
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=EfpS+gPZX+8k+6Jym0a0C0yPAtfRREWvgK7sDRbRxlw=;
 b=G7zJ30zFUuvz8IekCR9xhZadnAxJanum9YpOaggdLp7Qgm9H32OUG1YB43zKo0F973
 7B6P6gb7XQAF+lfpenvA/fuKObeGZBtn/8r3Ea4BbPhhe2M/+lif1191FkswrKXNV1sp
 JQ8epn708zAViBdjdvgAPY/IiiWYmjlBurMhBHMDnDZ3BrozeEVLqUqJJshtuFoxo7Kc
 XjKDk0eOc3vLukqgXoEDQznAevJW49FHakpcAKbNnQOCcD3MhkVk53d1VcWVigxkmHcT
 Kz1/ZG0fWMr8NTdrAXDyrYFaa8hb0OFXKCOuZXuzcqlJoZBTnoIprg72xSg6RuZQwpQT
 uKJg==
X-Gm-Message-State: APjAAAWOwPFgugEHzIW8udZU47t4W10s6CqDJyGayoT+qphYQZodH5cR
 khZaR2Ql6K+B3LHm/3TiTcpntULR7FGeEJLyW50fkw==
X-Google-Smtp-Source: APXvYqygDWB5uSojRfYRYZQ7VBRM77RAL22TEEvmj92/RZLevqeAKSgD75GxddvhIRRruYUJrEWcmKISmHcUZ1qgoH8=
X-Received: by 2002:a1f:35f:: with SMTP id 92mr9798235vkd.52.1551702947851;
 Mon, 04 Mar 2019 04:35:47 -0800 (PST)
MIME-Version: 1.0
References: <1551185824-5501-2-git-send-email-cernay@netcope.com>
 <1551451054-111249-1-git-send-email-cernay@netcope.com>
 <CAJFAV8yD8dzps6vu7Twp54SvPF9wqd9vgWR9FAzJBhTLdAhWCw@mail.gmail.com>
 <CAGS_BL=z7dHo0fTgaXC0w8rppF9k28kYmSQLDuAMVMK99nZw1A@mail.gmail.com>
In-Reply-To: <CAGS_BL=z7dHo0fTgaXC0w8rppF9k28kYmSQLDuAMVMK99nZw1A@mail.gmail.com>
From: David Marchand <david.marchand@redhat.com>
Date: Mon, 4 Mar 2019 13:35:36 +0100
Message-ID: <CAJFAV8xrMgTj1BrJO9QPQ3xwJUcsR91q0vPktJNpJVkV1=ULHg@mail.gmail.com>
To: =?UTF-8?Q?Rastislav_=C4=8Cernay?= <cernay@netcope.com>
Cc: dev <dev@dpdk.org>, "Yigit, Ferruh" <ferruh.yigit@intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Content-Filtered-By: Mailman/MimeDel 2.1.15
Subject: Re: [dpdk-dev] [PATCH v3] net/nfb: new netcope driver
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Mon, 04 Mar 2019 12:35:48 -0000

On Mon, Mar 4, 2019 at 1:30 PM Rastislav =C4=8Cernay <cernay@netcope.com> w=
rote:

> >>> What is the point of adding when i >=3D RTE_ETHDEV_QUEUE_STAT_CNTRS ?
>
> struct rte_eth_stats {
> ...
> uint64_t q_opackets[RTE_ETHDEV_QUEUE_STAT_CNTRS]
> ...
> }
>
> As there can be more queues (nb_tx) then RTE_ETHDEV_QUEUE_STAT_CNTRS (16)
> and struct rte_eth_stats eth_stats is allocated statically,
> there is need to check so it does not write garbage somewhere.
>

How about looping on min(nb_tx, RTE_ETHDEV_QUEUE_STAT_CNTRS) ?



> >>> Besides, q_errors[] is for reception errors.
> I will fix that, meanwhile could q_errors[] be renamed to q_ierrors[]?
> Also could there be a way to publish output errors per queue, for example
> q_oerrors[]?
>

At the moment, no, this would be a api breakage, and adding oerrors would
be a abi breakage.
This can be discussed yes, I just sent a series about q_errors[].

You can still export this via xstats.


--=20
David Marchand