Reviewed-by: Petri Savolainen <[email protected]>

> -----Original Message-----
> From: lng-odp [mailto:[email protected]] On Behalf Of Bill
> Fischofer
> Sent: Tuesday, July 05, 2016 5:32 AM
> To: [email protected]
> Subject: [lng-odp] [PATCHv2] linux-generic: schedule: simplify wait logic
> to avoid clang issues
> 
> Resolve Bug https://bugs.linaro.org/show_bug.cgi?id=2387 by ensuring that
> t1 variable is seen as initialized in all paths in the SP Scheduler.
> Previous logic failed due to an apparent issue with clang.
> 
> Signed-off-by: Bill Fischofer <[email protected]>
> ---
>  platform/linux-generic/odp_schedule_sp.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/platform/linux-generic/odp_schedule_sp.c b/platform/linux-
> generic/odp_schedule_sp.c
> index 8c45123..2e28aa4 100644
> --- a/platform/linux-generic/odp_schedule_sp.c
> +++ b/platform/linux-generic/odp_schedule_sp.c
> @@ -364,9 +364,9 @@ static uint64_t schedule_wait_time(uint64_t ns)
>  }
> 
>  static int schedule_multi(odp_queue_t *from, uint64_t wait,
> -                       odp_event_t events[], int max_events)
> +                       odp_event_t events[], int max_events ODP_UNUSED)
>  {
> -     (void)max_events;
> +     odp_time_t t1;
>       int update_t1 = 1;
> 
>       if (sched_local.cmd) {
> @@ -384,7 +384,6 @@ static int schedule_multi(odp_queue_t *from, uint64_t
> wait,
>               sched_cmd_t *cmd;
>               uint32_t qi;
>               int num;
> -             odp_time_t t1;
> 
>               cmd = sched_cmd(NUM_PRIO);
> 
> --
> 2.7.4
> 
> _______________________________________________
> lng-odp mailing list
> [email protected]
> https://lists.linaro.org/mailman/listinfo/lng-odp
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to