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

[Rene Herman]

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.