Skip to content

Commit

Permalink
md/raid5: export raid5 unplugging interface.
Browse files Browse the repository at this point in the history
Also remove remaining accesses to ->queue and ->gendisk when ->queue
is NULL (As it is in a DM target).

Signed-off-by: NeilBrown <neilb@suse.de>
  • Loading branch information
NeilBrown committed Jul 26, 2010
1 parent 252ac52 commit 9f7c222
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 19 deletions.
37 changes: 18 additions & 19 deletions drivers/md/raid5.c
Original file line number Diff line number Diff line change
Expand Up @@ -434,7 +434,6 @@ static int has_failed(raid5_conf_t *conf)
}

static void unplug_slaves(mddev_t *mddev);
static void raid5_unplug_device(raid5_conf_t *conf);

static struct stripe_head *
get_active_stripe(raid5_conf_t *conf, sector_t sector,
Expand Down Expand Up @@ -464,7 +463,7 @@ get_active_stripe(raid5_conf_t *conf, sector_t sector,
< (conf->max_nr_stripes *3/4)
|| !conf->inactive_blocked),
conf->device_lock,
raid5_unplug_device(conf)
md_raid5_unplug_device(conf)
);
conf->inactive_blocked = 0;
} else
Expand Down Expand Up @@ -3659,7 +3658,7 @@ static void unplug_slaves(mddev_t *mddev)
rcu_read_unlock();
}

static void raid5_unplug_device(raid5_conf_t *conf)
void md_raid5_unplug_device(raid5_conf_t *conf)
{
unsigned long flags;

Expand All @@ -3675,17 +3674,18 @@ static void raid5_unplug_device(raid5_conf_t *conf)

unplug_slaves(conf->mddev);
}
EXPORT_SYMBOL_GPL(md_raid5_unplug_device);

static void raid5_unplug(struct plug_handle *plug)
{
raid5_conf_t *conf = container_of(plug, raid5_conf_t, plug);
raid5_unplug_device(conf);
md_raid5_unplug_device(conf);
}

static void raid5_unplug_queue(struct request_queue *q)
{
mddev_t *mddev = q->queuedata;
raid5_unplug_device(mddev->private);
md_raid5_unplug_device(mddev->private);
}

int md_raid5_congested(mddev_t *mddev, int bits)
Expand Down Expand Up @@ -4095,7 +4095,7 @@ static int make_request(mddev_t *mddev, struct bio * bi)
* add failed due to overlap. Flush everything
* and wait a while
*/
raid5_unplug_device(conf);
md_raid5_unplug_device(conf);
release_stripe(sh);
schedule();
goto retry;
Expand Down Expand Up @@ -4991,7 +4991,7 @@ static int only_parity(int raid_disk, int algo, int raid_disks, int max_degraded
static int run(mddev_t *mddev)
{
raid5_conf_t *conf;
int working_disks = 0, chunk_size;
int working_disks = 0;
int dirty_parity_disks = 0;
mdk_rdev_t *rdev;
sector_t reshape_offset = 0;
Expand Down Expand Up @@ -5191,6 +5191,7 @@ static int run(mddev_t *mddev)
plugger_init(&conf->plug, raid5_unplug);
mddev->plug = &conf->plug;
if (mddev->queue) {
int chunk_size;
/* read-ahead size must cover two whole stripes, which
* is 2 * (datadisks) * chunksize where 'n' is the
* number of raid devices
Expand All @@ -5205,20 +5206,18 @@ static int run(mddev_t *mddev)

mddev->queue->backing_dev_info.congested_data = mddev;
mddev->queue->backing_dev_info.congested_fn = raid5_congested;
}

mddev->queue->queue_lock = &conf->device_lock;

mddev->queue->unplug_fn = raid5_unplug_queue;
mddev->queue->queue_lock = &conf->device_lock;
mddev->queue->unplug_fn = raid5_unplug_queue;

chunk_size = mddev->chunk_sectors << 9;
blk_queue_io_min(mddev->queue, chunk_size);
blk_queue_io_opt(mddev->queue, chunk_size *
(conf->raid_disks - conf->max_degraded));
chunk_size = mddev->chunk_sectors << 9;
blk_queue_io_min(mddev->queue, chunk_size);
blk_queue_io_opt(mddev->queue, chunk_size *
(conf->raid_disks - conf->max_degraded));

list_for_each_entry(rdev, &mddev->disks, same_set)
disk_stack_limits(mddev->gendisk, rdev->bdev,
rdev->data_offset << 9);
list_for_each_entry(rdev, &mddev->disks, same_set)
disk_stack_limits(mddev->gendisk, rdev->bdev,
rdev->data_offset << 9);
}

return 0;
abort:
Expand Down
1 change: 1 addition & 0 deletions drivers/md/raid5.h
Original file line number Diff line number Diff line change
Expand Up @@ -502,5 +502,6 @@ static inline int algorithm_is_DDF(int layout)
}

extern int md_raid5_congested(mddev_t *mddev, int bits);
extern void md_raid5_unplug_device(raid5_conf_t *conf);
extern int raid5_set_cache_size(mddev_t *mddev, int size);
#endif

0 comments on commit 9f7c222

Please sign in to comment.