Re: [dpdk-dev] [PATCH v2 05/32] net/i40e: set TX loopback from PF

2016-12-20 Thread Lu, Wenzhuo
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Wednesday, December 7, 2016 9:05 PM > To: Lu, Wenzhuo; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 05/32] net/i40e: set TX loopback from PF > > On 12/7/2016 3:31 AM, Wenzhuo Lu wrote: > > Support enabling/disabling TX loo

[dpdk-dev] [PATCH] net/ixgbe/base: clear redundant macro define

2016-12-20 Thread Chenghu Yao
In head file "ixgbe_mbx.h", macro define IXGBE_VF_API_NEGOTIATE and IXGBE_VF_GET_QUEUES appears two times with the same name,value, and notes. Version 2.0 VF requests can inherit the two macro defines in previous version 1.0 and 1.1. Otherwise, may be cause confusion. Signed-off-by: Chenghu Yao

Re: [dpdk-dev] [PATCH v2 02/32] net/i40e: add callback to user on VF to PF mbox msg

2016-12-20 Thread Lu, Wenzhuo
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Wednesday, December 7, 2016 8:45 PM > To: Lu, Wenzhuo; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 02/32] net/i40e: add callback to user on VF > to > PF mbox msg > > On 12/7/2016 3:31 AM, Wenzhuo Lu wrote: > > The callba

Re: [dpdk-dev] [PATCH v3 22/25] app/testpmd: add L4 items to flow command

