xen/blkback: Checkpatch.pl recommend against multiple assigments.
authorKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Thu, 12 May 2011 20:19:23 +0000 (16:19 -0400)
committerKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Thu, 12 May 2011 21:55:47 +0000 (17:55 -0400)
CHECK: multiple assignments should be avoided

Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
drivers/block/xen-blkback/blkback.c
drivers/block/xen-blkback/xenbus.c

index 453b51ac737f1bc93a66f952b2ec3fba99d1c117..362fbf6f656d84c71c78798cc76c7f115c0cfcc1 100644 (file)
@@ -620,10 +620,11 @@ static int dispatch_rw_block_io(struct blkif_st *blkif,
                                     seg[i].nsec << 9,
                                     seg[i].buf & ~PAGE_MASK) == 0)) {
 
-                       bio = biolist[nbio++] = bio_alloc(GFP_KERNEL, nseg-i);
+                       bio = bio_alloc(GFP_KERNEL, nseg-i);
                        if (unlikely(bio == NULL))
                                goto fail_put_bio;
 
+                       biolist[nbio++] = bio;
                        bio->bi_bdev    = preq.bdev;
                        bio->bi_private = pending_req;
                        bio->bi_end_io  = end_block_io_op;
@@ -636,10 +637,12 @@ static int dispatch_rw_block_io(struct blkif_st *blkif,
        /* This will be hit if the operation was a flush. */
        if (!bio) {
                BUG_ON(operation != WRITE_FLUSH);
-               bio = biolist[nbio++] = bio_alloc(GFP_KERNEL, 0);
+               
+               bio = bio_alloc(GFP_KERNEL, 0);
                if (unlikely(bio == NULL))
                        goto fail_put_bio;
 
+               biolist[nbio++] = bio;
                bio->bi_bdev    = preq.bdev;
                bio->bi_private = pending_req;
                bio->bi_end_io  = end_block_io_op;
@@ -677,7 +680,7 @@ static int dispatch_rw_block_io(struct blkif_st *blkif,
        return -EIO;
 
  fail_put_bio:
-       for (i = 0; i < (nbio-1); i++)
+       for (i = 0; i < nbio; i++)
                bio_put(biolist[i]);
        __end_block_io_op(pending_req, -EINVAL);
        msleep(1); /* back off a bit */
index 5d2bbf6240c8159bb385f8c339cd7dd06377e1cf..ba8d30662d1944731502815639bf0c7c49adff97 100644 (file)
@@ -538,7 +538,8 @@ static void backend_changed(struct xenbus_watch *watch,
                err = vbd_create(be->blkif, handle, major, minor,
                                 (NULL == strchr(be->mode, 'w')), cdrom);
                if (err) {
-                       be->major = be->minor = 0;
+                       be->major = 0;
+                       be->minor = 0;
                        xenbus_dev_fatal(dev, err, "creating vbd structure");
                        return;
                }
@@ -546,7 +547,8 @@ static void backend_changed(struct xenbus_watch *watch,
                err = xenvbd_sysfs_addif(dev);
                if (err) {
                        vbd_free(&be->blkif->vbd);
-                       be->major = be->minor = 0;
+                       be->major = 0;
+                       be->minor = 0;
                        xenbus_dev_fatal(dev, err, "creating sysfs entries");
                        return;
                }