Merge branch 'next' of git://git.kernel.org/pub/scm/linux/kernel/git/benh/powerpc
[linux-2.6.git] / block / blk-lib.c
index d7a98d3..2b461b4 100644 (file)
@@ -19,11 +19,8 @@ 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);
+       if (err && (err != -EOPNOTSUPP))
                clear_bit(BIO_UPTODATE, &bb->flags);
-       }
        if (atomic_dec_and_test(&bb->done))
                complete(bb->wait);
        bio_put(bio);
@@ -62,7 +59,10 @@ int blkdev_issue_discard(struct block_device *bdev, sector_t sector,
         * granularity
         */
        max_discard_sectors = min(q->limits.max_discard_sectors, UINT_MAX >> 9);
-       if (q->limits.discard_granularity) {
+       if (unlikely(!max_discard_sectors)) {
+               /* Avoid infinite loop below. Being cautious never hurts. */
+               return -EOPNOTSUPP;
+       } else if (q->limits.discard_granularity) {
                unsigned int disc_sects = q->limits.discard_granularity >> 9;
 
                max_discard_sectors &= ~(disc_sects - 1);
@@ -107,9 +107,7 @@ int blkdev_issue_discard(struct block_device *bdev, sector_t sector,
        if (!atomic_dec_and_test(&bb.done))
                wait_for_completion(&wait);
 
-       if (test_bit(BIO_EOPNOTSUPP, &bb.flags))
-               ret = -EOPNOTSUPP;
-       else if (!test_bit(BIO_UPTODATE, &bb.flags))
+       if (!test_bit(BIO_UPTODATE, &bb.flags))
                ret = -EIO;
 
        return ret;