On Mon, 2021-02-01 at 21:15 -0600, Benjamin Marzinski wrote: > On Thu, Jan 28, 2021 at 09:45:41PM +0100, mwilck@xxxxxxxx wrote: > > From: Martin Wilck <mwilck@xxxxxxxx> > > > > Hi Christophe, hi Ben, > > > > here are 3 patches I'd like to get reviewed before we create a pull > > request. The first two are related to complex SAS setups, the > > second > > one is to avoid accessing the root file system in a possible > > dangerous > > situation. > > > > Wrt 2/3: while testing it, I discovered that > > "I_T_nexus_loss_timeout" > > is a read-only sysfs attribute, therefore this code does nothing. > > I considered removing it altogether, observing that the attribute > > has > > been read-only as long as it existed (v2.6.17, 2006). I'd like some > > feedback beforehand, though, perhaps some distros use patched > > kernels > > that make this attribute r/w? > > I_T_nexus_loss_timeout appears to have always been read-only on RHEL > and > Fedora. Thanks. Anyway, given that we're preparing a submission to Christophe, I'd like to give people more time for responding. We can remove this code later. Regards Martin -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
