diff options
author | Tejun Heo <tj@kernel.org> | 2009-04-22 22:05:19 -0400 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2009-04-28 01:37:36 -0400 |
commit | f06d9a2b52e246a66b606130cea3f0d7b7be17a7 (patch) | |
tree | 020df1f9d54b00c72d8af02ac0827d496597e75a /drivers/block/swim3.c | |
parent | 40cbbb781d3eba5d6ac0860db078af490e5c7c6b (diff) |
block: replace end_request() with [__]blk_end_request_cur()
end_request() has been kept around for backward compatibility;
however, it's about time for it to go away.
* There aren't too many users left.
* Its use of @updtodate is pretty confusing.
* In some cases, newer code ends up using mixture of end_request() and
[__]blk_end_request[_all](), which is way too confusing.
So, add [__]blk_end_request_cur() and replace end_request() with it.
Most conversions are straightforward. Noteworthy ones are...
* paride/pcd: next_request() updated to take 0/-errno instead of 1/0.
* paride/pf: pf_end_request() and next_request() updated to take
0/-errno instead of 1/0.
* xd: xd_readwrite() updated to return 0/-errno instead of 1/0.
* mtd/mtd_blkdevs: blktrans_discard_request() updated to return
0/-errno instead of 1/0. Unnecessary local variable res
initialization removed from mtd_blktrans_thread().
[ Impact: cleanup ]
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Joerg Dorchain <joerg@dorchain.net>
Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Grant Likely <grant.likely@secretlab.ca>
Acked-by: Laurent Vivier <Laurent@lvivier.info>
Cc: Tim Waugh <tim@cyberelk.net>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Jeremy Fitzhardinge <jeremy@xensource.com>
Cc: Markus Lidel <Markus.Lidel@shadowconnect.com>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Pete Zaitcev <zaitcev@redhat.com>
Cc: unsik Kim <donari75@gmail.com>
Diffstat (limited to 'drivers/block/swim3.c')
-rw-r--r-- | drivers/block/swim3.c | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/drivers/block/swim3.c b/drivers/block/swim3.c index 612965307ba0..5904f7b73c6e 100644 --- a/drivers/block/swim3.c +++ b/drivers/block/swim3.c | |||
@@ -320,15 +320,15 @@ static void start_request(struct floppy_state *fs) | |||
320 | #endif | 320 | #endif |
321 | 321 | ||
322 | if (req->sector < 0 || req->sector >= fs->total_secs) { | 322 | if (req->sector < 0 || req->sector >= fs->total_secs) { |
323 | end_request(req, 0); | 323 | __blk_end_request_cur(req, -EIO); |
324 | continue; | 324 | continue; |
325 | } | 325 | } |
326 | if (req->current_nr_sectors == 0) { | 326 | if (req->current_nr_sectors == 0) { |
327 | end_request(req, 1); | 327 | __blk_end_request_cur(req, 0); |
328 | continue; | 328 | continue; |
329 | } | 329 | } |
330 | if (fs->ejected) { | 330 | if (fs->ejected) { |
331 | end_request(req, 0); | 331 | __blk_end_request_cur(req, -EIO); |
332 | continue; | 332 | continue; |
333 | } | 333 | } |
334 | 334 | ||
@@ -336,7 +336,7 @@ static void start_request(struct floppy_state *fs) | |||
336 | if (fs->write_prot < 0) | 336 | if (fs->write_prot < 0) |
337 | fs->write_prot = swim3_readbit(fs, WRITE_PROT); | 337 | fs->write_prot = swim3_readbit(fs, WRITE_PROT); |
338 | if (fs->write_prot) { | 338 | if (fs->write_prot) { |
339 | end_request(req, 0); | 339 | __blk_end_request_cur(req, -EIO); |
340 | continue; | 340 | continue; |
341 | } | 341 | } |
342 | } | 342 | } |
@@ -508,7 +508,7 @@ static void act(struct floppy_state *fs) | |||
508 | case do_transfer: | 508 | case do_transfer: |
509 | if (fs->cur_cyl != fs->req_cyl) { | 509 | if (fs->cur_cyl != fs->req_cyl) { |
510 | if (fs->retries > 5) { | 510 | if (fs->retries > 5) { |
511 | end_request(fd_req, 0); | 511 | __blk_end_request_cur(fd_req, -EIO); |
512 | fs->state = idle; | 512 | fs->state = idle; |
513 | return; | 513 | return; |
514 | } | 514 | } |
@@ -540,7 +540,7 @@ static void scan_timeout(unsigned long data) | |||
540 | out_8(&sw->intr_enable, 0); | 540 | out_8(&sw->intr_enable, 0); |
541 | fs->cur_cyl = -1; | 541 | fs->cur_cyl = -1; |
542 | if (fs->retries > 5) { | 542 | if (fs->retries > 5) { |
543 | end_request(fd_req, 0); | 543 | __blk_end_request_cur(fd_req, -EIO); |
544 | fs->state = idle; | 544 | fs->state = idle; |
545 | start_request(fs); | 545 | start_request(fs); |
546 | } else { | 546 | } else { |
@@ -559,7 +559,7 @@ static void seek_timeout(unsigned long data) | |||
559 | out_8(&sw->select, RELAX); | 559 | out_8(&sw->select, RELAX); |
560 | out_8(&sw->intr_enable, 0); | 560 | out_8(&sw->intr_enable, 0); |
561 | printk(KERN_ERR "swim3: seek timeout\n"); | 561 | printk(KERN_ERR "swim3: seek timeout\n"); |
562 | end_request(fd_req, 0); | 562 | __blk_end_request_cur(fd_req, -EIO); |
563 | fs->state = idle; | 563 | fs->state = idle; |
564 | start_request(fs); | 564 | start_request(fs); |
565 | } | 565 | } |
@@ -583,7 +583,7 @@ static void settle_timeout(unsigned long data) | |||
583 | return; | 583 | return; |
584 | } | 584 | } |
585 | printk(KERN_ERR "swim3: seek settle timeout\n"); | 585 | printk(KERN_ERR "swim3: seek settle timeout\n"); |
586 | end_request(fd_req, 0); | 586 | __blk_end_request_cur(fd_req, -EIO); |
587 | fs->state = idle; | 587 | fs->state = idle; |
588 | start_request(fs); | 588 | start_request(fs); |
589 | } | 589 | } |
@@ -615,7 +615,7 @@ static void xfer_timeout(unsigned long data) | |||
615 | fd_req->current_nr_sectors -= s; | 615 | fd_req->current_nr_sectors -= s; |
616 | printk(KERN_ERR "swim3: timeout %sing sector %ld\n", | 616 | printk(KERN_ERR "swim3: timeout %sing sector %ld\n", |
617 | (rq_data_dir(fd_req)==WRITE? "writ": "read"), (long)fd_req->sector); | 617 | (rq_data_dir(fd_req)==WRITE? "writ": "read"), (long)fd_req->sector); |
618 | end_request(fd_req, 0); | 618 | __blk_end_request_cur(fd_req, -EIO); |
619 | fs->state = idle; | 619 | fs->state = idle; |
620 | start_request(fs); | 620 | start_request(fs); |
621 | } | 621 | } |
@@ -646,7 +646,7 @@ static irqreturn_t swim3_interrupt(int irq, void *dev_id) | |||
646 | printk(KERN_ERR "swim3: seen sector but cyl=ff?\n"); | 646 | printk(KERN_ERR "swim3: seen sector but cyl=ff?\n"); |
647 | fs->cur_cyl = -1; | 647 | fs->cur_cyl = -1; |
648 | if (fs->retries > 5) { | 648 | if (fs->retries > 5) { |
649 | end_request(fd_req, 0); | 649 | __blk_end_request_cur(fd_req, -EIO); |
650 | fs->state = idle; | 650 | fs->state = idle; |
651 | start_request(fs); | 651 | start_request(fs); |
652 | } else { | 652 | } else { |
@@ -731,7 +731,7 @@ static irqreturn_t swim3_interrupt(int irq, void *dev_id) | |||
731 | printk("swim3: error %sing block %ld (err=%x)\n", | 731 | printk("swim3: error %sing block %ld (err=%x)\n", |
732 | rq_data_dir(fd_req) == WRITE? "writ": "read", | 732 | rq_data_dir(fd_req) == WRITE? "writ": "read", |
733 | (long)fd_req->sector, err); | 733 | (long)fd_req->sector, err); |
734 | end_request(fd_req, 0); | 734 | __blk_end_request_cur(fd_req, -EIO); |
735 | fs->state = idle; | 735 | fs->state = idle; |
736 | } | 736 | } |
737 | } else { | 737 | } else { |
@@ -740,7 +740,7 @@ static irqreturn_t swim3_interrupt(int irq, void *dev_id) | |||
740 | printk(KERN_ERR "swim3: fd dma: stat=%x resid=%d\n", stat, resid); | 740 | printk(KERN_ERR "swim3: fd dma: stat=%x resid=%d\n", stat, resid); |
741 | printk(KERN_ERR " state=%d, dir=%x, intr=%x, err=%x\n", | 741 | printk(KERN_ERR " state=%d, dir=%x, intr=%x, err=%x\n", |
742 | fs->state, rq_data_dir(fd_req), intr, err); | 742 | fs->state, rq_data_dir(fd_req), intr, err); |
743 | end_request(fd_req, 0); | 743 | __blk_end_request_cur(fd_req, -EIO); |
744 | fs->state = idle; | 744 | fs->state = idle; |
745 | start_request(fs); | 745 | start_request(fs); |
746 | break; | 746 | break; |
@@ -749,7 +749,7 @@ static irqreturn_t swim3_interrupt(int irq, void *dev_id) | |||
749 | fd_req->current_nr_sectors -= fs->scount; | 749 | fd_req->current_nr_sectors -= fs->scount; |
750 | fd_req->buffer += fs->scount * 512; | 750 | fd_req->buffer += fs->scount * 512; |
751 | if (fd_req->current_nr_sectors <= 0) { | 751 | if (fd_req->current_nr_sectors <= 0) { |
752 | end_request(fd_req, 1); | 752 | __blk_end_request_cur(fd_req, 0); |
753 | fs->state = idle; | 753 | fs->state = idle; |
754 | } else { | 754 | } else { |
755 | fs->req_sector += fs->scount; | 755 | fs->req_sector += fs->scount; |