diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-07-11 17:02:38 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-11 19:04:47 -0400 |
| commit | 8875408abd935a77b6e1cb11c21c438aa2e7ec75 (patch) | |
| tree | 25b8dc9d54b703651bc2ebabb2d6dea09f433eb0 /drivers/misc/sgi-xp/xp_uv.c | |
| parent | fea9f718b3d68147f162ed2d870183ce5e0ad8d8 (diff) | |
sgi-xp: nested calls to spin_lock_irqsave()
The code here has a nested spin_lock_irqsave(). It's not needed since
IRQs are already disabled and it causes a problem because it means that
IRQs won't be enabled again at the end. The second call to
spin_lock_irqsave() will overwrite the value of irq_flags and we can't
restore the proper settings.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Robin Holt <holt@sgi.com>
Cc: Jack Steiner <steiner@sgi.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/misc/sgi-xp/xp_uv.c')
0 files changed, 0 insertions, 0 deletions