[dpdk-dev] [PATCH v2 0/2] lpm6: speed improvement on delete rule

2016-09-16 Thread Nikita Kozlov
On 08/25/2016 00:59, Nikita Kozlov wrote: > This serie of pathes focus on improving the speed of deleting rules in lpm6. > > It also contains some other improvement like having a dynamic number of > rules in lpm6 and increasing the lpm6 nexthop size to 16bit for matching > the nexth

[dpdk-dev] LPM6 next hop size

2016-09-19 Thread Nikita Kozlov
On 09/19/2016 22:04, Shyam Sundar Govindaraj wrote: > Hi > > In IPv4 lpm implementation, the next hop size is increased from 8-bit to > 24-bit. Is there a plan to increase IPv6 lpm next hop size? > > Also, next hop size in this document needs to be updated, since it is not 1 > byte anymore. > htt

[dpdk-dev] LPM6 next hop size

2016-09-22 Thread Nikita Kozlov
On 09/21/2016 19:29, Matthew Hall wrote: > On Tue, Sep 20, 2016 at 10:11:04PM +0200, Thomas Monjalon wrote: >> Please, will you help reviewing this patch? > Sure. Sorry for being late to reply, I was a bit busy. > > 1. It adds a dependency on libbsd on Linux: bsd/sys/tree.h. Is this an > expecte

[dpdk-dev] [PATCH] Add CONFIG_RTE_KNI_KMOD build option.

2015-08-03 Thread Nikita Kozlov
This option permit to build librte_kni.so without building rte_kni.ko so you can build a sdk without building kernel drivers. Signed-off-by: Nikita Kozlov --- config/common_linuxapp | 1 + lib/librte_eal/linuxapp/Makefile | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff

[dpdk-dev] [PATCH 2/4] test: change lpm routes file from header to data file

2016-06-14 Thread Nikita Kozlov
Change the file extension of the test_lpm_routes file from .h to .dat. This makes the lines-of-code counts for DPDK more realistic as they are not affected by the huge counts from the lpm data. Signed-off-by: Nikita Kozlov Signed-off-by: Bruce Richardson --- app/test/Makefile

[dpdk-dev] [PATCH 1/4] test: make all lpm routes be of unsigned type

2016-06-14 Thread Nikita Kozlov
The one route that was different to the others in the test_lpm_routes.h file was the entry "{0, 8}" which was the only route without a "U" after the IP part. Add in the extra "U" to that entry so that it can be used as a check character when parsing routes manually.

[dpdk-dev] [PATCH 0/4] Convert lpm data from header to resource

2016-06-14 Thread Nikita Kozlov
%) python:1800 (0.34%) sh:1341 (0.25%) generated using David A. Wheeler's 'SLOCCount' [1] http://dpdk.org/ml/archives/dev/2016-April/038145.html Nikita Kozlov (4): test: make all lpm routes be of unsigned type test: change lpm routes file from header to data file

[dpdk-dev] [PATCH 4/4] test: change lpm perf test to use routes as resource

2016-06-14 Thread Nikita Kozlov
Change the lpm perf autotest to use the routes data from the resource data stored in the binary rather than including it directly into the C file as a C header. This speeds up compile and link time, without changing the test results. Signed-off-by: Nikita Kozlov Signed-off-by: Bruce Richardson

[dpdk-dev] [PATCH 3/4] test: change lpm test to use routes as resource

2016-06-14 Thread Nikita Kozlov
Change the lpm autotest to use the routes data from the resource data stored in the binary rather than including it directly into the C file as a C header. This speeds up compile and link time, without changing the test results. Signed-off-by: Nikita Kozlov Signed-off-by: Bruce Richardson

[dpdk-dev] [PATCH] lib/lpm:fix two issues in the delete_depth_small()

