aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlasdair G Kergon <agk@redhat.com>2008-10-10 08:37:05 -0400
committerAlasdair G Kergon <agk@redhat.com>2008-10-10 08:37:05 -0400
commit1e37bb8e557a186d327eb4d1387953880ffc2cdd (patch)
treedfd5c65642762a69ed94136cde90e92a09d4cc15
parentfc5a5e9aa878f86642c962b309f793fb2db0727e (diff)
dm crypt: remove inc_pending from write_io_submit
Make the caller reponsible for incrementing the pending count before calling kcryptd_crypt_write_io_submit() in the non-async case to bring it into line with the async case. Signed-off-by: Alasdair G Kergon <agk@redhat.com>
-rw-r--r--drivers/md/dm-crypt.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/drivers/md/dm-crypt.c b/drivers/md/dm-crypt.c
index 97b407582c03..0042636ad375 100644
--- a/drivers/md/dm-crypt.c
+++ b/drivers/md/dm-crypt.c
@@ -685,10 +685,8 @@ static void kcryptd_crypt_write_io_submit(struct dm_crypt_io *io,
685 685
686 if (async) 686 if (async)
687 kcryptd_queue_io(io); 687 kcryptd_queue_io(io);
688 else { 688 else
689 crypt_inc_pending(io);
690 generic_make_request(clone); 689 generic_make_request(clone);
691 }
692} 690}
693 691
694static void kcryptd_crypt_write_convert(struct dm_crypt_io *io) 692static void kcryptd_crypt_write_convert(struct dm_crypt_io *io)
@@ -724,9 +722,12 @@ static void kcryptd_crypt_write_convert(struct dm_crypt_io *io)
724 722
725 if (atomic_dec_and_test(&io->ctx.pending)) { 723 if (atomic_dec_and_test(&io->ctx.pending)) {
726 /* processed, no running async crypto */ 724 /* processed, no running async crypto */
725 crypt_inc_pending(io);
727 kcryptd_crypt_write_io_submit(io, r, 0); 726 kcryptd_crypt_write_io_submit(io, r, 0);
728 if (unlikely(r < 0)) 727 if (unlikely(r < 0)) {
728 crypt_dec_pending(io);
729 break; 729 break;
730 }
730 } else 731 } else
731 crypt_inc_pending(io); 732 crypt_inc_pending(io);
732 733