diff options
author | Henrique de Moraes Holschuh <hmh@hmh.eng.br> | 2007-04-24 11:48:15 -0300 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2007-04-25 02:00:27 -0400 |
commit | 40ca9fdf8aa7d929e2b8939be1e6380d107381e1 (patch) | |
tree | 285d9e5a577b87064ecb06ee7aea46e206d1a3ac /drivers/misc/thinkpad_acpi.h | |
parent | 7252374a39d794879f5e47bcfa0a16e7599b27b5 (diff) | |
download | op-kernel-dev-40ca9fdf8aa7d929e2b8939be1e6380d107381e1.zip op-kernel-dev-40ca9fdf8aa7d929e2b8939be1e6380d107381e1.tar.gz |
ACPI: thinkpad-acpi: protect fan and hotkey data structures
Add proper mutex locking to some data structures access subject to races
due to concurrent access of driver functions on the hotkey and fan
subdrivers.
Signed-off-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/misc/thinkpad_acpi.h')
-rw-r--r-- | drivers/misc/thinkpad_acpi.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/misc/thinkpad_acpi.h b/drivers/misc/thinkpad_acpi.h index 84fdefe..a9feb53 100644 --- a/drivers/misc/thinkpad_acpi.h +++ b/drivers/misc/thinkpad_acpi.h @@ -30,6 +30,7 @@ #include <linux/types.h> #include <linux/string.h> #include <linux/list.h> +#include <linux/mutex.h> #include <linux/proc_fs.h> #include <linux/sysfs.h> @@ -375,6 +376,8 @@ static enum fan_control_commands fan_control_commands; static u8 fan_control_initial_status; static int fan_watchdog_maxinterval; +struct mutex fan_mutex; + static acpi_handle fans_handle, gfan_handle, sfan_handle; static int fan_init(struct ibm_init_struct *iibm); @@ -403,6 +406,8 @@ static int fan_write_cmd_watchdog(const char *cmd, int *rc); static int hotkey_orig_status; static int hotkey_orig_mask; +static struct mutex hotkey_mutex; + static int hotkey_init(struct ibm_init_struct *iibm); static void hotkey_exit(void); static int hotkey_get(int *status, int *mask); |