[lm-sensors] [patch 1/5] hwmon: Convert fschmd to unlocked_ioctl
tglx at linutronix.de
Fri Nov 6 16:56:34 CET 2009
On Fri, 6 Nov 2009, Jean Delvare wrote:
> I'm not sure why we need to hold the mutex here? My understanding is
> that watchdog_data_mutex protects watchdog_data_list and each
> watchdog's kref. And the above code doesn't touch either.
> What I am more worried about is why ident is declared static. This
> looks like a bug to me. Instead of abusing watchdog_data_mutex to
> workaround this, I'd rather remove the "static". I guess that the
> current code happens to work because neither data->revision nor
> nowayout can change over time, but this looks needlessly fragile.
Right, we can change ident to be non static instead of abusing the
mutex. Stupid me. Will fix.
More information about the lm-sensors