> -----Original Message----- > From: EXT Ivan Khoronzhuk [mailto:[email protected]] > Sent: Thursday, September 10, 2015 12:49 PM > To: [email protected]; Savolainen, Petri (Nokia - FI/Espoo); > [email protected] > Cc: Ivan Khoronzhuk > Subject: [lng-odp] [Patch 2/2] validation: schedule: don't check > schedule time on 0 > > The ODP_SCHED_NO_WAIT now corresponds to 0, not 1. > So no need to check it anymore. > > Signed-off-by: Ivan Khoronzhuk <[email protected]> > --- > test/validation/scheduler/scheduler.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/test/validation/scheduler/scheduler.c > b/test/validation/scheduler/scheduler.c > index 1874889..94facea 100644 > --- a/test/validation/scheduler/scheduler.c > +++ b/test/validation/scheduler/scheduler.c > @@ -96,9 +96,6 @@ void scheduler_test_wait_time(void) > > wait_time = odp_schedule_wait_time(0);
This test is OK. > > - wait_time = odp_schedule_wait_time(1); This is OK. > - CU_ASSERT(wait_time > 0); This is not. The value returned is implementation specific. > - > wait_time = odp_schedule_wait_time((uint64_t)-1LL); This is OK. > CU_ASSERT(wait_time > 0); This is not. The value returned is implementation specific. So, both asserts should be removed. In addition, wait time should be tested with a schedule call ... but that's for another patch. -Petri > } > -- > 1.9.1 _______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp
