linux-kernel - Re: [PATCH 0/2] cpufreq: Upstream Android's Interactive governor

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 for Android: free password hash cracker in your pocket

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

Date:	Fri, 20 May 2016 13:26:53 +0200
From:	"Rafael J. Wysocki" <rafael@...nel.org>
To:	Viresh Kumar <viresh.kumar@...aro.org>
Cc:	Rafael Wysocki <rjw@...ysocki.net>,
	Lists linaro-kernel <linaro-kernel@...ts.linaro.org>,
	"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	Steven Rostedt <rostedt@...dmis.org>,
	Ingo Molnar <mingo@...hat.com>,
	Peter Zijlstra <peterz@...radead.org>, mike@...roid.com,
	toddpoynor@...gle.com,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	kandoiruchi@...gle.com, Juri Lelli <juri.lelli@....com>,
	Steve Muckle <steve.muckle@...aro.org>,
	Vincent Guittot <vincent.guittot@...aro.org>
Subject: Re: [PATCH 0/2] cpufreq: Upstream Android's Interactive governor

On Fri, May 20, 2016 at 8:39 AM, Viresh Kumar <viresh.kumar@...aro.org> wrote:
> Hi Rafael et. al.,
>
> We had some discussions [1] last week on the PM mailing about
> upstreaming the cpufreq governor most widely used on Andoid Mobile
> phones and tablets: Interactive governor. People (including Rafael)
> mostly agreed that we better get it upstreamed and here is an attempt to
> upstream most of it (idle notifiers aren't included in this series).
>
> I picked the latest code spread over 70-80 patches from [2].  The
> unmodified code, based over mainline is pushed [4] for reference.
>
> I have updated the governor to align it with the current practices
> followed with mainline governors, like using utilization hooks from the
> scheduler and handling kobject (for governor's sysfs directory) in a
> race free manner. And of course this included general cleanup of the
> governor as well. This version is pushed here [3] for testing.
>
> The Android version of interactive governor also uses idle EXIT
> notifiers, but that code isn't part of this series and will be sent
> separately later. For people interested in looking at that, those are 3
> minor patches on top of this series and are pushed here [5].
>
> I haven't changed the core logic of the governor intentionally, as that
> rather requires more in-depth knowledge of the use case for which the
> optimizations have been done. So, we should do any such thing later on
> with new patches, so that people can find things easily.
>
> I also haven't tried to change the userspace interface as that may have
> broken the userspace that already exists and uses this governor.
>
> This has been lightly tested on my Exynos board (dual ARM A15), where
> the governor gets inserted/removed multiple times, the sysfs files are
> all functional. The frequency gets changed with load, etc.
>
> This series is based of pm/bleeding-edge branch + few patches from
> Rafael [6] & [7] and few minor cleanups from me [8].

OK, thanks!

This may take some time to review, though.