From: Anand Jain <anand.jain@oracle.com>
To: linux-btrfs@vger.kernel.org
Subject: [PATCH 1/2] btrfs: add mount option read_mirror_policy
Date: Tue, 30 Jan 2018 14:30:19 +0800 [thread overview]
Message-ID: <20180130063020.14850-2-anand.jain@oracle.com> (raw)
In-Reply-To: <20180130063020.14850-1-anand.jain@oracle.com>
In case of RAID1 and RAID10 devices are mirror-ed, a read IO can
pick any device for reading. This choice of picking a device for
reading should be configurable. In short not one policy would
satisfy all types of workload and configs.
So before we add more policies, this patch-set makes existing
$pid policy configurable from the mount option.
For example..
mount -o read_mirror_policy=pid (which is also default)
Signed-off-by: Anand Jain <anand.jain@oracle.com>
---
fs/btrfs/ctree.h | 2 ++
fs/btrfs/super.c | 10 ++++++++++
fs/btrfs/volumes.c | 8 +++++++-
fs/btrfs/volumes.h | 5 +++++
4 files changed, 24 insertions(+), 1 deletion(-)
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h
index 1a462ab85c49..4759e988b0df 100644
--- a/fs/btrfs/ctree.h
+++ b/fs/btrfs/ctree.h
@@ -1100,6 +1100,8 @@ struct btrfs_fs_info {
spinlock_t ref_verify_lock;
struct rb_root block_tree;
#endif
+ /* Policy to balance read across mirrored devices */
+ int read_mirror_policy;
};
static inline struct btrfs_fs_info *btrfs_sb(struct super_block *sb)
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c
index 367ecbf477b9..dfe6b3c67df3 100644
--- a/fs/btrfs/super.c
+++ b/fs/btrfs/super.c
@@ -329,6 +329,7 @@ enum {
#ifdef CONFIG_BTRFS_FS_REF_VERIFY
Opt_ref_verify,
#endif
+ Opt_read_mirror_policy,
Opt_err,
};
@@ -393,6 +394,7 @@ static const match_table_t tokens = {
#ifdef CONFIG_BTRFS_FS_REF_VERIFY
{Opt_ref_verify, "ref_verify"},
#endif
+ {Opt_read_mirror_policy, "read_mirror_policy=%s"},
{Opt_err, NULL},
};
@@ -839,6 +841,14 @@ int btrfs_parse_options(struct btrfs_fs_info *info, char *options,
btrfs_set_opt(info->mount_opt, REF_VERIFY);
break;
#endif
+ case Opt_read_mirror_policy:
+ if (strcmp(args[0].from, "pid") == 0) {
+ info->read_mirror_policy =
+ BTRFS_READ_MIRROR_BY_PID;
+ break;
+ }
+ ret = -EINVAL;
+ goto out;
case Opt_err:
btrfs_info(info, "unrecognized mount option '%s'", p);
ret = -EINVAL;
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c
index a61715677b67..39ba59832f38 100644
--- a/fs/btrfs/volumes.c
+++ b/fs/btrfs/volumes.c
@@ -5269,7 +5269,13 @@ static int find_live_mirror(struct btrfs_fs_info *fs_info,
else
num = map->num_stripes;
- optimal = first + current->pid % num;
+ switch(fs_info->read_mirror_policy) {
+ case BTRFS_READ_MIRROR_DEFAULT:
+ case BTRFS_READ_MIRROR_BY_PID:
+ default:
+ optimal = first + current->pid % num;
+ break;
+ }
if (dev_replace_is_ongoing &&
fs_info->dev_replace.cont_reading_from_srcdev_mode ==
diff --git a/fs/btrfs/volumes.h b/fs/btrfs/volumes.h
index 28c28eeadff3..78f35d299a61 100644
--- a/fs/btrfs/volumes.h
+++ b/fs/btrfs/volumes.h
@@ -47,6 +47,11 @@ struct btrfs_pending_bios {
#define btrfs_device_data_ordered_init(device) do { } while (0)
#endif
+enum btrfs_read_mirror_type {
+ BTRFS_READ_MIRROR_DEFAULT,
+ BTRFS_READ_MIRROR_BY_PID,
+};
+
#define BTRFS_DEV_STATE_WRITEABLE (0)
#define BTRFS_DEV_STATE_IN_FS_METADATA (1)
#define BTRFS_DEV_STATE_MISSING (2)
--
2.7.0
next prev parent reply other threads:[~2018-01-30 6:29 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-30 6:30 [PATCH 0/2] Policy to balance read across mirrored devices Anand Jain
2018-01-30 6:30 ` Anand Jain [this message]
2018-01-31 8:06 ` [PATCH 1/2] btrfs: add mount option read_mirror_policy Nikolay Borisov
2018-01-31 9:06 ` Anand Jain
2018-01-30 6:30 ` [PATCH 2/2] btrfs: add read_mirror_policy parameter devid Anand Jain
2018-01-31 8:38 ` Nikolay Borisov
2018-01-31 9:28 ` Anand Jain
2018-01-31 9:54 ` Nikolay Borisov
2018-01-31 13:38 ` Anand Jain
2018-01-31 13:42 ` Nikolay Borisov
2018-01-31 14:36 ` Anand Jain
2018-02-01 5:26 ` Edmund Nadolski
2018-02-01 8:12 ` Anand Jain
2018-02-01 23:46 ` Edmund Nadolski
2018-02-02 12:36 ` Austin S. Hemmelgarn
2018-02-05 7:21 ` Anand Jain
2018-01-31 7:51 ` [PATCH 0/2] Policy to balance read across mirrored devices Peter Becker
2018-01-31 9:01 ` Anand Jain
2018-01-31 10:47 ` Peter Becker
2018-01-31 14:26 ` Anand Jain
2018-01-31 14:52 ` Peter Becker
2018-01-31 16:11 ` Austin S. Hemmelgarn
2018-01-31 16:40 ` Peter Becker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180130063020.14850-2-anand.jain@oracle.com \
--to=anand.jain@oracle.com \
--cc=linux-btrfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).