The code acted like AUTO_RESIZE_UNDEFINED didn't exist, but since conf->auto_resize was never set to AUTO_RESIZE_NEVER, the default was in fact AUTO_RESIZE_UNDEFINED, which ended up getting treated like AUTO_RESIZE_GROW_SHRINK. Remove AUTO_RESIZE_UNDEFINED, so AUTO_RESIZE_NEVER really is the default. Fixes: 981b83ad1 ("multipathd: Add auto_resize config option") Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx> --- libmultipath/defaults.h | 1 - libmultipath/structs.h | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/libmultipath/defaults.h b/libmultipath/defaults.h index 64b633f2..d01f9712 100644 --- a/libmultipath/defaults.h +++ b/libmultipath/defaults.h @@ -56,7 +56,6 @@ #define DEFAULT_UNKNOWN_FIND_MULTIPATHS_TIMEOUT 1 #define DEFAULT_ALL_TG_PT ALL_TG_PT_OFF #define DEFAULT_RECHECK_WWID RECHECK_WWID_OFF -#define DEFAULT_AUTO_RESIZE AUTO_RESIZE_NEVER /* Enable no foreign libraries by default */ #define DEFAULT_ENABLE_FOREIGN "NONE" diff --git a/libmultipath/structs.h b/libmultipath/structs.h index a1aac1b4..f133e78f 100644 --- a/libmultipath/structs.h +++ b/libmultipath/structs.h @@ -180,8 +180,7 @@ enum queue_mode_states { }; enum auto_resize_state { - AUTO_RESIZE_UNDEF = 0, - AUTO_RESIZE_NEVER, + AUTO_RESIZE_NEVER = 0, AUTO_RESIZE_GROW_ONLY, AUTO_RESIZE_GROW_SHRINK, }; -- 2.43.0