From: Gabriel C <nix.or.die@googlemail.com>
To: Jean Delvare <khali@linux-fr.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Zhang Rui <rui.zhang@intel.com>, Len Brown <len.brown@intel.com>,
Gabriel C <crazy@frugalware.org>
Subject: Re: Linux 2.6.25-rc6
Date: Mon, 17 Mar 2008 20:35:24 +0100 [thread overview]
Message-ID: <47DEC7FC.2060003@googlemail.com> (raw)
In-Reply-To: <20080317193941.6f247ea1@hyperion.delvare>
Jean Delvare wrote:
> Hi Gabriel,
>
> On Mon, 17 Mar 2008 15:34:04 +0100, Gabriel C wrote:
>> Linus Torvalds wrote:
>>
>>> Zhang, Rui (2):
>>> thermal: fix generic thermal I/F for hwmon
>> That commit broke lmsensros here ( found by bisect ).
>> ...
>>
>> Can't access procfs/sysfs file
>> Kernel interface access error
>> For 2.6 kernels, make sure you have mounted sysfs and libsensors
>> was compiled with sysfs support!
>>
>> ...
>>
>> Of course proc and sysfs is mounted and the lib has that support :)
>
> This has been reported as:
> http://bugzilla.kernel.org/show_bug.cgi?id=10259
> https://bugzilla.redhat.com/show_bug.cgi?id=437637
>
> The generic thermal zone device does something which is not
> fundamentally incorrect but that libsensors doesn't expect, and
> unfortunately libsensors was not made robust enough and dies instead of
> just ignoring the new unexpected device. libsensors 2.10.x is already
> fixed in lm-sensors' SVN [1] and a tentative patch is available for
> libsensors 3.0.x [2], however I am worried that kernel 2.6.25 will be
> released before any new version of lm-sensors so pretty much every user
> of lm-sensors will hit the problem if they upgrade to the new kernel.
> For this reason, I think we really should let the new generic thermal
> zone driver build as a module, and make it's hwmon support optional and
> disabled by default for 2.6.25 [3]. This will help mitigate the risk of
> massive breakage and complaints.
>
> [1] http://www.lm-sensors.org/changeset/5147
> [2] https://bugzilla.redhat.com/attachment.cgi?id=298270
> [3] http://lists.lm-sensors.org/pipermail/lm-sensors/2008-March/022724.html
>
Hmm , ok but then the situation is even worse.
Why something *known* to break _all_ existing lmsensors setups out there
is being pushed that late in game ?
IMO that should be just reverted from 2.6.25 and pushed again in the next merge window.
Best Regards,
Gabriel
next prev parent reply other threads:[~2008-03-17 19:35 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-17 0:01 Linux 2.6.25-rc6 Linus Torvalds
2008-03-17 14:34 ` Gabriel C
2008-03-17 18:39 ` Jean Delvare
2008-03-17 19:35 ` Gabriel C [this message]
2008-03-17 20:33 ` Jean Delvare
2008-03-18 4:57 ` Len Brown
2008-03-20 7:43 ` [BUG] Linux 2.6.25-rc6 - kernel BUG at fs/mpage.c:476! on powerpc Kamalesh Babulal
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=47DEC7FC.2060003@googlemail.com \
--to=nix.or.die@googlemail.com \
--cc=crazy@frugalware.org \
--cc=khali@linux-fr.org \
--cc=len.brown@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rui.zhang@intel.com \
--cc=torvalds@linux-foundation.org \
/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