From: Juri Lelli <juri.lelli@arm.com>
To: "Rafael J. Wysocki" <rafael@kernel.org>
Cc: Steve Muckle <steve.muckle@linaro.org>,
Viresh Kumar <viresh.kumar@linaro.org>,
"Rafael J. Wysocki" <rjw@rjwysocki.net>,
"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
Linaro Kernel Mailman List <linaro-kernel@lists.linaro.org>,
Ingo Molnar <mingo@kernel.org>,
Morten Rasmussen <Morten.Rasmussen@arm.com>,
Vincent Guittot <vincent.guittot@linaro.org>
Subject: Re: [RFC] Should we add Android's Interactive governor into mainline now?
Date: Fri, 13 May 2016 14:15:09 +0100 [thread overview]
Message-ID: <20160513131509.GM9525@e106622-lin> (raw)
In-Reply-To: <CAJZ5v0igpMqgrLdJdNRkb7qFG65UJ4XCUtr7fae1eoVCF-coig@mail.gmail.com>
On 13/05/16 13:56, Rafael J. Wysocki wrote:
> On Fri, May 13, 2016 at 1:44 PM, Juri Lelli <juri.lelli@arm.com> wrote:
> > Hi,
> >
> > On 12/05/16 14:10, Steve Muckle wrote:
> >> On Thu, May 12, 2016 at 10:34:39PM +0200, Rafael J. Wysocki wrote:
> >> > > That said my understanding is that the goal is to eventually deprecate
> >> > > the non-sched governors. That will be made harder by adding another one
> >> > > into the tree.
> >> >
> >> > We can't really deprecate any out-of-the-tree code, because it is
> >> > beyond the scope of our influence, so one can argue that taking it
> >> > into the tree may actually allow us to deprecate it at one point going
> >> > forward. :-)
> >> >
> >> > Seriously, though, "deprecation" may not be the right term to use
> >> > here. Our goal, IMO, should be to persuade the users of the code in
> >> > question to switch over to something else and I'm not really sure if
> >> > refusing to take that code into the tree is the best way to achieve
> >> > that goal.
> >> >
> >> > I do believe that if there's a significant piece of code living out of
> >> > the tree for a sufficiently long time, the tree is likely to be
> >> > missing something important. Then, it is really difficult to figure
> >> > out what is missing without even trying to take that code into the
> >> > tree.
> >>
> >> By far the biggest user of interactive is Android (I'm not aware of its
> >> use elsewhere). Persuading Google to switch is relatively doable once a
> >> viable alternative exists. After that point I'd expect the desire to
> >> merge and maintain interactive would almost immediately disappear. Folks
> >> wanting to run upstream kernels on already released devices have much
> >> bigger hurdles than merging the interactive governor.
> >>
> >> But if interactive is merged I'm worried that many other users on random
> >> platforms will adopt it, for whatever reason, introducing a support
> >> burden during a time that we're trying to develop and encourage an
> >> alternative.
> >>
> >
> > Right. It looks a bit odd to merge (and then maintain) something while
> > we are actively working on an alternative designed to suit the same kind
> > of needs.
>
> I'm not worried too much about the maintenance part. The code seems
> to be mostly self-contained and it shouldn't add too many additional
> constraints on the core as far as the interface is concerned.
>
Yeah, this is true.
> > Also considering that, with Viresh's effort, the merge
> > request will not come from the original authors.
>
> Which may not be a bad thing. ;-)
>
:-)
> >> Anyway that's just my $.02 - it'd actually be good for me as
> >> again it'd permit easier comparison with schedutil, so I won't complain
> >> if it goes in :) .
> >>
> >
> > Anyway, I can't say that comparison would not be much easier. :-)
> > Even though we won't run the workloads interactive has been designed for
> > on mainline, I guess.
>
> Think about running Android on top of the mainline, though. That
> would be somewhat easier if we had interactive in the mainline,
> wouldn't it?
>
I agree that there might be value in reducing the gap between Android
and mainline (even though I still fear that lacking interactive is not
the biggest problem out there). However, wouldn't it be better if we
convince Google folks to switch to something coming from mainline
instead of doing the other way around?
Anyway, as said, if Viresh wants and has time to work on this I think it
will still help doing comparisons, so I'm not saying no to it! :-)
Best,
- Juri
prev parent reply other threads:[~2016-05-13 13:14 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-12 2:04 [RFC] Should we add Android's Interactive governor into mainline now? Viresh Kumar
2016-05-12 13:19 ` Rafael J. Wysocki
2016-05-13 3:19 ` Viresh Kumar
2016-05-13 11:06 ` Rafael J. Wysocki
2016-05-13 11:09 ` Viresh Kumar
2016-05-12 16:36 ` Steve Muckle
2016-05-12 20:34 ` Rafael J. Wysocki
2016-05-12 21:10 ` Steve Muckle
2016-05-13 3:21 ` Viresh Kumar
2016-05-13 6:14 ` Steve Muckle
2016-05-13 6:17 ` Viresh Kumar
2016-05-13 11:44 ` Juri Lelli
2016-05-13 11:56 ` Rafael J. Wysocki
2016-05-13 13:15 ` Juri Lelli [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160513131509.GM9525@e106622-lin \
--to=juri.lelli@arm.com \
--cc=Morten.Rasmussen@arm.com \
--cc=linaro-kernel@lists.linaro.org \
--cc=linux-pm@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=peterz@infradead.org \
--cc=rafael@kernel.org \
--cc=rjw@rjwysocki.net \
--cc=steve.muckle@linaro.org \
--cc=vincent.guittot@linaro.org \
--cc=viresh.kumar@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox