[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-09-22 Thread Thomas Monjalon
ts are accepted. > > Best regards, > Jozef > > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, September 21, 2016 3:13 PM > To: Jozef Martiniak -X (jozmarti - PANTHEON TECHNOLOGIES at Cisco) at cisco.com> > Cc: dev at

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-09-22 Thread Jozef Martiniak -X (jozmarti - PANTHEON TECHNOLOGIES@Cisco)
[mailto:thomas.monja...@6wind.com] Sent: Wednesday, September 21, 2016 3:13 PM To: Jozef Martiniak -X (jozmarti - PANTHEON TECHNOLOGIES at Cisco) Cc: dev at dpdk.org Subject: Re: [dpdk-dev] [PATCH] rte_delay_us can be replaced with user function Hi, I think this feature should enter in the release 16.11.

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-09-21 Thread Thomas Monjalon
Hi, I think this feature should enter in the release 16.11. We just need to make sure it is implemented with the right API. Do you have any comment about managing several builtin handlers? 2016-09-13 22:04, Thomas Monjalon: > Hi, > > Sorry for late review. > This patch was in a summer hole :/ >

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-09-13 Thread Thomas Monjalon
Hi, Sorry for late review. This patch was in a summer hole :/ First a general comment: please check your patch with scripts/checkpatches.sh. In order to ease tracking of this patch, please increment the version when sending a new one in the same thread: git send-email -1 -v3 --annotate --

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-07-20 Thread jozma...@cisco.com
From: Jozef Martiniak When running single-core, some drivers tend to call rte_delay_us for a long time, and that is causing packet drops. To avoid this, rte_delay_us can be replaced with user-defined delay function with: void rte_delay_us_callback_register(void(*userfunc)(unsigned)); When userf

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-07-19 Thread Thomas Monjalon
Hi, 2016-07-19 14:42, jozmarti at cisco.com: > when running single-core, some drivers tend to call rte_delay_us for a > long time, and that is causing packet drops. > Attached patch introduces 2 new functions: > > void rte_delay_us_callback_register(void(*userfunc)(unsigned)); > void rte_delay_us

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-07-19 Thread jozma...@cisco.com
From: Jozef Martiniak when running single-core, some drivers tend to call rte_delay_us for a long time, and that is causing packet drops. Attached patch introduces 2 new functions: void rte_delay_us_callback_register(void(*userfunc)(unsigned)); void rte_delay_us_callback_unregister(void); First

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-07-19 Thread Jozef Martiniak -X (jozmarti - PANTHEON TECHNOLOGIES@Cisco)
Hi, On Tue, 2016-07-19 at 15:21 +0200, Thomas Monjalon wrote: Hi, > > 2016-07-19 14:42, jozmarti at cisco.com: > > when running single-core, some drivers tend to call rte_delay_us for a > > long time, and that is causing packet drops. > > Attached patch introduces 2 new functions: > > > > void r

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-07-19 Thread Wiles, Keith
Hi Jozef, I have two quick comments inline. > On Jul 19, 2016, at 7:42 AM, jozmarti at cisco.com wrote: > > From: Jozef Martiniak > > when running single-core, some drivers tend to call rte_delay_us for a > long time, and that is causing packet drops. > Attached patch introduces 2 new functions