[dm-devel] [PATCH v3 04/11] libmultipath: merge_hwe: fix queue_if_no_path logic
Hannes Reinecke
hare at suse.de
Thu Jun 22 06:04:56 UTC 2017
On 06/21/2017 05:06 PM, Martin Wilck wrote:
> The logic applied here should match the logic in select_features().
> This is achieved by calling reconcile_features_with_options().
>
> Signed-off-by: Martin Wilck <mwilck at suse.com>
> Acked-by: Benjamin Marzinski <bmarzins at redhat.com>
> ---
> libmultipath/config.c | 20 +++++++++++---------
> 1 file changed, 11 insertions(+), 9 deletions(-)
>
> diff --git a/libmultipath/config.c b/libmultipath/config.c
> index 60e345b3..a9b3eda2 100644
> --- a/libmultipath/config.c
> +++ b/libmultipath/config.c
> @@ -25,6 +25,7 @@
> #include "prio.h"
> #include "devmapper.h"
> #include "mpath_cmd.h"
> +#include "propsel.h"
>
> static int
> hwe_strmatch (struct hwentry *hwe1, struct hwentry *hwe2)
> @@ -318,6 +319,8 @@ set_param_str(char * str)
> static int
> merge_hwe (struct hwentry * dst, struct hwentry * src)
> {
> + int id_len;
> + char *id;
> merge_str(vendor);
> merge_str(product);
> merge_str(revision);
> @@ -353,15 +356,14 @@ merge_hwe (struct hwentry * dst, struct hwentry * src)
> merge_num(san_path_err_forget_rate);
> merge_num(san_path_err_recovery_time);
>
> - /*
> - * Make sure features is consistent with
> - * no_path_retry
> - */
> - if (dst->no_path_retry == NO_PATH_RETRY_FAIL)
> - remove_feature(&dst->features, "queue_if_no_path");
> - else if (dst->no_path_retry != NO_PATH_RETRY_UNDEF)
> - add_feature(&dst->features, "queue_if_no_path");
> -
> + id_len = strlen(dst->vendor) + strlen(dst->product) + 2;
> + id = MALLOC(id_len);
> + if (id != NULL)
> + snprintf(id, id_len, "%s/%s", dst->vendor, dst->product);
> + reconcile_features_with_options(id, &dst->features,
> + &dst->no_path_retry,
> + &dst->retain_hwhandler);
> + FREE(id);
> return 0;
> }
>
>
Ugh.
Allocating a string just for having 'nice' debugging messages in
reconcile_features_with_options?
Please, don't.
Cheers,
Hannes
--
Dr. Hannes Reinecke Teamlead Storage & Networking
hare at suse.de +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)
More information about the dm-devel
mailing list