Hi All,
> From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com]
> Sent: Friday, November 25, 2016 12:24 AM
> To: Thomas Monjalon
> Cc: dev at dpdk.org; Richardson, Bruce ; Van
> Haaren, Harry
> ; hemant.agrawal at nxp.com; Eads, Gage
>
> Subject: Re: [dpdk-d
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jerin Jacob
> Sent: Tuesday, October 25, 2016 6:49 PM
>
> Hi Community,
>
> So far, I have received constructive feedback from Intel, NXP and Linaro
> folks.
> Let me know, if anyone else interested in
> From: Vincent Jardin [mailto:vincent.jardin at 6wind.com]
> Sent: Wednesday, October 26, 2016 7:37 PM
> Le 26 octobre 2016 2:11:26 PM "Van Haaren, Harry"
> a ?crit :
>
> >> -Original Message-
> >> From: dev [mailto:dev-bounces at dpdk.org] O
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jerin Jacob
>
> So far, I have received constructive feedback from Intel, NXP and Linaro
> folks.
> Let me know, if anyone else interested in contributing to the definition of
> eventdev?
>
> If there are no
blob/master/src/collectd.conf.pod#plugin-dpdkstat
Keep in mind a for a simple setup the current defaults will just work, so
changing the default seems a bad idea to me.
Regards, -Harry
> On Thu, Oct 13, 2016 at 9:07 AM, Van Haaren, Harry intel.com> wrote:
>Hi John,
>> From: dev [
Hi John,
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of John Ousterhout
> Subject: Re: [dpdk-dev] [PATCH] eal: avoid unnecessary conflicts over
> rte_config file
> For example, it took me several hours
> to figure out why the problem was occurring and then to hunt down the
>
Hi Zhiyong,
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zhiyong Yang
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH v2] net/vhost: add pmd xstats
>
> +struct vhost_xstats {
> + uint64_t stat[16];
> +};
Perhaps we could create an enum to access the stat array?
enum
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maryam Tahhan
> Sent: Wednesday, September 7, 2016 11:38 AM
> To: dev at dpdk.org
> Cc: Tahhan, Maryam
> Subject: [dpdk-dev] [PATCH v2 1/1] dpdk_procinfo: check for primary process
>
> Add a check to see if the primary process is running
Hi Rasesh,
Some comments about xstats strings below, please refer to the docs here:
http://dpdk.org/doc/guides/prog_guide/poll_mode_drv.html#extended-statistics-api
Due to my familiarity with ixgbe, I will use it as an example: other PMDs should
be identical.
I understand these may seem trivial
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Rasesh Mody
> Subject: [dpdk-dev] [PATCH v3 3/4] bnx2x: Enhance stats get
Hi Rasesh,
> + snprintf(xstats[num].name, sizeof(xstats[num].name), "brb_drops");
I don't understand what a "brb" drop is.
> + snprintf(xstats[num].name,
> From: David Harton (dharton) [mailto:dharton at cisco.com]
> Subject: RE: [dpdk-dev] [PATCH] doc: announce xstats api change for 16.07
> > This should not be the normal process.
> > It is important to understand what should be the changes to decide of
> > announcing or not a deprecation.
> > In
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Subject: Re: [dpdk-dev] [PATCH] doc: announce xstats api change for 16.07
> > The issue we are going to fix is that currently PMDs copy strings when
> > retrieving
> statistics, which causes unnecessary overhead. The implementation
+ David Harton,
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Subject: Re: [dpdk-dev] [PATCH] doc: announce xstats api change for 16.07
> 2016-04-05 18:58, Harry van Haaren:
> > +* ABI change is planned for the xstats API
> Have you already submitted a RFC patch to let us have
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz
> Subject: [dpdk-dev] [PATCH] xstats: fix behavior when a null array is provided
>
> Coverity reports an issue in ethdev:
>
> *** CID 124562: Null pointer dereferences (FORWARD_NULL)
> /lib/librte_ether/rte_ethdev.c:
> From: David Marchand [mailto:david.marchand at 6wind.com]
> >> The issue is that if a secondary process is initialized, it holds a read
> >> lock on /var/run/.rte_config and this prevents a primary from starting.
> >
> > The new function is advertised as a monitoring feature.
> > But it seems
Hi David,
> From: David Marchand [mailto:david.marchand at 6wind.com]
> Subject: Re: [PATCH v6] eal: add function to check if primary proc alive
> When I look at this new api, I am under the impression that you are
> supposed to check for primary liveliness once dpdk init has finished
> (from
> From: Andralojc, WojciechX
> Sent: Monday, March 7, 2016 4:54 PM
> To: dev at dpdk.org
> Cc: Van Haaren, Harry ; Andralojc, WojciechX
> ; Kantecki, Tomasz intel.com>; Cornu,
> Marcel D
> Subject: [PATCH v3] examples/skeleton-cat: PQoS CAT and CDP, example of
> li
Hi All,
From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Wojciech Andralojc
> Subject: [dpdk-dev] [PATCH v2] examples/skeleton-cat: PQoS CAT and CDP,
> example of libpqos
> usage
Some minor comments/suggested-fixes in-line.
I've snipped away lots of code, below only that relevant to my
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Igor Ryzhov
> Sent: Friday, February 26, 2016 3:01 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH] virtio: don't count broadcast packets in
> multicast packets
> counter
>
> Signed-off-by: Igor Ryzhov
Acked-by: Harry van Haaren
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Igor Ryzhov
> I found some problem with virtio xstats counters.
>
> Example:
>
> rx_good_packets: 3
> rx_good_bytes: 180
> rx_errors: 0
>
> rx_q0_good_packets: 3
> rx_q0_good_bytes: 180
> rx_q0_errors: 0
>
> rx_q0_multicast_packets: 3
>
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Marc
> In file included from
> /home/marc/dpdk/lib/librte_eal/linuxapp/eal/eal_pci.c:42:
> /home/marc/dpdk/x86_64-native-linuxapp-clang/include/rte_memcpy.h:870:2:
> error: implicit declaration of function '_mm_alignr_epi8' is invalid in
>
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> 2016-02-08 15:42, Harry van Haaren:
> > This patch documents that the statistics of fm10k based NICs must be
> > read regularly in order to avoid an undetected 32 bit integer-overflow.
> >
> > Signed-off-by: Harry van Haaren
> > ---
+John,
> From: David Harton
> Subject: RE: [dpdk-dev] Future Direction for rte_eth_stats_get()
>
> Hi folks,
>
> I didn't see any follow up to this response.
I think you may have missed one:
http://dpdk.org/ml/archives/dev/2016-January/032211.html
>
> However, if modifying that API is not
> From: David Harton
>
> enum rte_eth_stat_e {
> /* accurate desc #1 */
> RTE_ETH_STAT_1,
> /* accurate desc #2 */
> RTE_ETH_STAT_2,
> ...
> }
> struct rte_eth_id_stat {
> rte_eth_stat_e id;
> uin64_t value;
> }
>
> int rte_eth_id_stats_num(uint8_t port_id, uint32_t
> From: Richardson, Bruce
> > Agreed, however hiding it totally removes the flexibility of waiting for a
> > primary
> > that is starting with --file-prefix (aka: in a non-default location).
> > Imposing
> > a limit on only monitoring primary procs in the default location seems
> > wrong.
>
>
> From: Qiu, Michael
> > Whatever work the secondary was performing (in its own address space)
> > won't be directly changed by the primary being killed, because the
> > shared config and hugepages stay (EAL "cleans up" when the primary
> > is re-launched, not on quit).
>
> OK, when primary quit
> From: Qiu, Michael
> On 1/25/2016 7:51 PM, Van Haaren, Harry wrote:
> > Not really, the secondary process will need some CPU,
> > however it can sleep so it doesn't have to use 100% of it.
> > It shouldn't be run on a core that is used by the primary
> >
> From: Qiu, Michael
> Subject: Re: [dpdk-dev] [PATCH] eal: add function to check if primary proc
> alive
>
> So secondary will waste a whole lcore to do such polling?
Not really, the secondary process will need some CPU,
however it can sleep so it doesn't have to use 100% of it.
It shouldn't be
> From: Richardson, Bruce
> The details of what the config file is should largely be hidden from the user
> IMHO.
Agreed, however hiding it totally removes the flexibility of waiting for a
primary
that is starting with --file-prefix (aka: in a non-default location). Imposing
a limit on only
+Jay, (@all, please keep everybody in the CCs :)
> From: David Harton (dharton) [mailto:dharton at cisco.com]
> To: Van Haaren, Harry ; Thomas Monjalon
> > > xstats are driver agnostic and have a well-defined naming scheme.
> >
> > Indeed, described here:
> > http
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Subject: Re: [dpdk-dev] Future Direction for rte_eth_stats_get()
> + Harry
Hey all,
> xstats are driver agnostic and have a well-defined naming scheme.
Indeed, described here:
> From: Qiu, Michael
> Sent: Thursday, January 21, 2016 6:14 AM
> To: Van Haaren, Harry ; david.marchand at
> 6wind.com
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] eal: add function to check if primary proc
> alive
>
> As we could start up many primari
Hi Kyle,
> From: Kyle Larose [mailto:eomereadig at gmail.com]
> On Wed, Jan 20, 2016 at 9:28 AM, Harry van Haaren
> wrote:
> > This patch exposes link duplex, speed, and status via the
> > existing xstats API.
>
> I'm slightly confused by this. Why are we exposing operational
> properties of
> From: Robin Jarry [mailto:robin.jarry at 6wind.com]
> Subject: [PATCH] ixgbe: restore imissed stat counter
>
> This counter was left unmodified. Restore it in ixgbe_dev_stats_get.
>
> The ierrors counter still includes imissed for ixgbe. This behaviour is
> not consistent amongst all drivers.
> Subject: [dpdk-dev] [PATCH] doc: cleanup doc index files
>
> Remove **Contents** and |Today| from the rst doc index files since
> these are already added automatically to PDF files and are of
> little value to the Html files where the Contents is shown in a
> sidebar.
>
> Signed-off-by: John
> Subject: [dpdk-dev] [PATCH] doc: fix minor rst doc issues
>
> Fix minor rst doc issues in the contributing/design.rst doc to correct
> rendering of notes and code blocks.
>
> Signed-off-by: John McNamara
> ---
> doc/guides/contributing/design.rst | 37 +++--
>
> Subject: [dpdk-dev] [PATCH] doc: fix spellings in docs
>
> Fix various spellings in rst docs.
>
> Signed-off-by: John McNamara
> ---
> doc/guides/contributing/coding_style.rst | 2 +-
> doc/guides/contributing/versioning.rst | 2 +-
>
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> It would be better with a colon:
> Fixes: line
Ok, will send a v2 with that fix.
> > git send-email -N --to
> Why -N?
-N sets the Number of patches to send, when not sending .patch files but
sending the latest commits of
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of John McNamara
> Add a document to explain the DPDK patch submission and review process.
>
> Signed-off-by: John McNamara
> +* New external functions should be added to the local ``version.map`` file.
These should be listed
> From: Stephen Hemminger [mailto:shemming at brocade.com]
> Harry van Haaren wrote:
>
> > + stats->tor -= (stats->tpr - old_tpr) * 4;
>
> Why not use ETHER_CRC_LEN rather than magic # 4?
That would work too. Will respin tomorrow to fix and to give time
for other comments.
Thanks, -Harry
Hi Stephen,
> From: Stephen Hemminger [mailto:shemming at brocade.com]
> Subject: Re: [PATCH] ixgbe: fix crc-strip enable changing rx bytes
>
> Harry van Haaren wrote:
> >
> > This patch reads the CRC strip register, and when enabled adds 4 bytes
> > to the total bytes recieved counter for
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Remy Horton
> Subject: [dpdk-dev] [PATCH v4 2/3] docs: add keep alive sample app guide &
> release notes
>
> Signed-off-by: Maryam Tahhan
> Signed-off-by: John J Browne
> Signed-off-by: Remy Horton
Acked-by: Harry van Haaren
> From: Martin Weiser [mailto:martin.weiser at allegro-packets.com]
> Subject: Re: [dpdk-dev] ixgbe: ierrors counter spuriously increasing in DPDK
> 2.1
> The
> rx-error which showed up immediately after starting the interface is
> gone since this was probably caused by mac_remote_errors.
> From: Sun, Xutao
> Sent: Wednesday, November 4, 2015 9:21 AM
> To: dev at dpdk.org
> Cc: Zhang, Helin; Van Haaren, Harry; Sun, Xutao
> Subject: [PATCH v6] i40e: Fix the statistics issue of i40e
>
> The old statistics on i40e only counted the packets on ports.
> So the d
Hi Keith,
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wiles, Keith
> Hmm, I just noticed I did not include the FCS bytes. Does the NIC include FCS
> bytes in the
> counters? Need to verify that one and if not then it becomes a bit more
> complex.
The Intel NICs count packet
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Martin Weiser
> Sent: Wednesday, October 21, 2015 9:38 AM
> To: dev at dpdk.org
> Subject: [dpdk-dev] ixgbe: ierrors counter spuriously increasing in DPDK 2.1
> with DPDK 2.1 we are seeing the ierrors counter increasing for 82599ES
> ports
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon
> Subject: Re: [dpdk-dev] [PATCH 1/2] scripts: add checkpatch wrapper
>
> 2015-10-30 16:16, Bruce Richardson:
> > Another request, can you perhaps also fix the script for situations where
> > checkpatch.pl is not in the
> Subject: [PATCH v5 00/10] Port XStats
Series-acked-by: Maryam Tahhan
Same code as previous v4, same Ack. Will include in commit message in future.
-Harry
Hi,
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Monday, November 2, 2015 7:59 AM
> > + /* if xstats_get() is implemented by the PMD, the Q stats are done
> > */
> > + if (dev->dev_ops->xstats_get != NULL)
> > + return count + xcount;
> > +
> >
Hi All,
(Please keep everyone on To/CC - a few people were dropped)
Note: During my writing a reply to Kyle, Thomas has also replied to Kyle.
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Kyle Larose
> Sent: Thursday, October 29, 2015 12:55 PM
> To:
Hi Xutao,
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Xutao Sun
> Sent: Thursday, October 29, 2015 8:02 AM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH v3 0/2] i40e: Fix the statistics issue of i40e
>
> The old statistics on i40e only counted the packets on ports.
> So the
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Remy Horton
> Sent: Wednesday, October 28, 2015 8:52 AM
> To: dev at dpdk.org
> Cc: Browne, John J
> Subject: [dpdk-dev] [PATCH v3 2/3] docs: add keep alive sample app guide
>
> Signed-off-by: Maryam Tahhan
> Signed-off-by: John J Browne
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Van Haaren, Harry
> I'll look into it [mspdc error register stats], and post back later.
Hi guys,
Attempts to generate a 32 byte packet and make mspdc increment have not been
successful - however from the datasheet and a theor
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of John McNamara
> Sent: Wednesday, October 21, 2015 10:39 AM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH v2] doc: change sphinx theme to the read the docs
> theme
>
> Change the Sphinx default theme from "alabaster" to the
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of John McNamara
> Sent: Monday, October 12, 2015 1:55 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH] doc: fix pdf build warning
>
> Fix a pdf doc build warning where a link wasn't recognised:
>
>
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of John McNamara
> Sent: Tuesday, October 20, 2015 12:32 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH] doc: remove dejavu font from doc build
>
> Remove requirement for occasionally hard to find/install
> DejaVuSansMono font. It
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of John McNamara
> Sent: Thursday, October 22, 2015 11:36 AM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH] tools: exit setup script without prompt
>
> Exit tools/setup.sh script without prompting "Press enter to continue".
>
> The
Hi Wenzhuo,
> From: Lu, Wenzhuo
> > Remove receive jabber count from ierrors count as the registers overlap,
> > previously causing some packets to be counted twice.
>
> Would you like to let us know which register RJC overlaps into? Thanks.
It seems that the RJC register overlaps with CRC
> From: Stephen Hemminger [mailto:stephen at networkplumber.org]
> > +/* [rt]x_qX_ is prepended to the name string here */ static const
> > +struct rte_virtio_xstats_name_off rte_virtio_q_stat_strings[] = {
> > + {"packets", offsetof(struct virtqueue, packets)},
> > + {"bytes", offsetof(struct
> -Original Message-
> From: Van Haaren, Harry
> Sent: Tuesday, September 29, 2015 3:30 PM
> To: Van Haaren, Harry
> Subject: [Patch 00/11] Port XStats
Self-NACK:
Sent outdated version of patches, new version tomorrow.
Sorry for the noise, -Harry
> From: Kyle Larose [mailto:eomereadig at gmail.com]
> Can you disable the MAC padding on the NIC through a backdoor register
> access?
Good idea - yes it looks like that's possible, TXPADEN register.
I'll look into it, and post back later. Thanks, -Harry
Hi Serguei,
> From: Serguei Bezverkhi
> I would appreciate if somebody could share an example of getting access to
> ixgbe_hw structure for already initialized ixgbe nic. I tried to find any
> references but could not find any API returning pointer for this structure.
The ixgbe_hw struct is part
> From: Igor Ryzhov [mailto:iryzhov at arccn.ru]
> Thank you, I'll wait for result of mspdc testing.
Hi Igor,
Regarding your original question:
The datasheet says that a packet with total size < 32 bytes is
discarded by MAC layer and counted in the mspdc register.
If possible, I would like to
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Igor Ryzhov
> Hello everyone.
Hi Igor,
> Investigating IXGBE driver I found an mspdc counter (MAC Short Packet
> Discard). And I am wondering why this counter is not used in the calculation
> of total RX errors (ierrors field in
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jesper Wramberg
> Hi all,
Hi Jesper,
> I got a Dell PowerEdge R630 with an X520 NIC. I want to set up two VFs that
> use the first port on the NIC. I've done the following:
Your process is correct - nothing
65 matches
Mail list logo
