From: Matthew Garrett <mjg59@srcf.ucam.org>
To: Thomas Renninger <trenn@suse.de>
Cc: Dave Jones <davej@codemonkey.org.uk>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
mzahor@dtech.sk, Dominik Brodowski <linux@dominikbrodowski.net>,
cpufreq <cpufreq@lists.linux.org.uk>
Subject: Re: [PATCH] Remove p4_clockmode driver
Date: Mon, 26 May 2008 13:05:07 +0100 [thread overview]
Message-ID: <20080526120507.GA23310@srcf.ucam.org> (raw)
In-Reply-To: <1211729971.3422.52.camel@linux-2bdv.site>
On Sun, May 25, 2008 at 05:39:31PM +0200, Thomas Renninger wrote:
> Could you explain this in two easy sentences again, why this should not
> use the throttling interface?
Because it involves rewriting code for purely aesthetic benefit. I agree
that, in an ideal world, p4-clockmod would integrate into the throttling
system. But unless someone wants to rewrite it, that doesn't look
likely...
> (former /proc/acpi/processor/*/throttling, now somewhere located in
> sysfs. Also used for passive cooling if no cpufreq is available).
> Throttling is what this driver is doing and with which it will interfere
> and slow down the machine to be closed to unresponsive if both
> (throttling and cpufreq interface) are using it?
The ACPI throttling interface will only be used if the minimum cpufreq
state has already been reached. Users really shouldn't play with the
throttling interface themselves, given its lack of useful functionality.
--
Matthew Garrett | mjg59@srcf.ucam.org
next prev parent reply other threads:[~2008-05-26 12:05 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-03 19:47 [PATCH] Remove p4_clockmode driver Thomas Renninger
2008-05-11 16:19 ` Dave Jones
2008-05-15 10:55 ` Dominik Brodowski
2008-05-16 18:32 ` Pallipadi, Venkatesh
2008-05-22 4:01 ` Dmitry Torokhov
2008-05-23 11:21 ` Thomas Renninger
2008-05-23 12:55 ` Dmitry Torokhov
2008-05-23 15:19 ` Thomas Renninger
2008-05-23 15:54 ` Dmitry Torokhov
2008-05-23 16:16 ` Thomas Renninger
2008-05-23 17:40 ` Dmitry Torokhov
2008-05-24 11:03 ` Dominik Brodowski
2008-05-25 15:39 ` Thomas Renninger
2008-05-26 12:05 ` Matthew Garrett [this message]
2008-05-26 12:56 ` Thomas Renninger
2008-05-12 22:30 ` Cesar Eduardo Barros
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=20080526120507.GA23310@srcf.ucam.org \
--to=mjg59@srcf.ucam.org \
--cc=cpufreq@lists.linux.org.uk \
--cc=davej@codemonkey.org.uk \
--cc=dmitry.torokhov@gmail.com \
--cc=linux@dominikbrodowski.net \
--cc=mzahor@dtech.sk \
--cc=trenn@suse.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.