From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751844AbdJFITp (ORCPT ); Fri, 6 Oct 2017 04:19:45 -0400 Received: from merlin.infradead.org ([205.233.59.134]:48122 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751797AbdJFITo (ORCPT ); Fri, 6 Oct 2017 04:19:44 -0400 Date: Fri, 6 Oct 2017 10:19:37 +0200 From: Peter Zijlstra To: Sebastian Andrzej Siewior Cc: linux-kernel@vger.kernel.org, Ingo Molnar , Jens Axboe , tglx@linutronix.de Subject: Re: [PATCH 1/2] sched/rt: Add a helper to test for a RT task Message-ID: <20171006081937.eshxl36hitrd4j22@hirez.programming.kicks-ass.net> References: <20171004154901.26904-1-bigeasy@linutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171004154901.26904-1-bigeasy@linutronix.de> User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 04, 2017 at 05:49:00PM +0200, Sebastian Andrzej Siewior wrote: > This helper returns true if a task has elevated priority which is true > for RT tasks (SCHED_RR and SCHED_FIFO) and also for SCHED_DEADLINE. > A task which runs at RT priority due to PI-boosting is not considered as > one with elevated priority. > > Signed-off-by: Sebastian Andrzej Siewior > --- > include/linux/sched/rt.h | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/include/linux/sched/rt.h b/include/linux/sched/rt.h > index f93329aba31a..30a86b8178f8 100644 > --- a/include/linux/sched/rt.h > +++ b/include/linux/sched/rt.h > @@ -17,6 +17,17 @@ static inline int rt_task(struct task_struct *p) > return rt_prio(p->prio); > } > > +static inline bool task_is_elevated(struct task_struct *tsk) Would you be terribly upset if I rename that? I'm thinking something like task_is_realtime() better reflects what it does. > +{ > + int policy = tsk->policy; > + > + if (policy == SCHED_FIFO || policy == SCHED_RR) > + return true; > + if (policy == SCHED_DEADLINE) > + return true; > + return false; > +} > + > #ifdef CONFIG_RT_MUTEXES > /* > * Must hold either p->pi_lock or task_rq(p)->lock. > -- > 2.14.2 >