diff options
author | Alasdair G Kergon <agk@redhat.com> | 2008-10-10 08:37:05 -0400 |
---|---|---|
committer | Alasdair G Kergon <agk@redhat.com> | 2008-10-10 08:37:05 -0400 |
commit | 1e37bb8e557a186d327eb4d1387953880ffc2cdd (patch) | |
tree | dfd5c65642762a69ed94136cde90e92a09d4cc15 /drivers/md/dm-crypt.c | |
parent | fc5a5e9aa878f86642c962b309f793fb2db0727e (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>
Diffstat (limited to 'drivers/md/dm-crypt.c')
-rw-r--r-- | drivers/md/dm-crypt.c | 9 |
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 | ||
694 | static void kcryptd_crypt_write_convert(struct dm_crypt_io *io) | 692 | static 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 | ||