From: Jean Delvare <khali@linux-fr.org>
To: "Zhang, Rui" <rui.zhang@intel.com>, Len Brown <lenb@kernel.org>
Cc: LM Sensors <lm-sensors@lm-sensors.org>, linux-acpi@vger.kernel.org
Subject: Dependency chain of new thermal driver
Date: Sun, 16 Mar 2008 15:56:59 +0100 [thread overview]
Message-ID: <20080316155659.7ab2446d@hyperion.delvare> (raw)
Hi Rui,
I am testing your new thermal driver and I am not very happy with the
dependencies introduced by this driver. The ACPI thermal driver selects
the generic thermal driver, which in turn selects the hwmon base
module. As the generic thermal driver's configuration option is a
boolean, this means that as soon as one selects the ACPI thermal driver
(built-in or modular), the hwmon thermal driver has to be built into
the kernel. This is a problem especially when both the ACPI_THERMAL and
THERMAL options default to y.
I fail to see why we are using select at all. The ACPI thermal driver
clearly works without the generic thermal driver (even though the
generic interface is preferred now.) Likewise, the generic thermal
interface driver doesn't need the hwmon base module to work; the hwmon
interface is only an extension, so it should be possible to build the
generic thermal driver without hwmon support. On top of that, I really
would like to be able to build the generic thermal driver as a module.
One of the reasons why I would like this to change is this bug:
https://bugzilla.redhat.com/show_bug.cgi?id=437637
http://bugzilla.kernel.org/show_bug.cgi?id=10259
(At the moment I think both reports are the exact same bug.)
We will have to fix this bug of course (not sure how...) but the fact
that the users can't temporarily remove the generic thermal driver is a
problem both for bug investigation and for working around the bug until
it's fixed. I really would like to be able to tell the user "rmmod this
module until we come up with a fix", but right now I can't.
A more modular setup would give us much more flexibility both at build
time and when bugs are reported. Please think about it.
--
Jean Delvare
next reply other threads:[~2008-03-16 15:01 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-16 14:56 Jean Delvare [this message]
2008-03-18 5:31 ` [PATCH] Re: Dependency chain of new thermal driver Len Brown
2008-03-18 9:11 ` Jean Delvare
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=20080316155659.7ab2446d@hyperion.delvare \
--to=khali@linux-fr.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=lm-sensors@lm-sensors.org \
--cc=rui.zhang@intel.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