mbox series

[v2,0/5] multipath-tools: udev rule fixes

Message ID 20241103224349.42582-1-mwilck@suse.com (mailing list archive)
Headers show
Series multipath-tools: udev rule fixes | expand

Message

Martin Wilck Nov. 3, 2024, 10:43 p.m. UTC 
Here are more fixes for the multipath udev rules. 2/5 is an actual fix,
1/5, 3/5 and 4/5 are minor cleanups / code clarifications.

5/5 implements the fix originally submitted by Ben [1].

Reviews and comments welcome.
Martin

[1] https://lore.kernel.org/dm-devel/20241022233349.247087-1-bmarzins@redhat.com/

Martin Wilck (5):
  11-dm-mpath.rules.in: import DM_COLDPLUG_SUSPENDED only once
  11-dm-mpath.rules.in: handle inactive suspended devices correctly
  11-dm-mpath.rules.in: clarify DM_ACTIVATION logic
  11-dm-mpath-rules.in: skip one .DM_NOSCAN check
  11-dm-mpath.rules.in: set .DM_NOSCAN if MPATH_UNCHANGED is set

 multipath/11-dm-mpath.rules.in | 44 ++++++++++++++++++++--------------
 1 file changed, 26 insertions(+), 18 deletions(-)

Comments

Benjamin Marzinski Nov. 5, 2024, 5:09 a.m. UTC | #1 
On Sun, Nov 03, 2024 at 11:43:44PM +0100, Martin Wilck wrote:
> Here are more fixes for the multipath udev rules. 2/5 is an actual fix,
> 1/5, 3/5 and 4/5 are minor cleanups / code clarifications.
> 
> 5/5 implements the fix originally submitted by Ben [1].
> 
> Reviews and comments welcome.
> Martin

For the set:
Reviewed-by: Benjamin Marzinski <bmarzins@redhat.com>

> 
> [1] https://lore.kernel.org/dm-devel/20241022233349.247087-1-bmarzins@redhat.com/
> 
> Martin Wilck (5):
>   11-dm-mpath.rules.in: import DM_COLDPLUG_SUSPENDED only once
>   11-dm-mpath.rules.in: handle inactive suspended devices correctly
>   11-dm-mpath.rules.in: clarify DM_ACTIVATION logic
>   11-dm-mpath-rules.in: skip one .DM_NOSCAN check
>   11-dm-mpath.rules.in: set .DM_NOSCAN if MPATH_UNCHANGED is set
> 
>  multipath/11-dm-mpath.rules.in | 44 ++++++++++++++++++++--------------
>  1 file changed, 26 insertions(+), 18 deletions(-)
> 
> -- 
> 2.47.0