On Thu, May 11, 2017 at 4:48 AM, Matias Elo <[email protected]> wrote:
> If CONFIG_BURST_SIZE was set to one low priority events were never
> scheduled.
>
> Signed-off-by: Matias Elo <[email protected]>

Reviewed-by: Bill Fischofer <[email protected]>

> ---
>  platform/linux-generic/odp_schedule.c        | 3 ++-
>  platform/linux-generic/odp_schedule_iquery.c | 2 +-
>  2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/platform/linux-generic/odp_schedule.c 
> b/platform/linux-generic/odp_schedule.c
> index f366e7e..f680ac4 100644
> --- a/platform/linux-generic/odp_schedule.c
> +++ b/platform/linux-generic/odp_schedule.c
> @@ -799,7 +799,8 @@ static inline int do_schedule_grp(odp_queue_t *out_queue, 
> odp_event_t out_ev[],
>
>                         /* Low priorities have smaller batch size to limit
>                          * head of line blocking latency. */
> -                       if (odp_unlikely(prio > ODP_SCHED_PRIO_DEFAULT))
> +                       if (odp_unlikely(MAX_DEQ > 1 &&
> +                                        prio > ODP_SCHED_PRIO_DEFAULT))
>                                 max_deq = MAX_DEQ / 2;
>
>                         ordered = sched_cb_queue_is_ordered(qi);
> diff --git a/platform/linux-generic/odp_schedule_iquery.c 
> b/platform/linux-generic/odp_schedule_iquery.c
> index b692457..b8a4001 100644
> --- a/platform/linux-generic/odp_schedule_iquery.c
> +++ b/platform/linux-generic/odp_schedule_iquery.c
> @@ -1457,7 +1457,7 @@ static inline int consume_queue(int prio, unsigned int 
> queue_index)
>         /* Low priorities have smaller batch size to limit
>          * head of line blocking latency.
>          */
> -       if (odp_unlikely(prio > ODP_SCHED_PRIO_DEFAULT))
> +       if (odp_unlikely(MAX_DEQ > 1 && prio > ODP_SCHED_PRIO_DEFAULT))
>                 max = MAX_DEQ / 2;
>
>         /* For ordered queues we want consecutive events to
> --
> 2.7.4
>

Reply via email to