diff options
author | Martin K. Petersen <martin.petersen@oracle.com> | 2011-03-17 06:11:05 -0400 |
---|---|---|
committer | Jens Axboe <jaxboe@fusionio.com> | 2011-03-17 06:11:05 -0400 |
commit | a91a2785b200864aef2270ed6a3babac7a253a20 (patch) | |
tree | 3d21d7ef82ddabbdbfb74499d68a73f15adc819f /drivers/md/raid0.c | |
parent | 82f04ab47e1d94d78503591a7460b2cad9601ede (diff) |
block: Require subsystems to explicitly allocate bio_set integrity mempool
MD and DM create a new bio_set for every metadevice. Each bio_set has an
integrity mempool attached regardless of whether the metadevice is
capable of passing integrity metadata. This is a waste of memory.
Instead we defer the allocation decision to MD and DM since we know at
metadevice creation time whether integrity passthrough is needed or not.
Automatic integrity mempool allocation can then be removed from
bioset_create() and we make an explicit integrity allocation for the
fs_bio_set.
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Reported-by: Zdenek Kabelac <zkabelac@redhat.com>
Acked-by: Mike Snitzer <snizer@redhat.com>
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'drivers/md/raid0.c')
-rw-r--r-- | drivers/md/raid0.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/md/raid0.c b/drivers/md/raid0.c index 95916fd6394a..e86bf3682e1e 100644 --- a/drivers/md/raid0.c +++ b/drivers/md/raid0.c | |||
@@ -379,8 +379,7 @@ static int raid0_run(mddev_t *mddev) | |||
379 | 379 | ||
380 | blk_queue_merge_bvec(mddev->queue, raid0_mergeable_bvec); | 380 | blk_queue_merge_bvec(mddev->queue, raid0_mergeable_bvec); |
381 | dump_zones(mddev); | 381 | dump_zones(mddev); |
382 | md_integrity_register(mddev); | 382 | return md_integrity_register(mddev); |
383 | return 0; | ||
384 | } | 383 | } |
385 | 384 | ||
386 | static int raid0_stop(mddev_t *mddev) | 385 | static int raid0_stop(mddev_t *mddev) |