Re: [RFC PATCH 8/8] arm64: add sysfs cpu_capacity attribute

From	Juri Lelli <juri.lelli@arm.com>
Newsgroups	linux.kernel
Subject	Re: [RFC PATCH 8/8] arm64: add sysfs cpu_capacity attribute
Date	2015-12-10 19:10 +0100
Message-ID	<qEdPd-1ez-35@gated-at.bofh.it> (permalink)
References	<qy0hX-2Wb-3@gated-at.bofh.it> <qy0rE-30z-17@gated-at.bofh.it> <qEaeB-7jl-15@gated-at.bofh.it> <qEbNp-8az-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi,

On 10/12/15 15:59, Mark Brown wrote:
> On Thu, Dec 10, 2015 at 02:15:04PM +0000, Dietmar Eggemann wrote:
> > On 23/11/15 14:28, Juri Lelli wrote:
> 
> > > The new attribute shows up as:
> 
> > >  /sys/devices/system/cpu/cpu*/cpu_capacity
> 
> > This sysfs interface is not really needed for arm or arm64. People can
> > build the dt blob if they want to change the values. Less code to carry
> > ... Let's focus on the core functionality, which is parsing cpu capacity
> > from dt file to task scheduler for heterogeneous systems.
> 
> That does make the tuning process much more cumbersome - users have to
> rebuild and reboot to tweak the numbers rather than just tweaking the
> numbers and rerunning the benchmark (which seems like something people
> would want to automate).

IMHO, this is not a tuning interface. It is an alternative interface,
w.r.t. DTs, that we could use to provide default capacity values to the
kernel. I'm proposing both here as they make both sense to me. Then we
might dedice for which one to go (or if we need some other way) or to
keep both for flexibility.

Best,

- Juri
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

Re: [RFC PATCH 8/8] arm64: add sysfs cpu_capacity attribute Dietmar Eggemann <dietmar.eggemann@arm.com> - 2015-12-10 15:20 +0100
  Re: [RFC PATCH 8/8] arm64: add sysfs cpu_capacity attribute Mark Brown <broonie@kernel.org> - 2015-12-10 17:00 +0100
    Re: [RFC PATCH 8/8] arm64: add sysfs cpu_capacity attribute Juri Lelli <juri.lelli@arm.com> - 2015-12-10 19:10 +0100
      Re: [RFC PATCH 8/8] arm64: add sysfs cpu_capacity attribute Mark Brown <broonie@kernel.org> - 2015-12-11 19:00 +0100

csiph-web