Re: [lm-sensors] [REGRESSION, ABI] Re: LMSENSORS: 2.6.26-rc, enablingACPI Termal Zone support costs sensors

From: Rene Herman
Date: Mon Jun 23 2008 - 16:07:19 EST


This is a multi-part message in MIME format.On 23-06-08 19:54, Len Brown wrote:

Rene,
Thank you for reporting this.

I agree that this failure is an unwelcome surprise to those users
who upgrade to 2.6.26 but are still using libsensors <= 2.10.6.

Jean, Mark, Hans,

I'm actually fine with adding a temporary kernel config option
along the lines Rene suggested to ease the migration
to linux-2.6.26 for those users.

But the config option would need to be scheduled for removal
after a certain period (say 6 months) so we don't have to maintain
it forever.

More importantly, I think it would also have to be disabled by default
so that it would not have a negative impact on what we think are the
majority of properly configured systems. After all, we fixed this
bug in user-space about out 4 months ago and as you point out, the
distro upgrade path is actually quite well looked after.

So I'm not sure how useful it would be to the target users.
After they run into the problem, they'd probably google it
and find that they can either tweak a kernel config option
or upgrade libsensors. And we'd prefer that they do the
later rather than the former, yes?

just let me know.

Thank you for the reason. Yes, people upgrading libsensors would be preferred and this issue should show up in google now.

Frankly, if I had gone to the lm-sensors homepage and had seen a fixed 2.x version available I'd no doubt have shrugged it off then and there so I might as well do so now that Delvare announced that it's _going_ to be there real soon. 2.10.7 should be painless enough as an upgrade, it's needing lm-sensors 3.0(.2) and all its listed accompanying patches to programs depending on libsensors which triggered this thread.

So, <shrug>. I know how to fix my own systems, others will be able to find out and once lm-sensors 2.10.7 is available, it's not a big deal anymore.

That said, here's the "make it optional" patch tweaked according to your comments. Feel free to drop it on the floor...

Rene.