From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-vk0-f51.google.com (mail-vk0-f51.google.com [209.85.213.51]) by dpdk.org (Postfix) with ESMTP id 0D9723787 for ; Thu, 8 Oct 2015 17:42:25 +0200 (CEST) Received: by vkfp126 with SMTP id p126so34456716vkf.3 for ; Thu, 08 Oct 2015 08:42:24 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=4amsALhmgAvV4CGDyWS+0PipzWBup+Zcz9xK1Gt+pPI=; b=ikvF1Nkraah96vHlwGoHTKiPd5WdSaZ1rGdd5Ltf610slMRjyoynrSFJf5aGihzkUx lKutyb94QZsSCZx6XLvtnMa5cZ8fmpdo8srI2L6TVXGhIjsgXEDgcDdcLxEPDcWiuSFc nV6VOeryWrWEVaboYyzJCsCQEYgaKEoJ3IsrR9buOlpZ3gN8pw228tn39xdpn82PR8Te U9oGnFePe/bqhtXu3su8ns+bOYNNJLdqsiHzR5VRbdfFMGU6N+XQZhqtdJ79gsXYeUNL M1uIirkM5GtRbAQLfxMlvUd93Lqz2CEQNSeV+rrj3aVUcc4+DRY6iy0myfInXKUJz3Y9 n9qQ== X-Gm-Message-State: ALoCoQntvlJCHehZ+Qc56bQngXVsS4tjfh9Zu5Me8EBjOvoiVkgDaZ9OHDdfhorEvH+g+tIOiY0M MIME-Version: 1.0 X-Received: by 10.31.151.84 with SMTP id z81mr5700624vkd.14.1444318944567; Thu, 08 Oct 2015 08:42:24 -0700 (PDT) Received: by 10.103.69.17 with HTTP; Thu, 8 Oct 2015 08:42:24 -0700 (PDT) In-Reply-To: <1A27633A6DA49C4A92FCD5D4312DBF536A5B392B@IRSMSX109.ger.corp.intel.com> References: <1443790078-15834-1-git-send-email-maryam.tahhan@intel.com> <1A27633A6DA49C4A92FCD5D4312DBF536A5B392B@IRSMSX109.ger.corp.intel.com> Date: Thu, 8 Oct 2015 10:42:24 -0500 Message-ID: From: Jay Rolette To: "Tahhan, Maryam" Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: DPDK Subject: Re: [dpdk-dev] [PATCH] ethdev: distinguish between drop and error stats 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, 08 Oct 2015 15:42:25 -0000 On Thu, Oct 8, 2015 at 9:37 AM, Tahhan, Maryam wrote: > Hi Jay > > Yeah, I will do this to make the distinction clear for the counters I=E2= =80=99m > modifying in my patch. But please note that these aren=E2=80=99t counters= that are > defined in the NIC datasheets, these are the high level DPDK general > counters for ethdevs. Their descriptions exist in the code and the > generated DPDK docs. > Yeah, I know. The problem is that the generated docs don't make it clear what the counters really are and you end up having to trace through the code and, frequently, go look at NIC datasheets. I just went through this a few months ago when I was integrating DPDK port stats into our stat tracking system. I appreciate you making the extra effort to make this clearer for app developers :-) Jay > *From:* Jay Rolette [mailto:rolette@infiniteio.com] > *Sent:* Friday, October 2, 2015 2:25 PM > *To:* Tahhan, Maryam > *Cc:* DPDK > *Subject:* Re: [dpdk-dev] [PATCH] ethdev: distinguish between drop and > error stats > > > > Can you improve the comments on these counters? If you didn't happen to > follow this thread, there's no way to reasonably figure out what the > difference is from looking at the code without chasing it all the way dow= n > and cross-referencing the NIC datasheet. > > > > Thanks, > > Jay > > > > On Fri, Oct 2, 2015 at 7:47 AM, Maryam Tahhan > wrote: > > Make a distniction between dropped packets and error statistics to allow > a higher level fault management entity to interact with DPDK and take > appropriate measures when errors are detected. It will also provide > valuable information for any applications that collects/extracts DPDK > stats, such applications include Open vSwitch. > After this patch the distinction is: > ierrors =3D Total number of packets dropped by hardware (malformed > packets, ...) Where the # of drops can ONLY be <=3D the packets received > (without overlap between registers). > Rx_pkt_errors =3D Total number of erroneous received packets. Where the # > of errors can be >=3D the packets received (without overlap between > registers), this is because there may be multiple errors associated with > a packet. > > Signed-off-by: Maryam Tahhan > --- > lib/librte_ether/rte_ethdev.h | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.= h > index 8a8c82b..53dd55d 100644 > --- a/lib/librte_ether/rte_ethdev.h > +++ b/lib/librte_ether/rte_ethdev.h > @@ -200,8 +200,9 @@ struct rte_eth_stats { > /**< Deprecated; Total of RX packets with CRC error. */ > uint64_t ibadlen; > /**< Deprecated; Total of RX packets with bad length. */ > - uint64_t ierrors; /**< Total number of erroneous received > packets. */ > + uint64_t ierrors; /**< Total number of dropped received packets= . > */ > uint64_t oerrors; /**< Total number of failed transmitted > packets. */ > + uint64_t ipkterrors; /**< Total number of erroneous received > packets. */ > uint64_t imcasts; > /**< Deprecated; Total number of multicast received packets. */ > uint64_t rx_nombuf; /**< Total number of RX mbuf allocation > failures. */ > -- > 2.4.3 > > >