Re: [dpdk-dev] Guidelines for moving PMDs to new ethdev offloads API

2017-12-09 Thread Shahaf Shuler
Sunday, December 10, 2017 7:52 AM, Stephen Hemminger: > > On Sun, 10 Dec 2017 05:35:45 + > Shahaf Shuler wrote: > > > In 17.11 the ethdev offloads API has changed: > > > > commit cba7f53b717d ("ethdev: introduce Tx queue offloads > > API") > > > > commit ce17

Re: [dpdk-dev] Guidelines for moving PMDs to new ethdev offloads API

2017-12-09 Thread Stephen Hemminger
On Sun, 10 Dec 2017 05:35:45 + Shahaf Shuler wrote: > In 17.11 the ethdev offloads API has changed: > > commit cba7f53b717d ("ethdev: introduce Tx queue offloads > API") > > commit ce17eddefc20 ("ethdev: introduce Rx queue offloads > API") The new API is do

[dpdk-dev] Guidelines for moving PMDs to new ethdev offloads API

2017-12-09 Thread Shahaf Shuler
In 17.11 the ethdev offloads API has changed: commit cba7f53b717d ("ethdev: introduce Tx queue offloads API") commit ce17eddefc20 ("ethdev: introduce Rx queue offloads API") The new API is documented in the programmer's guide: https://emea01.safe

[dpdk-dev] [PATCH v2] mbuf: check sanity of data_len and pkt_len as well

2017-12-09 Thread Ilya V. Matveychikov
Update rte_mbuf_sanity_check() to check sanity of data_len and pkt_len fields. For segmented packets it is supposed that head's pkt_len field should be the sum of all segments data_len values. Signed-off-by: Ilya V. Matveychikov --- lib/librte_mbuf/rte_mbuf.c | 23 +++ 1 file

[dpdk-dev] [PATCH v2] mbuf: cleanup rte_pktmbuf_lastseg function

2017-12-09 Thread Ilya V. Matveychikov
There is no reason to have local variable m2. Fixes: af75078fece3 ("first public release") Signed-off-by: Ilya V. Matveychikov --- lib/librte_mbuf/rte_mbuf.h | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h index

[dpdk-dev] [PATCH v2] lib/cmdline: init CLI parsing memory

2017-12-09 Thread Xueming Li
Initialize result memory every time before parsing. Also save successfully parsed result before further ambiguous command detection to avoid result being tainted by later parsing. Signed-off-by: Xueming Li --- lib/librte_cmdline/cmdline_parse.c | 12 +++- 1 file changed, 7 insertions(+),

[dpdk-dev] [PATCH] eventdev: fix doxygen comments

2017-12-09 Thread Pavan Nikhilesh
Fix doxygen return values and indentation. Fixes: 64103dbcd673 ("eventdev: add dev attribute get function") Signed-off-by: Pavan Nikhilesh --- lib/librte_eventdev/rte_eventdev.h | 50 ++ 1 file changed, 29 insertions(+), 21 deletions(-) diff --git a/lib/libr

[dpdk-dev] [PATCH v2 1/3] eal: update legacy modules dynamic logs regex

2017-12-09 Thread Pavan Nikhilesh
Update legacy log types regex strings used for registering dynamic logs. Signed-off-by: Pavan Nikhilesh Reviewed-by: Ferruh Yigit Reviewed-by: Olivier Matz --- v2 Changes: - Corrected dynamic log regex strings of 'librte.flow_classify' and 'librte.member' to 'lib.flow_classify' and 'lib.m

[dpdk-dev] [PATCH v2 2/3] eal: update default log levels

2017-12-09 Thread Pavan Nikhilesh
Use global default loglevel to DEBUG(8) and dynamic default loglevel to INFO(7). Signed-off-by: Pavan Nikhilesh Reviewed-by: Olivier Matz --- lib/librte_eal/common/eal_common_log.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/lib/librte_eal/common/eal_common_log.c

[dpdk-dev] [PATCH v2 3/3] logs: remove log level config option

