[dm-devel] [PATCH v2 2/3] Add 'show groups' command to multipathd
Todd Gill
tgill at redhat.com
Tue Feb 9 19:49:17 UTC 2016
Command allows user to display pathgroups. This is a step towards
allowing a user to parse the output of multipathd and be able to
understand the groups.
Changes by Gris: (changes since v1)
* Remove 'update_multipath()' in show_groups() as it only update path
status which is not required for path group query.
* Disable white space padding in raw mode.
* Display first group as 1 not 0
Signed-off-by: Todd Gill <tgill at redhat.com>
Signed-off-by: Gris Ge <fge at redhat.com>
---
libmultipath/print.c | 76 ++++++++++++++++++++++++++++++++++++++++++-----
libmultipath/print.h | 9 ++++--
multipathd/cli.c | 2 ++
multipathd/cli.h | 4 ++-
multipathd/cli_handlers.c | 58 ++++++++++++++++++++++++++++++++++--
multipathd/cli_handlers.h | 1 +
multipathd/main.c | 1 +
7 files changed, 138 insertions(+), 13 deletions(-)
diff --git a/libmultipath/print.c b/libmultipath/print.c
index c313e3e..e89c736 100644
--- a/libmultipath/print.c
+++ b/libmultipath/print.c
@@ -405,13 +405,13 @@ snprint_pri (char * buff, size_t len, struct path * pp)
}
static int
-snprint_pg_selector (char * buff, size_t len, struct pathgroup * pgp)
+snprint_pg_selector (char * buff, size_t len, struct pathgroup * pgp, int group)
{
return snprint_str(buff, len, pgp->selector);
}
static int
-snprint_pg_pri (char * buff, size_t len, struct pathgroup * pgp)
+snprint_pg_pri (char * buff, size_t len, struct pathgroup * pgp, int group)
{
/*
* path group priority is not updated for every path prio change,
@@ -424,7 +424,7 @@ snprint_pg_pri (char * buff, size_t len, struct pathgroup * pgp)
}
static int
-snprint_pg_state (char * buff, size_t len, struct pathgroup * pgp)
+snprint_pg_state (char * buff, size_t len, struct pathgroup * pgp, int group)
{
switch (pgp->status) {
case PGSTATE_ENABLED:
@@ -647,6 +647,30 @@ get_path_layout (vector pathvec, int header)
}
}
+void
+get_pathgroup_layout (vector mpvec, int header)
+{
+ int i, j, k;
+ char buff[MAX_FIELD_LEN];
+ struct pathgroup * pgp;
+ struct multipath * mpp;
+
+ for (i = 0; pgd[i].header; i++) {
+ if (header)
+ pgd[i].width = strlen(pgd[i].header);
+ else
+ pgd[i].width = 0;
+
+ vector_foreach_slot (mpvec, mpp, j) {
+ vector_foreach_slot (mpp->pg, pgp, k) {
+ pgp->selector = mpp->selector; /* hack */
+ pgd[i].snprint(buff, MAX_FIELD_LEN, pgp, k);
+ pgd[i].width = MAX(pgd[i].width, strlen(buff));
+ }
+ }
+ }
+}
+
static void
reset_multipath_layout (void)
{
@@ -855,7 +879,7 @@ snprint_path (char * line, int len, char * format,
int
snprint_pathgroup (char * line, int len, char * format,
- struct pathgroup * pgp)
+ struct pathgroup * pgp, int group, int pad)
{
char * c = line; /* line cursor */
char * s = line; /* for padding */
@@ -880,8 +904,42 @@ snprint_pathgroup (char * line, int len, char * format,
if (!(data = pgd_lookup(*f)))
continue;
- data->snprint(buff, MAX_FIELD_LEN, pgp);
+ data->snprint(buff, MAX_FIELD_LEN, pgp, group);
PRINT(c, TAIL, "%s", buff);
+ if (pad)
+ PAD(data->width);
+ } while (*f++);
+
+ ENDLINE;
+ return (c - line);
+}
+
+int
+snprint_pathgroup_header (char * line, int len, char * format)
+{
+ char * c = line; /* line cursor */
+ char * s = line; /* for padding */
+ char * f = format; /* format string cursor */
+ int fwd;
+ struct pathgroup_data * data;
+
+ memset(line, 0, len);
+
+ do {
+ if (!TAIL)
+ break;
+
+ if (*f != '%') {
+ *c++ = *f;
+ NOPAD;
+ continue;
+ }
+ f++;
+
+ if (!(data = pgd_lookup(*f)))
+ continue; /* unknown wildcard */
+
+ PRINT(c, TAIL, "%s", data->header);
PAD(data->width);
} while (*f++);
@@ -975,7 +1033,8 @@ snprint_multipath_topology (char * buff, int len, struct multipath * mpp,
strcpy(f, "|-+- " PRINT_PG_INDENT);
} else
strcpy(f, "`-+- " PRINT_PG_INDENT);
- fwd += snprint_pathgroup(buff + fwd, len - fwd, fmt, pgp);
+ fwd += snprint_pathgroup(buff + fwd, len - fwd, fmt, pgp, j,
+ 1/*padding*/);
if (fwd > len)
return len;
@@ -1565,12 +1624,13 @@ print_multipath (struct multipath * mpp, char * style)
}
extern void
-print_pathgroup (struct pathgroup * pgp, char * style)
+print_pathgroup (struct pathgroup * pgp, char * style, int group)
{
char line[MAX_LINE_LEN];
memset(&line[0], 0, MAX_LINE_LEN);
- snprint_pathgroup(&line[0], MAX_LINE_LEN, style, pgp);
+ snprint_pathgroup(&line[0], MAX_LINE_LEN, style, pgp, group,
+ 1/*padding*/);
printf("%s", line);
}
diff --git a/libmultipath/print.h b/libmultipath/print.h
index 9344271..fb53e8c 100644
--- a/libmultipath/print.h
+++ b/libmultipath/print.h
@@ -1,6 +1,7 @@
#define PRINT_PATH_LONG "%w %i %d %D %p %t %T %s %o"
#define PRINT_PATH_INDENT "%i %d %D %t %T %o"
#define PRINT_PATH_CHECKER "%i %d %D %p %t %T %o %C"
+#define PRINT_GROUP_CHECKER "%s %p %t"
#define PRINT_MAP_STATUS "%n %F %Q %N %t %r"
#define PRINT_MAP_STATS "%n %0 %1 %2 %3 %4"
#define PRINT_MAP_NAMES "%n %d %w"
@@ -30,14 +31,18 @@ struct pathgroup_data {
char wildcard;
char * header;
int width;
- int (*snprint)(char * buff, size_t len, struct pathgroup * pgp);
+ int (*snprint)(char * buff, size_t len, struct pathgroup * pgp, int group);
};
void get_path_layout (vector pathvec, int header);
void get_multipath_layout (vector mpvec, int header);
+void get_pathgroup_layout (vector mpvec, int header);
int snprint_path_header (char *, int, char *);
int snprint_multipath_header (char *, int, char *);
+int snprint_pathgroup_header (char *, int, char *);
int snprint_path (char *, int, char *, struct path *, int);
+int snprint_pathgroup (char * line, int len, char * format,
+ struct pathgroup * pgp, int group, int pad);
int snprint_multipath (char *, int, char *, struct multipath *, int);
int snprint_multipath_topology (char *, int, struct multipath * mpp,
int verbosity);
@@ -55,7 +60,7 @@ int snprint_overrides (char *, int, struct hwentry *);
void print_multipath_topology (struct multipath * mpp, int verbosity);
void print_path (struct path * pp, char * style);
void print_multipath (struct multipath * mpp, char * style);
-void print_pathgroup (struct pathgroup * pgp, char * style);
+void print_pathgroup (struct pathgroup * pgp, char * style, int group);
void print_map (struct multipath * mpp, char * params);
void print_all_paths (vector pathvec, int banner);
void print_all_paths_custo (vector pathvec, int banner, char *fmt);
diff --git a/multipathd/cli.c b/multipathd/cli.c
index e0a6bec..284741a 100644
--- a/multipathd/cli.c
+++ b/multipathd/cli.c
@@ -168,6 +168,7 @@ load_keys (void)
r += add_key(keys, "paths", PATHS, 0);
r += add_key(keys, "maps", MAPS, 0);
r += add_key(keys, "multipaths", MAPS, 0);
+ r += add_key(keys, "groups", GROUPS, 0);
r += add_key(keys, "path", PATH, 1);
r += add_key(keys, "map", MAP, 1);
r += add_key(keys, "multipath", MAP, 1);
@@ -474,6 +475,7 @@ cli_init (void) {
add_handler(LIST+MAPS+FMT, NULL);
add_handler(LIST+MAPS+RAW+FMT, NULL);
add_handler(LIST+MAPS+TOPOLOGY, NULL);
+ add_handler(LIST+GROUPS, NULL);
add_handler(LIST+TOPOLOGY, NULL);
add_handler(LIST+MAP+TOPOLOGY, NULL);
add_handler(LIST+CONFIG, NULL);
diff --git a/multipathd/cli.h b/multipathd/cli.h
index f6d2726..2757087 100644
--- a/multipathd/cli.h
+++ b/multipathd/cli.h
@@ -15,6 +15,7 @@ enum {
__RESTOREQ,
__PATHS,
__MAPS,
+ __GROUPS,
__PATH,
__MAP,
__GROUP,
@@ -55,6 +56,7 @@ enum {
#define PATH (1 << __PATH)
#define MAP (1 << __MAP)
#define GROUP (1 << __GROUP)
+#define GROUPS (1 << __GROUPS)
#define RECONFIGURE (1 << __RECONFIGURE)
#define DAEMON (1 << __DAEMON)
#define STATUS (1 << __STATUS)
@@ -67,7 +69,7 @@ enum {
#define COUNT (1 << __COUNT)
#define WILDCARDS (1 << __WILDCARDS)
#define QUIT (1 << __QUIT)
-#define SHUTDOWN (1 << __SHUTDOWN)
+#define SHUTDOWN (1UL << __SHUTDOWN)
#define GETPRSTATUS (1UL << __GETPRSTATUS)
#define SETPRSTATUS (1UL << __SETPRSTATUS)
#define UNSETPRSTATUS (1UL << __UNSETPRSTATUS)
diff --git a/multipathd/cli_handlers.c b/multipathd/cli_handlers.c
index a44281f..012ee76 100644
--- a/multipathd/cli_handlers.c
+++ b/multipathd/cli_handlers.c
@@ -24,6 +24,50 @@
#include "uevent.h"
int
+show_groups (char ** r, int * len, struct vectors * vecs, char * style,
+ int pretty)
+{
+ int i, j;
+ struct multipath * mpp;
+ struct pathgroup * pgp;
+ char * c;
+ char * reply;
+ unsigned int maxlen = INITIAL_REPLY_LEN;
+ int again = 1;
+
+ reply = MALLOC(maxlen);
+
+ while (again) {
+ if (!reply)
+ return 1;
+
+ c = reply;
+
+ get_pathgroup_layout(vecs->mpvec, pretty);
+
+ if (pretty && VECTOR_SIZE(vecs->pathvec) > 0)
+ c += snprint_pathgroup_header(c, reply + maxlen - c,
+ style);
+
+ vector_foreach_slot(vecs->mpvec, mpp, i) {
+ vector_foreach_slot(mpp->pg, pgp, j) {
+ pgp->selector = mpp->selector; /* hack */
+ c += snprint_pathgroup(c, reply + maxlen - c,
+ style, pgp, j + 1,
+ pretty);
+ }
+ }
+
+ again = ((c - reply) == (maxlen - 1));
+
+ REALLOC_REPLY(reply, again, maxlen);
+ }
+ *r = reply;
+ *len = (int)(c - reply + 1);
+ return 0;
+}
+
+int
show_paths (char ** r, int * len, struct vectors * vecs, char * style,
int pretty)
{
@@ -127,7 +171,7 @@ show_maps_topology (char ** r, int * len, struct vectors * vecs)
char * reply;
unsigned int maxlen = INITIAL_REPLY_LEN;
int again = 1;
-
+
get_path_layout(vecs->pathvec, 0);
reply = MALLOC(maxlen);
@@ -267,7 +311,7 @@ cli_list_map_topology (void * v, char ** reply, int * len, void * data)
struct multipath * mpp;
struct vectors * vecs = (struct vectors *)data;
char * param = get_keyparam(v, MAP);
-
+
param = convert_dev(param, 0);
get_path_layout(vecs->pathvec, 0);
mpp = find_mp_by_str(vecs->mpvec, param);
@@ -385,6 +429,16 @@ show_maps (char ** r, int *len, struct vectors * vecs, char * style,
}
int
+cli_list_groups (void * v, char ** reply, int * len, void * data)
+{
+ struct vectors * vecs = (struct vectors *)data;
+
+ condlog(3, "list groups (operator)");
+
+ return show_groups(reply, len, vecs, PRINT_GROUP_CHECKER, 1);
+}
+
+int
cli_list_maps_fmt (void * v, char ** reply, int * len, void * data)
{
struct vectors * vecs = (struct vectors *)data;
diff --git a/multipathd/cli_handlers.h b/multipathd/cli_handlers.h
index 799f8da..844c553 100644
--- a/multipathd/cli_handlers.h
+++ b/multipathd/cli_handlers.h
@@ -4,6 +4,7 @@ int cli_list_paths_raw (void * v, char ** reply, int * len, void * data);
int cli_list_path (void * v, char ** reply, int * len, void * data);
int cli_list_status (void * v, char ** reply, int * len, void * data);
int cli_list_daemon (void * v, char ** reply, int * len, void * data);
+int cli_list_groups (void * v, char ** reply, int * len, void * data);
int cli_list_maps (void * v, char ** reply, int * len, void * data);
int cli_list_maps_fmt (void * v, char ** reply, int * len, void * data);
int cli_list_maps_raw (void * v, char ** reply, int * len, void * data);
diff --git a/multipathd/main.c b/multipathd/main.c
index 04f6d02..f1a297c 100644
--- a/multipathd/main.c
+++ b/multipathd/main.c
@@ -904,6 +904,7 @@ uxlsnrloop (void * ap)
set_handler_callback(LIST+PATHS+RAW+FMT, cli_list_paths_raw);
set_handler_callback(LIST+PATH, cli_list_path);
set_handler_callback(LIST+MAPS, cli_list_maps);
+ set_handler_callback(LIST+GROUPS, cli_list_groups);
set_handler_callback(LIST+STATUS, cli_list_status);
set_handler_callback(LIST+DAEMON, cli_list_daemon);
set_handler_callback(LIST+MAPS+STATUS, cli_list_maps_status);
--
2.5.0
More information about the dm-devel
mailing list