[dpdk-dev] DPDK2.1 (rc3 & rc4) major performance drop.

2015-08-12 Thread Flavio Leitner
On Tue, Aug 11, 2015 at 01:10:10PM +, Mcnamara, John wrote: > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Weglicki, MichalX > > Sent: Tuesday, August 11, 2015 11:40 AM > > To: dev at dpdk.org > > Subject: [dpdk-dev] DPDK2.1 (rc3 & rc4) major performa

[dpdk-dev] ieee1588fwd.c implementation

2015-08-12 Thread Stefan Binna
Hi John, thanks very much for the big help :) I'll try it tomorrow and if I run into any problems I'll contact you again. Best regards, Stefan. Am 12.08.2015 um 19:09 schrieb Mcnamara, John: >> -Original Message- >> From: Stefan Binna [mailto:stefan.binna at salzburgresearch.at] >> Sent

[dpdk-dev] ieee1588fwd.c implementation

2015-08-12 Thread Stefan Binna
Hi John, thanks for the quick response. I don't understand how to implement ieee1588fwd. Is there any manual, sample application or similar available? Or do I have to modify the testpmd.c file by myself? I don't really know how to implement the code available in ieee1588fwd.c. Thanks, Stefan.

[dpdk-dev] Performance issues with Mellanox Connectx-3 EN

2015-08-12 Thread Xiaozhou Li
Hi folks, I am getting performance scalability issues with DPDK on Mellanox Connectx-3 . Each of our machine has 16 cores and a single-port 40G Mellanox Connectx-3 EN. We find out the server throughput *does not scale* with number of cores. With a single thread on one core, we can get about 2 Mp

[dpdk-dev] Lookup mechanim in DPDK HASH table.

2015-08-12 Thread Yeddula, Avinash
Hello All, I'm using DPDK extendable hash tables. This question is with respect to the lookup aspect of the hash table. I see that there is just one "t->key_offset" that is pre-defined for the hash table. I also understand that the frame needs to carry the "lookup_key / keys" in the meta data.

[dpdk-dev] ieee1588fwd.c implementation

2015-08-12 Thread Stefan Binna
Hi, I'm currently trying to implement the ieee1588fwd.c function into my testpmd programm. I'm using DPDK-2.0.0. Is there any installation guide for ieee1588fwd implementation, i.e. any source code where the function is already implemented? I already changed the CONFIG_RTE_LIBRTE_IEEE1588=y fla

[dpdk-dev] ieee1588fwd.c implementation

2015-08-12 Thread Mcnamara, John
> -Original Message- > From: Stefan Binna [mailto:stefan.binna at salzburgresearch.at] > Sent: Wednesday, August 12, 2015 5:42 PM > To: Mcnamara, John; dev at dpdk.org > Subject: Re: [dpdk-dev] ieee1588fwd.c implementation > > Hi John, > > > I don't understand how to implement ieee1588fw

[dpdk-dev] ieee1588fwd.c implementation

2015-08-12 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stefan Binna > Sent: Wednesday, August 12, 2015 5:01 PM > To: dev at dpdk.org > Subject: [dpdk-dev] ieee1588fwd.c implementation > > > I'm currently trying to implement the ieee1588fwd.c function into my > tes

[dpdk-dev] [PATCH v2] doc: announce ABI change of rte_eth_fdir_filter, rte_eth_fdir_masks

2015-08-12 Thread Thomas Monjalon
> > For x550 supports 2 new flow director modes, MAC VLAN and Cloud. The MAC > > VLAN mode means the MAC and VLAN are monitored. The Cloud mode is for > > VxLAN and NVGRE, and the tunnel type, TNI/VNI, inner MAC and inner VLAN > > are monitored. So, there're a few new lookup fields for these 2 new

[dpdk-dev] [PATCH v4 12/12] doc: update doc for vhost multiple queues

2015-08-12 Thread Ouyang Changchun
Update the sample guide doc for vhost multiple queues; Update the prog guide doc for vhost lib multiple queues feature; Signed-off-by: Changchun Ouyang --- It is added since v3 doc/guides/prog_guide/vhost_lib.rst | 38 doc/guides/sample_app_ug/vhost.rst | 113

[dpdk-dev] [PATCH v4 11/12] vhost: alloc core to virtq

