mbox series

[v2,0/3] alarm test fixes

Message ID 20240809152540.9568-1-stephen@networkplumber.org (mailing list archive)
Headers
Series alarm test fixes |

Message

Stephen Hemminger Aug. 9, 2024, 3:24 p.m. UTC 
This is the set of things which stood out as needing fixing when
looking at the rte_eal_alarm tests.

v2 - drop cleanup of eal alarm code for later
   - add RTE_ATOMIC() to fix clang build

Stephen Hemminger (3):
  eal: add missing parameter check to rte_eal_alarm_set on Windows
  test: support alarm test on FreeBSD
  test/alarm: rewrite the alarm test

 app/test/test_alarm.c       | 85 ++++++++++++++++++++++---------------
 lib/eal/windows/eal_alarm.c |  7 +++
 2 files changed, 57 insertions(+), 35 deletions(-)

Comments

fengchengwen Sept. 23, 2024, 2:17 a.m. UTC | #1 
LGTM
Series-acked-by: Chengwen Feng <fengchengwen@huawei.com>

On 2024/8/9 23:24, Stephen Hemminger wrote:
> This is the set of things which stood out as needing fixing when
> looking at the rte_eal_alarm tests.
> 
> v2 - drop cleanup of eal alarm code for later
>    - add RTE_ATOMIC() to fix clang build
> 
> Stephen Hemminger (3):
>   eal: add missing parameter check to rte_eal_alarm_set on Windows
>   test: support alarm test on FreeBSD
>   test/alarm: rewrite the alarm test
> 
>  app/test/test_alarm.c       | 85 ++++++++++++++++++++++---------------
>  lib/eal/windows/eal_alarm.c |  7 +++
>  2 files changed, 57 insertions(+), 35 deletions(-)
>