[dpdk-dev] [PATCH v3] app/test: fix a segfault when lpm_perf_autotest is run more than 1 time

2016-11-06 Thread Thomas Monjalon
> > num_route_entries need to be reseted. > > > > Fixes: 17d60f5b5eea ("app/test: remove large IPv4 LPM data file") > > > > Signed-off-by: Nikita Kozlov > Tested-by: Bruce Richardson > Tested-by: Wei Dai > Acked-by: Wei Dai Appplied, thanks

[dpdk-dev] [PATCH] lpm: fix freeing memory

2016-11-06 Thread Thomas Monjalon
2016-11-03 18:15, Wei Dai: > The memory pointed by lpm->rules_tbl should also be freed > when memory malloc for tbl8 fails in rte_lpm_create_v1604( ). > And the memory pointed by lpm->tbl8 should also be freed > when the lpm object is freed in rte_lpm_free_v1604( ). > > Fixes: f1f7261838b3 ("lpm:

[dpdk-dev] [PATCH] cfgfile: fix comments - style and return value

2016-11-06 Thread Thomas Monjalon
2016-10-19 03:33, Dmitriy Yakovlev: > Fixed style and return values in Doxygen comments. > > Signed-off-by: Dmitriy Yakovlev Applied, thanks

[dpdk-dev] [PATCH] net: remove mempool as a dependency

2016-11-06 Thread Thomas Monjalon
> Signed-off-by: Ferruh Yigit Applied, thanks

[dpdk-dev] [PATCH] doc: add missing library to release notes

2016-11-06 Thread Thomas Monjalon
> > Signed-off-by: Ferruh Yigit > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH] pci: Don't call probe callback if driver already loaded.

2016-11-06 Thread Thomas Monjalon
2016-10-25 14:50, Ben Walker: > If the user asks to probe multiple times, the probe > callback should only be called on devices that don't have > a driver already loaded. > > This is useful if a driver is registered after the > execution of a program has started and the list of devices > needs to

[dpdk-dev] [PATCH v2 2/2] arch/ppc: fix file descriptor leakage when getting CPU features

2016-11-06 Thread Thomas Monjalon
2016-11-04 12:18, Jan Viktorin: > On Fri, 4 Nov 2016 15:16:43 +0530 > Jianbo Liu wrote: > > > close the file descriptor after finish using it. > > s/close/Close/ > > Please include my ack (below). > > Jan > > > > > Fixes: 9ae15538 (eal/ppc: cpu flag checks for IBM Power) > > > >

[dpdk-dev] [PATCH] eal: fix bug in x86 cmpset

2016-11-06 Thread Thomas Monjalon
2016-09-29 18:34, Thomas Monjalon: > 2016-09-30 02:54, Nikhil Rao: > > The original code used movl instead of xchgl, this caused > > rte_atomic64_cmpset to use ebx as the lower dword of the source > > to cmpxchg8b instead of the lower dword of function argument "src". > > Could you please start

[dpdk-dev] [PATCH] ethdev: rename library for consistency

2016-11-06 Thread Thomas Monjalon
2016-11-06 19:21, Thomas Monjalon: > The library was named libethdev without rte_ prefix. > It is now fixed, the library namespace is consistent. > > Note: the ABI version has already been changed in this release cycle. > > Signed-off-by: Thomas Monjalon Applied for 16.11, as announced in the

[dpdk-dev] [PATCH] scripts: remove zlib dependency to test qede build

2016-11-06 Thread Thomas Monjalon
> >The driver qede does not depend on zlib anymore. > > > >Fixes: 7d178e9ef614 ("net/qede: remove zlib dependency and enable PMD") > > > >Signed-off-by: Thomas Monjalon > > Acked-by: Harish Patil Applied

[dpdk-dev] [PATCH] scripts: remove zlib dependency to test qede build

2016-11-06 Thread Thomas Monjalon
The driver qede does not depend on zlib anymore. Fixes: 7d178e9ef614 ("net/qede: remove zlib dependency and enable PMD") Signed-off-by: Thomas Monjalon --- scripts/test-build.sh | 2 -- 1 file changed, 2 deletions(-) diff --git a/scripts/test-build.sh b/scripts/test-build.sh index

[dpdk-dev] [PATCH] ethdev: rename library for consistency

2016-11-06 Thread Thomas Monjalon
The library was named libethdev without rte_ prefix. It is now fixed, the library namespace is consistent. Note: the ABI version has already been changed in this release cycle. Signed-off-by: Thomas Monjalon --- doc/guides/rel_notes/deprecation.rst | 3 ---

[dpdk-dev] [PATCH] scripts: remove zlib dependency to test qede build

2016-11-06 Thread Harish Patil
> >The driver qede does not depend on zlib anymore. > >Fixes: 7d178e9ef614 ("net/qede: remove zlib dependency and enable PMD") > >Signed-off-by: Thomas Monjalon >--- > scripts/test-build.sh | 2 -- > 1 file changed, 2 deletions(-) > >diff --git a/scripts/test-build.sh b/scripts/test-build.sh

[dpdk-dev] [PATCH] mk: fix install-doc without html

2016-11-06 Thread Thomas Monjalon
2016-11-06 17:53, Thomas Monjalon: > When trying to install PDF, man pages or examples without having built > neither HTML API nor HTML guides, there was an error: > > % make install-doc > tar: html: Cannot stat: No such file or directory > > The fix is to check the html directory before

[dpdk-dev] [PATCH v4] eal: fix lib version for device generalization patches

2016-11-06 Thread Thomas Monjalon
2016-10-27 16:59, Shreyansh Jain: > rte_device/driver generalization patches [1] were merged without a change > in the LIBABIVER variable. This patches bumps the macro of affected libs: > > - libcryptodev and libetherdev have been bumped > - librte_eal version changed in > d7e61ad3ae36 ("log:

[dpdk-dev] [PATCH] mk: fix install-doc without html

2016-11-06 Thread Thomas Monjalon
When trying to install PDF, man pages or examples without having built neither HTML API nor HTML guides, there was an error: % make install-doc tar: html: Cannot stat: No such file or directory The fix is to check the html directory before installing HTML files. Fixes: e4552b9cc603 ("mk:

[dpdk-dev] [PATCH] lib/ip_frag: fix IP reassembly not working issue

2016-11-06 Thread Wenzhuo Lu
After changing pkt[0] to pkt[], the example IP reassembly is not working. It's weird because this change is fine. There should be no difference between them. As a workaround, revert this change. Fixes: 347a1e037fd3 (lib: use C99 syntax for zero-size arrays) Reported-by: Huilong Xu

[dpdk-dev] [PATCH] ixgbe: fix wrong RX DMA registers

2016-11-06 Thread Wenzhuo Lu
Some VF RX DMA registers are using PF's addresses by mistake. Although some of them are sharing the same addresses, we should use the right ones. Fixes: 0198848a47f5 ("ixgbe: add access to specific device info") Signed-off-by: Wenzhuo Lu --- drivers/net/ixgbe/ixgbe_regs.h | 18

[dpdk-dev] [PATCH] ixgbe: correct the VF general registers

2016-11-06 Thread Wenzhuo Lu
Some VF registers are using PF's name by mistake. Although the addresses of the VF's and PF's are the same, we should use the right names. Fixes: 0198848a47f5 ("ixgbe: add access to specific device info") Signed-off-by: Wenzhuo Lu --- drivers/net/ixgbe/ixgbe_regs.h | 6 +++--- 1 file changed,

[dpdk-dev] [PATCH] ixgbe: fix wrong VF TX registers

2016-11-06 Thread Wenzhuo Lu
The VF TX registers are using PF's addresses by mistake, correct them. Fixes: 0198848a47f5 ("ixgbe: add access to specific device info") Reported-by: Xuekun Hu Signed-off-by: Wenzhuo Lu --- drivers/net/ixgbe/ixgbe_regs.h | 16 1 file changed, 8 insertions(+), 8 deletions(-)

[dpdk-dev] [PATCH] scripts: ignore const struct checks

2016-11-06 Thread Thomas Monjalon
2016-11-05 16:42, Thomas Monjalon: > checkpatch raise some false positives when run outside of the kernel tree. > > The script checkpatch.pl from the kernel checks a list > of structs known to be const. > It is obviously not relevant as DPDK has not such list. > > Signed-off-by: Thomas Monjalon

[dpdk-dev] [PATCH] scripts: show full headline with checkpatch

2016-11-06 Thread Thomas Monjalon
2016-11-05 16:28, Thomas Monjalon: > The Subject in an email can be split on two lines. > In such case, the second line (starting with a space) can be > joined to the first one. > > It is handled differently when parsing stdin or a file. > > Signed-off-by: Thomas Monjalon Applied