2015-08-12 Thread Ouyang Changchun
This patch allocates the core on the granularity of virtq instead of virtio device. This allows vhost having the capability of polling different virtq with different core, which shows better performance on vhost/virtio ports with more cores. Add 2 API: rte_vhost_core_id_get and rte_vhost_core_id

[dpdk-dev] [PATCH v4 10/12] vhost: add per queue stats info

2015-08-12 Thread Ouyang Changchun
Add per queue stats info Signed-off-by: Changchun Ouyang --- Changes in v3 - fix coding style and displaying format - check stats_enable to alloc mem for queue pair Changes in v2 - fix the stats issue in tx_local - dynamically alloc mem for queue pair stats info - fix checkpatch errors

[dpdk-dev] [PATCH v4 09/12] virtio: resolve for control queue

2015-08-12 Thread Ouyang Changchun
Fix the max virtio queue pair read issue. Control queue can't work for vhost-user mulitple queue mode, so introduce a counter to void the dead loop when polling the control queue(removed in v4). Signed-off-by: Changchun Ouyang --- Changes in v4: - revert the workaround - fix the max virtio

[dpdk-dev] [PATCH v4 08/12] vhost: support multiple queues

2015-08-12 Thread Ouyang Changchun
Sample vhost leverage the VMDq+RSS in HW to receive packets and distribute them into different queue in the pool according to 5 tuples. And enable multiple queues mode in vhost/virtio layer. HW queue numbers in pool exactly same with the queue number in virtio device, e.g. rxq = 4, the queue numb

[dpdk-dev] [PATCH v4 07/12] vhost: add new command line option: rxq

2015-08-12 Thread Ouyang Changchun
Sample vhost need know the queue number user want to enable for each virtio device, so add the new option '--rxq' into it. Signed-off-by: Changchun Ouyang --- Changes in v3 - fix coding style Changes in v2 - refine help info - check if rxq = 0 - fix checkpatch errors examples/vhost/ma

[dpdk-dev] [PATCH v4 06/12] vhost: support protocol feature

2015-08-12 Thread Ouyang Changchun
Support new protocol feature to communicate with qemu: Add set and get protocol feature bits; Add VRING_FLAG for mq feature to set vring flag, which indicates the vq is enabled or disabled. Reserve values as follows: VHOST_USER_SEND_RARP = 17 (merge from qemu community) VHOST_USER_SET_VRING_FLAG =

[dpdk-dev] [PATCH v4 05/12] vhost: check the virtqueue address's validity

2015-08-12 Thread Ouyang Changchun
This is added since v3. Check the virtqueue address's validity. Signed-off-by: Changchun Ouyang --- Changes in v4: - remove unnecessary code lib/librte_vhost/vhost_user/vhost-net-user.c | 4 +++- lib/librte_vhost/virtio-net.c| 10 ++ 2 files changed, 13 insertions(+),

[dpdk-dev] [PATCH v4 04/12] vhost: set memory layout for multiple queues mode

2015-08-12 Thread Ouyang Changchun
QEMU sends separate commands orderly to set the memory layout for each queue in one virtio device, accordingly vhost need keep memory layout information for each queue of the virtio device. This also need adjust the interface a bit for function gpa_to_vva by introducing the queue index to specify

[dpdk-dev] [PATCH v4 03/12] vhost: update version map file

2015-08-12 Thread Ouyang Changchun
From: Changchun Ouyang it is added in v4. Signed-off-by: Changchun Ouyang --- lib/librte_vhost/rte_vhost_version.map | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_vhost/rte_vhost_version.map b/lib/librte_vhost/rte_vhost_version.map index 3d8709e..0bb1c0f 10064

[dpdk-dev] [PATCH v4 02/12] vhost: support multiple queues in virtio dev

2015-08-12 Thread Ouyang Changchun
Each virtio device could have multiple queues, say 2 or 4, at most 8. Enabling this feature allows virtio device/port on guest has the ability to use different vCPU to receive/transmit packets from/to each queue. In multiple queues mode, virtio device readiness means all queues of this virtio devi

[dpdk-dev] [PATCH v4 01/12] ixgbe: support VMDq RSS in non-SRIOV environment

2015-08-12 Thread Ouyang Changchun
In non-SRIOV environment, VMDq RSS could be enabled by MRQC register. In theory, the queue number per pool could be 2 or 4, but only 2 queues are available due to HW limitation, the same limit also exist in Linux ixgbe driver. Signed-off-by: Changchun Ouyang --- Changes in v2: - fix checkpatch

