Skip to content

hwmon updates for 2.6.19

Greg KH lm-sensors 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 14 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 14 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [lm-sensors] [PATCH 00/14] hwmon updates for 2.6.19
Quoted 7 lines · Jean Delvare
Quoted 2 lines

Care to work on all of the __must_check warnings that the hwmon code now spits out in the -mm and my trees? It's quite ugly :(

Mark M. Hoffman and others are already working on that:
http://lists.lm-sensors.org/pipermail/lm-sensors/2006-August/017463.html
http://lists.lm-sensors.org/pipermail/lm-sensors/2006-August/017204.html
http://lists.lm-sensors.org/pipermail/lm-sensors/2006-August/017462.html

Great.

Quoted 3 lines

Note that we decided to abuse the fact that removing a sysfs file which doesn't exist is not considered an error. It makes our code much more simple. I hope there is no plan to change this behavior.

Hm, that works? Ah, nice, I need to remember it, as it would make my other cleanups easier too.

Quoted 4 lines

I still wonder how the sysfs file creation functions did qualify for __must_check while nothing bad actually happens if the files are not created. OK, the files are missing, but kernel-wise this doesn't cause any broken state as far as I can see.

Why would you not check the return value of a function that provides you that information? :)

All of the driver core functions that return errors were marked in this manner due to a wide range of problems that we have been having for a while with drivers ignoring problems that the driver core has been reporting. We need to handle error conditions better, and this is finally forcing it to happen.

thanks,

greg k-h