|
 |
b7f731 |
From 5e4ca8bb82e98400c9258cb3d7e4d030576f21df Mon Sep 17 00:00:00 2001
|
|
|
b7f731 |
From: Jes Sorensen <jsorensen@fb.com>
|
|
|
b7f731 |
Date: Wed, 19 Apr 2017 23:27:58 -0400
|
|
|
b7f731 |
Subject: [RHEL7.5 PATCH 081/169] sysfs: Parse array_state in sysfs_read()
|
|
|
b7f731 |
|
|
|
b7f731 |
Rather than copying in the array_state string, parse it and use an
|
|
|
b7f731 |
enum to indicate the state.
|
|
|
b7f731 |
|
|
|
b7f731 |
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
|
|
|
b7f731 |
---
|
|
|
b7f731 |
Manage.c | 2 +-
|
|
|
b7f731 |
maps.c | 17 +++++++++++++++++
|
|
|
b7f731 |
mdadm.h | 17 ++++++++++++++---
|
|
|
b7f731 |
sysfs.c | 9 +++++----
|
|
|
b7f731 |
4 files changed, 37 insertions(+), 8 deletions(-)
|
|
|
b7f731 |
|
|
|
b7f731 |
diff --git a/Manage.c b/Manage.c
|
|
|
b7f731 |
index bb84d28..8966e33 100644
|
|
|
b7f731 |
--- a/Manage.c
|
|
|
b7f731 |
+++ b/Manage.c
|
|
|
b7f731 |
@@ -929,7 +929,7 @@ int Manage_add(int fd, int tfd, struct mddev_dev *dv,
|
|
|
b7f731 |
return -1;
|
|
|
b7f731 |
}
|
|
|
b7f731 |
|
|
|
b7f731 |
- if (strncmp(mdp->sysfs_array_state, "readonly", 8) != 0) {
|
|
|
b7f731 |
+ if (mdp->array_state != ARRAY_READONLY) {
|
|
|
b7f731 |
sysfs_free(mdp);
|
|
|
b7f731 |
pr_err("%s is not readonly, cannot add journal.\n", devname);
|
|
|
b7f731 |
return -1;
|
|
|
b7f731 |
diff --git a/maps.c b/maps.c
|
|
|
b7f731 |
index d9ee7de..a8a4639 100644
|
|
|
b7f731 |
--- a/maps.c
|
|
|
b7f731 |
+++ b/maps.c
|
|
|
b7f731 |
@@ -139,6 +139,23 @@ mapping_t consistency_policies[] = {
|
|
|
b7f731 |
{ NULL, 0}
|
|
|
b7f731 |
};
|
|
|
b7f731 |
|
|
|
b7f731 |
+mapping_t sysfs_array_states[] = {
|
|
|
b7f731 |
+ /*
|
|
|
b7f731 |
+ * Beware map_name() uses strcmp() so active-idle must come before
|
|
|
b7f731 |
+ * active, to be detected correctly.
|
|
|
b7f731 |
+ */
|
|
|
b7f731 |
+ { "active-idle", ARRAY_ACTIVE_IDLE },
|
|
|
b7f731 |
+ { "active", ARRAY_ACTIVE },
|
|
|
b7f731 |
+ { "clear", ARRAY_CLEAR },
|
|
|
b7f731 |
+ { "inactive", ARRAY_INACTIVE },
|
|
|
b7f731 |
+ { "suspended", ARRAY_SUSPENDED },
|
|
|
b7f731 |
+ { "readonly", ARRAY_READONLY },
|
|
|
b7f731 |
+ { "read-auto", ARRAY_READ_AUTO },
|
|
|
b7f731 |
+ { "clean", ARRAY_CLEAN },
|
|
|
b7f731 |
+ { "write-pending", ARRAY_WRITE_PENDING },
|
|
|
b7f731 |
+ { NULL, 0 }
|
|
|
b7f731 |
+};
|
|
|
b7f731 |
+
|
|
|
b7f731 |
char *map_num(mapping_t *map, int num)
|
|
|
b7f731 |
{
|
|
|
b7f731 |
while (map->name) {
|
|
|
b7f731 |
diff --git a/mdadm.h b/mdadm.h
|
|
|
b7f731 |
index f1f643c..a379973 100644
|
|
|
b7f731 |
--- a/mdadm.h
|
|
|
b7f731 |
+++ b/mdadm.h
|
|
|
b7f731 |
@@ -335,8 +335,18 @@ struct mdinfo {
|
|
|
b7f731 |
int prev_state, curr_state, next_state;
|
|
|
b7f731 |
|
|
|
b7f731 |
/* info read from sysfs */
|
|
|
b7f731 |
- char sysfs_array_state[20];
|
|
|
b7f731 |
-
|
|
|
b7f731 |
+ enum {
|
|
|
b7f731 |
+ ARRAY_CLEAR,
|
|
|
b7f731 |
+ ARRAY_INACTIVE,
|
|
|
b7f731 |
+ ARRAY_SUSPENDED,
|
|
|
b7f731 |
+ ARRAY_READONLY,
|
|
|
b7f731 |
+ ARRAY_READ_AUTO,
|
|
|
b7f731 |
+ ARRAY_CLEAN,
|
|
|
b7f731 |
+ ARRAY_ACTIVE,
|
|
|
b7f731 |
+ ARRAY_WRITE_PENDING,
|
|
|
b7f731 |
+ ARRAY_ACTIVE_IDLE,
|
|
|
b7f731 |
+ ARRAY_UNKNOWN_STATE,
|
|
|
b7f731 |
+ } array_state;
|
|
|
b7f731 |
struct md_bb bb;
|
|
|
b7f731 |
};
|
|
|
b7f731 |
|
|
|
b7f731 |
@@ -716,7 +726,8 @@ extern int restore_stripes(int *dest, unsigned long long *offsets,
|
|
|
b7f731 |
|
|
|
b7f731 |
extern char *map_num(mapping_t *map, int num);
|
|
|
b7f731 |
extern int map_name(mapping_t *map, char *name);
|
|
|
b7f731 |
-extern mapping_t r5layout[], r6layout[], pers[], modes[], faultylayout[], consistency_policies[];
|
|
|
b7f731 |
+extern mapping_t r5layout[], r6layout[], pers[], modes[], faultylayout[];
|
|
|
b7f731 |
+extern mapping_t consistency_policies[], sysfs_array_states[];
|
|
|
b7f731 |
|
|
|
b7f731 |
extern char *map_dev_preferred(int major, int minor, int create,
|
|
|
b7f731 |
char *prefer);
|
|
|
b7f731 |
diff --git a/sysfs.c b/sysfs.c
|
|
|
b7f731 |
index 51deb23..c6df9b0 100644
|
|
|
b7f731 |
--- a/sysfs.c
|
|
|
b7f731 |
+++ b/sysfs.c
|
|
|
b7f731 |
@@ -247,11 +247,12 @@ struct mdinfo *sysfs_read(int fd, char *devnm, unsigned long options)
|
|
|
b7f731 |
|
|
|
b7f731 |
if (options & GET_ARRAY_STATE) {
|
|
|
b7f731 |
strcpy(base, "array_state");
|
|
|
b7f731 |
- if (load_sys(fname, sra->sysfs_array_state,
|
|
|
b7f731 |
- sizeof(sra->sysfs_array_state)))
|
|
|
b7f731 |
+ if (load_sys(fname, buf, sizeof(buf)))
|
|
|
b7f731 |
goto abort;
|
|
|
b7f731 |
- } else
|
|
|
b7f731 |
- sra->sysfs_array_state[0] = 0;
|
|
|
b7f731 |
+ sra->array_state = map_name(sysfs_array_states, buf);
|
|
|
b7f731 |
+ if (sra->array_state == UnSet)
|
|
|
b7f731 |
+ sra->array_state = ARRAY_UNKNOWN_STATE;
|
|
|
b7f731 |
+ }
|
|
|
b7f731 |
|
|
|
b7f731 |
if (options & GET_CONSISTENCY_POLICY) {
|
|
|
b7f731 |
strcpy(base, "consistency_policy");
|
|
|
b7f731 |
--
|
|
|
b7f731 |
2.7.4
|
|
|
b7f731 |
|