Re: [dm-devel] [PATCH 5/7] multipathd: fully initialize paths added by update_pathvec_from_dm

2021-11-08 Thread Benjamin Marzinski
On Mon, Nov 08, 2021 at 04:55:37PM +, Martin Wilck wrote: > On Mon, 2021-11-08 at 10:29 -0600, Benjamin Marzinski wrote: > > On Fri, Nov 05, 2021 at 11:20:01PM +, Martin Wilck wrote: > > > > 1: udev hasn't gotten an event for a device > > > > > > I don't think we can detect this without

Re: [dm-devel] [PATCH 5/7] multipathd: fully initialize paths added by update_pathvec_from_dm

2021-11-08 Thread Benjamin Marzinski
On Fri, Nov 05, 2021 at 11:20:01PM +, Martin Wilck wrote: > On Fri, 2021-11-05 at 16:49 -0500, Benjamin Marzinski wrote: > > On Fri, Nov 05, 2021 at 10:55:11AM +, Martin Wilck wrote: > > > Hi Ben, > > > > > > On Thu, 2021-11-04 at 23:10 +0100, Martin Wilck wrote: > > > > On Wed,

Re: [dm-devel] [PATCH 5/7] multipathd: fully initialize paths added by update_pathvec_from_dm

2021-11-05 Thread Benjamin Marzinski
On Fri, Nov 05, 2021 at 10:55:11AM +, Martin Wilck wrote: > Hi Ben, > > On Thu, 2021-11-04 at 23:10 +0100, Martin Wilck wrote: > > On Wed, 2021-10-20 at 14:15 -0500, Benjamin Marzinski wrote: > > > ... > > > > > > > > Multipath now has a new state to deal with these devices, > > >

Re: [dm-devel] [PATCH 5/7] multipathd: fully initialize paths added by update_pathvec_from_dm

2021-11-05 Thread Benjamin Marzinski
On Thu, Nov 04, 2021 at 10:10:18PM +, Martin Wilck wrote: > On Wed, 2021-10-20 at 14:15 -0500, Benjamin Marzinski wrote: > > When paths are added by update_pathvec_from_dm(), udev may not have > > initialized them. This means that it's possible that they are > > supposed > > to be blacklisted

[dm-devel] [PATCH 5/7] multipathd: fully initialize paths added by update_pathvec_from_dm

2021-10-20 Thread Benjamin Marzinski
When paths are added by update_pathvec_from_dm(), udev may not have initialized them. This means that it's possible that they are supposed to be blacklisted by udev properties, but weren't. Also, in order to avoid doing potentially stalling IO, update_pathvec_from_dm() doesn't get all the path