diff options
| author | Mike Miller <mike.miller@hp.com> | 2005-09-13 04:25:23 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-09-13 11:22:30 -0400 |
| commit | 6a445d3ba6b90ce13a843ad5d1a0867388b08096 (patch) | |
| tree | 30a07d018d74fbd99d323d6d6a6e08cac3a1b767 | |
| parent | 33079b21978f478865068ee6a3c5807b6c6ecdbe (diff) | |
[PATCH] cciss: bug fix in cciss_remove_one
This patch fixes a bug in cciss_remove_one. A set of braces was missing for
the if statement causing an Oops on driver unload.
Signed-off-by: Mike Miller <mike.miller@hp.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | drivers/block/cciss.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c index a12b95eef18e..49f05410a117 100644 --- a/drivers/block/cciss.c +++ b/drivers/block/cciss.c | |||
| @@ -3095,9 +3095,10 @@ static void __devexit cciss_remove_one (struct pci_dev *pdev) | |||
| 3095 | /* remove it from the disk list */ | 3095 | /* remove it from the disk list */ |
| 3096 | for (j = 0; j < NWD; j++) { | 3096 | for (j = 0; j < NWD; j++) { |
| 3097 | struct gendisk *disk = hba[i]->gendisk[j]; | 3097 | struct gendisk *disk = hba[i]->gendisk[j]; |
| 3098 | if (disk->flags & GENHD_FL_UP) | 3098 | if (disk->flags & GENHD_FL_UP) { |
| 3099 | blk_cleanup_queue(disk->queue); | ||
| 3100 | del_gendisk(disk); | 3099 | del_gendisk(disk); |
| 3100 | blk_cleanup_queue(disk->queue); | ||
| 3101 | } | ||
| 3101 | } | 3102 | } |
| 3102 | 3103 | ||
| 3103 | pci_free_consistent(hba[i]->pdev, NR_CMDS * sizeof(CommandList_struct), | 3104 | pci_free_consistent(hba[i]->pdev, NR_CMDS * sizeof(CommandList_struct), |
