Re: [Cluster-devel] [GFS2 PATCH] gfs2: Panic when an io error occurs writing to the journal

2018-12-17 Thread David Teigland
On Mon, Dec 17, 2018 at 09:58:47AM -0500, Bob Peterson wrote: > Dave Teigland recommended. Unless I'm mistaken, Dave has said that GFS2 > should never withdraw; it should always just kernel panic (Dave, correct > me if I'm wrong). At least this patch confines that behavior to a small > subset of

Re: [Cluster-devel] [GFS2 PATCH] gfs2: Panic when an io error occurs writing to the journal

2018-12-17 Thread Bob Peterson
Hi, - Original Message - > On 17/12/2018 09:04, Edwin Török wrote: > >> If we get an io error writing to the journal, the only correct > >> thing to do is to kernel panic. > > Hi, > > > > That may be required for correctness, however are we sure there is no > > other way to force the DLM

Re: [Cluster-devel] [GFS2 PATCH] gfs2: Panic when an io error occurs writing to the journal

2018-12-17 Thread Steven Whitehouse
Hi, On 17/12/2018 09:04, Edwin Török wrote: On 17/12/2018 13:54, Bob Peterson wrote: Hi, Before this patch, gfs2 would try to withdraw when it encountered io errors writing to its journal. That's incorrect behavior because if it can't write to the journal, it cannot write revokes for the

Re: [Cluster-devel] [GFS2 PATCH] gfs2: Panic when an io error occurs writing to the journal

2018-12-17 Thread Edwin Török
On 17/12/2018 13:54, Bob Peterson wrote: > Hi, > > Before this patch, gfs2 would try to withdraw when it encountered > io errors writing to its journal. That's incorrect behavior > because if it can't write to the journal, it cannot write revokes > for the metadata it sends down. A withdraw will

[Cluster-devel] [GFS2 PATCH] gfs2: Panic when an io error occurs writing to the journal

2018-12-17 Thread Bob Peterson
Hi, Before this patch, gfs2 would try to withdraw when it encountered io errors writing to its journal. That's incorrect behavior because if it can't write to the journal, it cannot write revokes for the metadata it sends down. A withdraw will cause gfs2 to unmount the file system from dlm, which