lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 21 Aug 2006 11:11:27 +0200 From: Jean Delvare <khali@...ux-fr.org> To: "Mark M. Hoffman" <mhoffman@...htlink.com>, Michal Piotrowski <michal.k.k.piotrowski@...il.com> Cc: Andrew Morton <akpm@...l.org>, LKML <linux-kernel@...r.kernel.org>, lm-sensors@...sensors.org Subject: Re: [lm-sensors] [RFC][PATCH] hwmon:fix sparse warnings + error handling Mark, Michal, > Thanks for doing this... but Andrew please don't apply it. The sensors project > people are working on these even now, and we already have a patch for the > w83627hf driver... > > http://lists.lm-sensors.org/pipermail/lm-sensors/2006-August/017204.html > > Jean Delvare (hwmon maintainer) should be sending these up the chain soon. > > Michal: if you're interested in fixing any of the rest of them, please take > a look at the patch above to see the mechanism we intend to use. It actually > makes the drivers *smaller* than they were. The size change really depends on the driver. For older drivers with individual file registration (sometimes hidden behind macros) the driver size will indeed shrink, but for newer drivers with loop-based file registration, this would be a slight increase in size. Not that it really matters anyway, what matters is that we handle errors and file deletion properly from now on. Michal, if you go on working on this (and this is welcome), please follow what Mark did, as this is what we agreed was the best approach. Here is a quick status summary for drivers/hwmon: Done by Mark M. Hoffman: o asb100 o lm75 o lm78 o smsc47b397 o w83627hf Done by Jim Cromie: o pc87360 Will be done by David Hubbard: o w83627ehf Will be done by me: o f71805f o it87 o lm63 o lm83 o lm90 This leaves the following list: o abituguru o adm1021 o adm1025 o adm1026 o adm1031 o adm9240 o atxp1 o ds1621 o fscher o fscpos o gl518sm o gl520sm o lm77 o lm80 o lm85 o lm87 o lm92 o max1619 o sis5595 o smsc47m1 o smsc47m192 o via686a o vt8231 o w83781d o w83791d o w83792d o w83l785ts Almost 1000 warnings for drivers/hwmon alone... OTOH I wonder how device_create_file and friends qualified for __must_check given that nothing wrong can happen if they fail, from the kernel's point of view. The files are not created and that's about it. If you are going to fix some of the drivers listed above, please advertise on the lm-sensors list so that your work is not duplicated. As a side note, I have patches ready for everything under drivers/i2c already, I sent them on the i2c list last week and will push them soon now. Thanks, -- Jean Delvare - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists