diff options
author | Mikulas Patocka <mpatocka@redhat.com> | 2011-05-29 08:02:58 -0400 |
---|---|---|
committer | Alasdair G Kergon <agk@redhat.com> | 2011-05-29 08:02:58 -0400 |
commit | a705a34a565a5445bf731bd8006d51ea4d2b4236 (patch) | |
tree | 84d4761ff62398110a44e630085bc43a97bae805 | |
parent | 6f13f6fba76edc7d0e7580c5deee829d59a41b2f (diff) |
dm kcopyd: avoid pointless job splitting
Don't split SUB_JOB_SIZE jobs
If the job size equals SUB_JOB_SIZE, there is no point in splitting it.
Splitting it just unnecessarily wastes time, because the split job size
is SUB_JOB_SIZE too.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
-rw-r--r-- | drivers/md/dm-kcopyd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/dm-kcopyd.c b/drivers/md/dm-kcopyd.c index 1bb73a13ca40..505b6f5cd385 100644 --- a/drivers/md/dm-kcopyd.c +++ b/drivers/md/dm-kcopyd.c | |||
@@ -578,7 +578,7 @@ int dm_kcopyd_copy(struct dm_kcopyd_client *kc, struct dm_io_region *from, | |||
578 | job->fn = fn; | 578 | job->fn = fn; |
579 | job->context = context; | 579 | job->context = context; |
580 | 580 | ||
581 | if (job->source.count < SUB_JOB_SIZE) | 581 | if (job->source.count <= SUB_JOB_SIZE) |
582 | dispatch_job(job); | 582 | dispatch_job(job); |
583 | 583 | ||
584 | else { | 584 | else { |