From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Rostedt Subject: Re: [RFC PATCH v1 3/8] sched/cpufreq_schedutil: make worker kthread be SCHED_DEADLINE Date: Fri, 7 Jul 2017 18:15:49 -0400 Message-ID: <20170707181549.5ca5cf31@gandalf.local.home> References: <20170705085905.6558-1-juri.lelli@arm.com> <20170707105316.whutk3h6x2h6xvyt@e106622-lin> <4582085.aKtjXTl7qR@aspire.rjw.lan> <20170707175805.6bd8f34d@gandalf.local.home> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Joel Fernandes Cc: "Rafael J. Wysocki" , Juri Lelli , Thomas Gleixner , Peter Zijlstra , Ingo Molnar , Viresh Kumar , LKML , Linux PM , Vincent Guittot , Luca Abeni , claudio@evidence.eu.com, Tommaso Cucinotta , bristot@redhat.com, Mathieu Poirier , Todd Kjos , Andres Oportus , Morten Rasmussen , Dietmar Eggemann , Patrick Bellasi Ingo Molnar List-Id: linux-pm@vger.kernel.org On Fri, 7 Jul 2017 15:07:09 -0700 Joel Fernandes wrote: > > It is rather long. Although I actually hate the SUGOV, it is easily > > grepable. Just comment what it stands for. We can always change it > > later. > > I was thinking why not just SCHED_FLAG_CPUFREQ. That says its for > cpufreq purposes, and is a bit self-documenting. "WORKER" is a bit > redundant and can be dropped in my opinion. I was thinking that too, but was wondering how tightly coupled is this with SCHED_DEADLINE? I like the searchability of SUGOV, where as CPUFREQ is still quite broad. -- Steve