Re: CB_LAYOUTRECALL "deadlock" with in-kernel flexfiles server and XFS

[Jeff Layton <jlayton@redhat.com>]

On Thu, 2016-08-11 at 18:59 +0200, hch wrote:
> On Thu, Aug 11, 2016 at 12:33:47PM -0400, Jeff Layton wrote:
> > 
> > On Thu, 2016-08-11 at 18:25 +0200, hch wrote:
> > > 
> > > Yeah, for file-like layouts there should be a flag in
> > > struct nfsd4_layout_ops to disable recalls.
> > 
> > I don't think disabling recalls would be enough, would it? XFS
> > still
> > wants to break_layout and won't proceed until the layout list is
> > empty,
> > AFAICT. We need some way to indicate to the lower filesystem not to
> > call break_layout in this case.
> 
> XFS only cares about block-like layours where the client has direct
> access to the file blocks.  I'd need to look how to propagate the
> flag into break_layout, but in principle we don't need to do any
> recalls on truncate every for file and flexfile layouts.
> 

Hmm...if we aren't ever going to recall files and flexfiles layouts,
then do we even need to set a FL_LAYOUT lease for them at all?

I think I'll try hacking something up that takes that approach and see
if that might be a reasonable fix.

-- 
Jeff Layton <jlayton@redhat.com>