mbox series

[0/3] Don't enter recovery move over pending paths

Message ID 1640223155-1678-1-git-send-email-bmarzins@redhat.com (mailing list archive)
Headers show
Series Don't enter recovery move over pending paths | expand

Message

Benjamin Marzinski Dec. 23, 2021, 1:32 a.m. UTC 
After commit 2e61b8fb, I started seeing multipath devices switching into
recovery mode briefly on startup.  They come back out shortly, so unless
no_path_retry is set very low, this won't be an issue, but multipathd
shouldn't enter recovery mode at all over pending paths.

Benjamin Marzinski (3):
  libmultipath: fix DI_NOIO pathinfo PATH_PENDING handling
  libmultipath: simplify out __set_no_path_retry()
  libmultipath: don't enter recovery mode with pending paths

 libmultipath/discovery.c          |  5 ++++-
 libmultipath/libmultipath.version |  2 +-
 libmultipath/structs_vec.c        | 13 ++++++-------
 libmultipath/structs_vec.h        |  3 +--
 4 files changed, 12 insertions(+), 11 deletions(-)

Comments

Martin Wilck Dec. 23, 2021, 8:45 a.m. UTC | #1 
On Wed, 2021-12-22 at 19:32 -0600, Benjamin Marzinski wrote:
> After commit 2e61b8fb, I started seeing multipath devices switching
> into
> recovery mode briefly on startup.  They come back out shortly, so
> unless
> no_path_retry is set very low, this won't be an issue, but multipathd
> shouldn't enter recovery mode at all over pending paths.
> 
> Benjamin Marzinski (3):
>   libmultipath: fix DI_NOIO pathinfo PATH_PENDING handling
>   libmultipath: simplify out __set_no_path_retry()
>   libmultipath: don't enter recovery mode with pending paths
> 
>  libmultipath/discovery.c          |  5 ++++-
>  libmultipath/libmultipath.version |  2 +-
>  libmultipath/structs_vec.c        | 13 ++++++-------
>  libmultipath/structs_vec.h        |  3 +--
>  4 files changed, 12 insertions(+), 11 deletions(-)
> 

For the set:

Reviewed-by: Martin Wilck <mwilck@suse.com>


--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel