raid5: separate header for log functions
authorArtur Paszkiewicz <artur.paszkiewicz@intel.com>
Thu, 9 Mar 2017 08:59:58 +0000 (09:59 +0100)
committerShaohua Li <shli@fb.com>
Thu, 16 Mar 2017 23:55:54 +0000 (16:55 -0700)
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: Artur Paszkiewicz <artur.paszkiewicz@intel.com>
Signed-off-by: Shaohua Li <shli@fb.com>
drivers/md/raid5-cache.c
drivers/md/raid5-log.h [new file with mode: 0644]
drivers/md/raid5.c
drivers/md/raid5.h

index 5c8640c86b90eb487ead2c65a1781f09a3a1b2ef..5f82dabdda6f54b698dcdd752d1b353f9917be4d 100644 (file)
@@ -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 */
 void r5c_check_stripe_cache_usage(struct r5conf *conf)
 {
@@ -2749,9 +2751,7 @@ void r5c_finish_stripe_write_out(struct r5conf *conf,
        }
 }
 
-int
-r5c_cache_data(struct r5l_log *log, struct stripe_head *sh,
-              struct stripe_head_state *s)
+int r5c_cache_data(struct r5l_log *log, struct stripe_head *sh)
 {
        struct r5conf *conf = sh->raid_conf;
        int pages = 0;
@@ -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 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)
                return -EINVAL;
@@ -3016,8 +3020,13 @@ io_kc:
        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);
        md_unregister_thread(&log->reclaim_thread);
        mempool_destroy(log->meta_pool);
diff --git a/drivers/md/raid5-log.h b/drivers/md/raid5-log.h
new file mode 100644 (file)
index 0000000..2da4bd3
--- /dev/null
@@ -0,0 +1,81 @@
+#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
index 013398ce2080bf10c038552729e9dc9cff895675..f575f40d2acbefa87ad6a998e5cd8a4159ce0d85 100644 (file)
@@ -64,6 +64,7 @@
 #include "raid5.h"
 #include "raid0.h"
 #include "bitmap.h"
+#include "raid5-log.h"
 
 #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)
 
        might_sleep();
 
-       if (!test_bit(STRIPE_R5C_CACHING, &sh->state)) {
-               /* writing out phase */
-               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;
-               }
-       }
+       if (log_stripe(sh, s) == 0)
+               return;
 
        should_defer = conf->batch_bio_dispatch && conf->group_cnt;
 
@@ -3345,7 +3336,7 @@ handle_failed_stripe(struct r5conf *conf, struct stripe_head *sh,
                if (bi)
                        bitmap_end = 1;
 
-               r5l_stripe_write_finished(sh);
+               log_stripe_write_finished(sh);
 
                if (test_and_clear_bit(R5_Overlap, &sh->dev[i].flags))
                        wake_up(&conf->wait_for_overlap);
@@ -3764,7 +3755,7 @@ returnbi:
                                discard_pending = 1;
                }
 
-       r5l_stripe_write_finished(sh);
+       log_stripe_write_finished(sh);
 
        if (!discard_pending &&
            test_bit(R5_Discard, &sh->dev[sh->pd_idx].flags)) {
@@ -4754,7 +4745,7 @@ static void handle_stripe(struct stripe_head *sh)
 
        if (s.just_cached)
                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
         * parity, or to satisfy requests
@@ -6168,7 +6159,7 @@ static int handle_active_stripes(struct r5conf *conf, int group,
 
        for (i = 0; i < batch_size; i++)
                handle_stripe(batch[i]);
-       r5l_write_stripe_run(conf->log);
+       log_write_stripe_run(conf);
 
        cond_resched();
 
@@ -6745,8 +6736,8 @@ static void free_conf(struct r5conf *conf)
 {
        int i;
 
-       if (conf->log)
-               r5l_exit_log(conf->log);
+       log_exit(conf);
+
        if (conf->shrinker.nr_deferred)
                unregister_shrinker(&conf->shrinker);
 
@@ -7436,14 +7427,8 @@ static int raid5_run(struct mddev *mddev)
                blk_queue_max_hw_sectors(mddev->queue, UINT_MAX);
        }
 
-       if (journal_dev) {
-               char b[BDEVNAME_SIZE];
-
-               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;
-       }
+       if (log_init(conf, journal_dev))
+               goto abort;
 
        return 0;
 abort:
@@ -7557,17 +7542,13 @@ static int raid5_remove_disk(struct mddev *mddev, struct md_rdev *rdev)
 
        print_raid5_conf(conf);
        if (test_bit(Journal, &rdev->flags) && conf->log) {
-               struct r5l_log *log;
                /*
                 * we can't wait pending write here, as this is called in
                 * raid5d, wait will deadlock.
                 */
                if (atomic_read(&mddev->writes_pending))
                        return -EBUSY;
-               log = conf->log;
-               conf->log = NULL;
-               synchronize_rcu();
-               r5l_exit_log(log);
+               log_exit(conf);
                return 0;
        }
        if (rdev == p->rdev)
@@ -7636,7 +7617,6 @@ static int raid5_add_disk(struct mddev *mddev, struct md_rdev *rdev)
        int last = conf->raid_disks - 1;
 
        if (test_bit(Journal, &rdev->flags)) {
-               char b[BDEVNAME_SIZE];
                if (conf->log)
                        return -EBUSY;
 
@@ -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
                 * write requests running. We should be safe
                 */
-               r5l_init_log(conf, rdev);
-               pr_debug("md/raid:%s: using device %s as journal\n",
-                        mdname(mddev), bdevname(rdev->bdev, b));
+               log_init(conf, rdev);
                return 0;
        }
        if (mddev->recovery_disabled == conf->recovery_disabled)
index 985cdc4850c2a30c3143761089fe991f40d5463f..6dd295a80ee14a31f2bd2472fae25837b0a227b9 100644 (file)
@@ -779,34 +779,4 @@ extern struct stripe_head *
 raid5_get_active_stripe(struct r5conf *conf, sector_t sector,
                        int previous, int noblock, int noquiesce);
 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