[dpdk-dev] [PATCH v4 00/12] Support multiple queues in vhost

2015-08-12 Thread Ouyang Changchun
This patch targets for R2.2, please ignore it in R2.1. Send them out a bit earlier just for seeking more comments. This patch set supports the multiple queues for each virtio device in vhost. Currently the multiple queues feature is supported only for vhost-user, not yet for vhost-cuse. The new

[dpdk-dev] [PATCH 1/1] test_table: added ACL table test to the list

2015-08-12 Thread Thomas Monjalon
2015-08-12 14:41, Maciej Gajdzica: > test_table_ACL wasn't invoked when running table_autotest. Added this > test to table_tests array to make it run with other table tests. > > Signed-off-by: Maciej Gajdzica [...] > --- a/app/test/test_table_tables.h > +++ b/app/test/test_table_tables.h > @@ -36

[dpdk-dev] [PATCH 1/1] test_table: fixed failing unit tests checking offset

2015-08-12 Thread Maciej Gajdzica
In commit: 1129992baa61d72c5 checking for offset alignment was removed. Unit tests wasn't updated to reflect that change. This patch changes checks with unaligned offsets to make tests pass. Signed-off-by: Maciej Gajdzica --- app/test/test_table_tables.c | 10 +- 1 file changed, 5 inse

[dpdk-dev] [PATCH 1/1] test_table: added ACL table test to the list

2015-08-12 Thread Maciej Gajdzica
test_table_ACL wasn't invoked when running table_autotest. Added this test to table_tests array to make it run with other table tests. Signed-off-by: Maciej Gajdzica --- app/test/test_table_tables.c |3 +++ app/test/test_table_tables.h |2 +- 2 files changed, 4 insertions(+), 1 deletion(

[dpdk-dev] [PATCH 1/1] test_table: added ACL table test to the list

2015-08-12 Thread Gajdzica, MaciejX T
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, August 12, 2015 2:53 PM > To: Gajdzica, MaciejX T > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/1] test_table: added ACL table test to the > list > > 2015-08-12 14:41, Maciej

[dpdk-dev] [PATCH] doc: announce ABI change for rte_eth_fdir_filter

2015-08-12 Thread Thomas Monjalon
> > To fix the FVL's flow director issue for SCTP flow, rte_eth_fdir_filter > > need to be change to support SCTP flow keys extension. Here announce the > > ABI deprecation. > > > > Signed-off-by: jingjing.wu > > Acked-by: John McNamara Reworded and applied, thanks

[dpdk-dev] [PATCH] abi: announce abi changes plan for struct rte_eth_fdir_flow_ext

2015-08-12 Thread Thomas Monjalon
2015-06-17 11:36, Jingjing Wu: > It announces the planned ABI change to support flow director filtering in VF > on v2.2. > > Signed-off-by: Jingjing Wu Reworded and applied, thanks

[dpdk-dev] [PATCH] mbuf & mempool: explicit typecast on function return

2015-08-12 Thread Sergey Balabanov
When DPDK is being compiled in C++ project using g++ then 'invalid conversion from' error appears. Added explicit typecast on function return to get rid of the error. Signed-off-by: Sergey Balabanov --- lib/librte_mbuf/rte_mbuf.h | 2 +- lib/librte_mempool/rte_mempool.h | 4 ++-- 2 files c

[dpdk-dev] [PATCH v4 03/12] vhost: update version map file

2015-08-12 Thread Panu Matilainen
On 08/12/2015 11:02 AM, Ouyang Changchun wrote: > From: Changchun Ouyang > > it is added in v4. > > Signed-off-by: Changchun Ouyang > --- > lib/librte_vhost/rte_vhost_version.map | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/librte_vhost/rte_vhost_version.map >

[dpdk-dev] [PATCH] doc: announce ABI change for old flow director APIs removing

2015-08-12 Thread Thomas Monjalon
> > APIs for flow director filters has been replaced by rte_eth_dev_filter_ctrl > > by > > previous releases. Enic, ixgbe and i40e are switched to support filter_ctrl > > APIs, so > > the old APIs are useless, and ready to be removed now. > > This patch announces the ABI change for these APIs rem

[dpdk-dev] [PATCH v3] doc: announce abi change for interrupt mode

2015-08-12 Thread Thomas Monjalon
> > The patch announces the planned ABI changes for interrupt mode. > > > > Signed-off-by: Cunming Liang > Acked-by: Helin Zhang Applied, thanks

[dpdk-dev] [PATCH v4 01/12] ixgbe: support VMDq RSS in non-SRIOV environment

2015-08-12 Thread Vincent JARDIN
On 12/08/2015 10:02, Ouyang Changchun wrote: > +#define VMDQ_RSS_RX_QUEUE_NUM_MAX 4 > + > +static int > +rte_eth_dev_check_vmdq_rss_rxq_num(__rte_unused uint8_t port_id, uint16_t > nb_rx_q) > +{ > + if (nb_rx_q > VMDQ_RSS_RX_QUEUE_NUM_MAX) > + return -EINVAL; > + return 0; > +}

[dpdk-dev] Getuid check in multiprocess_autotest

2015-08-12 Thread Aashima Arora
Hi, This snippet of code is from test_mp_secondary.c /* Run a test_pci call */ if (test_pci() != 0) { printf("PCI scan failed in secondary\n"); if (getuid() == 0) /* pci scans can fail as non-root */ { return -1; } } else printf("PCI scan su

[dpdk-dev] [PATCH] doc: simplify release notes cover

2015-08-12 Thread Thomas Monjalon
> > One hierarchical level is enough for this table of content. > > > > Use generated release number. > > > > Signed-off-by: Thomas Monjalon > > Acked-by: John McNamara Applied

[dpdk-dev] [PATCH] abi: Announce abi changes plan for vhost-user multiple queues

2015-08-12 Thread Thomas Monjalon
> It announces the planned ABI changes for vhost-user multiple queues feature > on v2.2. > > Signed-off-by: Changchun Ouyang > --- > +* The ABI changes are planned for struct virtio_net in order to support > vhost-user multiple queues feature. The upcoming release 2.1 will not contain > these

[dpdk-dev] [PATCH] vchost: Notify application of ownership change

2015-08-12 Thread Jan Kiszka
On 2015-08-12 05:34, Xie, Huawei wrote: > On 8/8/2015 1:21 AM, Jan Kiszka wrote: >> On VHOST_*_RESET_OWNER, we reinitialize the device but without telling >> the application. That will cause crashes when it continues to invoke >> vhost services on the device. Fix it by calling the destruction hook

[dpdk-dev] [PATCH] doc: simplify release notes cover

2015-08-12 Thread Mcnamara, John
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Tuesday, August 11, 2015 11:59 PM > To: Mcnamara, John > Cc: dev at dpdk.org > Subject: [PATCH] doc: simplify release notes cover > > One hierarchical level is enough for this table of content. > >

[dpdk-dev] [PATCH 2/2] doc: add mlx4 release notes to DPDK 2.1.0

2015-08-12 Thread Mcnamara, John
> -Original Message- > From: Adrien Mazarguil [mailto:adrien.mazarguil at 6wind.com] > Sent: Tuesday, August 11, 2015 3:43 PM > To: Mcnamara, John > Cc: dev at dpdk.org > Subject: [PATCH 2/2] doc: add mlx4 release notes to DPDK 2.1.0 > > Signed-off-by: Adrien Mazarguil Acked-by: John McN

[dpdk-dev] [PATCH 1/2] doc: add missing release notes for mlx4 availability in DPDK 2.0.0

2015-08-12 Thread Mcnamara, John
> -Original Message- > From: Adrien Mazarguil [mailto:adrien.mazarguil at 6wind.com] > Sent: Tuesday, August 11, 2015 3:43 PM > To: Mcnamara, John > Cc: dev at dpdk.org > Subject: [PATCH 1/2] doc: add missing release notes for mlx4 availability > in DPDK 2.0.0 > > Signed-off-by: Adrien Maz

[dpdk-dev] [PATCH] vchost: Notify application of ownership change

2015-08-12 Thread Xie, Huawei
On 8/8/2015 1:21 AM, Jan Kiszka wrote: > On VHOST_*_RESET_OWNER, we reinitialize the device but without telling > the application. That will cause crashes when it continues to invoke > vhost services on the device. Fix it by calling the destruction hook if > the device is still in use. > > Signed-o

[dpdk-dev] [PATCH] doc: simplify release notes cover

2015-08-12 Thread Thomas Monjalon
One hierarchical level is enough for this table of content. Use generated release number. Signed-off-by: Thomas Monjalon --- doc/guides/rel_notes/index.rst | 2 +- doc/guides/rel_notes/rel_description.rst | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/doc/gu