diff options
| author | Mikulas Patocka <mpatocka@redhat.com> | 2011-05-29 14:03:11 +0200 |
|---|---|---|
| committer | Alasdair G Kergon <agk@redhat.com> | 2011-05-29 14:03:11 +0200 |
| commit | 5f43ba2950414dc0abf4ac44c397d88069056746 (patch) | |
| tree | f80df9947ef30f965d9c33e649d1856f1e47220f /drivers/md/dm-snap.c | |
| parent | dm io: use fixed initial mempool size (diff) | |
| download | linux-5f43ba2950414dc0abf4ac44c397d88069056746.tar.xz linux-5f43ba2950414dc0abf4ac44c397d88069056746.zip | |
dm kcopyd: reserve fewer pages
Reserve just the minimum of pages needed to process one job.
Because we allocate pages from page allocator, we don't need to reserve
a large number of pages. The maximum job size is SUB_JOB_SIZE and we
calculate the number of reserved pages based on this.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Diffstat (limited to 'drivers/md/dm-snap.c')
| -rw-r--r-- | drivers/md/dm-snap.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/md/dm-snap.c b/drivers/md/dm-snap.c index a2d330942cb2..5a2296de84a3 100644 --- a/drivers/md/dm-snap.c +++ b/drivers/md/dm-snap.c @@ -40,11 +40,6 @@ static const char dm_snapshot_merge_target_name[] = "snapshot-merge"; #define SNAPSHOT_COPY_PRIORITY 2 /* - * Reserve 1MB for each snapshot initially (with minimum of 1 page). - */ -#define SNAPSHOT_PAGES (((1UL << 20) >> PAGE_SHIFT) ? : 1) - -/* * The size of the mempool used to track chunks in use. */ #define MIN_IOS 256 @@ -1116,7 +1111,7 @@ static int snapshot_ctr(struct dm_target *ti, unsigned int argc, char **argv) goto bad_hash_tables; } - r = dm_kcopyd_client_create(SNAPSHOT_PAGES, &s->kcopyd_client); + r = dm_kcopyd_client_create(&s->kcopyd_client); if (r) { ti->error = "Could not create kcopyd client"; goto bad_kcopyd; |