[dpdk-dev] [PATCH v4] eal: Enable Port Hotplug as default in Linux and BSD

2015-07-09 Thread Thomas Monjalon
2015-06-30 17:26, Tetsuya Mukawa: > This patch removes CONFIG_RTE_LIBRTE_EAL_HOTPLUG option, and enables it > as default in both Linux and BSD. > Also, to support port hotplug, rte_eal_pci_scan() and below missing > symbols should be exported to ethdev library. > - rte_eal_parse_devargs_str() > -

[dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline

2015-07-09 Thread Thomas Monjalon
2015-07-07 10:09, Maciej Gajdzica: > From: Jasvinder Singh > > Master pipeline is responsible for command line handling and > communicationg with all other pipelines via message queues. Removed > cmdline.c file, as its functionality will be split over multiple > pipeline files. > > Signed-off-by

[dpdk-dev] [PATCH v4] pci: save list of detached devices, and re-probe during driver unload

2015-07-09 Thread Thomas Monjalon
This patch received no more comment about a common BSD/Linux behaviour. Maybe that some documentation is needed. 2015-03-16 17:29, Thomas Monjalon: > Hi, > > 2015-03-12 14:24, Raz Amir: > > Thank you. > > Can you tell when will the patch be pushed to the source code? > > This is changing the beh

[dpdk-dev] [PATCH v2 0/2] next abi option

2015-07-09 Thread Thomas Monjalon
2015-07-08 12:50, Neil Horman: > On Wed, Jul 08, 2015 at 04:55:21PM +0200, Thomas Monjalon wrote: > > This is the second version of the NEXT_ABI policy. > > It can now be used for shared and static libraries. > > > > While updating rte.lib.mk, it appeared that some useless > > (and not consistent)

[dpdk-dev] [PATCH v3 0/6] query hash key size in byte

2015-07-09 Thread Thomas Monjalon
> > As different hardware has different hash key sizes, querying it (in byte) > > per port was asked by users. Otherwise there is no convenient way to know > > the size of hash key which should be prepared. > > > > v2 changes: > > * Disabled the code changes by default, to avoid breaking ABI compa

[dpdk-dev] [PATCH v3 00/10] vmxnet3: multisegment and bugfixes

2015-07-09 Thread Thomas Monjalon
Ping Stephen 2015-06-22 14:22, Thomas Monjalon: > 2015-03-10 14:42, Thomas Monjalon: > > 2015-03-05 16:10, Stephen Hemminger: > > > From: Stephen Hemminger > > > > > > Revised version of earlier patches. > > > Incorporate the small packet optimization > > > Add more cleanups > > > > > > Stephen

[dpdk-dev] [PATCH v3 00/11] Cuckoo hash

2015-07-09 Thread Thomas Monjalon
Bruce, what is the status of this series? 2015-06-28 23:25, Pablo de Lara: > This patchset is to replace the existing hash library with > a more efficient and functional approach, using the Cuckoo hash > method to deal with collisions. This method is based on using > two different hash functions t

[dpdk-dev] [RFC PATCHv3 0/7] add OSv support

2015-07-09 Thread Thomas Monjalon
Hi Takuya, It has been difficult to receive comments on this series during 2.1 cycle. I see 3 reasons: - few people know OSv - the series is mixing C++ support and OSv environment - you are adding a third system to support while some work is in progress to reduce differences between Linux and BS

[dpdk-dev] [PATCH v4 1/7] ether: add function to query for link state interrupt

2015-07-09 Thread Thomas Monjalon
2015-04-21 10:32, Stephen Hemminger: > Allow application to query whether link state will work. > This is also part of abstracting dependency on PCI. > > Signed-off-by: Stephen Hemminger > --- > lib/librte_ether/rte_ethdev.c | 14 ++ > lib/librte_ether/rte_ethdev.h | 12

[dpdk-dev] [PATCH v4 3/7] hv: add basic vmbus support

2015-07-09 Thread Thomas Monjalon
2015-04-21 10:32, Stephen Hemminger: > The hyper-v device driver forces the base EAL code to change > to support multiple bus types. This is done changing the pci_device > in ether driver to a generic union. > > As much as possible this is done in a backwards source compatiable > way. It will brea

[dpdk-dev] [PATCH v4 4/7] hv: uio driver

2015-07-09 Thread Thomas Monjalon
2015-04-21 10:32, Stephen Hemminger: > Add new UIO driver in kernel to support DPDK Poll Mode Driver. > > Signed-off-by: Stas Egorov > Signed-off-by: Stephen Hemminger We should not add a kernel driver in DPDK. Stephen, you worked on upstreaming things in kernel and you often say that maintaini

[dpdk-dev] [PATCH v4 6/7] hv: enable driver in common config

2015-07-09 Thread Thomas Monjalon
2015-04-21 10:32, Stephen Hemminger: > Add hyperv driver config to enable it. > > Signed-off-by: Stephen Hemminger > --- > config/common_linuxapp | 9 + It would be clearer to add a disabled config option in bsdapp with the comment that it is not supported on FreeBSD.

[dpdk-dev] [PATCH v4 5/7] hv: poll mode driver

2015-07-09 Thread Thomas Monjalon
2015-04-21 14:35, Stephen Hemminger: > On Tue, 21 Apr 2015 19:34:39 + > "Butler, Siobhan A" wrote: > > > Hi Stephen > > Will you have documentation to go along with these changes? > > Thanks > > Siobhan > > Unlikely. Microsoft or other contributors might add something > in a later version.

[dpdk-dev] [PATCH v4 5/7] hv: poll mode driver

2015-07-09 Thread Thomas Monjalon
2015-04-21 10:32, Stephen Hemminger: > From: Stephen Hemminger > > This is new Poll Mode driver for using hyper-v virtual network > interface. > > Signed-off-by: Stas Egorov > Signed-off-by: Stephen Hemminger > --- > lib/Makefile |1 + > lib/librte_pmd_hyperv/Make

[dpdk-dev] [PATCH 1/5] xen: allow choosing dom0 support at runtime

2015-07-09 Thread Thomas Monjalon
Please, could you try to rebase it?

[dpdk-dev] [PATCH 0/5] receive IRQ related patches

2015-07-09 Thread Thomas Monjalon
2015-05-18 10:40, Stephen Hemminger: > These are some of the patches to enhance the still as not yet > merged receive interrupt functionality. Have you made these comments in the interrupt mode thread? > The big piece is support of UIO-MSI interrupts which is required > to make the virtio and vmx

[dpdk-dev] [PATCH v3 04/12] eal: allow empty compile time flags

2015-07-09 Thread Thomas Monjalon
2015-07-06 16:51, Zhigang Lu: > The rte_cpu_check_supported() code breaks with a "comparison is always > false due to limited range of data type" when the compile_time_flags[] > array is empty. Assigning the array dimension to a local variable > apparently solves this. How is it related to the ti

[dpdk-dev] [PATCH v3 06/12] memzone: refactor rte_memzone_reserve() variants

2015-07-09 Thread Thomas Monjalon
2015-07-06 16:51, Zhigang Lu: > The definitions of rte_memzone_reserve_aligned() and > rte_memzone_reserve_bounded() were identical with the exception of the > bound argument passed into rte_memzone_reserve_thread_safe(). > > This patch removes this replication of code by unifying it into > rte_me

[dpdk-dev] [PATCH v3] doc: announce ABI changes planned for unified packet type

2015-07-09 Thread Wu, Jingjing
Acked-by: Jingjing Wu > -Original Message- > From: Zhang, Helin > Sent: Wednesday, July 08, 2015 1:45 AM > To: dev at dpdk.org > Cc: Liu, Jijiang; Wu, Jingjing; nhorman at tuxdriver.com; Zhang, Helin > Subject: [PATCH v3] doc: announce ABI changes planned for unified packet > type > > Th

[dpdk-dev] [PATCH v5 4/5] vhost: eventfd_link: replace copy-pasted sys_close

2015-07-09 Thread Thomas Monjalon
Ping 2015-06-17 17:24, Thomas Monjalon: > 2015-05-07 06:54, Xie, Huawei: > > On 4/16/2015 7:48 PM, Pavel Boldin wrote: > > > + /* Closing the source_fd */ > > > + ret = sys_close(eventfd_copy.source_fd); > > Pavel: > > Here we close the fd and re-install a new file on this fd later. > > sys_close

[dpdk-dev] [PATCH v9 01/12] eal: Fix coding style of eal_pci.c and eal_pci_uio.c

2015-07-09 Thread Tetsuya Mukawa
On 2015/07/09 5:30, Thomas Monjalon wrote: > 2015-07-08 09:09, Stephen Hemminger: >> On Wed, 8 Jul 2015 19:34:42 +0900 >> Tetsuya Mukawa wrote: >> >>> diff --git a/lib/librte_eal/bsdapp/eal/eal_pci.c >>> b/lib/librte_eal/bsdapp/eal/eal_pci.c >>> index 2df5c1c..a63d450 100644 >>> --- a/lib/librte

[dpdk-dev] [PATCH] [PATCH v2] librte_hash: miss parentheses in hash function

2015-07-09 Thread Thomas Monjalon
2015-07-08 16:28, Bruce Richardson: > On Wed, Jul 08, 2015 at 11:15:47PM +0800, damu wrote: > > In the no-x86 branch, a parentheses is missing. > > And need to include header to use CHAR_BIT. > > > > Signed-off-by: Bruce Liu <2103458176 at qq.com> > > Previously also reported here: http://dpdk.o

[dpdk-dev] [PATCH v4 8/8] app: add a new app proc_info

2015-07-09 Thread Thomas Monjalon
2015-07-05 18:40, Maryam Tahhan: > proc_info displays statistics information including extened stats for typo: extended > given DPDK ports and dumps the memory information for DPDK. > > Signed-off-by: Maryam Tahhan > --- > MAINTAINERS| 4 + > app/Makefile | 1 + > app

[dpdk-dev] [PATCH v3 5/8] ring: queue stats mapping set dummy implementation

2015-07-09 Thread Thomas Monjalon
2015-06-29 16:50, Tomasz Kulasek: > Per queue statistics are already implemented for ring device, but with > static mapping (stat_idx == queue_id). > > This fix is required, if you want to use ring device in test application > and is used only to point that per queue statistics are provided for th

[dpdk-dev] [PATCH v4 4/4] test-pmd: remove call to rte_eth_promiscuous_disable() from detach_port()

2015-07-09 Thread Tetsuya Mukawa
On 2015/07/08 9:42, Ouyang, Changchun wrote: >> -Original Message- >> From: Iremonger, Bernard >> Sent: Tuesday, July 7, 2015 5:18 PM >> To: dev at dpdk.org >> Cc: Ouyang, Changchun; Iremonger, Bernard >> Subject: [PATCH v4 4/4] test-pmd: remove call to >> rte_eth_promiscuous_disable() from

[dpdk-dev] [PATCH 2/6] eal/linux: remove useless casts

2015-07-09 Thread Thomas Monjalon
David, there is a 32-bit failure. See below 2015-07-07 11:00, David Marchand: > Rather than cast the huge pages number returned by get_num_hugepages, rework > this function so that it returns 0 when something goes wrong. > And no need for casts in log. > --- a/lib/librte_eal/linuxapp/eal/eal_huge

[dpdk-dev] [PATCH] doc: announce ABI change of rte_fdir_filter, rte_fdir_masks

2015-07-09 Thread Wenzhuo Lu
For x550 supports 2 new flow director modes, MAC VLAN and Cloud. There're several new lookup fields for these 2 new modes, like MAC, tunnel type, TNI, VNI. So, we have to change the ABI to support these new lookup fields. Signed-off-by: Wenzhuo Lu --- doc/guides/rel_notes/abi.rst | 1 + 1 file

[dpdk-dev] [PATCH v3] i40e:Fix the Descriptor Done check mechanism for i40e

2015-07-09 Thread Zhe Tao
If a descriptor the device drive is handling is the context descriptor, its type value will be 0x1. When using the not operator ! to do the conditional check, if the expression value is zero, the device driver will consider the transaction for this descriptor has been completed, even its DD fiel

[dpdk-dev] [PATCH v2] librte_ether: release memory in uninit function.

2015-07-09 Thread Tetsuya Mukawa
On 2015/07/08 18:59, Thomas Monjalon wrote: > 2015-07-08 09:49, Iremonger, Bernard: >> From: Tetsuya Mukawa [mailto:mukawa at igel.co.jp] >>> On 2015/07/07 19:53, Iremonger, Bernard wrote: From: Tetsuya Mukawa [mailto:mukawa at igel.co.jp] > On 2015/07/06 20:35, Qiu, Michael wrote: >>

[dpdk-dev] [PATCH v3] i40e:Fix the Descriptor Done check mechanism for i40e

2015-07-09 Thread Wu, Jingjing
Acked-by: Jingjing Wu > -Original Message- > From: Tao, Zhe > Sent: Thursday, July 09, 2015 10:58 AM > To: dev at dpdk.org > Cc: Tao, Zhe; Wu, Jingjing > Subject: [PATCH v3] i40e:Fix the Descriptor Done check mechanism for i40e > > If a descriptor the device drive is handling is the cont

[dpdk-dev] [PATCH] ixgbe: fix the issue that auto negotiation for flow control cannot be disabled

2015-07-09 Thread Ouyang, Changchun
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Wednesday, July 8, 2015 9:14 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] ixgbe: fix the issue that auto negotiation for > flow control cannot be disabled > > There's a parameter "

[dpdk-dev] [PATCH v3 04/12] eal: allow empty compile time flags

2015-07-09 Thread Tony Lu
>-Original Message- >From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] >Sent: Thursday, July 09, 2015 8:46 AM >To: Zhigang Lu >Cc: dev at dpdk.org >Subject: Re: [dpdk-dev] [PATCH v3 04/12] eal: allow empty compile time flags > >2015-07-06 16:51, Zhigang Lu: >> The rte_cpu_check_

[dpdk-dev] [PATCH v4 00/11] Introducing the TILE-Gx platform

2015-07-09 Thread Zhigang Lu
This series adds support for the EZchip TILE-Gx family of SoCs. The architecture port in itself is fairly straight forward due to its reliance on generics for the most part. In addition to adding TILE-Gx architecture specific code, this series includes a few cross-platform fixes for DPDK (cpuflag

[dpdk-dev] [PATCH v4 01/11] test: limit x86 cpuflags checks to x86 builds

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy The original code mistakenly defaulted to X86 when RTE_ARCH_PPC_64 was left undefined. This did not accomodate other non-PPC/non-X86 architectures. This patch fixes this issue. Signed-off-by: Zhigang Lu Acked-by: Bruce Richardson --- app/test/test_cpuflags.c | 6 +++-

[dpdk-dev] [PATCH v4 02/11] hash: check SSE flags only on x86 builds

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This is necessary because the required CPU flags may not be defined on other architectures. Signed-off-by: Zhigang Lu Acked-by: Bruce Richardson --- lib/librte_hash/rte_hash_crc.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/librte_hash/rte_hash_crc.h b/li

[dpdk-dev] [PATCH v4 03/11] eal: allow empty compile time flags RTE_COMPILE_TIME_CPUFLAGS

2015-07-09 Thread Zhigang Lu
When RTE_COMPILE_TIME_CPUFLAGS is empty, the rte_cpu_check_supported() code breaks with a "comparison is always false due to limited range of data type". This is because the compile_time_flags[] array is empty. Assigning the array dimension to a local variable apparently solves this. Signed-off-b

[dpdk-dev] [PATCH v4 04/11] config: remove RTE_LIBNAME definition.

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy The library name is now being pinned to "dpdk" instead of intel_dpdk, powerpc_dpdk, etc. As a result, we no longer need this config item. This patch removes it. Signed-off-by: Zhigang Lu Acked-by: Bruce Richardson --- config/common_bsdapp| 1 -

[dpdk-dev] [PATCH v4 05/11] memzone: refactor rte_memzone_reserve() variants

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy The definitions of rte_memzone_reserve_aligned() and rte_memzone_reserve_bounded() were identical with the exception of the bound argument passed into rte_memzone_reserve_thread_safe(). This patch removes this replication of code by unifying it into rte_memzone_reserve_th

[dpdk-dev] [PATCH v4 06/11] memzone: allow multiple pagesizes to be requested

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This patch extends the memzone allocator to remove the restriction that prevented callers from specifying multiple page sizes in the flags argument. In doing so, we also sanitize the free segment matching logic to get rid of architecture specific disjunctions (2MB vs 1GB

[dpdk-dev] [PATCH v4 07/11] mempool: allow config override on element alignment

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy On TILE-Gx and TILE-Mx platforms, the buffers fed into the hardware buffer manager require a 128-byte alignment. With this change, we allow configuration based override of the element alignment, and default to RTE_CACHE_LINE_SIZE if left unspecified. Signed-off-by: Zhiga

[dpdk-dev] [PATCH v4 08/11] tile: add page sizes for TILE-Gx/Mx platforms

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This patch adds a few new page sizes that are supported on the TILE-Gx and TILE-Mx platforms. Signed-off-by: Zhigang Lu --- lib/librte_eal/common/eal_common_memzone.c | 8 + lib/librte_eal/common/include/rte_memory.h | 16 + lib/librte_eal/common/include/

[dpdk-dev] [PATCH v4 09/11] tile: initial TILE-Gx support.

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This commit adds support for the TILE-Gx platform, as well as the TILE CPU architecture. This architecture port is fairly simple due to its reliance on generics for most arch stuff. Signed-off-by: Zhigang Lu --- config/defconfig_tile-tilegx-linuxapp-gcc | 69 +

[dpdk-dev] [PATCH v4 10/11] tile: Add TILE-Gx mPIPE poll mode driver.

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This commit adds a poll mode driver for the mPIPE hardware present on TILE-Gx SoCs. Signed-off-by: Zhigang Lu --- config/defconfig_tile-tilegx-linuxapp-gcc |1 + drivers/net/Makefile |1 + drivers/net/mpipe/Makefile| 46 +

[dpdk-dev] [PATCH v4 11/11] maintainers: claim responsibility for TILE-Gx platform

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy Signed-off-by: Zhigang Lu --- MAINTAINERS | 4 1 file changed, 4 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 5476a73..6ffa01b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -114,6 +114,10 @@ M: Bruce Richardson M: Konstantin Ananyev F: lib/librte_

[dpdk-dev] [PATCH] doc: Fix doxygen comments of rte_eth_dev_close() and rte_eth_dev_detach()

2015-07-09 Thread Tetsuya Mukawa
The patch fixes doxygen comments of below functions. - rte_eth_dev_close() Add description about what kind of resources will be freed. - rte_eth_dev_detach() Add description about when the function can be called. Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.h | 4 1

[dpdk-dev] [PATCH] i40e: Fix the issue when resetting the stats for i40e, the stats information will be lost before next time getting the stats

2015-07-09 Thread Zhe Tao
Issue: When calling the reset API to reset all the stats for i40e, all the stats will be untracked before the next time calling get stats API. Once calling the get stats API it will show all the stats equal zero. Changes: Calling the get stats API in the reset API will force the stats_offset upda

[dpdk-dev] [PATCH v3 04/12] eal: allow empty compile time flags

2015-07-09 Thread damu
-- Original -- From: "Tony Lu";; Date: Jul 9, 2015 To: "'Thomas Monjalon'"; Cc: "dev"; Subject: Re: [dpdk-dev] [PATCH v3 04/12] eal: allow empty compile time flags >-Original Message- >From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] >Sen

[dpdk-dev] Exception Path issue with AF Packet PMD for fragmented packets

2015-07-09 Thread Prashant Upadhyaya
Hi, I have a DPDK based application where the core 0 is handling the exception path and rest of the cores are bringing in the data from the NIC via the PMD. For the exception path handling I use the tun/tap interface. So the flow is like this ? Fast path cores bring in the data from NIC,

[dpdk-dev] [PATCH] doc: Fix doxygen comments of rte_eth_dev_close() and rte_eth_dev_detach()

2015-07-09 Thread Thomas Monjalon
2015-07-09 14:46, Tetsuya Mukawa: > The patch fixes doxygen comments of below functions. > - rte_eth_dev_close() >Add description about what kind of resources will be freed. > - rte_eth_dev_detach() >Add description about when the function can be called. > > Signed-off-by: Tetsuya Mukawa

[dpdk-dev] [PATCH] doc: announce ABI change of rte_fdir_filter, rte_fdir_masks

2015-07-09 Thread Thomas Monjalon
2015-07-09 10:47, Wenzhuo Lu: > +* The ABI changes are planned for struct rte_fdir_filter and rte_fdir_masks > in order to support new flow director modes, MAC VLAN and Cloud on x550. The > upcoming release 2.1 will not contain these ABI changes, but release 2.2 > will, and no backwards compatib

[dpdk-dev] How to prevent KNI interface from getting deleted on application termination?

2015-07-09 Thread Marc Sune
On 09/07/15 08:36, Gopakumar Choorakkot Edakkunni wrote: > Reading through the KNI module source, doesnt look like there is a way > to do this. For my requirement, I will make some patch tomorrow to > have a module option to just keep the KNI data structures around even > if /dev/kni is closed, l

[dpdk-dev] [PATCH v3 00/11] Cuckoo hash

2015-07-09 Thread Bruce Richardson
On Thu, Jul 09, 2015 at 01:23:54AM +0200, Thomas Monjalon wrote: > Bruce, what is the status of this series? > The parts of the series can that are independent of the cuckoo hash update have already been sent out as updates so changes to those can quicker be resolved and merged. Pablo should be s

[dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline

2015-07-09 Thread Gajdzica, MaciejX T
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, July 09, 2015 12:33 AM > To: Gajdzica, MaciejX T; Singh, Jasvinder > Cc: dev at dpdk.org; Dumitrescu, Cristian > Subject: Re: [dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline >

[dpdk-dev] [PATCH] doc: Fix doxygen comments of rte_eth_dev_close() and rte_eth_dev_detach()

2015-07-09 Thread Tetsuya Mukawa
On 2015/07/09 16:28, Thomas Monjalon wrote: > 2015-07-09 14:46, Tetsuya Mukawa: >> The patch fixes doxygen comments of below functions. >> - rte_eth_dev_close() >>Add description about what kind of resources will be freed. >> - rte_eth_dev_detach() >>Add description about when the functio

[dpdk-dev] [PATCH] hash: move rte_hash structure to C file and make it internal

2015-07-09 Thread Bruce Richardson
On Wed, Jul 08, 2015 at 09:57:03AM -0700, Matthew Hall wrote: > On Wed, Jul 08, 2015 at 02:21:42PM +0100, Bruce Richardson wrote: > > Irrespective of whether or not we change the underlying hash table > > implementation > > this looks a good change to me. The rte_hash structure should not be used

[dpdk-dev] [PATCH 2/2] pipeline: added missing symbols to map file

2015-07-09 Thread Maciej Gajdzica
Exported symbols for port in, port out and table stats. Signed-off-by: Maciej Gajdzica --- lib/librte_pipeline/rte_pipeline_version.map | 10 ++ 1 file changed, 10 insertions(+) diff --git a/lib/librte_pipeline/rte_pipeline_version.map b/lib/librte_pipeline/rte_pipeline_version.map i

[dpdk-dev] [PATCH 1/2] port: added missing symbols to map file

2015-07-09 Thread Maciej Gajdzica
Exported symbols for nodrop, fragmentation, reassembly ports Signed-off-by: Maciej Gajdzica --- lib/librte_port/rte_port_version.map | 11 +++ 1 file changed, 11 insertions(+) diff --git a/lib/librte_port/rte_port_version.map b/lib/librte_port/rte_port_version.map index f8abbc1..b7ac

[dpdk-dev] [PATCH v2] doc: Fix doxygen comments of rte_eth_dev_close() and rte_eth_dev_detach()

2015-07-09 Thread Tetsuya Mukawa
The patch fixes doxygen comments of below functions. - rte_eth_dev_close() Add description about when the function can be called and also about what kind of resources will be freed. - rte_eth_dev_detach() Add description about when the function can be called. Signed-off-by: Tetsuya Muka

[dpdk-dev] [PATCH v5 00/11] Introducing the TILE-Gx platform

2015-07-09 Thread Zhigang Lu
This series adds support for the EZchip TILE-Gx family of SoCs. The architecture port in itself is fairly straight forward due to its reliance on generics for the most part. In addition to adding TILE-Gx architecture specific code, this series includes a few cross-platform fixes for DPDK (cpuflag

[dpdk-dev] [PATCH v5 01/11] test: limit x86 cpuflags checks to x86 builds

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy The original code mistakenly defaulted to X86 when RTE_ARCH_PPC_64 was left undefined. This did not accomodate other non-PPC/non-X86 architectures. This patch fixes this issue. Signed-off-by: Cyril Chemparathy Signed-off-by: Zhigang Lu Acked-by: Bruce Richardson ---

[dpdk-dev] [PATCH v5 02/11] hash: check SSE flags only on x86 builds

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This is necessary because the required CPU flags may not be defined on other architectures. Signed-off-by: Cyril Chemparathy Signed-off-by: Zhigang Lu Acked-by: Bruce Richardson --- lib/librte_hash/rte_hash_crc.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/l

[dpdk-dev] [PATCH v5 03/11] eal: allow empty compile time flags RTE_COMPILE_TIME_CPUFLAGS

2015-07-09 Thread Zhigang Lu
When RTE_COMPILE_TIME_CPUFLAGS is empty, the rte_cpu_check_supported() code breaks with a "comparison is always false due to limited range of data type". This is because the compile_time_flags[] array is empty. Assigning the array dimension to a local variable apparently solves this. Signed-off-b

[dpdk-dev] [PATCH v5 04/11] config: remove RTE_LIBNAME definition.

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy The library name is now being pinned to "dpdk" instead of intel_dpdk, powerpc_dpdk, etc. As a result, we no longer need this config item. This patch removes it. Signed-off-by: Cyril Chemparathy Signed-off-by: Zhigang Lu Acked-by: Bruce Richardson --- config/common_bs

[dpdk-dev] [PATCH v5 05/11] memzone: refactor rte_memzone_reserve() variants

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy The definitions of rte_memzone_reserve_aligned() and rte_memzone_reserve_bounded() were identical with the exception of the bound argument passed into rte_memzone_reserve_thread_safe(). This patch removes this replication of code by unifying it into rte_memzone_reserve_th

[dpdk-dev] [PATCH v5 06/11] memzone: allow multiple pagesizes to be requested

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This patch extends the memzone allocator to remove the restriction that prevented callers from specifying multiple page sizes in the flags argument. In doing so, we also sanitize the free segment matching logic to get rid of architecture specific disjunctions (2MB vs 1GB

[dpdk-dev] [PATCH v5 07/11] mempool: allow config override on element alignment

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy On TILE-Gx and TILE-Mx platforms, the buffers fed into the hardware buffer manager require a 128-byte alignment. With this change, we allow configuration based override of the element alignment, and default to RTE_CACHE_LINE_SIZE if left unspecified. Signed-off-by: Cyril

[dpdk-dev] [PATCH v5 08/11] tile: add page sizes for TILE-Gx/Mx platforms

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This patch adds a few new page sizes that are supported on the TILE-Gx and TILE-Mx platforms. Signed-off-by: Cyril Chemparathy Signed-off-by: Zhigang Lu --- lib/librte_eal/common/eal_common_memzone.c | 8 + lib/librte_eal/common/include/rte_memory.h | 16 +--

[dpdk-dev] [PATCH v5 09/11] tile: initial TILE-Gx support.

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This commit adds support for the TILE-Gx platform, as well as the TILE CPU architecture. This architecture port is fairly simple due to its reliance on generics for most arch stuff. Signed-off-by: Cyril Chemparathy Signed-off-by: Zhigang Lu --- config/defconfig_tile-t

[dpdk-dev] [PATCH v5 10/11] tile: Add TILE-Gx mPIPE poll mode driver.

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy This commit adds a poll mode driver for the mPIPE hardware present on TILE-Gx SoCs. Signed-off-by: Cyril Chemparathy Signed-off-by: Zhigang Lu --- config/defconfig_tile-tilegx-linuxapp-gcc |1 + drivers/net/Makefile |1 + drivers/net/mpipe/

[dpdk-dev] [PATCH v5 11/11] maintainers: claim responsibility for TILE-Gx platform

2015-07-09 Thread Zhigang Lu
From: Cyril Chemparathy Signed-off-by: Cyril Chemparathy Signed-off-by: Zhigang Lu --- MAINTAINERS | 4 1 file changed, 4 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 5476a73..6ffa01b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -114,6 +114,10 @@ M: Bruce Richardson M: K

[dpdk-dev] [PATCH] doc: announce ABI change of rte_fdir_filter, rte_fdir_masks

2015-07-09 Thread Lu, Wenzhuo
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, July 9, 2015 3:33 PM > To: Lu, Wenzhuo > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] doc: announce ABI change of rte_fdir_filter, > rte_fdir_masks > > 2015-07-09 10:47

[dpdk-dev] [PATCH] test/mempool: decrease size of requested mempool

2015-07-09 Thread Olivier Matz
In test application, the default size of allocated mempool is calculated as following: (RTE_MAX_LCORE * (RTE_MEMPOOL_CACHE_MAX_SIZE + max_kept_objects)) - 1 The objective is to ensure that all cores can fill their cache and keep 'max_kept_objects' at the same time. As RTE_MAX_LCORE is 128 and R

[dpdk-dev] [PATCH v2 0/6] eal/linux: cleanup hugepage code

2015-07-09 Thread David Marchand
Here is a little patchset cleaning eal_hugepage_info.c file and fixing a bug when having more than two hugepage sizes. No functional change to be expected. Changes since v1 - fix build on 32 bits system -- David Marchand David Marchand (6): eal/linux: remove useless check on process type ea

[dpdk-dev] [PATCH v2 1/6] eal/linux: remove useless check on process type

2015-07-09 Thread David Marchand
The code in eal_hugepage_info.c is not reachable by secondary processes. Signed-off-by: David Marchand Acked-by: Sergio Gonzalez Monroy --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/lib/librte_eal/lin

[dpdk-dev] [PATCH v2 2/6] eal/linux: remove useless casts

2015-07-09 Thread David Marchand
Rather than cast the huge pages number returned by get_num_hugepages, rework this function so that it returns 0 when something goes wrong. And no need for casts in log. Signed-off-by: David Marchand Acked-by: Sergio Gonzalez Monroy --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 26

[dpdk-dev] [PATCH v2 3/6] eal/linux: cosmetic change

2015-07-09 Thread David Marchand
Prepare for checkpatch compliance. Signed-off-by: David Marchand Acked-by: Sergio Gonzalez Monroy --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 28 +-- 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal_hugepage_info.c

[dpdk-dev] [PATCH v2 4/6] eal/linux: rework while loop

2015-07-09 Thread David Marchand
Replace this while loop with a for loop and simplify error handling. Indent is broken on purpose, fixed in next commit. Signed-off-by: David Marchand Acked-by: Sergio Gonzalez Monroy --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 28 --- 1 file changed, 15 insertion

[dpdk-dev] [PATCH v2 5/6] eal/linux: indent file

2015-07-09 Thread David Marchand
With this, we should be checkpatch compliant. Signed-off-by: David Marchand Acked-by: Sergio Gonzalez Monroy --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 73 --- 1 file changed, 39 insertions(+), 34 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal_hugepa

[dpdk-dev] [PATCH v2 6/6] eal/linux: avoid out of bound access

2015-07-09 Thread David Marchand
Using IBM advance toolchain on Ubuntu 14.04 (package 8.0-3), gcc is complaining about out of bound accesses. CC eal_hugepage_info.o lib/librte_eal/linuxapp/eal/eal_hugepage_info.c: In function ?eal_hugepage_info_init?: lib/librte_eal/linuxapp/eal/eal_hugepage_info.c:350:35: error: array subscrip

[dpdk-dev] [PATCH 1/3] cxgbe: Fix RX performance for cxgbe PMD.

2015-07-09 Thread Rahul Lakkireddy
Hi Thomas, On Tue, Jul 07, 2015 at 23:30:38 +0200, Thomas Monjalon wrote: > 2015-07-07 22:42, Rahul Lakkireddy: > > CXGBE PMD rx allocates a new mbuf everytime, which could lead to performance > > hit. Instead, do bulk allocation of mbufs and re-use them. > > > > Also, simplify the overall rx-ha

[dpdk-dev] [PATCH 2/2] pipeline: added missing symbols to map file

2015-07-09 Thread Jastrzebski, MichalX K
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maciej Gajdzica > Sent: Thursday, July 09, 2015 10:17 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 2/2] pipeline: added missing symbols to map file > > Exported symbols for port in, port out and table s

[dpdk-dev] [PATCH 1/2] port: added missing symbols to map file

2015-07-09 Thread Jastrzebski, MichalX K
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maciej Gajdzica > Sent: Thursday, July 09, 2015 10:17 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 1/2] port: added missing symbols to map file > > Exported symbols for nodrop, fragmentation, reassembly

[dpdk-dev] [PATCH v3 5/8] ring: queue stats mapping set dummy implementation

2015-07-09 Thread Kulasek, TomaszX
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, July 9, 2015 03:58 > To: Kulasek, TomaszX > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 5/8] ring: queue stats mapping set dummy > implementation > > 2015-06-29 16:50, Tomasz Kulasek: > > Per queue statisti

[dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline

2015-07-09 Thread Gajdzica, MaciejX T
> -Original Message- > From: Gajdzica, MaciejX T > Sent: Thursday, July 09, 2015 10:03 AM > To: 'Thomas Monjalon'; Singh, Jasvinder > Cc: dev at dpdk.org; Dumitrescu, Cristian > Subject: RE: [dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline > > > -Original Message- >

[dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline

2015-07-09 Thread Dumitrescu, Cristian
> -Original Message- > From: Gajdzica, MaciejX T > Sent: Thursday, July 9, 2015 12:58 PM > To: Thomas Monjalon; Singh, Jasvinder > Cc: dev at dpdk.org; Dumitrescu, Cristian > Subject: RE: [dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline > > > > > -Original Message

[dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline

2015-07-09 Thread Gajdzica, MaciejX T
> -Original Message- > From: Dumitrescu, Cristian > Sent: Thursday, July 09, 2015 12:01 PM > To: Gajdzica, MaciejX T; Thomas Monjalon; Singh, Jasvinder > Cc: dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline > > > > > -Original Message---

[dpdk-dev] dev Digest, Vol 45, Issue 9

2015-07-09 Thread Bruce Richardson
On Wed, Jul 08, 2015 at 11:18:04PM +, Assaad, Sami (Sami) wrote: > Hello Bruce, > > As you stated previously, I cannot guarantee the proper mapping of the > virtual addresses between my primary and secondary processes. I have one > primary process and up to 42 secondary processes. The appli

[dpdk-dev] [PATCH v3 5/8] ring: queue stats mapping set dummy implementation

2015-07-09 Thread Thomas Monjalon
2015-07-09 09:55, Kulasek, TomaszX: > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Thursday, July 9, 2015 03:58 > > To: Kulasek, TomaszX > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v3 5/8] ring: queue stats mapping set dummy > > implementation > > > > 2015

[dpdk-dev] [PATCH] test/mempool: decrease size of requested mempool

2015-07-09 Thread Bruce Richardson
On Thu, Jul 09, 2015 at 11:01:29AM +0200, Olivier Matz wrote: > In test application, the default size of allocated mempool is calculated > as following: > > (RTE_MAX_LCORE * (RTE_MEMPOOL_CACHE_MAX_SIZE + max_kept_objects)) - 1 > > The objective is to ensure that all cores can fill their cache a

[dpdk-dev] [PATCH 1/2] port: added missing symbols to map file

2015-07-09 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maciej Gajdzica > Sent: Thursday, July 9, 2015 11:17 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 1/2] port: added missing symbols to map file > > Exported symbols for nodrop, fragmentation, reassembl

[dpdk-dev] [PATCH 2/2] pipeline: added missing symbols to map file

2015-07-09 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maciej Gajdzica > Sent: Thursday, July 9, 2015 11:17 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 2/2] pipeline: added missing symbols to map file > > Exported symbols for port in, port out and table

[dpdk-dev] [PATCH v3] Improve hash unit tests - Cuckoo hash part 2

2015-07-09 Thread Pablo de Lara
This patch improves the unit tests for the hash library, showing more readable results, in preparation for the new hash implementation to show performance differences. Changes in v3: - Increase table utilization in performance unit tests Changes in v2: - Add missing error check - Simplify avera

[dpdk-dev] [PATCH v3] test/hash: improve hash unit tests

2015-07-09 Thread Pablo de Lara
Add new unit test for calculating the average table utilization, using random keys, based on number of entries that can be added until we encounter one that cannot be added (bucket if full) Also, replace current hash_perf unit test to see performance more clear. The current hash_perf unit test tak

[dpdk-dev] [PATCH v2 0/6] eal/linux: cleanup hugepage code

2015-07-09 Thread Thomas Monjalon
2015-07-09 11:19, David Marchand: > Here is a little patchset cleaning eal_hugepage_info.c file and fixing a bug > when having more than two hugepage sizes. > No functional change to be expected. > > Changes since v1 > - fix build on 32 bits system Applied, thanks

[dpdk-dev] [PATCH] mbuf: clarify comment on adding TX offload flags

2015-07-09 Thread Bruce Richardson
The comment for TX offload flags stated that those flags started at bit 55 and then were added to the right of that, leaving 8 bits reserved for generic mbuf (i.e. non-offload) use. This comment may not have been clear as 5 of the 8 flags which were reserved have now been used for TX offloads. This

[dpdk-dev] [PATCH v4 0/7] ethdev: add support for ieee1588 timestamping

2015-07-09 Thread John McNamara
This patchset adds ethdev API to enable and read IEEE1588/802.1AS PTP timestamps from devices that support it. The following functions are added: rte_eth_timesync_enable() rte_eth_timesync_disable() rte_eth_timesync_read_rx_timestamp() rte_eth_timesync_read_tx_timestamp() The "iee

[dpdk-dev] [PATCH v4 1/7] ethdev: add support for ieee1588 timestamping

2015-07-09 Thread John McNamara
Add ethdev API to enable and read IEEE1588/802.1AS PTP timestamps from devices that support it. The following functions are added: rte_eth_timesync_enable() rte_eth_timesync_disable() rte_eth_timesync_read_rx_timestamp() rte_eth_timesync_read_tx_timestamp() Signed-off-by: John McN

[dpdk-dev] [PATCH v4 3/7] e1000: add support for ieee1588 timestamping

2015-07-09 Thread John McNamara
Add e1000/igb support for new ethdev APIs to enable and read IEEE1588 PTP timestamps. Signed-off-by: John McNamara --- drivers/net/e1000/igb_ethdev.c | 115 + 1 file changed, 115 insertions(+) diff --git a/drivers/net/e1000/igb_ethdev.c b/drivers/net/e100

[dpdk-dev] [PATCH v4 4/7] ixgbe: add support for ieee1588 timestamping

2015-07-09 Thread John McNamara
Add ixgbe support for new ethdev APIs to enable and read IEEE1588 PTP timestamps. Signed-off-by: John McNamara --- drivers/net/ixgbe/ixgbe_ethdev.c | 122 +++ 1 file changed, 122 insertions(+) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/

[dpdk-dev] [PATCH v4 7/7] doc: document ieee1588 forwarding mode

2015-07-09 Thread John McNamara
Document the optional ieee1588 forwarding mode. Signed-off-by: John McNamara --- doc/guides/testpmd_app_ug/run_app.rst | 2 +- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/doc/guides/testpmd_app_ug/run_app.rst b/doc/guid

  1   2   >