Commit ff875738 authored by Artur Paszkiewicz's avatar Artur Paszkiewicz Committed by Shaohua Li

raid5: separate header for log functions

Move raid5-cache declarations from raid5.h to raid5-log.h, add inline
wrappers for functions which will be shared with ppl and use them in
raid5 core instead of direct calls to raid5-cache.

Remove unused parameter from r5c_cache_data(), move two duplicated
pr_debug() calls to r5l_init_log().
Signed-off-by: default avatarArtur Paszkiewicz <artur.paszkiewicz@intel.com>
Signed-off-by: default avatarShaohua Li <shli@fb.com>
parent ea0213e0
...@@ -344,6 +344,8 @@ void r5c_handle_cached_data_endio(struct r5conf *conf, ...@@ -344,6 +344,8 @@ void r5c_handle_cached_data_endio(struct r5conf *conf,
} }
} }
void r5l_wake_reclaim(struct r5l_log *log, sector_t space);
/* Check whether we should flush some stripes to free up stripe cache */ /* Check whether we should flush some stripes to free up stripe cache */
void r5c_check_stripe_cache_usage(struct r5conf *conf) void r5c_check_stripe_cache_usage(struct r5conf *conf)
{ {
...@@ -2749,9 +2751,7 @@ void r5c_finish_stripe_write_out(struct r5conf *conf, ...@@ -2749,9 +2751,7 @@ void r5c_finish_stripe_write_out(struct r5conf *conf,
} }
} }
int int r5c_cache_data(struct r5l_log *log, struct stripe_head *sh)
r5c_cache_data(struct r5l_log *log, struct stripe_head *sh,
struct stripe_head_state *s)
{ {
struct r5conf *conf = sh->raid_conf; struct r5conf *conf = sh->raid_conf;
int pages = 0; int pages = 0;
...@@ -2914,6 +2914,10 @@ int r5l_init_log(struct r5conf *conf, struct md_rdev *rdev) ...@@ -2914,6 +2914,10 @@ int r5l_init_log(struct r5conf *conf, struct md_rdev *rdev)
{ {
struct request_queue *q = bdev_get_queue(rdev->bdev); struct request_queue *q = bdev_get_queue(rdev->bdev);
struct r5l_log *log; struct r5l_log *log;
char b[BDEVNAME_SIZE];
pr_debug("md/raid:%s: using device %s as journal\n",
mdname(conf->mddev), bdevname(rdev->bdev, b));
if (PAGE_SIZE != 4096) if (PAGE_SIZE != 4096)
return -EINVAL; return -EINVAL;
...@@ -3016,8 +3020,13 @@ int r5l_init_log(struct r5conf *conf, struct md_rdev *rdev) ...@@ -3016,8 +3020,13 @@ int r5l_init_log(struct r5conf *conf, struct md_rdev *rdev)
return -EINVAL; return -EINVAL;
} }
void r5l_exit_log(struct r5l_log *log) void r5l_exit_log(struct r5conf *conf)
{ {
struct r5l_log *log = conf->log;
conf->log = NULL;
synchronize_rcu();
flush_work(&log->disable_writeback_work); flush_work(&log->disable_writeback_work);
md_unregister_thread(&log->reclaim_thread); md_unregister_thread(&log->reclaim_thread);
mempool_destroy(log->meta_pool); mempool_destroy(log->meta_pool);
......
#ifndef _RAID5_LOG_H
#define _RAID5_LOG_H
extern int r5l_init_log(struct r5conf *conf, struct md_rdev *rdev);
extern void r5l_exit_log(struct r5conf *conf);
extern int r5l_write_stripe(struct r5l_log *log, struct stripe_head *head_sh);
extern void r5l_write_stripe_run(struct r5l_log *log);
extern void r5l_flush_stripe_to_raid(struct r5l_log *log);
extern void r5l_stripe_write_finished(struct stripe_head *sh);
extern int r5l_handle_flush_request(struct r5l_log *log, struct bio *bio);
extern void r5l_quiesce(struct r5l_log *log, int state);
extern bool r5l_log_disk_error(struct r5conf *conf);
extern bool r5c_is_writeback(struct r5l_log *log);
extern int
r5c_try_caching_write(struct r5conf *conf, struct stripe_head *sh,
struct stripe_head_state *s, int disks);
extern void
r5c_finish_stripe_write_out(struct r5conf *conf, struct stripe_head *sh,
struct stripe_head_state *s);
extern void r5c_release_extra_page(struct stripe_head *sh);
extern void r5c_use_extra_page(struct stripe_head *sh);
extern void r5l_wake_reclaim(struct r5l_log *log, sector_t space);
extern void r5c_handle_cached_data_endio(struct r5conf *conf,
struct stripe_head *sh, int disks, struct bio_list *return_bi);
extern int r5c_cache_data(struct r5l_log *log, struct stripe_head *sh);
extern void r5c_make_stripe_write_out(struct stripe_head *sh);
extern void r5c_flush_cache(struct r5conf *conf, int num);
extern void r5c_check_stripe_cache_usage(struct r5conf *conf);
extern void r5c_check_cached_full_stripe(struct r5conf *conf);
extern struct md_sysfs_entry r5c_journal_mode;
extern void r5c_update_on_rdev_error(struct mddev *mddev);
extern bool r5c_big_stripe_cached(struct r5conf *conf, sector_t sect);
static inline int log_stripe(struct stripe_head *sh, struct stripe_head_state *s)
{
struct r5conf *conf = sh->raid_conf;
if (conf->log) {
if (!test_bit(STRIPE_R5C_CACHING, &sh->state)) {
/* writing out phase */
if (s->waiting_extra_page)
return 0;
return r5l_write_stripe(conf->log, sh);
} else if (test_bit(STRIPE_LOG_TRAPPED, &sh->state)) {
/* caching phase */
return r5c_cache_data(conf->log, sh);
}
}
return -EAGAIN;
}
static inline void log_stripe_write_finished(struct stripe_head *sh)
{
struct r5conf *conf = sh->raid_conf;
if (conf->log)
r5l_stripe_write_finished(sh);
}
static inline void log_write_stripe_run(struct r5conf *conf)
{
if (conf->log)
r5l_write_stripe_run(conf->log);
}
static inline void log_exit(struct r5conf *conf)
{
if (conf->log)
r5l_exit_log(conf);
}
static inline int log_init(struct r5conf *conf, struct md_rdev *journal_dev)
{
if (journal_dev)
return r5l_init_log(conf, journal_dev);
return 0;
}
#endif
...@@ -64,6 +64,7 @@ ...@@ -64,6 +64,7 @@
#include "raid5.h" #include "raid5.h"
#include "raid0.h" #include "raid0.h"
#include "bitmap.h" #include "bitmap.h"
#include "raid5-log.h"
#define UNSUPPORTED_MDDEV_FLAGS (1L << MD_FAILFAST_SUPPORTED) #define UNSUPPORTED_MDDEV_FLAGS (1L << MD_FAILFAST_SUPPORTED)
...@@ -997,18 +998,8 @@ static void ops_run_io(struct stripe_head *sh, struct stripe_head_state *s) ...@@ -997,18 +998,8 @@ static void ops_run_io(struct stripe_head *sh, struct stripe_head_state *s)
might_sleep(); might_sleep();
if (!test_bit(STRIPE_R5C_CACHING, &sh->state)) { if (log_stripe(sh, s) == 0)
/* writing out phase */ return;
if (s->waiting_extra_page)
return;
if (r5l_write_stripe(conf->log, sh) == 0)
return;
} else { /* caching phase */
if (test_bit(STRIPE_LOG_TRAPPED, &sh->state)) {
r5c_cache_data(conf->log, sh, s);
return;
}
}
should_defer = conf->batch_bio_dispatch && conf->group_cnt; should_defer = conf->batch_bio_dispatch && conf->group_cnt;
...@@ -3345,7 +3336,7 @@ handle_failed_stripe(struct r5conf *conf, struct stripe_head *sh, ...@@ -3345,7 +3336,7 @@ handle_failed_stripe(struct r5conf *conf, struct stripe_head *sh,
if (bi) if (bi)
bitmap_end = 1; bitmap_end = 1;
r5l_stripe_write_finished(sh); log_stripe_write_finished(sh);
if (test_and_clear_bit(R5_Overlap, &sh->dev[i].flags)) if (test_and_clear_bit(R5_Overlap, &sh->dev[i].flags))
wake_up(&conf->wait_for_overlap); wake_up(&conf->wait_for_overlap);
...@@ -3764,7 +3755,7 @@ static void handle_stripe_clean_event(struct r5conf *conf, ...@@ -3764,7 +3755,7 @@ static void handle_stripe_clean_event(struct r5conf *conf,
discard_pending = 1; discard_pending = 1;
} }
r5l_stripe_write_finished(sh); log_stripe_write_finished(sh);
if (!discard_pending && if (!discard_pending &&
test_bit(R5_Discard, &sh->dev[sh->pd_idx].flags)) { test_bit(R5_Discard, &sh->dev[sh->pd_idx].flags)) {
...@@ -4754,7 +4745,7 @@ static void handle_stripe(struct stripe_head *sh) ...@@ -4754,7 +4745,7 @@ static void handle_stripe(struct stripe_head *sh)
if (s.just_cached) if (s.just_cached)
r5c_handle_cached_data_endio(conf, sh, disks, &s.return_bi); r5c_handle_cached_data_endio(conf, sh, disks, &s.return_bi);
r5l_stripe_write_finished(sh); log_stripe_write_finished(sh);
/* Now we might consider reading some blocks, either to check/generate /* Now we might consider reading some blocks, either to check/generate
* parity, or to satisfy requests * parity, or to satisfy requests
...@@ -6168,7 +6159,7 @@ static int handle_active_stripes(struct r5conf *conf, int group, ...@@ -6168,7 +6159,7 @@ static int handle_active_stripes(struct r5conf *conf, int group,
for (i = 0; i < batch_size; i++) for (i = 0; i < batch_size; i++)
handle_stripe(batch[i]); handle_stripe(batch[i]);
r5l_write_stripe_run(conf->log); log_write_stripe_run(conf);
cond_resched(); cond_resched();
...@@ -6745,8 +6736,8 @@ static void free_conf(struct r5conf *conf) ...@@ -6745,8 +6736,8 @@ static void free_conf(struct r5conf *conf)
{ {
int i; int i;
if (conf->log) log_exit(conf);
r5l_exit_log(conf->log);
if (conf->shrinker.nr_deferred) if (conf->shrinker.nr_deferred)
unregister_shrinker(&conf->shrinker); unregister_shrinker(&conf->shrinker);
...@@ -7436,14 +7427,8 @@ static int raid5_run(struct mddev *mddev) ...@@ -7436,14 +7427,8 @@ static int raid5_run(struct mddev *mddev)
blk_queue_max_hw_sectors(mddev->queue, UINT_MAX); blk_queue_max_hw_sectors(mddev->queue, UINT_MAX);
} }
if (journal_dev) { if (log_init(conf, journal_dev))
char b[BDEVNAME_SIZE]; goto abort;
pr_debug("md/raid:%s: using device %s as journal\n",
mdname(mddev), bdevname(journal_dev->bdev, b));
if (r5l_init_log(conf, journal_dev))
goto abort;
}
return 0; return 0;
abort: abort:
...@@ -7557,17 +7542,13 @@ static int raid5_remove_disk(struct mddev *mddev, struct md_rdev *rdev) ...@@ -7557,17 +7542,13 @@ static int raid5_remove_disk(struct mddev *mddev, struct md_rdev *rdev)
print_raid5_conf(conf); print_raid5_conf(conf);
if (test_bit(Journal, &rdev->flags) && conf->log) { if (test_bit(Journal, &rdev->flags) && conf->log) {
struct r5l_log *log;
/* /*
* we can't wait pending write here, as this is called in * we can't wait pending write here, as this is called in
* raid5d, wait will deadlock. * raid5d, wait will deadlock.
*/ */
if (atomic_read(&mddev->writes_pending)) if (atomic_read(&mddev->writes_pending))
return -EBUSY; return -EBUSY;
log = conf->log; log_exit(conf);
conf->log = NULL;
synchronize_rcu();
r5l_exit_log(log);
return 0; return 0;
} }
if (rdev == p->rdev) if (rdev == p->rdev)
...@@ -7636,7 +7617,6 @@ static int raid5_add_disk(struct mddev *mddev, struct md_rdev *rdev) ...@@ -7636,7 +7617,6 @@ static int raid5_add_disk(struct mddev *mddev, struct md_rdev *rdev)
int last = conf->raid_disks - 1; int last = conf->raid_disks - 1;
if (test_bit(Journal, &rdev->flags)) { if (test_bit(Journal, &rdev->flags)) {
char b[BDEVNAME_SIZE];
if (conf->log) if (conf->log)
return -EBUSY; return -EBUSY;
...@@ -7645,9 +7625,7 @@ static int raid5_add_disk(struct mddev *mddev, struct md_rdev *rdev) ...@@ -7645,9 +7625,7 @@ static int raid5_add_disk(struct mddev *mddev, struct md_rdev *rdev)
* The array is in readonly mode if journal is missing, so no * The array is in readonly mode if journal is missing, so no
* write requests running. We should be safe * write requests running. We should be safe
*/ */
r5l_init_log(conf, rdev); log_init(conf, rdev);
pr_debug("md/raid:%s: using device %s as journal\n",
mdname(mddev), bdevname(rdev->bdev, b));
return 0; return 0;
} }
if (mddev->recovery_disabled == conf->recovery_disabled) if (mddev->recovery_disabled == conf->recovery_disabled)
......
...@@ -779,34 +779,4 @@ extern struct stripe_head * ...@@ -779,34 +779,4 @@ extern struct stripe_head *
raid5_get_active_stripe(struct r5conf *conf, sector_t sector, raid5_get_active_stripe(struct r5conf *conf, sector_t sector,
int previous, int noblock, int noquiesce); int previous, int noblock, int noquiesce);
extern int raid5_calc_degraded(struct r5conf *conf); extern int raid5_calc_degraded(struct r5conf *conf);
extern int r5l_init_log(struct r5conf *conf, struct md_rdev *rdev);
extern void r5l_exit_log(struct r5l_log *log);
extern int r5l_write_stripe(struct r5l_log *log, struct stripe_head *head_sh);
extern void r5l_write_stripe_run(struct r5l_log *log);
extern void r5l_flush_stripe_to_raid(struct r5l_log *log);
extern void r5l_stripe_write_finished(struct stripe_head *sh);
extern int r5l_handle_flush_request(struct r5l_log *log, struct bio *bio);
extern void r5l_quiesce(struct r5l_log *log, int state);
extern bool r5l_log_disk_error(struct r5conf *conf);
extern bool r5c_is_writeback(struct r5l_log *log);
extern int
r5c_try_caching_write(struct r5conf *conf, struct stripe_head *sh,
struct stripe_head_state *s, int disks);
extern void
r5c_finish_stripe_write_out(struct r5conf *conf, struct stripe_head *sh,
struct stripe_head_state *s);
extern void r5c_release_extra_page(struct stripe_head *sh);
extern void r5c_use_extra_page(struct stripe_head *sh);
extern void r5l_wake_reclaim(struct r5l_log *log, sector_t space);
extern void r5c_handle_cached_data_endio(struct r5conf *conf,
struct stripe_head *sh, int disks, struct bio_list *return_bi);
extern int r5c_cache_data(struct r5l_log *log, struct stripe_head *sh,
struct stripe_head_state *s);
extern void r5c_make_stripe_write_out(struct stripe_head *sh);
extern void r5c_flush_cache(struct r5conf *conf, int num);
extern void r5c_check_stripe_cache_usage(struct r5conf *conf);
extern void r5c_check_cached_full_stripe(struct r5conf *conf);
extern struct md_sysfs_entry r5c_journal_mode;
extern void r5c_update_on_rdev_error(struct mddev *mddev);
extern bool r5c_big_stripe_cached(struct r5conf *conf, sector_t sect);
#endif #endif
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment