Commit a356e426 authored by Jose Castillo's avatar Jose Castillo Committed by Mike Snitzer

dm mpath: print more useful warnings in multipath_message()

The warning message "Unrecognised multipath message received" is
displayed in two different situations in multipath_message(): when the
number of arguments passed is invalid and when the string passed in
argv[0] is not recognized.

Make it easier to identify where the problem is by making these warnings
more specific with additional context for each case.
Signed-off-by: default avatarJose Castillo <jcastillo@redhat.com>
Signed-off-by: default avatarMike Snitzer <snitzer@redhat.com>
parent 3a017509
...@@ -1478,7 +1478,7 @@ static int multipath_message(struct dm_target *ti, unsigned argc, char **argv) ...@@ -1478,7 +1478,7 @@ static int multipath_message(struct dm_target *ti, unsigned argc, char **argv)
} }
if (argc != 2) { if (argc != 2) {
DMWARN("Unrecognised multipath message received."); DMWARN("Invalid multipath message arguments. Expected 2 arguments, got %d.", argc);
goto out; goto out;
} }
...@@ -1496,7 +1496,7 @@ static int multipath_message(struct dm_target *ti, unsigned argc, char **argv) ...@@ -1496,7 +1496,7 @@ static int multipath_message(struct dm_target *ti, unsigned argc, char **argv)
else if (!strcasecmp(argv[0], "fail_path")) else if (!strcasecmp(argv[0], "fail_path"))
action = fail_path; action = fail_path;
else { else {
DMWARN("Unrecognised multipath message received."); DMWARN("Unrecognised multipath message received: %s", argv[0]);
goto out; goto out;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment