<div dir="ltr">Merged,<div>thanks.</div></div><div class="gmail_extra"> <div class="gmail_quote">On Tue, May 15, 2018 at 2:32 PM, Martin Wilck <<a href="mailto:mwilck@suse.com" target="_blank">mwilck@suse.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If "find_multipaths strict" is set in multipath.conf, the error message "illegal value for find_multipaths: strict" is printed. This causes no functional problem, as "strict" happens to be the default, fallback value. It should be fixed nonetheless. FIND_MULTIPATHS_STRICT, having the highest numeric value, must be last in the enum. Fixes: c36f2f42 "libmultipath: change find_multipaths option to multi-value" Signed-off-by: Martin Wilck <<a href="mailto:mwilck@suse.com">mwilck@suse.com</a>> --- libmultipath/structs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libmultipath/structs.h b/libmultipath/structs.h index eb6a1788..e424b150 100644 --- a/libmultipath/structs.h +++ b/libmultipath/structs.h @@ -122,9 +122,9 @@ enum find_multipaths_states { FIND_MULTIPATHS_UNDEF = YNU_UNDEF, FIND_MULTIPATHS_OFF = YNU_NO, FIND_MULTIPATHS_ON = _FIND_MULTIPATHS_F, - FIND_MULTIPATHS_STRICT = _FIND_MULTIPATHS_F|_FIND_MULTIPATHS_N, FIND_MULTIPATHS_GREEDY = _FIND_MULTIPATHS_I, FIND_MULTIPATHS_SMART = _FIND_MULTIPATHS_F|_FIND_MULTIPATHS_I, + FIND_MULTIPATHS_STRICT = _FIND_MULTIPATHS_F|_FIND_MULTIPATHS_N, __FIND_MULTIPATHS_LAST, }; -- 2.16.3 </blockquote></div> </div>