2015-10-28 Thread Nikita Kozlov
On 10/28/2015 03:40 PM, Bruce Richardson wrote: > On Wed, Oct 28, 2015 at 11:44:15AM +0800, Jijiang Liu wrote: >> Fix two issues in the delete_depth_small() function. >> >> 1> The control is not strict in this function. >> >> In the following structure, >> struct rte_lpm_tbl24_entry { >>

[dpdk-dev] Release of Packet Journey

2015-10-29 Thread Nikita Kozlov
Hello, We have opensourced our dpdk-based project, Packet Journey https://github.com/Gandi/packet-journey . Packet Journey is a combinationof Linux RT_NETLINK and severalparts ofDPDK (rte_kni, rte_lpm, rte_acl, rte_cmdline) and is intended to serveas an edge router. Our use case is: - pktj start

[dpdk-dev] libdpdk upstream changes for ecosystem best practices

2015-09-03 Thread Nikita Kozlov
On 09/02/2015 03:49 PM, Robie Basak wrote: > Hi, > > We?re looking at packaging DPDK in Ubuntu. We?d like to discuss upstream > changes to better integrate DPDK into Linux distributions. > ... > > Thanks, > > Robie Hello, For your interest, we have also done some kind of packaging of dpdk for debi

[dpdk-dev] [PATCH 0/3] lpm6: speed improvement on delete rule

2016-06-09 Thread Nikita Kozlov
same test. This patch was written in collaboration with Baptiste Daroussin from Gandi. Nikita Kozlov (3): lpm6 speed inmprovement on delete rule librte_eal: Import FreeBSD sys/tree.h into librte_eal/common test_lpm6: make test_lpm6* compatible with the new rte_lpm6.c lib app/test

[dpdk-dev] [PATCH 3/3] test_lpm6: make test_lpm6* compatible with the new rte_lpm6.c lib

2016-06-09 Thread Nikita Kozlov
Modify of test_lpm6.c to reflect that we no longer have a maximum number of rules. Check in some places that we are using the same number of tbl8 as the previous implementation after a rte_lpm6_delete. Signed-off-by: Nikita Kozlov --- app/test/test_lpm6.c | 131

[dpdk-dev] [PATCH 1/3] lpm6 speed inmprovement on delete rule

2016-06-09 Thread Nikita Kozlov
written in collaboration with Baptiste Daroussin from Gandi. Signed-off-by: Nikita Kozlov --- lib/librte_lpm/Makefile| 2 +- lib/librte_lpm/rte_lpm6.c | 576 ++--- lib/librte_lpm/rte_lpm6.h | 49 +++- lib/librte_lpm

[dpdk-dev] [PATCH 2/3] librte_eal: Import FreeBSD sys/tree.h into librte_eal/common

2016-06-09 Thread Nikita Kozlov
This structure is used inside the rte_lpm6 lib for storing added rules. It's imported from FreeBSD-10.3 from /usr/include/sys/tree.h, another solution could have been to use on Linux the version from libbsd but it would create an external dependency. Signed-off-by: Nikita Kozlov ---

[dpdk-dev] [PATCH 2/3] librte_eal: Import FreeBSD sys/tree.h into librte_eal/common

2016-06-09 Thread Nikita Kozlov
On 06/ 9/16 02:58 AM, Stephen Hemminger wrote: > On Thu, 9 Jun 2016 02:53:53 +0200 > Nikita Kozlov wrote: > >> This structure is used inside the rte_lpm6 lib for storing added rules. >> It's imported from FreeBSD-10.3 from /usr/include/sys/tree.h, another >> so

[dpdk-dev] no packets with i40e when setting up more than 1 rx queue

2015-06-02 Thread Nikita Kozlov
Hello, To introduce myself, I am working on a slightly enhanced version of l3fwd which support some more features like dynamically adding new routes in a lpm table, adding some acls, rate-limiting per dest, a command line through unixsock etc. We hope to open source it as soon as it is shiny enoug

[dpdk-dev] [PATCH v2 0/2] lpm6: speed improvement on delete rule

2016-08-25 Thread Nikita Kozlov
same test. This patch was written in collaboration with Baptiste Daroussin from Gandi. Changes since V1: - use system bsd-tree.h - fix a bug when valid_group field was overwritten Nikita Kozlov (2): lpm6: speed inmprovement on delete rule test_lpm6: make test_lpm6* compatible with the new

[dpdk-dev] [PATCH v2 2/2] test_lpm6: make test_lpm6* compatible with the new rte_lpm6.c lib

2016-08-25 Thread Nikita Kozlov
Modify of test_lpm6.c to reflect that we no longer have a maximum number of rules. Check in some places that we are using the same number of tbl8 as the previous implementation after a rte_lpm6_delete. Signed-off-by: Nikita Kozlov Signed-off-by: Baptiste Daroussin --- app/test/test_lpm6.c

[dpdk-dev] [PATCH v2 1/2] lpm6: speed inmprovement on delete rule

2016-08-25 Thread Nikita Kozlov
written in collaboration with Baptiste Daroussin from Gandi. Signed-off-by: Nikita Kozlov Signed-off-by: Baptiste Daroussin --- lib/librte_lpm/Makefile| 2 +- lib/librte_lpm/rte_lpm6.c | 626 +++-- lib/librte_lpm/rte_lpm6.h | 50

[dpdk-dev] lpm patches

2016-02-01 Thread Nikita Kozlov
On 10/30/2015 11:25 PM, Matthew Hall wrote: > On Fri, Oct 30, 2015 at 09:55:16PM +, Bruce Richardson wrote: >> We'll see what we can do in 2.3 timeframe. > Thanks for taking all of this up and helping us make a solid plan. I'll do my > best to help out from the community side. I have done some

[dpdk-dev] [PATCH v1] net: i40e: add VLAN tag size to RXMAX

2016-05-06 Thread Nikita Kozlov
Hello, On 09/ 1/15 07:34 AM, Zhang, Helin wrote: >> -Original Message- >> From: Vlad Zolotarov [mailto:vladz at cloudius-systems.com] >> Sent: Monday, August 31, 2015 2:42 PM >> To: dev at dpdk.org >> Cc: Zhang, Helin; Ananyev, Konstantin; avi at cloudius-systems.com; Vlad >> Zolotarov >>

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

2016-11-01 Thread Nikita Kozlov
num_route_entries need to be reseted. --- app/test/test_lpm_perf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/app/test/test_lpm_perf.c b/app/test/test_lpm_perf.c index 608e17a..e7e1281 100644 --- a/app/test/test_lpm_perf.c +++ b/app/test/test_lpm_perf.c @@ -301,6 +301,7 @@ static void gen

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

2016-11-01 Thread Nikita Kozlov
num_route_entries need to be reseted. V2 : resubmitting the patch with a signed-off Signed-off-by: Nikita Kozlov --- app/test/test_lpm_perf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/app/test/test_lpm_perf.c b/app/test/test_lpm_perf.c index 608e17a..e7e1281 100644 --- a/app/test

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

2016-11-01 Thread Nikita Kozlov
num_route_entries need to be reseted. Fixes: 17d60f5b5eea ("app/test: remove large IPv4 LPM data file") Signed-off-by: Nikita Kozlov --- app/test/test_lpm_perf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/app/test/test_lpm_perf.c b/app/test/test_lpm_perf.c index 608e17