diff options
| author | Krzysztof Helt <krzysztof.h1@wp.pl> | 2008-01-11 21:50:46 +0100 |
|---|---|---|
| committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-01-23 18:29:25 +0100 |
| commit | d9aa3af09cdc5d3ae0e67bed4107bcf7e25b9f31 (patch) | |
| tree | 82d2699430907da4947022c406feabf855414bdf /drivers/scsi/u14-34f.c | |
| parent | [SCSI] aacraid: add call to flush_kernel_dcache_page (diff) | |
| download | linux-d9aa3af09cdc5d3ae0e67bed4107bcf7e25b9f31.tar.xz linux-d9aa3af09cdc5d3ae0e67bed4107bcf7e25b9f31.zip | |
[SCSI] sym53c8xx: fixes two bugs related to chip reset
This patch fixes two bugs pointed by James Bottomley:
1. the if (!sym_data->io_reset). That variable is only ever filled
by a stack based completion. If we find it non empty it means
this code has been entered twice and we have a severe problem,
so that should just become a BUG_ON(sym_data->io_reset).
2. sym_data->io_reset should be set to NULL before the routine is
exited otherwise the PCI recovery code could end up completing
what will be a bogus pointer into the stack.
Big thanks to James Bottomley for help with the patch.
Signed-off-by: Krzysztof Helt <krzysztof.h1@w.pl>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/scsi/u14-34f.c')
0 files changed, 0 insertions, 0 deletions