2016-12-20 Thread Pei, Yulong
Hi Adrien, For SCTP, it may need support to set 'tag' since in FDIR (--pkt-filter-mode=perfect) need set it when filter sctp flow. struct sctp_hdr { uint16_t src_port; /**< Source port. */ uint16_t dst_port; /**< Destin port. */ uint32_t tag; /**< Validation tag. *

Re: [dpdk-dev] [PATCH v3 21/25] app/testpmd: add items ipv4/ipv6 to flow command

2016-12-20 Thread Pei, Yulong
Hi adrien, Is it possible to support to set ipv4 TOS, ipv4 PROTO, ipv4 TTL and ipv6 tc, ipv6 next-header, ipv6 hop-limits since previous FDIR for i40e already support it. Best Regards Yulong Pei -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Adrien Mazarguil Se

Re: [dpdk-dev] [PATCH 1/4] eal/common: introduce rte_memset on IA platform

2016-12-20 Thread Yang, Zhiyong
Hi, Konstantin: > -Original Message- > From: Ananyev, Konstantin > Sent: Friday, December 16, 2016 7:48 PM > To: Yang, Zhiyong ; Thomas Monjalon > > Cc: dev@dpdk.org; yuanhan@linux.intel.com; Richardson, Bruce > ; De Lara Guarch, Pablo > > Subject: RE: [dpdk-dev] [PATCH 1/4] eal/comm

Re: [dpdk-dev] [PATCH v2 06/25] app/testpmd: implement basic support for rte_flow

2016-12-20 Thread Adrien Mazarguil
On Tue, Dec 20, 2016 at 01:57:46AM +, Xing, Beilei wrote: > > -Original Message- > > From: Adrien Mazarguil [mailto:adrien.mazarg...@6wind.com] > > Sent: Monday, December 19, 2016 6:20 PM > > To: Xing, Beilei > > Cc: dev@dpdk.org; Pei, Yulong > > Subject: Re: [dpdk-dev] [PATCH v2 06/2

Re: [dpdk-dev] [PATCH v2] doc: fix required tools list layout

2016-12-20 Thread Mcnamara, John
> -Original Message- > From: Baruch Siach [mailto:bar...@tkos.co.il] > Sent: Monday, December 19, 2016 7:29 PM > To: dev@dpdk.org > Cc: Mcnamara, John ; Baruch Siach > > Subject: [PATCH v2] doc: fix required tools list layout > > The Python requirement should appear in the bullet list.

Re: [dpdk-dev] [PATCH v3 10/25] app/testpmd: add flow flush command

2016-12-20 Thread Adrien Mazarguil
Hi Wei, On Tue, Dec 20, 2016 at 07:32:29AM +, Zhao1, Wei wrote: > Hi, Adrien > > > -Original Message- > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Adrien Mazarguil > > Sent: Tuesday, December 20, 2016 1:49 AM > > To: dev@dpdk.org > > Subject: [dpdk-dev] [PATCH v3 10/25] a

Re: [dpdk-dev] [PATCH v3 22/25] app/testpmd: add L4 items to flow command

2016-12-20 Thread Adrien Mazarguil
Hi Yulong, On Tue, Dec 20, 2016 at 09:14:55AM +, Pei, Yulong wrote: > Hi Adrien, > > For SCTP, it may need support to set 'tag' since in FDIR > (--pkt-filter-mode=perfect) need set it when filter sctp flow. > > struct sctp_hdr { > uint16_t src_port; /**< Source port. */ >

Re: [dpdk-dev] [PATCH v3 21/25] app/testpmd: add items ipv4/ipv6 to flow command

2016-12-20 Thread Adrien Mazarguil
Hi Yulong, On Tue, Dec 20, 2016 at 09:21:28AM +, Pei, Yulong wrote: > Hi adrien, > > Is it possible to support to set ipv4 TOS, ipv4 PROTO, ipv4 TTL and ipv6 tc, > ipv6 next-header, ipv6 hop-limits since > previous FDIR for i40e already support it. I suggest we add them later (like for SCT

Re: [dpdk-dev] [PATCH v2] doc: introduce PVP reference benchmark

2016-12-20 Thread Thomas Monjalon
> > >> Signed-off-by: Maxime Coquelin > > > > > > There is one trailing whitespace warning but apart from that: > > > > > > Acked-by: John McNamara > > > > Thanks John, > > > > Do you want me to send a v3, fixing the trailing whitespace & collecting > > the acks? > > No need (unless the tree m

[dpdk-dev] [PATCH] nfp: add tso support

2016-12-20 Thread Alejandro Lucero
This patch implements NFP PMD support for TSO but it also requires a firmware advertising the capability. Signed-off-by: Alejandro Lucero --- doc/guides/nics/features/nfp.ini | 1 + drivers/net/nfp/nfp_net.c| 40 2 files changed, 33 insertions(+)

Re: [dpdk-dev] [PATCH v2] doc: add details of sub-trees and maintainers

2016-12-20 Thread Thomas Monjalon
2016-12-16 17:50, John McNamara: > Add a new section to the Code Contributors Guide to outline > the roles of tree and component maintainers and how they > can be added and removed. > > Signed-off-by: John McNamara Applied, thanks

Re: [dpdk-dev] [PATCH v2] doc: fix mistakes in contribution guide

2016-12-20 Thread Thomas Monjalon
> > Signed-off-by: Yong Wang > > --- > > v2: > > * modify some redundant descriptions. > > > Thanks, > > Acked-by: John McNamara Applied, thanks

Re: [dpdk-dev] [PATCH] nfp: extend speed capabilities advertised

2016-12-20 Thread Ferruh Yigit
On 12/19/2016 6:00 PM, Alejandro Lucero wrote: > I forgot one thing: to update the features file with this new one. > > I will wait for your feedback regarding the discussed problem for > sending another version. I think it is good to go, please send updated version. > <...> > > Sorr

Re: [dpdk-dev] [PATCH v2] doc: fix required tools list layout

2016-12-20 Thread Thomas Monjalon
> > The Python requirement should appear in the bullet list. > > > > Also, indent the x32 note, since it is related to the previous bullet. > > > > Signed-off-by: Baruch Siach > > Acked-by: John McNamara Applied, thanks

Re: [dpdk-dev] [PATCH] nfp: extend speed capabilities advertised

2016-12-20 Thread Alejandro Lucero
On Tue, Dec 20, 2016 at 10:25 AM, Ferruh Yigit wrote: > On 12/19/2016 6:00 PM, Alejandro Lucero wrote: > > I forgot one thing: to update the features file with this new one. > > > > I will wait for your feedback regarding the discussed problem for > > sending another version. > > I think it is go

Re: [dpdk-dev] [PATCH] doc: correct source extract command

2016-12-20 Thread Thomas Monjalon
> > DPDK source archives are .tar.xz or .tar.gz, not .zip. Use .tar.xz in the > > instructions, since that is what the main download page links to. > > > > Also, correct the archive file and directory name capitalization. > > > > Signed-off-by: Baruch Siach > > Acked-by: John McNamara Applied

Re: [dpdk-dev] [PATCH] doc: simplify L3fwd user guide examples

2016-12-20 Thread Mcnamara, John
> -Original Message- > From: De Lara Guarch, Pablo > Sent: Monday, December 19, 2016 4:34 PM > To: Mcnamara, John > Cc: dev@dpdk.org; De Lara Guarch, Pablo > Subject: [PATCH] doc: simplify L3fwd user guide examples > > L3 Forwarding sample app user guides have some inconsistencies betw

Re: [dpdk-dev] [PATCH v2] doc: fix required tools list layout

2016-12-20 Thread Mcnamara, John
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > Sent: Tuesday, December 20, 2016 10:27 AM > To: Baruch Siach > Cc: dev@dpdk.org; Mcnamara, John > Subject: Re: [dpdk-dev] [PATCH v2] doc: fix required tools list layout > > > > The Python requirement should

Re: [dpdk-dev] [PATCH v2 8/8] ethdev: Decouple interrupt handling from PCI device

2016-12-20 Thread Jan Blunck
On Tue, Nov 22, 2016 at 1:57 PM, Shreyansh Jain wrote: > On Monday 21 November 2016 10:25 PM, Jan Blunck wrote: >> >> The struct rte_intr_handle is an abstraction layer for different types of >> interrupt mechanisms. It is embedded in the low-level device (e.g. PCI). >> On allocation of a struct r

Re: [dpdk-dev] [PATCH v3 25/25] doc: describe testpmd flow command

2016-12-20 Thread Adrien Mazarguil
On Mon, Dec 19, 2016 at 08:44:07PM +, Mcnamara, John wrote: > > > > -Original Message- > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Adrien Mazarguil > > Sent: Monday, December 19, 2016 5:49 PM > > To: dev@dpdk.org > > Subject: [dpdk-dev] [PATCH v3 25/25] doc: describe test

Re: [dpdk-dev] [PATCH] nfp: extend speed capabilities advertised

2016-12-20 Thread Alejandro Lucero
On Tue, Dec 20, 2016 at 10:29 AM, Alejandro Lucero < alejandro.luc...@netronome.com> wrote: > > > On Tue, Dec 20, 2016 at 10:25 AM, Ferruh Yigit > wrote: > >> On 12/19/2016 6:00 PM, Alejandro Lucero wrote: >> > I forgot one thing: to update the features file with this new one. >> > >> > I will wa

[dpdk-dev] [PATCH v3 0/9] Decouple ethdev from PCI device

2016-12-20 Thread Jan Blunck
This is a repost of the series I sent in November. I've addressed Shreyansh's review comments about the PCI device users I've missed. Jan Blunck (9): eal: define container_of macro ethdev: Helper to convert to struct rte_pci_device drivers: Use ETH_DEV_PCI_DEV() helper virtio: Don't fill d

[dpdk-dev] [PATCH v3 1/9] eal: define container_of macro

2016-12-20 Thread Jan Blunck
This macro is based on Jan Viktorin's original patch but also checks the type of the passed pointer against the type of the member. Signed-off-by: Jan Viktorin Signed-off-by: Shreyansh Jain [jblu...@infradead.org: add type checking and __extension__] Signed-off-by: Jan Blunck --- lib/librte_ea

[dpdk-dev] [PATCH v3 2/9] ethdev: Helper to convert to struct rte_pci_device

2016-12-20 Thread Jan Blunck
Signed-off-by: Jan Blunck Acked-by: Shreyansh Jain --- lib/librte_ether/rte_ethdev.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.h index 9678179..3adbb2b 100644 --- a/lib/librte_ether/rte_ethdev.h +++ b/lib/librte_ether/rt

[dpdk-dev] [PATCH v3 3/9] drivers: Use ETH_DEV_PCI_DEV() helper

2016-12-20 Thread Jan Blunck
The drivers should not directly access the rte_eth_dev->pci_dev but use a macro instead. This is a preparation for replacing the pci_dev with a struct rte_device member in the future. Signed-off-by: Jan Blunck --- drivers/net/bnx2x/bnx2x_ethdev.c| 12 +++--- drivers/net/bnxt/bnxt_ethdev.

[dpdk-dev] [PATCH v3 4/9] virtio: Don't fill dev_info->driver_name

2016-12-20 Thread Jan Blunck
This is overwritten in rte_eth_dev_info_get(). Signed-off-by: Jan Blunck Reviewed-by: David Marchand --- drivers/net/virtio/virtio_ethdev.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c index 079fd6c..741688e 1006

[dpdk-dev] [PATCH v3 5/9] virtio: Add vtpci_intr_handle() helper to get rte_intr_handle

2016-12-20 Thread Jan Blunck
This adds a helper to get the rte_intr_handle from the virtio_hw. This is safe to do since the usage of the helper is guarded by RTE_ETH_DEV_INTR_LSC which is only set if we found a PCI device during initialization. Signed-off-by: Jan Blunck --- drivers/net/virtio/virtio_ethdev.c | 12 +++---

[dpdk-dev] [PATCH v3 8/9] ethdev: Decouple interrupt handling from PCI device

2016-12-20 Thread Jan Blunck
The struct rte_intr_handle is an abstraction layer for different types of interrupt mechanisms. It is embedded in the low-level device (e.g. PCI). On allocation of a struct rte_eth_dev a reference to the intr_handle should be stored for devices supporting interrupts. Signed-off-by: Jan Blunck ---

[dpdk-dev] [PATCH v3 7/9] ethdev: Move filling of rte_eth_dev_info->pci_dev to dev_infos_get()

2016-12-20 Thread Jan Blunck
Only the device itself can decide its PCI or not. Signed-off-by: Jan Blunck --- drivers/net/bnx2x/bnx2x_ethdev.c| 1 + drivers/net/bnxt/bnxt_ethdev.c | 2 ++ drivers/net/cxgbe/cxgbe_ethdev.c| 2 ++ drivers/net/e1000/em_ethdev.c | 1 + drivers/net/e1000/igb_ethd

[dpdk-dev] [PATCH v3 6/9] virtio: Don't depend on struct rte_eth_dev's pci_dev

2016-12-20 Thread Jan Blunck
We don't need to depend on rte_eth_dev->pci_dev to differentiate between the virtio_user and the virtio_pci case. Instead we can use the private virtio_hw struct to get that information. Signed-off-by: Jan Blunck --- drivers/net/virtio/virtio_ethdev.c | 21 ++--- 1 file changed,

Re: [dpdk-dev] [PATCH 02/13] rte_device: make driver pointer const

2016-12-20 Thread Jan Blunck
On Mon, Dec 19, 2016 at 10:59 PM, Stephen Hemminger wrote: > The info in rte_device about driver is immutable and > shouldn't change. > --- > lib/librte_eal/common/include/rte_dev.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/librte_eal/common/include/rte_dev.h >

Re: [dpdk-dev] [PATCH v3 0/6] libeventdev API and northbound implementation

2016-12-20 Thread Bruce Richardson
On Sun, Dec 18, 2016 at 07:51:29PM +0530, Jerin Jacob wrote: > As previously discussed in RFC v1 [1], RFC v2 [2], with changes > described in [3] (also pasted below), here is the first non-draft series > for this new API. > > [1] http://dpdk.org/ml/archives/dev/2016-August/045181.html > [2] http:/

[dpdk-dev] [PATCH v3 9/9] ethdev: Decouple struct rte_eth_dev from struct rte_pci_device

2016-12-20 Thread Jan Blunck
Signed-off-by: Jan Blunck --- app/test/virtual_pmd.c | 4 ++-- drivers/net/bonding/rte_eth_bond_args.c | 12 ++-- drivers/net/cxgbe/cxgbe_main.c | 4 ++-- drivers/net/fm10k/fm10k_ethdev.c| 6 +++--- drivers/net/virtio/virtio_user_ethdev.c | 1 - lib/l

Re: [dpdk-dev] [PATCH 05/13] pmd: remove useless reset of dev_info->dev_pci

2016-12-20 Thread Jan Blunck
On Mon, Dec 19, 2016 at 10:59 PM, Stephen Hemminger wrote: > Since rte_eth_dev_info_get does memset() on dev_info before > calling device specific code, the explicit assignment of NULL > in all these virtual drivers has no effect. > > Signed-off-by: Stephen Hemminger > --- > app/test/virtual_pmd

Re: [dpdk-dev] [PATCH 06/13] ethdev: make dev_info generic (not just PCI)

2016-12-20 Thread Jan Blunck
On Mon, Dec 19, 2016 at 10:59 PM, Stephen Hemminger wrote: > The result from rte_eth_dev_info_get should have pointer to > device not PCI device. This breaks ABI but is necessary. > > Signed-off-by: Stephen Hemminger > --- > app/test-pmd/config.c | 32 ++

Re: [dpdk-dev] [PATCH 1/3] ethdev: New API to free consumed buffers in TX ring

2016-12-20 Thread Adrien Mazarguil
Hi Billy, On Fri, Dec 16, 2016 at 07:48:49AM -0500, Billy McFall wrote: > Add a new API to force free consumed buffers on TX ring. API will return > the number of packets freed (0-n) or error code if feature not supported > (-ENOTSUP) or input invalid (-ENODEV). > > Because rte_eth_tx_buffer() ma

Re: [dpdk-dev] [PATCH v3 4/9] virtio: Don't fill dev_info->driver_name

2016-12-20 Thread Ferruh Yigit
On 12/20/2016 11:11 AM, Jan Blunck wrote: > This is overwritten in rte_eth_dev_info_get(). > > Signed-off-by: Jan Blunck > Reviewed-by: David Marchand > --- > drivers/net/virtio/virtio_ethdev.c | 4 > 1 file changed, 4 deletions(-) > > diff --git a/drivers/net/virtio/virtio_ethdev.c > b/

Re: [dpdk-dev] [PATCH 1/3] ethdev: New API to free consumed buffers in TX ring

2016-12-20 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Adrien Mazarguil > Sent: Tuesday, December 20, 2016 11:28 AM > To: Billy McFall > Cc: thomas.monja...@6wind.com; Lu, Wenzhuo ; > dev@dpdk.org; Stephen Hemminger > > Subject: Re: [dpdk-dev] [PATCH 1/3] ethdev: N

Re: [dpdk-dev] [PATCH v3 02/12] eal/bus: introduce bus abstraction

2016-12-20 Thread Hemant Agrawal
On 12/16/2016 6:40 PM, Shreyansh Jain wrote: This patch introduces the rte_bus abstraction for devices and drivers in EAL framework. The model is: - One or more buses are connected to a CPU (or core) - One or more devices are conneted to a Bus - Drivers are running instances which manage one o

Re: [dpdk-dev] [PATCH v3 4/9] virtio: Don't fill dev_info->driver_name

2016-12-20 Thread Jan Blunck
On Tue, Dec 20, 2016 at 1:17 PM, Ferruh Yigit wrote: > On 12/20/2016 11:11 AM, Jan Blunck wrote: >> This is overwritten in rte_eth_dev_info_get(). >> >> Signed-off-by: Jan Blunck >> Reviewed-by: David Marchand >> --- >> drivers/net/virtio/virtio_ethdev.c | 4 >> 1 file changed, 4 deletions

Re: [dpdk-dev] [PATCH v2 8/8] ethdev: Decouple interrupt handling from PCI device

2016-12-20 Thread Shreyansh Jain
On Tuesday 20 December 2016 04:21 PM, Jan Blunck wrote: On Tue, Nov 22, 2016 at 1:57 PM, Shreyansh Jain wrote: On Monday 21 November 2016 10:25 PM, Jan Blunck wrote: The struct rte_intr_handle is an abstraction layer for different types of interrupt mechanisms. It is embedded in the low-level

Re: [dpdk-dev] [RFC v2 00/13] Generalize rte_eth_dev model

2016-12-20 Thread Jan Blunck
On Mon, Dec 19, 2016 at 10:59 PM, Stephen Hemminger wrote: > This set of patches generalize the ethernet device model sufficently > to support virtual, pci, and vmbus devices. > > All this is based on the XP principal of only building what is > necessary and no more than that. The rationale is NOT

[dpdk-dev] [PATCH v2] nfp: extend speed capabilities advertised

2016-12-20 Thread Alejandro Lucero
NFP supports more speeds than just 40 and 100GB, which were what was advertised before. v2: add feature to nfp.ini Signed-off-by: Alejandro Lucero --- doc/guides/nics/features/nfp.ini | 1 + drivers/net/nfp/nfp_net.c| 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a

Re: [dpdk-dev] [PATCH v3 4/9] virtio: Don't fill dev_info->driver_name

2016-12-20 Thread Ferruh Yigit
On 12/20/2016 12:40 PM, Jan Blunck wrote: > On Tue, Dec 20, 2016 at 1:17 PM, Ferruh Yigit wrote: >> On 12/20/2016 11:11 AM, Jan Blunck wrote: >>> This is overwritten in rte_eth_dev_info_get(). >>> >>> Signed-off-by: Jan Blunck >>> Reviewed-by: David Marchand >>> --- >>> drivers/net/virtio/virti

Re: [dpdk-dev] [PATCH 1/3] ethdev: New API to free consumed buffers in TX ring

2016-12-20 Thread Adrien Mazarguil
On Tue, Dec 20, 2016 at 12:17:10PM +, Ananyev, Konstantin wrote: > > > > -Original Message- > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Adrien Mazarguil > > Sent: Tuesday, December 20, 2016 11:28 AM > > To: Billy McFall > > Cc: thomas.monja...@6wind.com; Lu, Wenzhuo ; >

Re: [dpdk-dev] [PATCH 04/13] eal: introduce driver type

2016-12-20 Thread Jan Blunck
On Mon, Dec 19, 2016 at 10:59 PM, Stephen Hemminger wrote: > Since multiple buses and device types need to be supported. > Provide type field in driver. > --- > lib/librte_eal/common/include/rte_dev.h | 15 --- > lib/librte_eal/common/include/rte_pci.h | 1 + > lib/librte_eal/commo

Re: [dpdk-dev] [PATCH 04/13] eal: introduce driver type

2016-12-20 Thread Ferruh Yigit
On 12/19/2016 9:59 PM, Stephen Hemminger wrote: > Since multiple buses and device types need to be supported. > Provide type field in driver. > --- > lib/librte_eal/common/include/rte_dev.h | 15 --- > lib/librte_eal/common/include/rte_pci.h | 1 + > lib/librte_eal/common/include/rt

Re: [dpdk-dev] [PATCH v3 0/6] libeventdev API and northbound implementation

2016-12-20 Thread Jerin Jacob
On Tue, Dec 20, 2016 at 11:13:42AM +, Bruce Richardson wrote: > On Sun, Dec 18, 2016 at 07:51:29PM +0530, Jerin Jacob wrote: > > As previously discussed in RFC v1 [1], RFC v2 [2], with changes > > described in [3] (also pasted below), here is the first non-draft series > > for this new API. > >

Re: [dpdk-dev] [PATCH v3 02/12] eal/bus: introduce bus abstraction

2016-12-20 Thread Jan Blunck
On Fri, Dec 16, 2016 at 2:10 PM, Shreyansh Jain wrote: > This patch introduces the rte_bus abstraction for devices and drivers in > EAL framework. The model is: > - One or more buses are connected to a CPU (or core) > - One or more devices are conneted to a Bus > - Drivers are running instances

[dpdk-dev] [PATCH v6] nfp: report link speed using hardware info

2016-12-20 Thread Alejandro Lucero
Previous reported speed was hardcoded because there was not firmware support for getting this information. This change needs also to support old firmware versions, but instead of the previous hardcoded report, no speed is reported to the user avoiding to give the wrong speed when link is not config

Re: [dpdk-dev] [PATCH v3 0/6] libeventdev API and northbound implementation

2016-12-20 Thread Bruce Richardson
On Tue, Dec 20, 2016 at 06:39:30PM +0530, Jerin Jacob wrote: > On Tue, Dec 20, 2016 at 11:13:42AM +, Bruce Richardson wrote: > > On Sun, Dec 18, 2016 at 07:51:29PM +0530, Jerin Jacob wrote: > > > As previously discussed in RFC v1 [1], RFC v2 [2], with changes > > > described in [3] (also pasted

Re: [dpdk-dev] [PATCH v5 29/29] net/i40e: set/clear VF stats from PF

2016-12-20 Thread Ferruh Yigit
On 12/16/2016 7:02 PM, Ferruh Yigit wrote: > From: Qi Zhang > > This patch add support to get/clear VF statistics > from PF side. > Two APIs are added: > rte_pmd_i40e_get_vf_stats. > rte_pmd_i40e_reset_vf_stats. > > Signed-off-by: Qi Zhang > --- <...> > diff --git a/drivers/net/i40e/rte_pmd_i

Re: [dpdk-dev] [PATCH v13 6/7] vmxnet3: add Tx preparation

2016-12-20 Thread Ferruh Yigit
On 12/13/2016 5:41 PM, Tomasz Kulasek wrote: > From: "Ananyev, Konstantin" > > Signed-off-by: Konstantin Ananyev > --- <...> > > uint16_t > +vmxnet3_prep_pkts(__rte_unused void *tx_queue, struct rte_mbuf **tx_pkts, > + uint16_t nb_pkts) > +{ <...> > + > +#ifdef RTE_LIBRTE_ETHDEV_DEBUG >

Re: [dpdk-dev] [PATCH v5 29/29] net/i40e: set/clear VF stats from PF

2016-12-20 Thread Iremonger, Bernard
Hi Ferruh, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ferruh Yigit > Sent: Tuesday, December 20, 2016 1:25 PM > To: dev@dpdk.org > Cc: Wu, Jingjing ; Zhang, Helin > ; Zhang, Qi Z ; Lu, Wenzhuo > ; Chen, Jing D > Subject: Re: [dpdk-dev] [PATCH v5 29/29] net

Re: [dpdk-dev] [PATCH] doc: fix environment variable typo

2016-12-20 Thread Thomas Monjalon
> > Signed-off-by: Baruch Siach > > --- > > doc/guides/sample_app_ug/ethtool.rst | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > Missing fixline > > Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample > application") > > Otherwise, well spotted.. :) > > Acked-

Re: [dpdk-dev] [PATCH v6] nfp: report link speed using hardware info

2016-12-20 Thread Ferruh Yigit
On 12/20/2016 1:22 PM, Alejandro Lucero wrote: > Previous reported speed was hardcoded because there was not firmware > support for getting this information. This change needs also to support > old firmware versions, but instead of the previous hardcoded report, no > speed is reported to the user a

Re: [dpdk-dev] [PATCH v2] nfp: extend speed capabilities advertised

2016-12-20 Thread Ferruh Yigit
On 12/20/2016 12:52 PM, Alejandro Lucero wrote: > NFP supports more speeds than just 40 and 100GB, which were > what was advertised before. > > v2: add feature to nfp.ini > > Signed-off-by: Alejandro Lucero > --- Applied to dpdk-next-net/master, thanks.

Re: [dpdk-dev] [PATCH v3 02/12] eal/bus: introduce bus abstraction

2016-12-20 Thread Shreyansh Jain
> -Original Message- > From: jblu...@gmail.com [mailto:jblu...@gmail.com] On Behalf Of Jan Blunck > Sent: Tuesday, December 20, 2016 6:47 PM > To: Shreyansh Jain > Cc: dev@dpdk.org; David Marchand ; Thomas Monjalon > ; Ferruh Yigit ; > jianbo@linaro.org > Subject: Re: [dpdk-dev] [PATCH

Re: [dpdk-dev] [PATCH] doc: Fix a typo in testpmd application user guide.

2016-12-20 Thread Thomas Monjalon
> > This patch fixes a trivial typo in testpmd application user guide. > > > > Signed-off-by: Rami Rosen > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH] nfp: add support for new metadata api

2016-12-20 Thread Alejandro Lucero
NFP is a smart programmable NIC and firmware is deployed for specific system needs, like offloading OVS, vRouter, contrack or eBPF into the hardware. This often requires to give metadata to the host within packets delivered. Last NFP firmware implementations support richer metadata api facilitating

Re: [dpdk-dev] [PATCH 1/3] ethdev: New API to free consumed buffers in TX ring

2016-12-20 Thread Billy McFall
Thank you for your responses, see inline. On Tue, Dec 20, 2016 at 7:58 AM, Adrien Mazarguil wrote: > On Tue, Dec 20, 2016 at 12:17:10PM +, Ananyev, Konstantin wrote: >> >> >> > -Original Message- >> > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Adrien Mazarguil >> > Sent: Tue

Re: [dpdk-dev] [PATCH] net/af_packet: initialize link interrupt callback queue

2016-12-20 Thread Ferruh Yigit
On 12/17/2016 6:03 PM, Chas Williams wrote: > This patch initializes the eth_dev->link_intr_cbs queue which is > used when af_packet is passed into rte_eth_ev_callback_register(). Why do you want to register callback to af_packet PMD, it won't be calling them? > > Fixes: 4dc294158cac ("ethdev: s

Re: [dpdk-dev] [PATCH v2] doc: introduce PVP reference benchmark

2016-12-20 Thread Maxime Coquelin
On 12/20/2016 11:03 AM, Thomas Monjalon wrote: Signed-off-by: Maxime Coquelin There is one trailing whitespace warning but apart from that: Acked-by: John McNamara Thanks John, Do you want me to send a v3, fixing the trailing whitespace & collecting the acks? No need (unless the tree

Re: [dpdk-dev] [PATCH v1] examples/ethtool: fix segfault querying non-PCI devices

2016-12-20 Thread Thomas Monjalon
2016-11-30 10:47, Remy Horton: > Doing a device information query on a non-PCI device such as > vhost was resulting in the dereferencing of a NULL pointer > (the absent PCI data), causing a segmentation fault. > > Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample > application

Re: [dpdk-dev] [PATCH v5 00/29] Support VFD and DPDK PF + kernel VF on i40e

2016-12-20 Thread Vincent JARDIN
Le 20/12/2016 à 05:48, Chen, Jing D a écrit : That's a collaboration with another team. we'll follow-up that but not guarantee it will happen. May I ask if my reply make it clear? Still NAC for this patch? Yes still nack, I am not confident with this PF approach since you are breaking Linux PF

Re: [dpdk-dev] [PATCH v5 18/29] app/testpmd: use VFD APIs on i40e

2016-12-20 Thread Vincent JARDIN
Le 19/12/2016 à 12:03, Ferruh Yigit a écrit : And it is always possible to move these into ethdev layer, when multiple PMDs supports same feature. I agree this is something that needs to keep an eye on it, and be sure if an API is generic, move it into eth_dev layer. you are right, you have a g

Re: [dpdk-dev] [PATCH 0/3] buildtools/devtools/usertools

2016-12-20 Thread Ferruh Yigit
On 12/15/2016 9:59 PM, Thomas Monjalon wrote: > The current tools/ and scripts/ directory names are not > self describing. > These patches create devtools/ and usertools/ directories. I think classifying scripts is a good idea. > > Thomas Monjalon (3): > scripts: move to buildtools > scripts

Re: [dpdk-dev] [PATCH v3 2/9] ethdev: Helper to convert to struct rte_pci_device

2016-12-20 Thread Stephen Hemminger
On Tue, 20 Dec 2016 12:11:48 +0100 Jan Blunck wrote: > Signed-off-by: Jan Blunck > Acked-by: Shreyansh Jain > --- > lib/librte_ether/rte_ethdev.h | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.h > index 9678179..3adbb

Re: [dpdk-dev] [PATCH v5 00/29] Support VFD and DPDK PF + kernel VF on i40e

2016-12-20 Thread Ferruh Yigit
On 12/20/2016 3:18 PM, Vincent JARDIN wrote: > Le 20/12/2016 à 05:48, Chen, Jing D a écrit : >> That's a collaboration with another team. we'll follow-up that but not >> guarantee >> it will happen. >> May I ask if my reply make it clear? Still NAC for this patch? > > Yes still nack, I am not con

Re: [dpdk-dev] [PATCH v3 7/9] ethdev: Move filling of rte_eth_dev_info->pci_dev to dev_infos_get()

2016-12-20 Thread Stephen Hemminger
On Tue, 20 Dec 2016 12:11:53 +0100 Jan Blunck wrote: > Only the device itself can decide its PCI or not. > > Signed-off-by: Jan Blunck > --- > drivers/net/bnx2x/bnx2x_ethdev.c| 1 + > drivers/net/bnxt/bnxt_ethdev.c | 2 ++ > drivers/net/cxgbe/cxgbe_ethdev.c| 2 ++ > dr

Re: [dpdk-dev] [PATCH v3 00/31] net/i40e: base code update

2016-12-20 Thread Gregory Etelson
Hello, I have several XL710-Q2 interfaces that fail with `PMD: eth_i40e_dev_init(): Failed to init adminq: -54' Rebase to the latest dpdk-next-net/master did not fix the fault Firmware version is 5.04 0x80002505 0.0.0 lspci output looks the same for working and failed interfaces Full traces with

Re: [dpdk-dev] [PATCH v3 07/25] app/testpmd: add flow command

2016-12-20 Thread Ferruh Yigit
Hi Adrien, On 12/19/2016 5:48 PM, Adrien Mazarguil wrote: > Managing generic flow API functions from command line requires the use of > dynamic tokens for convenience as flow rules are not fixed and cannot be > defined statically. > > This commit adds specific flexible parser code and object for

Re: [dpdk-dev] [PATCH v3 00/31] net/i40e: base code update

2016-12-20 Thread Gregory Etelson
add dmesg output: dmar: DRHD: handling fault status reg 502 dmar: DMAR:[DMA Read] Request device [04:00.0] fault addr 865bcc000 DMAR:[fault reason 02] Present bit in context entry is clear dmar: DRHD: handling fault status reg 602 dmar: DMAR:[DMA Read] Request device [04:00.0] fault addr 865bcc00

Re: [dpdk-dev] [PATCH v3 02/25] doc: add rte_flow prog guide

2016-12-20 Thread Mcnamara, John
Hi Adrien, There seems to be an issue when building the PDF version of the docs due to the width of some of the fields in 2 of the tables. I think this may be a Sphinx/Latex bug since it doesn't happen without the table:: directive. If you replace the following tables with the format below (the t

Re: [dpdk-dev] [PATCH 01/18] net/ixgbe: store SYN filter

2016-12-20 Thread Ferruh Yigit
On 12/2/2016 10:42 AM, Wei Zhao wrote: > From: wei zhao1 > > Add support for storing SYN filter in SW. Do you think does it makes more clear to refer as TCP SYN filter? Or SYN filter is clear enough? > > Signed-off-by: Wenzhuo Lu > Signed-off-by: wei zhao1 Can you please update sign-off to

Re: [dpdk-dev] [PATCH 02/18] net/ixgbe: store flow director filter

2016-12-20 Thread Ferruh Yigit
On 12/2/2016 10:42 AM, Wei Zhao wrote: > From: wei zhao1 > > Add support for storing flow director filter in SW. > > Signed-off-by: Wenzhuo Lu > Signed-off-by: wei zhao1 > --- > drivers/net/ixgbe/ixgbe_ethdev.c | 48 ++ > drivers/net/ixgbe/ixgbe_ethdev.h | 19 ++- > driv

Re: [dpdk-dev] [PATCH 04/18] net/ixgbe: restore n-tuple filter

2016-12-20 Thread Ferruh Yigit
On 12/2/2016 10:43 AM, Wei Zhao wrote: > From: wei zhao1 > > Add support for restoring n-tuple filter in SW. > > Signed-off-by: Wenzhuo Lu > Signed-off-by: wei zhao1 > --- > drivers/net/ixgbe/ixgbe_ethdev.c | 131 > +-- > 1 file changed, 83 insertions(+),

Re: [dpdk-dev] [PATCH 15/18] net/ixgbe: parse flow director filter

2016-12-20 Thread Ferruh Yigit
On 12/2/2016 10:43 AM, Wei Zhao wrote: > From: wei zhao1 > > check if the rule is a flow director rule, and get the flow director info. > > Signed-off-by: wei zhao1 > Signed-off-by: Wenzhuo Lu > --- <...> > + PATTERN_SKIP_VOID(rule, struct ixgbe_fdir_rule, > + RTE_F

Re: [dpdk-dev] [PATCH v3 25/25] doc: describe testpmd flow command

2016-12-20 Thread Ferruh Yigit
On 12/19/2016 5:49 PM, Adrien Mazarguil wrote: > Document syntax, interaction with rte_flow and provide usage examples. > > Signed-off-by: Adrien Mazarguil > Acked-by: Olga Shern > --- <...> > + > +Check whether redirecting any Ethernet packet received on port 0 to RX queue > +index 6 is suppo

Re: [dpdk-dev] [PATCH 04/13] eal: introduce driver type

2016-12-20 Thread Stephen Hemminger
On Tue, 20 Dec 2016 14:00:55 +0100 Jan Blunck wrote: > On Mon, Dec 19, 2016 at 10:59 PM, Stephen Hemminger > wrote: > > Since multiple buses and device types need to be supported. > > Provide type field in driver. > > --- > > lib/librte_eal/common/include/rte_dev.h | 15 --- > > li

Re: [dpdk-dev] [PATCH v3 02/12] eal/bus: introduce bus abstraction

2016-12-20 Thread Stephen Hemminger
On Tue, 20 Dec 2016 14:17:14 +0100 Jan Blunck wrote: > On Fri, Dec 16, 2016 at 2:10 PM, Shreyansh Jain > wrote: > > This patch introduces the rte_bus abstraction for devices and drivers in > > EAL framework. The model is: > > - One or more buses are connected to a CPU (or core) > > - One or m

Re: [dpdk-dev] [PATCH 01/13] ethdev: increase length ethernet device internal name

2016-12-20 Thread Stephen Hemminger
On Tue, 20 Dec 2016 12:23:25 +0530 Shreyansh Jain wrote: > On Tuesday 20 December 2016 03:29 AM, Stephen Hemminger wrote: > > Allow sufficicent space for UUID in string form (36+1). > > Needed to use UUID with Hyper-V > > > > Signed-off-by: Stephen Hemminger > > --- > > doc/guides/rel_notes/dep

Re: [dpdk-dev] [PATCH 04/13] eal: introduce driver type

2016-12-20 Thread Stephen Hemminger
On Tue, 20 Dec 2016 12:46:17 +0530 Shreyansh Jain wrote: > On Tuesday 20 December 2016 03:29 AM, Stephen Hemminger wrote: > > Since multiple buses and device types need to be supported. > > Provide type field in driver. > > --- > > lib/librte_eal/common/include/rte_dev.h | 15 --- >

Re: [dpdk-dev] [PATCH v2] examples/ethtool: fix bug in drvinfo callback

2016-12-20 Thread Thomas Monjalon
2016-11-23 09:00, Remy Horton: > Not sure this is a problem in practice, as the same set if fields is > updated each time...at least for now. > > On 22/11/2016 09:41, Qiming Yang wrote: > > Function pcmd_drvinfo_callback uses struct info to get > > the ethtool information of each port. Struct inf

Re: [dpdk-dev] [PATCH 11/18] net/ixgbe: parse n-tuple filter

2016-12-20 Thread Ferruh Yigit
On 12/2/2016 10:43 AM, Wei Zhao wrote: > From: wei zhao1 > > Add rule validate function and check if the rule is a n-tuple rule, > and get the n-tuple info. > > Signed-off-by: wei zhao1 > Signed-off-by: Wenzhuo Lu > --- > drivers/net/ixgbe/ixgbe_ethdev.c | 349 > +

Re: [dpdk-dev] [PATCH 16/18] net/ixgbe: create consistent filter

2016-12-20 Thread Ferruh Yigit
On 12/2/2016 10:43 AM, Wei Zhao wrote: > From: wei zhao1 > > This patch adds a function to create the flow directory filter. > > Signed-off-by: wei zhao1 > Signed-off-by: Wenzhuo Lu <...> > +/** > + * Create or destroy a flow rule. > + * Theorically one rule can match more than one filters.

Re: [dpdk-dev] [PATCH v3] drivers: advertise kmod dependencies in pmdinfo

2016-12-20 Thread Thomas Monjalon
> > Add a new macro RTE_PMD_REGISTER_KMOD_DEP() that allows a driver to > > declare the list of kernel modules required to run properly. > > > > Today, most PCI drivers require uio/vfio. > > > > Signed-off-by: Olivier Matz > > Acked-by: Fiona Trahe > > Acked-by: Adrien Mazarguil Applied in m

Re: [dpdk-dev] [PATCH 10/24] ethdev: parse ethertype filter

2016-12-20 Thread Ferruh Yigit
On 12/2/2016 11:53 AM, Beilei Xing wrote: > Check if the rule is a ethertype rule, and get the ethertype > info BTW. > > Signed-off-by: Wenzhuo Lu > Signed-off-by: Beilei Xing > --- CC: Adrien Mazarguil > lib/librte_ether/rte_flow.c| 136 > + > li

[dpdk-dev] [PATCH v4 00/25] Generic flow API (rte_flow)

2016-12-20 Thread Adrien Mazarguil
As previously discussed in RFC v1 [1], RFC v2 [2], with changes described in [3] (also pasted below), here is the first non-draft series for this new API. Its capabilities are so generic that its name had to be vague, it may be called "Generic flow API", "Generic flow interface" (possibly shortene

[dpdk-dev] [PATCH v4 01/25] ethdev: introduce generic flow API

2016-12-20 Thread Adrien Mazarguil
This new API supersedes all the legacy filter types described in rte_eth_ctrl.h. It is slightly higher level and as a result relies more on PMDs to process and validate flow rules. Benefits: - A unified API is easier to program for, applications do not have to be written for a specific filter t

[dpdk-dev] [PATCH v4 03/25] doc: announce deprecation of legacy filter types

2016-12-20 Thread Adrien Mazarguil
They are superseded by the generic flow API (rte_flow). Target release is not defined yet. Suggested-by: Kevin Traynor Signed-off-by: Adrien Mazarguil Acked-by: Olga Shern --- doc/guides/rel_notes/deprecation.rst | 8 1 file changed, 8 insertions(+) diff --git a/doc/guides/rel_notes/

[dpdk-dev] [PATCH v4 02/25] doc: add rte_flow prog guide

2016-12-20 Thread Adrien Mazarguil
This documentation is based on the latest RFC submission, subsequently updated according to feedback from the community. Signed-off-by: Adrien Mazarguil Acked-by: Olga Shern --- doc/guides/prog_guide/index.rst|1 + doc/guides/prog_guide/rte_flow.rst | 2042 ++

[dpdk-dev] [PATCH v4 04/25] cmdline: add support for dynamic tokens

2016-12-20 Thread Adrien Mazarguil
Considering tokens must be hard-coded in a list part of the instruction structure, context-dependent tokens cannot be expressed. This commit adds support for building dynamic token lists through a user-provided function, which is called when the static token list is empty (a single NULL entry). B

[dpdk-dev] [PATCH v4 05/25] cmdline: add alignment constraint

2016-12-20 Thread Adrien Mazarguil
This prevents sigbus errors on architectures that cannot handle unexpected unaligned accesses to the output buffer. Signed-off-by: Adrien Mazarguil Acked-by: Olga Shern --- lib/librte_cmdline/cmdline_parse.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/lib/librte

[dpdk-dev] [PATCH v4 06/25] app/testpmd: implement basic support for rte_flow

2016-12-20 Thread Adrien Mazarguil
Add basic management functions for the generic flow API (validate, create, destroy, flush, query and list). Flow rule objects and properties are arranged in lists associated with each port. Signed-off-by: Adrien Mazarguil Acked-by: Olga Shern --- app/test-pmd/cmdline.c | 1 + app/test-pmd

[dpdk-dev] [PATCH v4 08/25] app/testpmd: add rte_flow integer support

2016-12-20 Thread Adrien Mazarguil
Parse all integer types and handle conversion to network byte order in a single function. Signed-off-by: Adrien Mazarguil Acked-by: Olga Shern --- app/test-pmd/cmdline_flow.c | 148 +++ 1 file changed, 148 insertions(+) diff --git a/app/test-pmd/cmdline_flow

[dpdk-dev] [PATCH v4 07/25] app/testpmd: add flow command

2016-12-20 Thread Adrien Mazarguil
Managing generic flow API functions from command line requires the use of dynamic tokens for convenience as flow rules are not fixed and cannot be defined statically. This commit adds specific flexible parser code and object for a new "flow" command in separate file. Signed-off-by: Adrien Mazargu

  1   2   >