mbox series

[0/2] Fix unit tests execution for ENA PMD

Message ID 20210326104759.280175-1-mk@semihalf.com (mailing list archive)
Headers
Series Fix unit tests execution for ENA PMD |

Message

Michal Krawczyk March 26, 2021, 10:47 a.m. UTC 
  Hi,

ENA PMD uses timer service to implement various periodic device status
check routine. Because of that, it already initializes the timer
subsystem. As unit tests also initalizes the timer subsystem, return
value is equal to -EAGAIN, as it was already done by the ENA PMD.

This patch set adds missing documentation that this kind of return code
may appear and this is not a failure and also changes unit tests tool,
to do not fail if timer subsystem was already initialized.

Stanislaw Kardach (2):
  timer: clarify subsystem_init return value
  test: proceed if timer subsystem was initialized

 app/test/test.c              | 11 ++++++-----
 lib/librte_timer/rte_timer.h |  1 +
 2 files changed, 7 insertions(+), 5 deletions(-)

Comments

Thomas Monjalon April 8, 2021, 9:15 p.m. UTC | #1 
26/03/2021 11:47, Michal Krawczyk:
> ENA PMD uses timer service to implement various periodic device status
> check routine. Because of that, it already initializes the timer
> subsystem. As unit tests also initalizes the timer subsystem, return
> value is equal to -EAGAIN, as it was already done by the ENA PMD.
> 
> This patch set adds missing documentation that this kind of return code
> may appear and this is not a failure and also changes unit tests tool,
> to do not fail if timer subsystem was already initialized.
> 
> Stanislaw Kardach (2):
>   timer: clarify subsystem_init return value
>   test: proceed if timer subsystem was initialized

Applied, thanks