* [PATCH] btrfs-progs: replace fails start but in the background
2013-08-21 13:10 [PATCH] btrfs: introduce BTRFS_IOC_CHECK_DEV_EXCL_OPS ioctl to check dev excl op Anand Jain
@ 2013-08-21 13:10 ` Anand Jain
2013-09-06 9:38 ` Anand Jain
0 siblings, 1 reply; 3+ messages in thread
From: Anand Jain @ 2013-08-21 13:10 UTC (permalink / raw)
To: linux-btrfs
when the balance is running, the replace start ioctl
fails (for the right reasons). but since the cli has
put ioctl thread to background (for right reasons)
the user won't know that cli failed to start.
so before cli goes to the background, it should check
if mutually_exclusive_operation_running is not held.
this is done by newly introduced ioctl
BTRFS_IOC_CHECK_DEV_EXCL_OPS by the following kernel patch:
btrfs: introduce BTRFS_IOC_CHECK_DEV_EXCL_OPS ioctl to check dev excl op
Signed-off-by: Anand Jain <anand.jain@oracle.com>
---
cmds-replace.c | 14 ++++++++++++++
ioctl.h | 1 +
utils.c | 15 +++++++++++++++
utils.h | 1 +
4 files changed, 31 insertions(+), 0 deletions(-)
diff --git a/cmds-replace.c b/cmds-replace.c
index e3ff695..0b2cbc8 100644
--- a/cmds-replace.c
+++ b/cmds-replace.c
@@ -203,6 +203,20 @@ static int cmd_start_replace(int argc, char **argv)
goto leave_with_error;
}
+ /* check if there is some other device exclusive
+ * operation running in the FS which won't let this replace
+ * to run. And ENOTTY is when older kernel doesn't support
+ * lock checking ioctl
+ */
+ ret = is_dev_excl_op_free(fdmnt);
+ if (ret && ret != -ENOTTY) {
+ fprintf(stderr,
+ "ERROR: replace start failed on \"%s\" - %s\n",
+ path,
+ ret > 0 ? btrfs_err_str(ret) : strerror(-ret));
+ goto leave_with_error;
+ }
+
srcdev = argv[optind];
dstdev = argv[optind + 1];
diff --git a/ioctl.h b/ioctl.h
index e959720..afadcbc 100644
--- a/ioctl.h
+++ b/ioctl.h
@@ -598,6 +598,7 @@ struct btrfs_ioctl_clone_range_args {
#define BTRFS_IOC_DEV_REPLACE _IOWR(BTRFS_IOCTL_MAGIC, 53, \
struct btrfs_ioctl_dev_replace_args)
#define BTRFS_IOC_DEDUP_CTL _IOW(BTRFS_IOCTL_MAGIC, 55, int)
+#define BTRFS_IOC_CHECK_DEV_EXCL_OPS _IO(BTRFS_IOCTL_MAGIC, 56)
#ifdef __cplusplus
}
diff --git a/utils.c b/utils.c
index 422530d..f0f7719 100644
--- a/utils.c
+++ b/utils.c
@@ -1992,3 +1992,18 @@ int is_vol_small(char *file)
return 0;
}
}
+
+/* Returns:
+ * BTRFS_ERROR_DEV_EXCL_RUN_IN_PROGRESS:
+ * If the device is locked to prevent other device operations
+ * from the user cli like device add remove replace balance etc..
+ * < 0:
+ * For any other error including if kernel don't support the
+ * ioctl (-ENOTTY)
+ */
+int is_dev_excl_op_free(int fd)
+{
+ int ret;
+ ret = ioctl(fd, BTRFS_IOC_CHECK_DEV_EXCL_OPS, NULL);
+ return ret > 0 ? ret : -errno;
+}
diff --git a/utils.h b/utils.h
index eb6fba3..a1e5d67 100644
--- a/utils.h
+++ b/utils.h
@@ -83,4 +83,5 @@ int test_num_disk_vs_raid(u64 metadata_profile, u64 data_profile,
u64 dev_cnt, int mixed, char *estr);
int get_btrfs_mount(const char *dev, char *mp, size_t mp_size);
int is_vol_small(char *file);
+int is_dev_excl_op_free(int fd);
#endif
--
1.7.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] btrfs-progs: replace fails start but in the background
2013-08-21 13:10 ` [PATCH] btrfs-progs: replace fails start but in the background Anand Jain
@ 2013-09-06 9:38 ` Anand Jain
0 siblings, 0 replies; 3+ messages in thread
From: Anand Jain @ 2013-09-06 9:38 UTC (permalink / raw)
To: linux-btrfs; +Cc: dsterba
when the balance is running, the replace start ioctl
fails (for the right reasons). but since the cli has
put ioctl thread to background (for right reasons)
the user won't know that cli failed to start.
so before cli goes to the background, it should check
if mutually_exclusive_operation_running is not held.
this is done by newly introduced ioctl
BTRFS_IOC_CHECK_DEV_EXCL_OPS by the following kernel patch:
btrfs: introduce BTRFS_IOC_CHECK_DEV_EXCL_OPS ioctl to check dev excl op
Signed-off-by: Anand Jain <anand.jain@oracle.com>
---
cmds-replace.c | 14 ++++++++++++++
ioctl.h | 1 +
utils.c | 15 +++++++++++++++
utils.h | 1 +
4 files changed, 31 insertions(+), 0 deletions(-)
diff --git a/cmds-replace.c b/cmds-replace.c
index e89f8cc..916d997 100644
--- a/cmds-replace.c
+++ b/cmds-replace.c
@@ -203,6 +203,20 @@ static int cmd_start_replace(int argc, char **argv)
goto leave_with_error;
}
+ /* check if there is some other device exclusive
+ * operation running in the FS which won't let this replace
+ * to run. And ENOTTY is when older kernel doesn't support
+ * lock checking ioctl
+ */
+ ret = is_dev_excl_op_free(fdmnt);
+ if (ret && ret != -ENOTTY) {
+ fprintf(stderr,
+ "ERROR: replace start failed on \"%s\" - %s\n",
+ path,
+ ret > 0 ? btrfs_err_str(ret) : strerror(-ret));
+ goto leave_with_error;
+ }
+
srcdev = argv[optind];
dstdev = argv[optind + 1];
diff --git a/ioctl.h b/ioctl.h
index e959720..afadcbc 100644
--- a/ioctl.h
+++ b/ioctl.h
@@ -598,6 +598,7 @@ struct btrfs_ioctl_clone_range_args {
#define BTRFS_IOC_DEV_REPLACE _IOWR(BTRFS_IOCTL_MAGIC, 53, \
struct btrfs_ioctl_dev_replace_args)
#define BTRFS_IOC_DEDUP_CTL _IOW(BTRFS_IOCTL_MAGIC, 55, int)
+#define BTRFS_IOC_CHECK_DEV_EXCL_OPS _IO(BTRFS_IOCTL_MAGIC, 56)
#ifdef __cplusplus
}
diff --git a/utils.c b/utils.c
index 82672a7..54a80a6 100644
--- a/utils.c
+++ b/utils.c
@@ -1998,3 +1998,18 @@ int is_vol_small(char *file)
return 0;
}
}
+
+/* Returns:
+ * BTRFS_ERROR_DEV_EXCL_RUN_IN_PROGRESS:
+ * If the device is locked to prevent other device operations
+ * from the user cli like device add remove replace balance etc..
+ * < 0:
+ * For any other error including if kernel don't support the
+ * ioctl (-ENOTTY)
+ */
+int is_dev_excl_op_free(int fd)
+{
+ int ret;
+ ret = ioctl(fd, BTRFS_IOC_CHECK_DEV_EXCL_OPS, NULL);
+ return ret > 0 ? ret : -errno;
+}
diff --git a/utils.h b/utils.h
index 6446e0a..5ef728e 100644
--- a/utils.h
+++ b/utils.h
@@ -83,4 +83,5 @@ int test_num_disk_vs_raid(u64 metadata_profile, u64 data_profile,
u64 dev_cnt, int mixed, char *estr);
int is_vol_small(char *file);
int get_btrfs_mount(const char *dev, char *mp, size_t mp_size);
+int is_dev_excl_op_free(int fd);
#endif
--
1.7.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] btrfs-progs: replace fails start but in the background
@ 2013-09-13 11:37 Anand Jain
0 siblings, 0 replies; 3+ messages in thread
From: Anand Jain @ 2013-09-13 11:37 UTC (permalink / raw)
To: linux-btrfs; +Cc: dsterba
when the balance is running, the replace start ioctl
fails (for the right reasons). but since the cli has
put ioctl thread to background (for right reasons)
the user won't know that cli failed to start.
so before cli goes to the background, it should check
if mutually_exclusive_operation_running is not held.
this is done by newly introduced ioctl
BTRFS_IOC_CHECK_DEV_EXCL_OPS by the following kernel patch:
btrfs: introduce BTRFS_IOC_CHECK_DEV_EXCL_OPS ioctl to check dev excl op
Signed-off-by: Anand Jain <anand.jain@oracle.com>
---
cmds-replace.c | 14 ++++++++++++++
ioctl.h | 1 +
utils.c | 15 +++++++++++++++
utils.h | 1 +
4 files changed, 31 insertions(+)
diff --git a/cmds-replace.c b/cmds-replace.c
index a31d77e..f69c5ce 100644
--- a/cmds-replace.c
+++ b/cmds-replace.c
@@ -203,6 +203,20 @@ static int cmd_start_replace(int argc, char **argv)
goto leave_with_error;
}
+ /* check if there is some other device exclusive
+ * operation running in the FS which won't let this replace
+ * to run. And ENOTTY is when older kernel doesn't support
+ * lock checking ioctl
+ */
+ ret = is_dev_excl_op_free(fdmnt);
+ if (ret && ret != -ENOTTY) {
+ fprintf(stderr,
+ "ERROR: replace start failed on \"%s\" - %s\n",
+ path,
+ ret > 0 ? btrfs_err_str(ret) : strerror(-ret));
+ goto leave_with_error;
+ }
+
srcdev = argv[optind];
dstdev = argv[optind + 1];
diff --git a/ioctl.h b/ioctl.h
index c0dcc06..7df7fc9 100644
--- a/ioctl.h
+++ b/ioctl.h
@@ -598,6 +598,7 @@ struct btrfs_ioctl_clone_range_args {
#define BTRFS_IOC_DEV_REPLACE _IOWR(BTRFS_IOCTL_MAGIC, 53, \
struct btrfs_ioctl_dev_replace_args)
#define BTRFS_IOC_DEDUP_CTL _IOW(BTRFS_IOCTL_MAGIC, 55, int)
+#define BTRFS_IOC_CHECK_DEV_EXCL_OPS _IO(BTRFS_IOCTL_MAGIC, 56)
#ifdef __cplusplus
}
diff --git a/utils.c b/utils.c
index 02a2658..22c3310 100644
--- a/utils.c
+++ b/utils.c
@@ -2004,3 +2004,18 @@ int is_vol_small(char *file)
return 0;
}
}
+
+/* Returns:
+ * BTRFS_ERROR_DEV_EXCL_RUN_IN_PROGRESS:
+ * If the device is locked to prevent other device operations
+ * from the user cli like device add remove replace balance etc..
+ * < 0:
+ * For any other error including if kernel don't support the
+ * ioctl (-ENOTTY)
+ */
+int is_dev_excl_op_free(int fd)
+{
+ int ret;
+ ret = ioctl(fd, BTRFS_IOC_CHECK_DEV_EXCL_OPS, NULL);
+ return ret > 0 ? ret : -errno;
+}
diff --git a/utils.h b/utils.h
index 616bae1..6952d34 100644
--- a/utils.h
+++ b/utils.h
@@ -85,4 +85,5 @@ int is_vol_small(char *file);
int csum_tree_block(struct btrfs_root *root, struct extent_buffer *buf,
int verify);
int get_btrfs_mount(const char *dev, char *mp, size_t mp_size);
+int is_dev_excl_op_free(int fd);
#endif
--
1.8.4.rc4.1.g0d8beaa
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-09-13 11:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-13 11:37 [PATCH] btrfs-progs: replace fails start but in the background Anand Jain
-- strict thread matches above, loose matches on Subject: below --
2013-08-21 13:10 [PATCH] btrfs: introduce BTRFS_IOC_CHECK_DEV_EXCL_OPS ioctl to check dev excl op Anand Jain
2013-08-21 13:10 ` [PATCH] btrfs-progs: replace fails start but in the background Anand Jain
2013-09-06 9:38 ` Anand Jain
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).