From: Jens Axboe Date: Thu, 29 Apr 2010 07:28:21 +0000 (+0200) Subject: block: fix bad use of min() on different types X-Git-Tag: v2.6.35-rc1~442^2~62 X-Git-Url: https://bbs.cooldavid.org/git/?a=commitdiff_plain;h=0341aafb7f3313bcedc6811a098500be85f3fc77;p=net-next-2.6.git block: fix bad use of min() on different types Just cast the page size to sector_t, that will always fit. Signed-off-by: Jens Axboe --- diff --git a/block/blk-lib.c b/block/blk-lib.c index 886c3f9e1be..d0216b9f22d 100644 --- a/block/blk-lib.c +++ b/block/blk-lib.c @@ -124,6 +124,7 @@ struct bio_batch static void bio_batch_end_io(struct bio *bio, int err) { struct bio_batch *bb = bio->bi_private; + if (err) { if (err == -EOPNOTSUPP) set_bit(BIO_EOPNOTSUPP, &bb->flags); @@ -186,8 +187,8 @@ submit: if (flags & BLKDEV_IFL_WAIT) bio->bi_private = &bb; - while(nr_sects != 0) { - sz = min(PAGE_SIZE >> 9 , nr_sects); + while (nr_sects != 0) { + sz = min((sector_t) PAGE_SIZE >> 9 , nr_sects); if (sz == 0) /* bio has maximum size possible */ break;