mbox series

[v4,0/2] CPU non-blocking delay

Message ID 20181010141220.26907-1-i.maximets@samsung.com (mailing list archive)
Headers
Series CPU non-blocking delay |

Message

Ilya Maximets Oct. 10, 2018, 2:12 p.m. UTC 
  For meson build without deprecation warnings following
patch should be applied first:
    http://patches.dpdk.org/patch/46064/

Version 4:
	* Fixed return value checking for nanosleep().
	  It doesn't return EINTR, it sets the errno.

Version 3:
	* Minor rebase on top of current master.

Version 2:
	* Added comment for EINTR handling for nanosleep. [Stephen Hemminger]

Ilya Maximets (2):
  eal: add nanosleep based delay function
  drivers/net: use sleep delay by default for intel NICs

 drivers/net/avf/Makefile                      |  2 +-
 drivers/net/avf/base/avf_osdep.h              |  4 +-
 drivers/net/avf/meson.build                   |  2 +
 drivers/net/e1000/Makefile                    |  1 +
 drivers/net/e1000/base/e1000_osdep.h          |  2 +-
 drivers/net/e1000/meson.build                 |  2 +
 drivers/net/i40e/base/i40e_osdep.h            |  6 +--
 drivers/net/ifc/base/ifcvf_osdep.h            |  2 +-
 drivers/net/ixgbe/base/ixgbe_osdep.h          |  2 +-
 drivers/net/ixgbe/meson.build                 |  3 +-
 lib/librte_eal/common/eal_common_timer.c      | 24 +++++++++
 .../common/include/generic/rte_cycles.h       | 11 ++++
 lib/librte_eal/rte_eal_version.map            |  1 +
 test/test/autotest_data.py                    |  6 +++
 test/test/meson.build                         |  1 +
 test/test/test_cycles.c                       | 51 ++++++++++++++-----
 16 files changed, 96 insertions(+), 24 deletions(-)

Comments

Ferruh Yigit Oct. 25, 2018, 1:27 p.m. UTC | #1 
On 10/10/2018 3:12 PM, Ilya Maximets wrote:
> For meson build without deprecation warnings following
> patch should be applied first:
>     http://patches.dpdk.org/patch/46064/
> 
> Version 4:
> 	* Fixed return value checking for nanosleep().
> 	  It doesn't return EINTR, it sets the errno.
> 
> Version 3:
> 	* Minor rebase on top of current master.
> 
> Version 2:
> 	* Added comment for EINTR handling for nanosleep. [Stephen Hemminger]
> 
> Ilya Maximets (2):
>   eal: add nanosleep based delay function
>   drivers/net: use sleep delay by default for intel NICs

Looks like good idea.

For series,
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
Ferruh Yigit Oct. 26, 2018, 2:23 p.m. UTC | #2 
On 10/25/2018 2:27 PM, Ferruh Yigit wrote:
> On 10/10/2018 3:12 PM, Ilya Maximets wrote:
>> For meson build without deprecation warnings following
>> patch should be applied first:
>>     http://patches.dpdk.org/patch/46064/
>>
>> Version 4:
>> 	* Fixed return value checking for nanosleep().
>> 	  It doesn't return EINTR, it sets the errno.
>>
>> Version 3:
>> 	* Minor rebase on top of current master.
>>
>> Version 2:
>> 	* Added comment for EINTR handling for nanosleep. [Stephen Hemminger]
>>
>> Ilya Maximets (2):
>>   eal: add nanosleep based delay function
>>   drivers/net: use sleep delay by default for intel NICs
> 
> Looks like good idea.
> 
> For series,
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Series applied to dpdk-next-net/master, thanks.