diff options
author | Gabriel A. Devenyi <ace@staticwave.ca> | 2005-11-09 00:35:00 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-11-09 10:55:58 -0500 |
commit | e4a53cbabc81f04e24a5570b4aa6a6384bdbfc67 (patch) | |
tree | cfcfb5b0f93b30d035719ddffe069d3362155707 /drivers/block | |
parent | 55e64b3003f03355d8e950af58d456b6314a5a62 (diff) |
[PATCH] drivers/block/pktcdvd.c: remove write-only variable in pkt_iosched_process_queue()
Found this on Coverty's linux bug database (http://linuxbugsdb.coverity.com).
The function pkt_iosched_process_queue makes a call to bdev_get_queue and
stores the result but never uses it, so it looks like it can be safely
removed.
Signed-off-by: Gabriel A. Devenyi <ace@staticwave.ca>
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Acked-by: Peter Osterlund <petero2@telia.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/pktcdvd.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/block/pktcdvd.c b/drivers/block/pktcdvd.c index a280e679b1ca..59e5982a5db3 100644 --- a/drivers/block/pktcdvd.c +++ b/drivers/block/pktcdvd.c | |||
@@ -511,14 +511,11 @@ static void pkt_queue_bio(struct pktcdvd_device *pd, struct bio *bio) | |||
511 | */ | 511 | */ |
512 | static void pkt_iosched_process_queue(struct pktcdvd_device *pd) | 512 | static void pkt_iosched_process_queue(struct pktcdvd_device *pd) |
513 | { | 513 | { |
514 | request_queue_t *q; | ||
515 | 514 | ||
516 | if (atomic_read(&pd->iosched.attention) == 0) | 515 | if (atomic_read(&pd->iosched.attention) == 0) |
517 | return; | 516 | return; |
518 | atomic_set(&pd->iosched.attention, 0); | 517 | atomic_set(&pd->iosched.attention, 0); |
519 | 518 | ||
520 | q = bdev_get_queue(pd->bdev); | ||
521 | |||
522 | for (;;) { | 519 | for (;;) { |
523 | struct bio *bio; | 520 | struct bio *bio; |
524 | int reads_queued, writes_queued; | 521 | int reads_queued, writes_queued; |