From: Martin Wilck <mwilck@xxxxxxx> mpp->no_path_retry is already checked in setup_feature() itself, no need to do it here as well. This allows using setup_feature() for other features except queue_if_no_path. Signed-off-by: Martin Wilck <mwilck@xxxxxxxx> --- libmultipath/dmparser.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/libmultipath/dmparser.c b/libmultipath/dmparser.c index 274eb947..4940e8cf 100644 --- a/libmultipath/dmparser.c +++ b/libmultipath/dmparser.c @@ -183,10 +183,7 @@ int disassemble_map(vector pathvec, char *params, struct multipath *mpp, FREE(word); return 1; } - if ((mpp->no_path_retry == NO_PATH_RETRY_UNDEF) || - (mpp->no_path_retry == NO_PATH_RETRY_FAIL) || - (mpp->no_path_retry == NO_PATH_RETRY_QUEUE)) - setup_feature(mpp, word); + setup_feature(mpp, word); FREE(word); } -- 2.11.0 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel