On Thu, 2019-02-07 at 17:53 -0600, Benjamin Marzinski wrote:
> When a multipath device fails to get a wwid in pathinfo, it moves to
> the
> INIT_MISSING_UDEV state. After a device in this state sends
> retrigger_tries change uevents in check_path(), it moves to the
> INIT_FAILED state. However, when check_path() is run on a device in
> INIT_FAILED, it can call pathinfo, which will set the path back
> into INIT_MISSING_UDEV if it cannot get a wwid. The next call to
> check_path() will put the path back into INIT_FAILED. The device
> will
> continue to ping-pong between these states.
>
> To solve this a new pp->initialized state has been added
> INIT_NEW. New
> path devices start in this state, instead of INIT_FAILED. INIT_NEW
> and
> INIT_FAILED are treated exactly the same, with one exception. A
> device
> in INIT_FAILED cannot transition back to INIT_MISSING_UDEV.
>
> Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
Reviewed-by: Martin Wilck <mwilck@xxxxxxxx>
> ---
> libmultipath/discovery.c | 8 +++++---
> libmultipath/structs.h | 1 +
> multipathd/main.c | 4 +++-
> 3 files changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c
> index 1748eeb..6aef188 100644
> --- a/libmultipath/discovery.c
> +++ b/libmultipath/discovery.c
> @@ -1966,8 +1966,10 @@ int pathinfo(struct path *pp, struct config
> *conf, int mask)
> if ((mask & DI_WWID) && !strlen(pp->wwid)) {
> get_uid(pp, path_state, pp->udev);
> if (!strlen(pp->wwid)) {
> - pp->initialized = INIT_MISSING_UDEV;
> - pp->tick = conf->retrigger_delay;
> + if (pp->initialized != INIT_FAILED) {
> + pp->initialized = INIT_MISSING_UDEV;
> + pp->tick = conf->retrigger_delay;
> + }
> return PATHINFO_OK;
> }
> else
> @@ -2000,7 +2002,7 @@ blank:
> * Recoverable error, for example faulty or offline path
> */
> pp->chkrstate = pp->state = PATH_DOWN;
> - if (pp->initialized == INIT_FAILED)
> + if (pp->initialized == INIT_NEW || pp->initialized ==
> INIT_FAILED)
> memset(pp->wwid, 0, WWID_SIZE);
>
> return PATHINFO_OK;
> diff --git a/libmultipath/structs.h b/libmultipath/structs.h
> index 375c728..b794b0d 100644
> --- a/libmultipath/structs.h
> +++ b/libmultipath/structs.h
> @@ -202,6 +202,7 @@ enum ghost_delay_states {
> };
>
> enum initialized_states {
> + INIT_NEW,
> INIT_FAILED,
> INIT_MISSING_UDEV,
> INIT_REQUESTED_UDEV,
> diff --git a/multipathd/main.c b/multipathd/main.c
> index 1caa40f..4d0fa8c 100644
> --- a/multipathd/main.c
> +++ b/multipathd/main.c
> @@ -2028,7 +2028,9 @@ check_path (struct vectors * vecs, struct path
> * pp, int ticks)
> return 1;
> }
> if (!pp->mpp) {
> - if (!strlen(pp->wwid) && pp->initialized == INIT_FAILED
> &&
> + if (!strlen(pp->wwid) &&
> + (pp->initialized == INIT_FAILED ||
> + pp->initialized == INIT_NEW) &&
> (newstate == PATH_UP || newstate == PATH_GHOST)) {
> condlog(2, "%s: add missing path", pp->dev);
> conf = get_multipath_config();
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