2017-12-09 Thread Pavan Nikhilesh
Remove RTE_LOG_LEVEL config option, use existing RTE_LOG_DP_LEVEL config option for controlling datapath log level. RTE_LOG_LEVEL is no longer needed as dynamic logging can be used to control global and module specific log levels. Signed-off-by: Pavan Nikhilesh --- config/common_base

Re: [dpdk-dev] [PATCH 01/16] net/axgbe: add minimal dev init and uninit support

2017-12-09 Thread Kumar, Ravi1
>On 11/30/2017 5:10 AM, Ravi Kumar wrote: >> Signed-off-by: Ravi Kumar > ><...> > >> +/* >> + * For secondary processes, we don't initialise any further as primary >> + * has already done this work. >> + */ >> +if (rte_eal_process_type() != RTE_PROC_PRIMARY) { >> +P

[dpdk-dev] [PATCH v2 2/2] event/octeontx: fix Rx adapter port id mapping

2017-12-09 Thread Pavan Nikhilesh
When octeontx event dev receives a packet for the event Rx adapter, the mbuf port id should contain the appropriate ethdev id instead of internal channel info. Fixes: 45a914c5bd71 ("event/octeontx: support event Rx adapter") Signed-off-by: Pavan Nikhilesh --- drivers/event/octeontx/Makefile

[dpdk-dev] [PATCH v2 1/2] net/octeontx: add channel to port id mapping

2017-12-09 Thread Pavan Nikhilesh
The channel to port id map is used by event octeontx to map the received wqe to the respective ethdev port. Signed-off-by: Pavan Nikhilesh --- v2 changes: - Used extern instead of defining global variable drivers/net/octeontx/octeontx_ethdev.c| 6 ++ drivers/net/octeontx/oct

Re: [dpdk-dev] [PATCH] test/eventdev: use CPU event type

2017-12-09 Thread Jerin Jacob
-Original Message- > Date: Mon, 4 Dec 2017 14:14:45 +0530 > From: Pavan Nikhilesh Bhagavatula > To: Jerin Jacob > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] test/eventdev: use CPU event type > User-Agent: NeoMutt/20170609 (1.8.3) > > On Mon, Dec 04, 2017 at 01:41:18PM +0530, Jeri

Re: [dpdk-dev] [PATCH 2/2] event/sw: use dynamically-sized IQs

2017-12-09 Thread Jerin Jacob
-Original Message- > Date: Thu, 7 Dec 2017 17:15:35 + > From: "Van Haaren, Harry" > To: "Eads, Gage" , "dev@dpdk.org" > CC: "jerin.ja...@caviumnetworks.com" , > "Richardson, Bruce" , "hemant.agra...@nxp.com" > , "nipun.gu...@nxp.com" , > "santosh.shu...@caviumnetworks.com" , > "pb

Re: [dpdk-dev] [PATCH 1/2] net/octeontx: add channel to port id mapping

2017-12-09 Thread Pavan Nikhilesh Bhagavatula
On Fri, Dec 08, 2017 at 09:39:00AM -0800, Ferruh Yigit wrote: > On 12/8/2017 3:08 AM, Pavan Nikhilesh Bhagavatula wrote: > > On Thu, Dec 07, 2017 at 04:41:04PM -0800, Ferruh Yigit wrote: > >> On 11/28/2017 6:58 AM, Pavan Nikhilesh wrote: > >>> The channel to port id map is used by event octeontx to

Re: [dpdk-dev] [PATCH 1/2] event/sw: fix queue memory leak and multi-link bug

2017-12-09 Thread Jerin Jacob
-Original Message- > Date: Thu, 7 Dec 2017 17:15:16 + > From: "Van Haaren, Harry" > To: "Eads, Gage" , "dev@dpdk.org" > CC: "jerin.ja...@caviumnetworks.com" , > "Richardson, Bruce" , "hemant.agra...@nxp.com" > , "nipun.gu...@nxp.com" , > "santosh.shu...@caviumnetworks.com" , > "pb