Re: [PATCH 1/2] remove delay_(watch|wait)_checks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ben,

On Wed, 2017-12-13 at 12:08 -0600, Benjamin Marzinski wrote:
> These configuration options do essentially the same thing as the
> marginal paths options do, but since they don't do the increased IO
> testing, they are less likely to find the marginal paths. Guan, if
> you
> left these in for a reason (such as to deal with devices that don't
> do
> well with IO testing, such as passive paths) I'm open to keeping
> them.
> But it seems like the marginal paths options options do a better job
> than they do, and if there are cases that they doesn't handle well,
> they
> should be expanded to do so.

I think we should wait a bit more before removing this stuff. The
marginal paths code hasn't seen much testing yet AFAICS.

Regards
Martin

-- 
Dr. Martin Wilck <mwilck@xxxxxxxx>, Tel. +49 (0)911 74053 2107
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)

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




[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux