[dpdk-dev] [PATCH v2 0/5] Virtio-net PMD: QEMU QTest extension for container

2016-02-10 Thread Tetsuya Mukawa
The patches will work on below patch series. - [PATCH v2 0/5] virtio support for container - [PATCH 0/4] rework ioport access for virtio [Changes] v2 changes: - Rebase on above patch seiries. - Rebase on master - Add "--qtest-virtio" EAL option. - Fixes in qtest.c - Fix error handling for

[dpdk-dev] [PATCH v2 1/5] virtio: Retrieve driver name from eth_dev

2016-02-10 Thread Tetsuya Mukawa
Currently, virtio_dev_info_get() retrieves driver name from pci_drv. If the driver is virtual PMD, pci_drv will be invalid. So retrieves the name from eth_dev. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/virtio_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[dpdk-dev] [PATCH v2 2/5] EAL: Add new EAL "--qtest-virtio" option

2016-02-10 Thread Tetsuya Mukawa
To work with qtest virtio-net PMD, virtual address that maps hugepages should be between (1 << 31) to (1 << 44). This patch adds one more option to map like this. Also all hugepages should consists of one file. Because of this, the option will work only when '--single-file' option is specified.

[dpdk-dev] [PATCH v2 3/5] vhost: Add a function to check virtio device type

2016-02-10 Thread Tetsuya Mukawa
The patch adds below function to cleanup virtio code. - virtio_dev_check() Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/virtio_ethdev.c | 52 ++ drivers/net/virtio/virtio_ethdev.h | 32 +++ 2 files changed, 57 insertions(+), 27

[dpdk-dev] [PATCH v2 4/5] virtio: Add support for qtest virtio-net PMD

2016-02-10 Thread Tetsuya Mukawa
The patch adds a new virtio-net PMD configuration that allows the PMD to work on host as if the PMD is in VM. Here is new configuration for virtio-net PMD. - CONFIG_RTE_VIRTIO_VDEV_QTEST To use this mode, EAL needs map all hugepages as one file. Also the file should be mapped between (1 << 31)

[dpdk-dev] [PATCH v2 5/5] docs: add release note for qtest virtio container support

2016-02-10 Thread Tetsuya Mukawa
Signed-off-by: Tetsuya Mukawa --- doc/guides/rel_notes/release_2_3.rst | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/guides/rel_notes/release_2_3.rst b/doc/guides/rel_notes/release_2_3.rst index 1e7d51d..ab3baf9 100644 --- a/doc/guides/rel_notes/release_2_3.rst +++

[dpdk-dev] [PATCH v3] driver/net/mpipe: fix the crash/hung issue when testpmd quits

2016-02-10 Thread Liming Sun
Fixes: the hung/crash issue when quitting testpmd under high traffic rate. The following issue were found and fixed. 1. edesc->size is not initialized properly in mpipe_do_xmit() and could cause buffer leak or corruption when HW buffer return is used. 2. Check the 'idesc.be' error bit in

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread David Marchand
On Tue, Feb 9, 2016 at 6:05 PM, Jan Viktorin wrote: > Maybe, a better subject? > > drivers: init pdev drivers in constructors Why not, I will try to find a best one, and if I can't, I will go with this. > On Fri, 29 Jan 2016 15:08:30 +0100 > David Marchand wrote: >> -static int >>

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread David Marchand
On Wed, Feb 10, 2016 at 9:51 AM, David Marchand wrote: > On Tue, Feb 9, 2016 at 6:05 PM, Jan Viktorin > wrote: >> What about introducing a macro for this? >> >> RTE_REGISTER_PCI_DRIVER(rte_qad_pmd); > > Yes. The only problem here, is that rte_qad_pmd is a crypto structure (same problem with

[dpdk-dev] [PATCH v2 1/3] driver/net/mpipe: support native build on tilegx platform.

2016-02-10 Thread Bruce Richardson
On Tue, Feb 09, 2016 at 11:47:55PM +0100, Thomas Monjalon wrote: > 2016-02-09 21:15, Liming Sun: > > Looks like this patch serie has been merged into dpdk-next-net/rel_16_04. > > What would be the usual way to submit changes for new comments? Would it be > > incremental changes (new commit) based

[dpdk-dev] [PATCH v2 1/3] driver/net/mpipe: support native build on tilegx platform.

2016-02-10 Thread Thomas Monjalon
2016-02-10 09:49, Bruce Richardson: > On Tue, Feb 09, 2016 at 11:47:55PM +0100, Thomas Monjalon wrote: > > 2016-02-09 21:15, Liming Sun: > > > Looks like this patch serie has been merged into dpdk-next-net/rel_16_04. > > > What would be the usual way to submit changes for new comments? Would it >

[dpdk-dev] Regarding mbuf allocation/free in secondary process

2016-02-10 Thread Bruce Richardson
On Tue, Feb 09, 2016 at 11:43:19PM -0800, Saravana Kumar wrote: > Hi DPDK community, > > > > I'd like to have DPDK NIC IO operations in (primary) process and > execution logic in (secondary) processes. > Primary process pushes NIC Rx mbufs to Secondary process through S/W ring > > Seconary

[dpdk-dev] [PATCH 0/2] bonding fixes

2016-02-10 Thread Bernard Iremonger
These patches fix segmentation faults which were occurring when slave devices were detached before being removed from the bonding device. The slave devices must now be removed from the bonding device before they can be detached. The bonding device cannot be detached now until all slave devices

[dpdk-dev] [PATCH 2/2] bonding: fix detach of bonded slave devices

2016-02-10 Thread Bernard Iremonger
Ensure that a bonded slave device is not detached, until it is removed from the bonded device. Fixes: 2efb58cbab6e ("bond: new link bonding library") Fixes: a45b288ef21a ("bond: support link status polling") Fixes: 494adb7f63f2 ("ethdev: add device fields from PCI layer") Fixes: b1fb53a39d88

[dpdk-dev] [PATCH 1/2] bonding: fix detach of bonded device

2016-02-10 Thread Bernard Iremonger
Check that the bonded device has no slaves before detaching it. Fixes: 8d30fe7fa737 ("bonding: support port hotplug") Signed-off-by: Bernard Iremonger --- drivers/net/bonding/rte_eth_bond_api.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[dpdk-dev] Regarding mbuf allocation/free in secondary process

2016-02-10 Thread Saravana Kumar
Thanks for your response.. Sara On Wed, Feb 10, 2016 at 2:01 AM, Bruce Richardson < bruce.richardson at intel.com> wrote: > On Tue, Feb 09, 2016 at 11:43:19PM -0800, Saravana Kumar wrote: > > Hi DPDK community, > > > > > > > > I'd like to have DPDK NIC IO operations in (primary) process and > >

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread Jan Viktorin
On Wed, 10 Feb 2016 10:27:14 +0100 David Marchand wrote: > On Wed, Feb 10, 2016 at 9:51 AM, David Marchand > wrote: > > On Tue, Feb 9, 2016 at 6:05 PM, Jan Viktorin > > wrote: > >> What about introducing a macro for this? > >> > >> RTE_REGISTER_PCI_DRIVER(rte_qad_pmd); > > > > Yes. > >

[dpdk-dev] i40e: cannot change mtu to enable jumbo frame

2016-02-10 Thread Julien Meunier
On 02/09/2016 08:05 PM, Zhu, Heqing wrote: > Helin is still in Chinese New Year Vacation. Will the below command option > help ? > > 4.5.9. port config - max-pkt-len > Set the maximum packet length: > > testpmd> port config all max-pkt-len (value) > This is equivalent to the --max-pkt-len

[dpdk-dev] [dpdk-users] DPDK i40evf problem in receving packet

2016-02-10 Thread Muhammad Zain-ul-Abideen
Hi Saurabh, Can you tell me what card you are uing *Regards:* Zain ul Abideen *Disclaimer**: The information contained in this e-mail and any attachments is confidential; it is intended only for use of the individual or entity named above. If the reader of this message is not the intended

[dpdk-dev] [PATCH 6/9] eal: initialize vdevs right next to pci devices

2016-02-10 Thread Jan Viktorin
On Fri, 29 Jan 2016 15:08:33 +0100 David Marchand wrote: > This way, the resources probing happens in a common place. > > Signed-off-by: David Marchand > --- > lib/librte_eal/bsdapp/eal/eal.c | 7 +++ > lib/librte_eal/common/include/rte_dev.h | 2 +- >

[dpdk-dev] [PATCH 7/9] pci: add a helper for device name

2016-02-10 Thread Jan Viktorin
On Fri, 29 Jan 2016 15:08:34 +0100 David Marchand wrote: > eal is a better place than ethdev for naming resources. > Add a helper here, and make use of it in ethdev hotplug code. > > Signed-off-by: David Marchand > --- > lib/librte_eal/common/include/rte_pci.h | 28

[dpdk-dev] [dpdk-users] DPDK i40evf problem in receving packet

2016-02-10 Thread Thomas Monjalon
Formatting note: Please avoid top posting (inline replies are preferred), and do not include such disclaimer, not relevant on a mailing list. Thanks 2016-02-10 16:00, Muhammad Zain-ul-Abideen: > Hi Saurabh, > Can you tell me what card you are uing > > > *Regards:* > Zain ul Abideen > > >

[dpdk-dev] [PATCH 8/9] pci: add a helper to refresh a device

2016-02-10 Thread Jan Viktorin
On Fri, 29 Jan 2016 15:08:35 +0100 David Marchand wrote: > It will be used mainly for hotplug code. > > Signed-off-by: David Marchand > --- > lib/librte_eal/bsdapp/eal/eal_pci.c | 49 > +++ > lib/librte_eal/common/eal_private.h | 13 ++ >

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread David Marchand
On Wed, Feb 10, 2016 at 11:20 AM, Jan Viktorin wrote: > On Wed, 10 Feb 2016 10:27:14 +0100 > David Marchand wrote: >> #define RTE_EAL_PCI_REGISTER(name, d)\ >> void pciinitfn_ ##name(void);\ >> void __attribute__((constructor, used)) pciinitfn_ ##name(void)\ >> {\ >>

[dpdk-dev] [PATCH 6/9] eal: initialize vdevs right next to pci devices

2016-02-10 Thread David Marchand
On Wed, Feb 10, 2016 at 12:05 PM, Jan Viktorin wrote: > On Fri, 29 Jan 2016 15:08:33 +0100 > David Marchand wrote: >> --- a/lib/librte_eal/common/include/rte_dev.h >> +++ b/lib/librte_eal/common/include/rte_dev.h >> @@ -156,7 +156,7 @@ void rte_eal_driver_register(struct rte_driver *driver); >>

[dpdk-dev] [PATCH 8/9] pci: add a helper to refresh a device

2016-02-10 Thread David Marchand
On Wed, Feb 10, 2016 at 12:23 PM, Jan Viktorin wrote: > On Fri, 29 Jan 2016 15:08:35 +0100 > David Marchand wrote: > >> It will be used mainly for hotplug code. >> >> Signed-off-by: David Marchand >> --- >> lib/librte_eal/bsdapp/eal/eal_pci.c | 49 >> +++ >>

[dpdk-dev] [PATCH 7/9] pci: add a helper for device name

2016-02-10 Thread David Marchand
On Wed, Feb 10, 2016 at 12:10 PM, Jan Viktorin wrote: > On Fri, 29 Jan 2016 15:08:34 +0100 > David Marchand wrote: > >> eal is a better place than ethdev for naming resources. >> Add a helper here, and make use of it in ethdev hotplug code. >> >> Signed-off-by: David Marchand >> --- >>

[dpdk-dev] [PATCH v2] hash: fix CRC32c computation

2016-02-10 Thread De Lara Guarch, Pablo
Hi Didier, > -Original Message- > From: Didier Pallard [mailto:didier.pallard at 6wind.com] > Sent: Tuesday, February 09, 2016 9:34 AM > To: dev at dpdk.org; Richardson, Bruce; De Lara Guarch, Pablo > Cc: jean-mickael.guerin at 6wind.com; thomas.monjalon at 6wind.com > Subject: [PATCH v2]

[dpdk-dev] [PATCH 8/9] pci: add a helper to refresh a device

2016-02-10 Thread Jan Viktorin
On Wed, 10 Feb 2016 13:00:50 +0100 David Marchand wrote: > On Wed, Feb 10, 2016 at 12:23 PM, Jan Viktorin > wrote: > > On Fri, 29 Jan 2016 15:08:35 +0100 > > David Marchand wrote: > > > >> It will be used mainly for hotplug code. > >> > >> Signed-off-by: David Marchand > >> --- > >>

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread Jan Viktorin
On Wed, 10 Feb 2016 12:38:20 +0100 David Marchand wrote: > On Wed, Feb 10, 2016 at 11:20 AM, Jan Viktorin > wrote: > > On Wed, 10 Feb 2016 10:27:14 +0100 > > David Marchand wrote: > >> #define RTE_EAL_PCI_REGISTER(name, d)\ > >> void pciinitfn_ ##name(void);\ > >> void

[dpdk-dev] [PATCH v4 0/6] vmxnet3 TSO, tx cksum offload and cleanups

2016-02-10 Thread Bruce Richardson
On Tue, Jan 12, 2016 at 08:56:34PM -0800, Stephen Hemminger wrote: > On Tue, 12 Jan 2016 18:08:31 -0800 > Yong Wang wrote: > > > v4: > > * moved cleanups to separate patches > > * correctly handled multi-seg pkts with data ring used > > > > v3: > > * fixed comments from Stephen > > * added

[dpdk-dev] i40e: cannot change mtu to enable jumbo frame

2016-02-10 Thread Ananyev, Konstantin
> > On 02/09/2016 08:05 PM, Zhu, Heqing wrote: > > Helin is still in Chinese New Year Vacation. Will the below command option > > help ? > > > > 4.5.9. port config - max-pkt-len > > Set the maximum packet length: > > > > testpmd> port config all max-pkt-len (value) > > This is equivalent to the

[dpdk-dev] [PATCH] virtio: fix crashes in virtio stats functions

2016-02-10 Thread Bruce Richardson
On Thu, Dec 24, 2015 at 11:48:41AM +0800, Yuanhan Liu wrote: > On Wed, Dec 23, 2015 at 09:45:19AM +, Bernard Iremonger wrote: > > This initialisation of nb_rx_queues and nb_tx_queues has been removed > > from eth_virtio_dev_init. > > > > The nb_rx_queues and nb_tx_queues were being

[dpdk-dev] [PATCH v2 2/2] doc: rename release notes 2.3 to 16.04

2016-02-10 Thread John McNamara
From: "Richardson, Bruce" Updated release documentation to reflect new numbering scheme. Signed-off-by: Bruce Richardson Signed-off-by: John McNamara --- doc/guides/rel_notes/index.rst | 2 +- doc/guides/rel_notes/release_16_04.rst | 138

[dpdk-dev] [PATCH v2 1/2] version: switch to year/month version numbers

2016-02-10 Thread John McNamara
From: "Richardson, Bruce" As discussed on list, switch numbering scheme to be based on year/month. Release 2.3 then becomes 16.04. Ref: http://dpdk.org/ml/archives/dev/2015-December/030336.html Also, added zero padding to the month so that it appear as 16.04 and

[dpdk-dev] [PATCH] hash: fix CRC32c computation

2016-02-10 Thread Pattan, Reshma
Hi, Small typo. > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Didier Pallard > Sent: Tuesday, December 22, 2015 9:35 AM > To: dev at dpdk.org; Richardson, Bruce ; De > Lara > Guarch, Pablo > Subject: [dpdk-dev] [PATCH] hash: fix CRC32c computation >

[dpdk-dev] [PATCH v2 2/2] driver/net/mpipe: fix the crash/hung issue when testpmd quits

2016-02-10 Thread Liming Sun
I split the change and resubmitted it as v3 http://dpdk.org/dev/patchwork/patch/10456/ It includes one of the two fixes. The other one is optional for now and could be submitted separately later. Thanks, Liming -Original Message- From: Bruce Richardson

[dpdk-dev] [PATCH v3] remove extra parentheses in return statement

2016-02-10 Thread Thomas Monjalon
2016-01-27 21:58, Huawei Xie: > v3 changes: > remove other extra parentheses in 'return (logical expressions)' > which checkpatch doesn't report as error > remove extra parentheses in return statement which crosses > multiple line > fix the document > > v2 changes: > add missed commit message

[dpdk-dev] [PATCH v2 1/2] driver/net/mpipe: add rte_vect.h and enable CONFIG_RTE_LIBRTE_LPM

2016-02-10 Thread Liming Sun
Thanks Bruce. I split the changes as suggested and resubmitted as below. http://dpdk.org/dev/patchwork/patch/10454/ http://dpdk.org/dev/patchwork/patch/10455/ - Liming -Original Message- From: Bruce Richardson [mailto:bruce.richard...@intel.com] Sent: Tuesday, February 09, 2016 10:56

[dpdk-dev] [PATCH] doc: drop old naming of the project

2016-02-10 Thread Thomas Monjalon
2016-02-09 17:20, Bruce Richardson: > On Mon, Feb 08, 2016 at 02:39:02PM +0100, Thomas Monjalon wrote: > > It was requested by Intel, more than one year ago, to replace the name > > "Intel DPDK" by "DPDK". > > Some references to the old name were still in some docs and code comments, > > leading

[dpdk-dev] Cannot run DPDK applications in more than one Container simultaneously

2016-02-10 Thread Pushpasis Sarkar
Hi, I have tried the stops mentioned in the mail thread below.. http://dpdk.org/ml/archives/dev/2014-July/004031.html But I could not run the same applcation in two different containers. It seems once I start the application on the first container, the /mnt/huge filesystem cannot be locked from

[dpdk-dev] [PATCH] virtio: fix rx ring descriptor starvation

2016-02-10 Thread Bruce Richardson
On Tue, Jan 05, 2016 at 07:13:04AM +, Xie, Huawei wrote: > On 12/17/2015 7:18 PM, Tom Kiely wrote: > > > > > > On 11/25/2015 05:32 PM, Xie, Huawei wrote: > >> On 11/13/2015 5:33 PM, Tom Kiely wrote: > >>> If all rx descriptors are processed while transient > >>> mbuf exhaustion is present, the

[dpdk-dev] [PATCH v2 1/2] version: switch to year/month version numbers

2016-02-10 Thread Thomas Monjalon
2016-02-10 14:33, John McNamara: > /** > * Major version number i.e. the x in x.y.z > */ > -#define RTE_VER_MAJOR 2 > +#define RTE_VER_MAJOR 16 > > /** > * Minor version number i.e. the y in x.y.z > */ > -#define RTE_VER_MINOR 3 > +#define RTE_VER_MINOR 4 > I liked the idea of Bruce

[dpdk-dev] [PATCH v2 ] ixgbe: speed up transmit

2016-02-10 Thread Bruce Richardson
On Fri, Dec 11, 2015 at 06:52:36PM +, Ananyev, Konstantin wrote: > > -Original Message- > > From: Stephen Hemminger [mailto:shemming at brocade.com] > > Sent: Friday, December 11, 2015 4:48 PM > > To: dev at dpdk.org; Zhang, Helin; Ananyev, Konstantin > > Subject: Re: [PATCH v2 ]

[dpdk-dev] i40e: cannot change mtu to enable jumbo frame

2016-02-10 Thread Zhang, Helin
> -Original Message- > From: Julien Meunier [mailto:julien.meunier at 6wind.com] > Sent: Wednesday, February 10, 2016 12:36 AM > To: Zhang, Helin ; dev at dpdk.org > Subject: i40e: cannot change mtu to enable jumbo frame > > Hello Helin, > > I tried to send jumbo frames to a i40e card.

[dpdk-dev] [PATCH v2 1/2] version: switch to year/month version numbers

2016-02-10 Thread Bruce Richardson
On Wed, Feb 10, 2016 at 04:11:40PM +0100, Thomas Monjalon wrote: > 2016-02-10 14:33, John McNamara: > > /** > > * Major version number i.e. the x in x.y.z > > */ > > -#define RTE_VER_MAJOR 2 > > +#define RTE_VER_MAJOR 16 > > > > /** > > * Minor version number i.e. the y in x.y.z > > */

[dpdk-dev] [PATCH] virtio: fix virtio_net_hdr desc pointing to the same buffer

2016-02-10 Thread Bruce Richardson
On Mon, Dec 14, 2015 at 01:58:34PM +, Xie, Huawei wrote: > > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Monday, December 14, 2015 9:45 PM > > To: Xie, Huawei > > Cc: Yuanhan Liu; dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH]

[dpdk-dev] [PATCH v2] af_packet: make the device detachable

2016-02-10 Thread Wojciech Żmuda
Hi Bruce, >The use of "deinitialization" sounds awkward Thank you for your interest. I called it deinitialization in opposition to an initialization of a device. As I'm not a native English speaker, I trust your opinion and I'll try to rephrase this. Hi Bernard, >What parameters do you use

[dpdk-dev] [PATCH] ixgbe: Discard SRIOV transparent vlan packet headers.

2016-02-10 Thread Bruce Richardson
On Thu, Dec 17, 2015 at 10:30:42AM +, Tom Kiely wrote: > Sorry for the delay in replying to this thread. I was on vacation for the > last 3 days. Please see inline for my comments. > > On 12/15/2015 02:37 PM, Ananyev, Konstantin wrote: > > > >>-Original Message- > >>From: Stephen

[dpdk-dev] [PATCH] i40e: fix inverted check for ETH_TXQ_FLAGS_NOREFCOUNT

2016-02-10 Thread Bruce Richardson
On Wed, Dec 23, 2015 at 08:42:26AM +, Zhang, Helin wrote: > > > > -Original Message- > > From: Rich Lane [mailto:rich.lane at bigswitch.com] > > Sent: Wednesday, December 23, 2015 4:08 PM > > To: dev at dpdk.org > > Cc: Zhang, Helin > > Subject: [PATCH] i40e: fix inverted check for

[dpdk-dev] [PATCH] remove redundant __func__ in PMD_INIT_LOG and PMD_RX_LOG

2016-02-10 Thread Bruce Richardson
On Mon, Dec 28, 2015 at 09:39:44AM -0800, Stephen Hemminger wrote: > On Mon, 28 Dec 2015 01:28:28 +0800 > Huawei Xie wrote: > > > > > Signed-off-by: Huawei Xie > > --- > > drivers/net/virtio/virtio_ethdev.c | 12 +--- > > drivers/net/vmxnet3/vmxnet3_ethdev.c | 6 +++--- > >

[dpdk-dev] [PATCH v2 1/2] version: switch to year/month version numbers

2016-02-10 Thread Mcnamara, John
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, February 10, 2016 3:12 PM > To: Mcnamara, John > Cc: dev at dpdk.org; Richardson, Bruce > Subject: Re: [PATCH v2 1/2] version: switch to year/month version numbers > > ... > > I liked

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Olivier MATZ
Hi Keith, Thank you for adding the RTE_NEXT_ABI. I think this is the way described in the process. Your changes will be available in next version (16.4) for people compiling with RTE_NEXT_ABI=y, and in 16.7 without option (I'm just surprised that RTE_NEXT_ABI=y in default configs...). I think a

[dpdk-dev] [PATCH v3 2/2] doc: rename release notes 2.3 to 16.04

2016-02-10 Thread John McNamara
From: "Richardson, Bruce" Updated release documentation to reflect new numbering scheme. Signed-off-by: Bruce Richardson Signed-off-by: John McNamara --- doc/guides/rel_notes/index.rst | 2 +- doc/guides/rel_notes/release_16_04.rst | 138

[dpdk-dev] [PATCH v3 1/2] version: switch to year/month version numbers

2016-02-10 Thread Mcnamara, John
> -Original Message- > From: Mcnamara, John > Sent: Wednesday, February 10, 2016 5:02 PM > To: dev at dpdk.org > Cc: thomas.monjalon at 6wind.com; Richardson, Bruce > ; Mcnamara, John > Subject: [PATCH v3 1/2] version: switch to year/month version numbers > diff --git

[dpdk-dev] [PATCH] ixgbe: fix whitespace

2016-02-10 Thread Bruce Richardson
On Wed, Jan 13, 2016 at 05:23:22AM +, Zhang, Helin wrote: > > > -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Wednesday, January 13, 2016 12:54 PM > To: Zhang, Helin ; Ananyev, Konstantin > > Cc: dev at dpdk.org; Stephen Hemminger >

[dpdk-dev] [PATCH v3 1/2] version: switch to year/month version numbers

2016-02-10 Thread Thomas Monjalon
2016-02-10 17:11, Mcnamara, John: > Hi Alejandro, > > Please note the above change to the nfp_net.c code due to the name change > in the version macro names. > > > I was originally going to change it to this: > > #if RTE_VER_YEAR == 2 && RTE_VER_MONTH >= 1 > dev_info->hash_key_size

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Wiles, Keith
>Hi Keith, > >Thank you for adding the RTE_NEXT_ABI. I think this is the way >described in the process. Your changes will be available in next >version (16.4) for people compiling with RTE_NEXT_ABI=y, and in >16.7 without option (I'm just surprised that RTE_NEXT_ABI=y in >default configs...). > >I

[dpdk-dev] Regarding mbuf allocation/free in secondary process

2016-02-10 Thread Lawrence MacIntyre
Sara: The use of polling is to provide high throughput. At high bandwidths, interrupt processing is a great contributor to latency and can greatly decrease available bandwidth. Polling eliminates the overhead of interrupts but it will consume an entire CPU. That is why the DPDK library makes

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Wiles, Keith
>>Hi Keith, >> >>Thank you for adding the RTE_NEXT_ABI. I think this is the way >>described in the process. Your changes will be available in next >>version (16.4) for people compiling with RTE_NEXT_ABI=y, and in >>16.7 without option (I'm just surprised that RTE_NEXT_ABI=y in >>default

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Thomas Monjalon
2016-02-10 18:01, Wiles, Keith: > >>> --- a/config/defconfig_x86_64-native-linuxapp-gcc > >>> +++ b/config/defconfig_x86_64-native-linuxapp-gcc > >>> @@ -40,3 +40,8 @@ CONFIG_RTE_ARCH_64=y > >>> > >>> CONFIG_RTE_TOOLCHAIN="gcc" > >>> CONFIG_RTE_TOOLCHAIN_GCC=y > >>>

[dpdk-dev] [PATCH v2] af_packet: make the device detachable

2016-02-10 Thread John W. Linville
On Wed, Feb 10, 2016 at 04:42:53PM +0100, Wojciech ?muda wrote: > Bernard, Bruce, I have a question, if I may. Do you know what is the > reason that rte_pmd_af_packet_devinit() is the only non-static device > initialization function among all the dpdk drivers? There's even a > comment in the

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Wiles, Keith
>Hi Keith, > >Thank you for adding the RTE_NEXT_ABI. I think this is the way >described in the process. Your changes will be available in next >version (16.4) for people compiling with RTE_NEXT_ABI=y, and in >16.7 without option (I'm just surprised that RTE_NEXT_ABI=y in >default configs...). > >I

[dpdk-dev] [PATCH v3] vhost: fix leak of fds and mmaps

2016-02-10 Thread Rich Lane
The common vhost code only supported a single mmap per device. vhost-user worked around this by saving the address/length/fd of each mmap after the end of the rte_virtio_memory struct. This only works if the vhost-user code frees dev->mem, since the common code is unaware of the extra info. The

[dpdk-dev] [PATCH v3] cfgfile: support looking up sections by index

2016-02-10 Thread Rich Lane
This is useful when sections have duplicate names. Signed-off-by: Rich Lane --- v2->v3 - Added check for index < 0. v1->v2: - Added new symbol to version script. lib/librte_cfgfile/rte_cfgfile.c | 16 lib/librte_cfgfile/rte_cfgfile.h | 23

[dpdk-dev] [PATCH v2] cfgfile: support looking up sections by index

2016-02-10 Thread Rich Lane
On Tue, Feb 2, 2016 at 7:10 AM, Dumitrescu, Cristian < cristian.dumitrescu at intel.com> wrote: > > > > -Original Message- > > From: Rich Lane [mailto:rich.lane at bigswitch.com] > > Sent: Tuesday, January 19, 2016 4:42 AM > > To: dev at dpdk.org > > Cc: Dumitrescu, Cristian ; Panu >

[dpdk-dev] [PATCH] vhost: remove vhost_net_device_ops

2016-02-10 Thread Rich Lane
The indirection is unnecessary because there is only one implementation of the vhost common code. Removing it makes the code more readable. Signed-off-by: Rich Lane --- examples/vhost_xen/virtio-net.h | 2 - lib/librte_vhost/vhost-net.h | 40 +---

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Olivier MATZ
Hi Keith, On 02/10/2016 07:35 PM, Wiles, Keith wrote: >>> @@ -672,6 +704,24 @@ rte_mempool_count(const struct rte_mempool *mp) >>> static unsigned >>> rte_mempool_dump_cache(FILE *f, const struct rte_mempool *mp) >>> { >>> +#ifdef RTE_NEXT_ABI >>> + unsigned lcore_id; >>> + unsigned count

[dpdk-dev] [PATCH v3] mempool: reduce rte_mempool structure size

2016-02-10 Thread Keith Wiles
The rte_mempool structure is changed, which will cause an ABI change for this structure. Providing backward compat is not reasonable here as this structure is used in multiple defines/inlines. Allow mempool cache support to be dynamic depending on if the mempool being created needs cache support.

[dpdk-dev] [dpdk-users] DPDK i40evf problem in receving packet

2016-02-10 Thread Saurabh Mishra
Hi, *i40e card is :* 00:09.0 *Ether*net controller: Intel Corporation XL710/X710 Virtual Function (rev 01) 00:0a.0 *Ether*net controller: Intel Corporation XL710/X710 Virtual Function (rev 01) firmware-version: f4.40 a1.4 n04.53 e80001dc0 Is there any extra verbose/debug in DPDK which we

[dpdk-dev] [PATCH v3 1/2] version: switch to year/month version numbers

2016-02-10 Thread Thomas Monjalon
2016-02-10 17:02, John McNamara: > From: "Richardson, Bruce" > > As discussed on list, switch numbering scheme to be based on year/month. > Release 2.3 then becomes 16.04. > > Ref: http://dpdk.org/ml/archives/dev/2015-December/030336.html > > Also, added zero padding to the month so that

[dpdk-dev] [PATCH] config: remove useless explicit includes of generated header

2016-02-10 Thread Thomas Monjalon
2016-02-08 15:38, Thomas Monjalon: > The file rte_config.h is automatically generated and included. > No need to #include it. > > The example performance-thread needs a makefile fix to avoid > overwriting the default cflags. > > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] DPDK (and rte_*alloc family) friendly Valgrind

2016-02-10 Thread Luca Boccassi
Hello all, I created a set of patches for Valgrind that add support for the rte_*alloc family of functions. We use it for memcheck (I added support for other all the other Valgrind tools like cachegrind as well, but it's less tested), and find it extremely useful, since the vanilla version cannot

[dpdk-dev] [PATCH] Correcting upstream kernel version in driver

2016-02-10 Thread John Griffin
Fixing the version of the kernel required in the QAT documentation. Signed-off-by: John Griffin --- doc/guides/cryptodevs/qat.rst | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/doc/guides/cryptodevs/qat.rst b/doc/guides/cryptodevs/qat.rst index 1901842..23402b4

[dpdk-dev] [PATCH] virtio: prettify log messages

2016-02-10 Thread Stephen Hemminger
On Thu, 11 Feb 2016 00:45:34 +0100 Vincent JARDIN wrote: > PMD_TX_LOG() looks better with a \n > > Signed-off-by: Vincent JARDIN NAK. Yes the messages, are messed up, but this is not the right way to fix it. The logging wrappers are inconsistent in virtio. PMD_INIT_LOG adds newline, but

[dpdk-dev] thoughts on DPDK after a few days of reading sources

2016-02-10 Thread Seth Arnold
Hello, The Ubuntu distribution is looking at supporting DPDK in the 'main' component of the archive. As part of this process I spent a few days reading the DPDK sources to gauge if we can support it or not. I've taken some notes while reading the sources; I'm sharing them in the hopes that it's

[dpdk-dev] thoughts on DPDK after a few days of reading sources

2016-02-10 Thread Matthew Hall
On Wed, Feb 10, 2016 at 07:05:40PM -0800, Seth Arnold wrote: > - ixgbe driver in the package is very different from the driver in the > Linux kernel -- when bugs in one are found, who is in charge of copying > the fixes between the two code bases? It's not the Linux driver. It's from BSD