linux-kernel - Re: [RFC][PATCH] ipc: mqueue: wq_add priority changed to dynamic priority

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20171211151537.58212890@gandalf.local.home>
Date:   Mon, 11 Dec 2017 15:15:37 -0500
From:   Steven Rostedt <rostedt@...dmis.org>
To:     Jonathan Haws <jhaws@....usu.edu>
Cc:     "bigeasy@...utronix.de" <bigeasy@...utronix.de>,
        "mingo@...nel.org" <mingo@...nel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "tglx@...utronix.de" <tglx@...utronix.de>,
        "deepa.kernel@...il.com" <deepa.kernel@...il.com>,
        "viro@...iv.linux.org.uk" <viro@...iv.linux.org.uk>,
        "akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
        "linux-rt-users@...r.kernel.org" <linux-rt-users@...r.kernel.org>,
        "arnd@...db.de" <arnd@...db.de>
Subject: Re: [RFC][PATCH] ipc: mqueue: wq_add priority changed to dynamic
 priority

On Mon, 11 Dec 2017 19:54:33 +0000
Jonathan Haws <jhaws@....usu.edu> wrote:

> > Probably a better subject would be:
> > 
> >  ipc: mqueue: Have RT tasks queue in by priority in wq_add()  
> 
> Is the best way to change that just to change it in the email thread or
> resubmit the patch as v2?

That, or whoever takes it could change it before they pull it into git.

> 
> > 
> > Other than that (and that this doesn't handle DL tasks),
> > 
> > Reviewed-by: Steven Rostedt (VMware) <rostedt@...dmis.org>
> > 
> > -- Steve
> >   
> 
> I'm still new to kernel scheduling details - can you elaborate on what
> a DL task is?  I'm happy to update the patch to handle DL tasks as well
> if it makes sense to do so.

DL is a SCHED_DEADLINE task that has a dynamic priority that changes
over time. It's not a trivial task to compare them in these situations,
and they are currently not used much, and having DL tasks use this
queue is probably even more rare of an occurrence.

-- Steve