[dpdk-dev] [PATCH] ixgbe: prefetch packet headers in vector PMD receive function

2015-10-14 Thread Ananyev, Konstantin
> -Original Message- > From: Zoltan Kiss [mailto:zoltan.kiss at linaro.org] > Sent: Wednesday, October 14, 2015 5:11 PM > To: Ananyev, Konstantin; Richardson, Bruce; dev at dpdk.org > Subject: Re: [PATCH] ixgbe: prefetch packet headers in vector PMD receive > function > > > > On 28/09

[dpdk-dev] [PATCH] eal/bsd: reinitialize optind and optreset to 1

2015-10-14 Thread Tiwei Bie
On Wed, Oct 14, 2015 at 06:19:33PM +0800, Tiwei Bie wrote: > On Wed, Oct 14, 2015 at 10:31:28AM +0100, Bruce Richardson wrote: > > On Wed, Oct 14, 2015 at 10:28:44AM +0800, Tiwei Bie wrote: > > > On Tue, Oct 13, 2015 at 05:14:38PM +, Don Provan wrote: > > > > Actually, this is a good opportunit

[dpdk-dev] [PATCH] mem: Command line option to delete hugepage backing files

2015-10-14 Thread shesha Sreenivasamurthy (shesha)
When an application using huge-pages crash or exists, the hugetlbfs backing files are not cleaned up. This is a patch to clean those files. There are multi-process DPDK applications that may be benefited by those backing files. Therefore, I have made that configurable so that the application tha

[dpdk-dev] [PATCHv5 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Ananyev, Konstantin
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Wednesday, October 14, 2015 5:09 PM > To: Ananyev, Konstantin > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCHv5 1/8] ethdev: add new API to retrieve RX/TX > queue information > > On Thu,

[dpdk-dev] [PATCH] eal/bsd: reinitialize optind and optreset to 1

2015-10-14 Thread Tiwei Bie
On Wed, Oct 14, 2015 at 10:31:28AM +0100, Bruce Richardson wrote: > On Wed, Oct 14, 2015 at 10:28:44AM +0800, Tiwei Bie wrote: > > On Tue, Oct 13, 2015 at 05:14:38PM +, Don Provan wrote: > > > Actually, this is a good opportunity to fix a bug that's been in this > > > code forever: it shouldn'

[dpdk-dev] [PATCH v2] ixgbe: remove rx jabber from ierrors

2015-10-14 Thread Harry van Haaren
Remove receive jabber count from ierrors count as the registers overlap, previously causing some packets to be counted twice. Signed-off-by: Harry van Haaren --- v2: Fixed typo drivers/net/ixgbe/ixgbe_ethdev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b

[dpdk-dev] [PATCH] eal/bsd: reinitialize optind and optreset to 1

2015-10-14 Thread Don Provan
> > On Wed, Oct 14, 2015 at 10:28:44AM +0800, Tiwei Bie wrote: > > > It is designed to have DPDK's parameters specified in the front of the > > > cmd line and terminated by '--'. In other words, it is designed assuming the DPDK library can dictate the application's command line. This is an incorre

[dpdk-dev] DPDK User Space: Session onUseability and Ease of Use

2015-10-14 Thread Bruce Richardson
On Wed, Oct 14, 2015 at 04:36:27PM +0200, Vincent JARDIN wrote: > Thomas, John, > > thanks for your notes. > > Enclosed the contents of our chats, > > On 13/10/2015 18:36, Mcnamara, John wrote: > >* PMD lite > > > > - Do we need a lighter PMD model? Perhaps based on the Mellanox > > model.

[dpdk-dev] [PATCH v2 0/4] Support for Netronome´s NFP-6xxx card

2015-10-14 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Alejandro.Lucero > Sent: Tuesday, October 6, 2015 3:49 PM > To: dev at dpdk.org > Subject: [PATCH v2 0/4] Support for Netronome?s NFP-6xxx card > > From: "Alejandro.Lucero" > > This patchset adds a new PMD for

[dpdk-dev] [PATCH] ixgbe: prefetch packet headers in vector PMD receive function

2015-10-14 Thread Zoltan Kiss
On 28/09/15 00:19, Ananyev, Konstantin wrote: > > >> -Original Message- >> From: Zoltan Kiss [mailto:zoltan.kiss at linaro.org] >> Sent: Friday, September 25, 2015 7:29 PM >> To: Richardson, Bruce; dev at dpdk.org >> Cc: Ananyev, Konstantin >> Subject: Re: [PATCH] ixgbe: prefetch packet h

[dpdk-dev] [PATCH] mem: Command line option to delete hugepage backing files

2015-10-14 Thread shesha Sreenivasamurthy (shesha)
--- lib/librte_eal/common/eal_common_options.c | 12 ++ lib/librte_eal/common/eal_internal_cfg.h | 1 + lib/librte_eal/common/eal_options.h| 2 ++ lib/librte_eal/linuxapp/eal/eal_memory.c | 37 ++ 4 files changed, 52 insertions(+) diff --git a/lib/libr

