lkml.org 
[lkml]   [2017]   [Dec]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [RFC][PATCHv2] Fixing POSIX wait queue to insert in task priority order for real-time, including normal tasks
Date
Sorry for the initial email - forgot the signed-off-by in my commit.
 Still new to this...

I'd like feedback from the community on this patch.  I've tested this
against my applications that were exhibiting problematic behavior when
tasks would wake up in FIFO order instead of priority order when trying
to write to a POSIX message queue.  Without the patch, multiple RT
tasks waiting to write to a queue wake up in FIFO order (assuming nice
value untouched); with the patch they wake up in priority order (based
on RT priority setting via sched_setparam.

Is this the correct way to remedy this issue?  Using prio here pulls in
the current priority of the task, including boosts from priority
inheritance.  In that case, would normal_prio be more appropriate?

Anyway, if this patch is acceptable then I'd also love some feedback to
know what steps need to be taken from here to push it along.

Thanks!
Jon

> Signed-off-by: Jonathan Haws <jhaws@sdl.usu.edu>
> ---
>  ipc/mqueue.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ipc/mqueue.c b/ipc/mqueue.c
> index 9649ecd..cb96db9 100644
> --- a/ipc/mqueue.c
> +++ b/ipc/mqueue.c
> @@ -546,7 +546,7 @@ static void wq_add(struct mqueue_inode_info
> *info, int sr,
>   ewp->task = current;
>  
>   list_for_each_entry(walk, &info->e_wait_q[sr].list, list) {
> - if (walk->task->static_prio <= current->static_prio)
> {
> + if (walk->task->prio <= current->prio) {
>   list_add_tail(&ewp->list, &walk->list);
>   return;
>   }
\
 
 \ /
  Last update: 2017-12-06 02:24    [W:0.070 / U:0.476 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site