From: Mattia Dongili <malattia@gmail.com>
To: Dave Jones <davej@redhat.com>,
Jonathan Nieder <jrnieder@gmail.com>,
cpufreq@vger.kernel.org, linux-kernel@vger.kernel.org,
"Rafael J. Wysocki" <rjw@sisk.pl>,
Nicolas Pitre <nicolas.pitre@linaro.org>,
Russell King - ARM Linux <linux@arm.linux.org.uk>,
Arnd Bergmann <arnd@arndb.de>, Tony Lindgren <tony@atomide.com>,
Linus Walleij <linus.walleij@linaro.org>,
Mark Brown <broonie@opensource.wolfsonmicro.com>
Subject: Re: [BUG?] Moving drivers to drivers/cpufreq/ causes all to be loaded
Date: Sun, 14 Aug 2011 09:18:27 +0900 [thread overview]
Message-ID: <20110814001826.GB6432@kamineko.org> (raw)
In-Reply-To: <20110813211142.GA1536@redhat.com>
On Sat, Aug 13, 2011 at 05:11:42PM -0400, Dave Jones wrote:
> On Sat, Aug 13, 2011 at 02:02:46PM -0500, Jonathan Nieder wrote:
> > Jonathan Nieder wrote:
> >
> > > (1) This is still incredibly fragile. What *should* cpufrequtils
> > > be doing to get the modules it needs?
> > >
> > > (2) Using the 3.0 or later kernel with old userspace gives bad
> > > results (e.g., 30% increase in power consumption for one
> > > reporter). That's a regression.
> >
> > The "30% increase" part was an unrelated bug (i915.i915_enable_rc6=1
> > brings power consumption back to normal), for those who were
> > wondering. :)
> >
> > Old userspace automatically loading the wrong cpufreq drivers still
> > does not seem great to me, though I don't have any great ideas about
> > how to prevent that (a separate drivers/cpufreq-drivers/ directory
> > does not sound too appealing). I guess I'd be most interested in how
> > to fix (1) first.
>
> If we have to move stuff again, we could do drivers/cpufreq/x86/ etc..
> Even if we do that though, you really want to fix that userspace, because
> you're right that "load everything and see what sticks" is fragile,
> and pure luck that it ever did the right thing.
not sure why this bug landed here finally, it was clearly an
overlook in the Debian startup script and it's only specific to Debian.
The "load everything" part was not for cpu drivers but for governors
and helpers that used to sit into drivers/cpufreq alone.
The cpu driver loading part is fairly complex (or yes, messy as you say)
and not too dissimilar than the one from fedora.
--
mattia
:wq!
next prev parent reply other threads:[~2011-08-14 0:18 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20110414094447.GA1611@n2100.arm.linux.org.uk>
[not found] ` <alpine.LFD.2.00.1104211629300.24613@xanadu.home>
[not found] ` <201104261605.26791.arnd@arndb.de>
[not found] ` <201104261904.45923.rjw@sisk.pl>
[not found] ` <20110426181508.GB1908@redhat.com>
2011-08-13 15:46 ` Jonathan Nieder
2011-08-13 19:02 ` Jonathan Nieder
2011-08-13 21:11 ` Dave Jones
2011-08-14 0:18 ` Mattia Dongili [this message]
2011-08-14 17:01 ` Jonathan Nieder
2011-08-14 17:17 ` Kay Sievers
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=20110814001826.GB6432@kamineko.org \
--to=malattia@gmail.com \
--cc=arnd@arndb.de \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=cpufreq@vger.kernel.org \
--cc=davej@redhat.com \
--cc=jrnieder@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=nicolas.pitre@linaro.org \
--cc=rjw@sisk.pl \
--cc=tony@atomide.com \
/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
all inboxes | Powered by JetHome®