[dpdk-dev] [PATCH] ixgbe: remove rx jabber from ierrors

2015-10-14 Thread Harry van Haaren
Remove recieve jabber count from ierrors count as the registers overlap, previously causing some packets to be counted twice. Signed-off-by: Harry van Haaren --- drivers/net/ixgbe/ixgbe_ethdev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixg

[dpdk-dev] DPDK User Space: Session onUseability and Ease of Use

2015-10-14 Thread Vincent JARDIN
Thomas, John, thanks for your notes. Enclosed the contents of our chats, On 13/10/2015 18:36, Mcnamara, John wrote: > * PMD lite > >- Do we need a lighter PMD model? Perhaps based on the Mellanox > model. >- Vincent suggested be could remove 90% of the code. I'll leave > Vincen

[dpdk-dev] [PATCH v3 07/20] pcap: remove pci device driver

2015-10-14 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bernard Iremonger > Sent: Monday, October 12, 2015 5:25 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v3 07/20] pcap: remove pci device driver > > remove rte_pcap_pmd and pci_dev. > initialise dev_flags,

[dpdk-dev] [PATCH v3 02/20] librte_ether: add fields from rte_pci_driver to rte_eth_dev_data

2015-10-14 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bernard Iremonger > Sent: Monday, October 12, 2015 5:25 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v3 02/20] librte_ether: add fields from > ... > > lro : 1; /**< RX LRO is ON

[dpdk-dev] DPDK User Space: Session onUseability and Ease of Use

2015-10-14 Thread Vincent JARDIN
On 13/10/2015 18:36, Mcnamara, John wrote: > * Should DPDK applications be running as root > >- Clearly not a great option. > >- Currently required due to kernel. It is not 100% accurate. With some systems, it shall be possible to run DPDK application without running them as root. It was

[dpdk-dev] DPDK User Space: Session onUseability and Ease of Use

2015-10-14 Thread Vincent JARDIN
On 13/10/2015 22:06, Thomas Monjalon wrote: > 2015-10-13 16:36, Mcnamara, John: >> > - Move the EAL to the kernel. > Please explain what you mean here. > It's difficult to imagine. > +Martin at ARM on it. We were rising this topic with him last week.

[dpdk-dev] [PATCH] eal: default to one memory channel if not specified

2015-10-14 Thread Panu Matilainen
On 10/14/2015 02:45 PM, David Marchand wrote: > Hello Panu, > > On Wed, Oct 14, 2015 at 12:22 PM, Panu Matilainen > wrote: > > Obtaining the correct value, especially from a running system, can > be anything from difficult to plain impossible. Since the valu

[dpdk-dev] [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Amine Kherbouche
> > So your v6 will make all implemented queue_info_get()s to fill these two new > fields correctly, right? > Konstantin Yes. Amine Kherbouche

[dpdk-dev] [PATCH 34/34] e1000/base: some minor change

2015-10-14 Thread Wenzhuo Lu
Some minor code change. No functionality impact. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 27 --- drivers/net/e1000/base/e1000_ich8lan.h | 1 + 2 files changed, 13 insertions(+), 15 deletions(-) diff --git a/drivers/net/e1000/base/e1000_ich

[dpdk-dev] [PATCH 33/34] e1000/base: fix to enable both ulp and EEE in Sx state

2015-10-14 Thread Wenzhuo Lu
This patch implements a modified flow that allows both ULP and EEE in Sx(Sticky mode). Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.c b/drivers/net/e1000/base/e1000_ich8lan.c i

[dpdk-dev] [PATCH 32/34] e1000/base: synchronization of MAC-PHY interface only on non- ME systems

2015-10-14 Thread Wenzhuo Lu
On power up, the MAC - PHY interface needs to be set to PCIe, even if cable is disconnected. In ME systems, the ME handles this on exit from Sx(Sticky mode) state. In non-ME, the driver handles it. Added a check for non-ME system to the driver code that handles that. Signed-off-by: Wenzhuo Lu --

[dpdk-dev] [PATCH 31/34] e1000/base: move the print to the right position

2015-10-14 Thread Wenzhuo Lu
This info need not to be always printed. Move it into the if. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_phy.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/e1000/base/e1000_phy.c b/drivers/net/e1000/base/e1000_phy.c index 6bbb379..d43b7ce 1

[dpdk-dev] [PATCH 30/34] e1000/base: use the correct i210 register for EEMNGCTL

2015-10-14 Thread Wenzhuo Lu
The i210 has two EEPROM access registers that are located in non-standard offsets: EEARBC and EEMNGCTL. EEARBC was fixed previously and EEMNGCTL should also be corrected. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_i210.c | 30 ++ drivers/net/e1000/base

[dpdk-dev] [PATCH 29/34] e1000/base: implement 88E1543 PHY initialization

2015-10-14 Thread Wenzhuo Lu
The initializtion process for 88E1543 PHY. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 106 - drivers/net/e1000/base/e1000_82575.h | 1 + drivers/net/e1000/base/e1000_defines.h | 1 + 3 files changed, 107 insertions(+), 1 deletion(

[dpdk-dev] [PATCH 28/34] e1000/base: increase timeout of polling bit RSPCIPHY in check_reset_block

2015-10-14 Thread Wenzhuo Lu
Previously, in check_reset_block RSPCIPHY was polled for 100 ms before determining that the ME veto is set. This needed to be increased to 300 ms. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net

[dpdk-dev] [PATCH 27/34] e1000/base: fix for i354 88E1112 PHY using AutoMedia Detect

2015-10-14 Thread Wenzhuo Lu
e1000_check_for_link_media_swap() is supposed to check PHY page 0 for copper and PHY page 1 for "other" (fiber) link. We switched back from page 1 to page 0 too soon, before e1000_check_for_link_82575() is executed and we were never finding link on fiber (other). Note: The precedence of link type

[dpdk-dev] [PATCH 26/34] e1000/base: disable extension header parsing for IPv6

2015-10-14 Thread Wenzhuo Lu
All 1G Server products need to have IPv6 extension headers turned off. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/e1000/base/e1000_82575.c b/drivers/net/e1000/base/e1000_82575.c i

[dpdk-dev] [PATCH 25/34] e1000/base: set correct value of beacon duration

2015-10-14 Thread Wenzhuo Lu
Fix for I217 Packet Loss issue - The Management Engine sets the FEXTNVM4 Beacon Duration incorrectly. This fix ensures that the correct value will always be set. Correct value for this field is 8 usec. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 14 ++ 1 f

[dpdk-dev] [PATCH 24/34] e1000/base: remove FIXME comment

2015-10-14 Thread Wenzhuo Lu
The "FIXME" comment is revomed from e1000_acquire_swfw_sync_80003es2lan but forgotten being removed from e1000_acquire_swfw_sync_82575 while the similar changes were made to both. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 2 +- 1 file changed, 1 insertion(+), 1 deletio

[dpdk-dev] [PATCH 23/34] e1000/base: K1 flow fixes

2015-10-14 Thread Wenzhuo Lu
This patch is for the following updates to the K1 configurations: Tx idle period for entering K1 should be 128 ns. Minimum Tx idle period in K1 should be 256 ns. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 47 +- drivers/net/e1000/base/e

[dpdk-dev] [PATCH 22/34] e1000/base: add bit for disable packetbuffer read

2015-10-14 Thread Wenzhuo Lu
Added bit FEXTNVM7[18], that controls disabling MAC packet buffer read. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.h b/drivers/net/e1000/base/e1000_ich8lan.h index f5d8ab1..f7f66

[dpdk-dev] [PATCH 21/34] e1000/base: cleanup NAHUM6LP_HW tags

2015-10-14 Thread Wenzhuo Lu
Remove all NAHUM6LP_HW tags. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_hw.h | 6 +++--- drivers/net/e1000/base/e1000_ich8lan.c | 9 + drivers/net/e1000/base/e1000_ich8lan.h | 29 ++--- drivers/net/e1000/base/e1000_osdep.h | 1 - drivers/

[dpdk-dev] [PATCH 20/34] e1000/base: fix link detect flow

2015-10-14 Thread Wenzhuo Lu
In case that auto-negotiate is not enabled, call e1000_setup_copper_link_generic instead of e1000_phy_setup_autoneg. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/drivers/net/e1000/base/e1000

[dpdk-dev] [PATCH 19/34] e1000/base: add return value for resume workaround

2015-10-14 Thread Wenzhuo Lu
Add u32 return value to function e1000_resume_workarounds_pchlan, so that calling function can detect PHY access failure during resuming flow. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 11 ++- drivers/net/e1000/base/e1000_ich8lan.h | 2 +- 2 files changed, 7

[dpdk-dev] [PATCH 18/34] e1000/base: fix TIPG value for non 10 half duplex mode

2015-10-14 Thread Wenzhuo Lu
TIPG value is increased when setting speed to 10 half to prevent packet loss. However, it was never decreased again when speed changes. This caused performance issues in the NDIS driver. Fix this to restore TIPG to default value on non 10 half. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/bas

[dpdk-dev] [PATCH 17/34] e1000/base: prevent ulp flow if cable connected

2015-10-14 Thread Wenzhuo Lu
Enabling ulp on link down when cable is connect caused an infinite loop of linkup/down indications in the NDIS driver. After discussed, correct flow is to enable ULP only when cable is disconnected. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 3 +++ 1 file changed, 3 i

[dpdk-dev] [PATCH 16/34] e1000/base: add flags to set eee advertisement modes

2015-10-14 Thread Wenzhuo Lu
!!! REQUIRES DRIVER CHANGES !!! Change e1000_set_eee_i350 and e1000_set_eee_i354 to have flags allowing changes in the advertised EEE speeds. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 34 +++--- drivers/net/e1000/base/e1000_82575.h | 4 ++-

[dpdk-dev] [PATCH 15/34] e1000/base: apply paranoia to macro arguments

2015-10-14 Thread Wenzhuo Lu
Macro arguments need to be in parens since we can pass in expressions. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_api.h | 16 drivers/net/e1000/base/e1000_hw.h | 2 +- drivers/net/e1000/base/e1000_regs.h | 4 ++-- 3 files changed, 11 insertions(+), 11 deleti

[dpdk-dev] [PATCH 14/34] e1000/base: add EEARBC_I210 for i210

2015-10-14 Thread Wenzhuo Lu
EEARBC has changed on i210. It means EEARBC has a different address on i210 than on other NICs. So, add a new entity named EEARBC_I210 to the register list and make sure the right one is being used on i210. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_i210.c | 17 ++---

[dpdk-dev] [PATCH 13/34] e1000/base: add support for inverted format ETrackId

2015-10-14 Thread Wenzhuo Lu
There are some images which contain ETrackID in inverted format. This patch allows reading this format. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_nvm.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_nvm.c b/drivers/net/e100

[dpdk-dev] [PATCH 12/34] e1000/base: add return value handler for ESB2 controller init and reset

2015-10-14 Thread Wenzhuo Lu
Adding code where missing to handle case where calls to e1000_read_kmrn_reg_80003es2lan and e1000_write_kmrn_reg_80003es2lan return an error value. Also, when accessing the E1000_KMRNCTRLSTA_INBAND_PARAM offset to disable far-end loopback on 80003es2lan devices, make the handling of a read or write

[dpdk-dev] [PATCH 11/34] e1000/base: add return value handler when check manage mode

2015-10-14 Thread Wenzhuo Lu
Adding code, where missing, to handle the case when hw->nvm.ops.read returns an error value. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82571.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_82571.c b/drivers/net/e1000/base/

[dpdk-dev] [PATCH 10/34] e1000/base: change invariant return to not use variables

2015-10-14 Thread Wenzhuo Lu
Although this change should be optimized out by the compiler, just return a constant directly rather than declare a variable. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/net/e1000/bas

[dpdk-dev] [PATCH 09/34] e1000/base: add evaluation of e1000_nvm_read return value

2015-10-14 Thread Wenzhuo Lu
Adding code to a case where e1000_nvn_read is called, but there is no consideration for when the read fails (returns an error code). Also, this patch adds an error message to a base NVM reading function that is missing it for consistency. This patch is not covering all cases of these conditions, it

[dpdk-dev] [PATCH 08/34] e1000/base: remove E1000_WRITE_FLUSH for DH89XXCC_SGMII after commencing HW reset

2015-10-14 Thread Wenzhuo Lu
For DH89XXCC_SGMII, write flush leaves registers of this device trashed (0x). Added check for this device. Also, after both for Port SW Reset and Device Reset case, platform should wait at least 3ms before reading any registers. Since waiting is conditionally executed only for Device Reset

[dpdk-dev] [PATCH 07/34] e1000/base: add defaults for i210 TX/RX PBSIZE

2015-10-14 Thread Wenzhuo Lu
These are the defaults for the packet buffer size registers that need to be explicitly set back if someone changes them and comes back to a normal driver. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_defines.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/e1000/

[dpdk-dev] [PATCH 06/34] e1000/base: add return value to the functions of setting receive address register

2015-10-14 Thread Wenzhuo Lu
Previously, the rar_set functions were of type void, and when they failed to program an address register they would, at most, put a message into the log and end. The fact that they failed to program an address into a address register, if checked for, should be captured and passed back to the call

[dpdk-dev] [PATCH 05/34] e1000/base: redundant PHY power down for i210

2015-10-14 Thread Wenzhuo Lu
The wrong bit is being used in PHYREG16 for PHY power down. In addition, the use of PHYREG 16 is unnecessary if bit 11 of PHYREG 0 is used. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_phy.c | 15 --- drivers/net/e1000/base/e1000_phy.h | 1 - 2 files changed, 16 deleti

[dpdk-dev] [PATCH 04/34] e1000/base: fix issue with jumbo frame CRC failures in client

2015-10-14 Thread Wenzhuo Lu
This is a patch to change the value of register 776.20[11:2] for jumbo mode from 0x1A to 0x1F. This is to enlarge the gap between read and write pointers in the TX Fifo. And replace the magic number with a macro by the way. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_defines.h | 1

[dpdk-dev] [PATCH 03/34] e1000/base: fix issue with link flap on 82579

2015-10-14 Thread Wenzhuo Lu
Several customers have reported a link flap issue on 82579. The symptoms are random and intermittent link losses when 82579 is connected to specific switches. Issue has been root caused as interoperability problem between the NIC and at least some Broadcom PHYs in the Energy Efficient Ethernet wake

[dpdk-dev] [PATCH 02/34] e1000/base: add new devices

2015-10-14 Thread Wenzhuo Lu
Add some new i218 devices. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_api.c | 4 drivers/net/e1000/base/e1000_defines.h | 2 ++ drivers/net/e1000/base/e1000_hw.h | 4 drivers/net/e1000/base/e1000_ich8lan.c | 18 ++ 4 files changed, 24 inserti

[dpdk-dev] [PATCH 01/34] e1000/base: update readme and copyright

2015-10-14 Thread Wenzhuo Lu
Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/README | 4 ++-- drivers/net/e1000/base/e1000_80003es2lan.c | 2 +- drivers/net/e1000/base/e1000_80003es2lan.h | 2 +- drivers/net/e1000/base/e1000_82540.c | 2 +- drivers/net/e1000/base/e1000_82541.c | 2 +- drivers/ne

[dpdk-dev] [PATCH 00/34] update e1000 base driver

2015-10-14 Thread Wenzhuo Lu
Short summary: *update readme and copyright *add new devices *fix issue with link flap on 82579 *fix issue with jumbo frame CRC failures in client *redundant PHY power down for i210 *add return value to the functions of setting receive address register *add defaults for i210 TX/RX PBSIZE *remove E1

[dpdk-dev] [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Amine Kherbouche
Hi Konstantin > Well, in general I am not opposed to add these fields to the structures. > In fact, I think it is a good thing to have. > But if we'll add them without making queue_info_get() feeling them properly - > it might create a lot of controversy. > So unless you prepared to make changes

[dpdk-dev] [PATCH] eal: default to one memory channel if not specified

2015-10-14 Thread Bruce Richardson
On Wed, Oct 14, 2015 at 03:04:43PM +0300, Panu Matilainen wrote: > On 10/14/2015 02:45 PM, David Marchand wrote: > >Hello Panu, > > > >On Wed, Oct 14, 2015 at 12:22 PM, Panu Matilainen >> wrote: > > > >Obtaining the correct value, especially from a running system

[dpdk-dev] [PATCH 0/2] Add support for driver directories

2015-10-14 Thread David Marchand
On Wed, Oct 14, 2015 at 12:41 PM, Panu Matilainen wrote: > On 09/25/2015 04:00 PM, Panu Matilainen wrote: > >> On 09/25/2015 03:35 PM, David Marchand wrote: >> >>> Is there any reason why we should only have this for linux ? >>> >> >> I merely followed suite with the related, pre-existing -d opti

[dpdk-dev] [PATCHv5 8/8] testpmd: add new command to display RX/TX queue information

2015-10-14 Thread Remy Horton
On 01/10/2015 20:54, Konstantin Ananyev wrote: > Signed-off-by: Konstantin Ananyev Acked-by: Remy Horton

[dpdk-dev] [PATCHv5 7/8] vmxnet3: add HW specific desc_lim data into dev_info

2015-10-14 Thread Remy Horton
On 01/10/2015 20:54, Konstantin Ananyev wrote: > Signed-off-by: Konstantin Ananyev Acked-by: Remy Horton

[dpdk-dev] [PATCHv5 6/8] cxgbe: add HW specific desc_lim data into dev_info

2015-10-14 Thread Remy Horton
On 01/10/2015 20:54, Konstantin Ananyev wrote: > Signed-off-by: Konstantin Ananyev Acked-by: Remy Horton

[dpdk-dev] [PATCHv5 5/8] fm10k: add HW specific desc_lim data into dev_info

2015-10-14 Thread Remy Horton
On 01/10/2015 20:54, Konstantin Ananyev wrote: > Signed-off-by: Konstantin Ananyev Acked-by: Remy Horton

[dpdk-dev] [PATCHv5 4/8] e1000: add support for eth_(rxq|txq)_info_get and (rx|tx)_desc_lim

2015-10-14 Thread Remy Horton
On 01/10/2015 20:54, Konstantin Ananyev wrote: > Signed-off-by: Konstantin Ananyev Acked-by: Remy Horton

[dpdk-dev] [PATCHv5 3/8] ixgbe: add support for eth_(rxq|txq)_info_get and (rx|tx)_desc_lim

2015-10-14 Thread Remy Horton
On 01/10/2015 20:54, Konstantin Ananyev wrote: > Signed-off-by: Konstantin Ananyev Acked-by: Remy Horton

[dpdk-dev] [PATCHv5 2/8] i40e: add support for eth_(rxq|txq)_info_get and (rx|tx)_desc_lim

2015-10-14 Thread Remy Horton
On 01/10/2015 20:54, Konstantin Ananyev wrote: > From: "Ananyev, Konstantin" > > This patch assumes that the patch: > i40e: fix wrong alignment for the number of HW descriptors > already applied. > > Signed-off-by: Konstantin Ananyev Acked-by: Remy Horton

[dpdk-dev] [PATCH] eal: default to one memory channel if not specified

2015-10-14 Thread David Marchand
Hello Panu, On Wed, Oct 14, 2015 at 12:22 PM, Panu Matilainen wrote: > Obtaining the correct value, especially from a running system, can > be anything from difficult to plain impossible. Since the value is > merely an optimization and does not affect functionality otherwise, > its pointless to

[dpdk-dev] [PATCHv5 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Remy Horton
On 01/10/2015 20:54, Konstantin Ananyev wrote: > Signed-off-by: Konstantin Ananyev Acked-by: Remy Horton

[dpdk-dev] [PATCH 0/2] Add support for driver directories

2015-10-14 Thread Panu Matilainen
On 09/25/2015 04:00 PM, Panu Matilainen wrote: > On 09/25/2015 03:35 PM, David Marchand wrote: >> Hello Panu, >> >> On Fri, Sep 25, 2015 at 1:58 PM, Panu Matilainen > > wrote: >> >> This mini-series adds support for driver directory concept as >> described by

[dpdk-dev] [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Amine Kherbouche
Hi Konstantin > +/** > + * Ethernet device RX queue information structure. > + * Used to retieve information about configured queue. > + */ > +struct rte_eth_rxq_info { > + struct rte_mempool *mp; /**< mempool used by that queue. */ > + struct rte_eth_rxconf conf; /**< queue config pa

[dpdk-dev] [PATCH] eal: move interrupt init after device init

2015-10-14 Thread David Marchand
On Wed, Oct 14, 2015 at 11:50 AM, David Marchand wrote: > Fixes: "eal: set iopl only when needed" (565b85d) > Forgot the release note. If this current patch is ok for all, I will resubmit with an update to doc/guides/rel_notes/release_2_2.rst. -- David Marchand

[dpdk-dev] DPDK User Space: Session onUseability and Ease of Use

2015-10-14 Thread Panu Matilainen
On 10/13/2015 07:36 PM, Mcnamara, John wrote: >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas F Herbert >> Sent: Thursday, October 8, 2015 7:30 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] DPDK User Space: Session onUseability and Ease of Use >> >

[dpdk-dev] [PATCH] eal: default to one memory channel if not specified

2015-10-14 Thread Panu Matilainen
Obtaining the correct value, especially from a running system, can be anything from difficult to plain impossible. Since the value is merely an optimization and does not affect functionality otherwise, its pointless to force such a guess on users initially, such things belong to performance tuning

[dpdk-dev] I have a problem in setting up DPDK 2.1.0 in Fedora OS release 20 (Heisenbug). I cannot r

2015-10-14 Thread De Lara Guarch, Pablo
Hi Ick-Sung, > * Fedora OS version : Fedora release 20 (Heisenbug) > > * gcc version : gcc (GCC) 4.8.3 20140911 (Red Hat 4.8.3-7). > > * Intel(R) Xeon(R) CPU E5-2680 10 cores, 64G bytes system memory. > * 2 2-port NIC cards, Intel? 82599ES 10 Gigabit Ethernet 2 port Controller. > SFI/SFP+. opti

[dpdk-dev] [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Ananyev, Konstantin
> -Original Message- > From: Amine Kherbouche [mailto:amine.kherbouche at 6wind.com] > Sent: Wednesday, October 14, 2015 1:47 PM > To: Ananyev, Konstantin; dev at dpdk.org > Subject: Re: [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX > queue information > > > > > > > S

[dpdk-dev] [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Ananyev, Konstantin
> -Original Message- > From: Amine Kherbouche [mailto:amine.kherbouche at 6wind.com] > Sent: Wednesday, October 14, 2015 1:22 PM > To: Ananyev, Konstantin; dev at dpdk.org > Subject: Re: [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX > queue information > > > Hi Konstan

[dpdk-dev] [PATCH v5 5/9] null: export eth_dev_null_create

2015-10-14 Thread Jastrzebski, MichalX K
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Tetsuya Mukawa > Sent: Wednesday, October 14, 2015 3:35 AM > To: Kulasek, TomaszX; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v5 5/9] null: export eth_dev_null_create > > On 2015/09/30 23:05, Tomasz Kulase

[dpdk-dev] [PATCH v8 0/9] Dynamic memzones

2015-10-14 Thread Gonzalez Monroy, Sergio
On 14/10/2015 01:12, Stephen Hemminger wrote: > On Tue, 14 Jul 2015 09:57:04 +0100 > Sergio Gonzalez Monroy wrote: > >> Current implemetation allows reserving/creating memzones but not the opposite >> (unreserve/free). This affects mempools and other memzone based objects. >> >> From my point of

[dpdk-dev] [PATCH 1/2] eal/linux: move plugin load to very start of eal init

2015-10-14 Thread David Marchand
On Wed, Oct 14, 2015 at 2:05 AM, Stephen Hemminger < stephen at networkplumber.org> wrote: > Given all this, I recommend that Thomas not apply this patch. > Please resubmit if there is a real problem with drivers (something in > tree). > There are enough other bugs to fix without chasing ghosts. >

[dpdk-dev] [PATCH] eal: move interrupt init after device init

2015-10-14 Thread David Marchand
For virtio-net pmd, the interrupt management thread must be created after this driver has initialised so that iopl() has been properly called and its effects are inherited by all eal children threads. Before this change, changing link status on a virtio-net device would trigger a segfault in the i

[dpdk-dev] [PATCH 2/2] virtio: change io privilege level as early as possible

2015-10-14 Thread David Marchand
On Wed, Oct 14, 2015 at 10:00 AM, David Marchand wrote: > > I am testing Thomas proposal (just moving the intr_init() call), else I > will rebase this patch. > Thomas approach works and is minimal. I ran some autotest, did not see anything wrong (let aside the tests that already fail on current

[dpdk-dev] [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Ananyev, Konstantin
Hi Amine, > -Original Message- > From: Amine Kherbouche [mailto:amine.kherbouche at 6wind.com] > Sent: Wednesday, October 14, 2015 12:40 PM > To: Ananyev, Konstantin; dev at dpdk.org > Subject: Re: [dpdk-dev, PATCHv5, 1/8] ethdev: add new API to retrieve RX/TX > queue information > > >

[dpdk-dev] [PATCH v4 0/7] Add instalation rules for dpdk files.

2015-10-14 Thread Miguel Bernal Marin
Hi guys, Have you had any time to check this patchset version? On 10/5/2015 3:19 PM, Mario Carrillo wrote: > DPDK package lacks of a mechanism to install libraries, headers > applications, kernel modules and sdk files to a file system tree. > > This patch set allows to install files according to

[dpdk-dev] I have a problem in setting up DPDK 2.1.0 in Fedora OS release 20 (Heisenbug). I cannot r

2015-10-14 Thread 최익성
My DPDK setup shell script is as follows. #! /bin/bash ifconfig -a > ifconfig.out.txt sudo ifconfig p785p1 down sudo ifconfig p785p2 down sudo ifconfig p787p1 down sudo ifconfig p787p2 down setup_target() { option

[dpdk-dev] [PATCH v5 5/9] null: export eth_dev_null_create

2015-10-14 Thread Tetsuya Mukawa
On 2015/09/30 23:05, Tomasz Kulasek wrote: > Signed-off-by: Tomasz Kulasek > --- > drivers/net/null/Makefile |2 +- > drivers/net/null/rte_eth_null.c |2 +- > drivers/net/null/rte_eth_null.h | 40 > + > drivers/net/null/rt

[dpdk-dev] [PATCH v5 4/9] null: virtual dynamic rss configuration

2015-10-14 Thread Tetsuya Mukawa
On 2015/09/30 23:05, Tomasz Kulasek wrote: > This implementation allows to set and read RSS configuration for null > device, and is used to validate right values propagation over the slaves, > in test units for dynamic RSS configuration for bonding. > > v5 changes: > - replaced memcpy with rte_mem

[dpdk-dev] [PATCH v5 3/9] null: extend number of virtual queues

2015-10-14 Thread Tetsuya Mukawa
On 2015/09/30 23:04, Tomasz Kulasek wrote: > This patch adds a possibility to configure more than one queue on null > device. > > v5 changes: > - fixed queues number configuration (using internals->nb_*_queues instead >of dev->data->nb_*_queues) > > Signed-off-by: Tomasz Kulasek > --- > driv

[dpdk-dev] [PATCH v5 2/9] null: fix segfault when null_pmd added to bonding

2015-10-14 Thread Tetsuya Mukawa
On 2015/09/30 23:04, Tomasz Kulasek wrote: > This patch initializes eth_dev->link_intr_cbs queue used when null pmd is > added to the bonding. > > v5 changes: > - removed unnecessary malloc for eth_driver (rte_null_pmd) > > Signed-off-by: Tomasz Kulasek > --- > drivers/net/null/rte_eth_null.c |

[dpdk-dev] [PATCH] eal/bsd: reinitialize optind and optreset to 1

2015-10-14 Thread Bruce Richardson
On Wed, Oct 14, 2015 at 10:28:44AM +0800, Tiwei Bie wrote: > On Tue, Oct 13, 2015 at 05:14:38PM +, Don Provan wrote: > > Actually, this is a good opportunity to fix a bug that's been in this code > > forever: it shouldn't be resetting optind to some arbitrary value: it > > should be saving op

[dpdk-dev] IXGBE RX packet loss with 5+ cores

2015-10-14 Thread Bruce Richardson
On Tue, Oct 13, 2015 at 01:24:22PM -0700, Alexander Duyck wrote: > On 10/13/2015 07:47 AM, Sanford, Robert wrote: > [Robert:] > 1. The 82599 device supports up to 128 queues. Why do we see trouble > with as few as 5 queues? What could limit the system (and one port > controlled by 5

[dpdk-dev] I have a problem in setting up DPDK 2.1.0 in Fedora OS release 20 (Heisenbug). I cannot r

2015-10-14 Thread 최익성
Dear DPDK experts. Thank you very much for your best great efforts and precious answers. I have a problem in setting up DPDK 2.1.0 in Fedora OS release 20 (Heisenbug). My problem is that I cannot receive any traffics in test-pmd. There are 2 nics (2 port NIC), I connected port 0 to port 1(loopb

[dpdk-dev] [PATCH] eal/bsd: reinitialize optind and optreset to 1

2015-10-14 Thread Tiwei Bie
On Tue, Oct 13, 2015 at 05:14:38PM +, Don Provan wrote: > Actually, this is a good opportunity to fix a bug that's been in this code > forever: it shouldn't be resetting optind to some arbitrary value: it should > be saving optind (and optarg and optopt) at the beginning, initializing > opti

[dpdk-dev] [PATCH] Add error message when trying to use make option T= during build/clean

2015-10-14 Thread Francesco
Hi Olivier, thanks for reviewing the patch. However for me it is logical that T= option is used for "config" target and then for building/cleaning only O= is used. However it is not logical for me that install target requires T= option instead of O= option... or perhaps I'm missing something? Than

[dpdk-dev] [PATCH 2/2] virtio: change io privilege level as early as possible

2015-10-14 Thread David Marchand
On Wed, Oct 14, 2015 at 2:07 AM, Stephen Hemminger < stephen at networkplumber.org> wrote: > On Thu, 1 Oct 2015 07:25:45 -0400 > Neil Horman wrote: > > > On Wed, Sep 30, 2015 at 05:37:05PM +0200, Thomas Monjalon wrote: > > > 2015-09-30 10:52, Neil Horman: > > > > I think it may be solved by calli

[dpdk-dev] [PATCH v2] devargs: add blacklisting by linux interface name

2015-10-14 Thread Charles (Chas) Williams
On Tue, 2015-10-13 at 14:49 +0200, Olivier MATZ wrote: > Hi Chas, > > > @@ -352,6 +354,19 @@ pci_scan_one(const char *dirname, uint16_t domain, > > uint8_t bus, > > return -1; > > } > > > > + /* get network interface name */ > > + snprintf(filename, sizeof(filename), "%s/net

[dpdk-dev] [PATCHv5 1/8] ethdev: add new API to retrieve RX/TX queue information

2015-10-14 Thread Stephen Hemminger
On Thu, 1 Oct 2015 20:54:46 +0100 Konstantin Ananyev wrote: > + if (nb_rx_desc > dev_info.rx_desc_lim.nb_max || > + nb_rx_desc < dev_info.rx_desc_lim.nb_min || > + nb_rx_desc % dev_info.rx_desc_lim.nb_align != 0) { > + Preferred indentation style is t

[dpdk-dev] [PATCH 02/34] e1000/base: add new devices

2015-10-14 Thread Stephen Hemminger
On Wed, 14 Oct 2015 14:34:07 +0800 Wenzhuo Lu wrote: > Add some new i218 devices. > > Signed-off-by: Wenzhuo Lu > --- > drivers/net/e1000/base/e1000_api.c | 4 > drivers/net/e1000/base/e1000_defines.h | 2 ++ > drivers/net/e1000/base/e1000_hw.h | 4 > drivers/net/e1000/ba

[dpdk-dev] DPDK User Space: Session onUseability and Ease of Use

2015-10-14 Thread Dave Neary
Hi, On 10/13/2015 10:06 PM, Thomas Monjalon wrote: >> * Create a User Mailing List. - Needs support from 6Wind. > > OK > Name: user at dpdk.org or other suggestion? Other pages seem to have standardised on "users" - I don't think it matters much, either is fine. Thanks, Dave. -- Dave Neary -

[dpdk-dev] [PATCH] ixgbe: fix the wrong prompt for VF TSO

2015-10-14 Thread Wu, Jingjing
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Friday, September 11, 2015 10:00 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] ixgbe: fix the wrong prompt for VF TSO > > When setting TSO on VF ixgbe NICs, for example, 82599, x550

[dpdk-dev] i40e drops segmented traffic

2015-10-14 Thread Shaham Fridenberg
Hey all, I'm using dpdk 2.1.0 and segmented traffic via 40 giga NIC seems to fail (rte_i40e_pmd seems to drop all segments on RX). Is that a known issue? Any idea what might be the problem? Thanks, Shaham

[dpdk-dev] [PATCH] rte_alarm: modify it to make it not to be affected by discontinuous jumps in the system time

2015-10-14 Thread Jay Rolette
Back when this was first submitted in June, I mentioned that CLOCK_MONOTONIC_RAW was ~10x slower than CLOCK_MONOTONIC: http://dpdk.org/ml/archives/dev/2015-June/018687.html It's not completely free from NTP frequency adjustments, but it won't have any discontinuities. That's what we've been usin

[dpdk-dev] [PATCH 2/2] igb: fix VF statistic wraparound handling macro

2015-10-14 Thread Roger B. Melton
On 10/12/15 12:45 PM, Harry van Haaren wrote: > Fix a misinterpreatation of VF statistic macro in e1000/igb. > > Signed-off-by: Harry van Haaren Acked-by: Roger Melton

  1   2   >