diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2008-07-30 01:33:26 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-07-30 12:41:45 -0400 |
commit | d667b6ddbcdc036a27407c8b2c1243f1dfd69e26 (patch) | |
tree | 4c43d584afd7cd5b83d914ae3f4fd364b6c2da91 | |
parent | 26c131c71e31973e273adde4027e6a80bde164dc (diff) |
hpwdt: don't use static flags
Static (read: global) is potential problem. Two threads can corrupt each
other's interrupt status, better avoid this.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Cc: Wim Van Sebroeck <wim@iguana.be>
Cc: Thomas Mingarelli <thomas.mingarelli@hp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/watchdog/hpwdt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/watchdog/hpwdt.c b/drivers/watchdog/hpwdt.c index eaa3f2a79ff5..ccd6c530782d 100644 --- a/drivers/watchdog/hpwdt.c +++ b/drivers/watchdog/hpwdt.c | |||
@@ -420,7 +420,7 @@ static int __devinit detect_cru_service(void) | |||
420 | static int hpwdt_pretimeout(struct notifier_block *nb, unsigned long ulReason, | 420 | static int hpwdt_pretimeout(struct notifier_block *nb, unsigned long ulReason, |
421 | void *data) | 421 | void *data) |
422 | { | 422 | { |
423 | static unsigned long rom_pl; | 423 | unsigned long rom_pl; |
424 | static int die_nmi_called; | 424 | static int die_nmi_called; |
425 | 425 | ||
426 | if (ulReason != DIE_NMI && ulReason != DIE_NMI_IPI) | 426 | if (ulReason != DIE_NMI && ulReason != DIE_NMI_IPI) |