dm crypt: move dec_pending on error into write_io_submit
authorMilan Broz <mbroz@redhat.com>
Fri, 10 Oct 2008 12:37:06 +0000 (13:37 +0100)
committerAlasdair G Kergon <agk@redhat.com>
Fri, 10 Oct 2008 12:37:06 +0000 (13:37 +0100)
Make kcryptd_crypt_write_io_submit() responsible for decrementing
the pending count after an error.

Also fixes a bug in the async path that forgot to decrement it.

Signed-off-by: Milan Broz <mbroz@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
drivers/md/dm-crypt.c

index 0042636ad375695c5817b70e56d49da7a0b8aae6..d8126ac829604e131cfea1119467c6f08c433f64 100644 (file)
@@ -674,6 +674,7 @@ static void kcryptd_crypt_write_io_submit(struct dm_crypt_io *io,
                crypt_free_buffer_pages(cc, clone);
                bio_put(clone);
                io->error = -EIO;
+               crypt_dec_pending(io);
                return;
        }
 
@@ -724,10 +725,8 @@ static void kcryptd_crypt_write_convert(struct dm_crypt_io *io)
                        /* processed, no running async crypto  */
                        crypt_inc_pending(io);
                        kcryptd_crypt_write_io_submit(io, r, 0);
-                       if (unlikely(r < 0)) {
-                               crypt_dec_pending(io);
+                       if (unlikely(r < 0))
                                break;
-                       }
                } else
                        crypt_inc_pending(io);