Re: xfs: commit 6552321831dc "xfs: remove i_iolock and use i_rwsem in the VFS inode instead" change causes hang

From	Christoph Hellwig <hch@lst.de>
Newsgroups	linux.kernel
Subject	Re: xfs: commit 6552321831dc "xfs: remove i_iolock and use i_rwsem in the VFS inode instead" change causes hang
Date	2017-01-08 20:20 +0100
Message-ID	<sXrax-28V-9@gated-at.bofh.it> (permalink)
References	<sXmXf-7DZ-1@gated-at.bofh.it> <sXn6V-7Hu-5@gated-at.bofh.it> <sXpV8-16G-33@gated-at.bofh.it> <sXqeu-1ue-9@gated-at.bofh.it> <sXqRb-1KB-21@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Sun, Jan 08, 2017 at 10:57:28AM -0800, James Bottomley wrote:
> I'm unsure about the DIO case, so lets try defining the semantics and
> see if they're implementable for DIO, otherwise simply exclude it.

Let's start with the semantics.  First we need to write down what
IMA requires from the FS, and have an interface how the FS can declare
that it supports these features.  As far as I can tell there are not
proper feature checks anywhere right now.  Once we have done that
we can move forward from there.

As you seem to be interested in IMA how about you spearhead documenting
the requirements and adding xfstests support?

> OK, so how about we define it.  I think we need two vfs calls:
> 
> inode_block_local_writes(inode)
> inode_unblock_local_writes(inode)

No.  We need an ->ima_measure file_operation, guts of process_measurement
turned into a library function that the FS can call after taking fs-specific
locks.  And maybe also a small wrapper around it that takes ilock and
can be used directly for file systems not needing special locking.

> With semantics that between these two, all write attempts to the file
> backed by the inode on this system block but reads of the underlying
> file are allowed (I added local so we don't have to implement for
> remote filesystems).

How do you define local?  Are GFS2 and OCFS2 local?  Is XFS with
outstanding pNFS layout local?  Is NFS with the block or SCSI layout
local because it operates on a block device?

The only sane way is to make INA opt-in with a check list of features
that need to be supported, and declared to be supported by the fs,
similar to how we handle NFS exporting.

Thread

xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem in  the VFS inode instead"  change causes hang Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-01-08 15:50 +0100
  Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use         i_rwsem in the VFS inode instead"  change causes hang Christoph Hellwig <hch@lst.de> - 2017-01-08 16:00 +0100
    Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use    i_rwsem  in the VFS inode instead"  change causes hang Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-01-08 16:10 +0100
      Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use         i_rwsem in the VFS inode instead"  change causes hang Christoph Hellwig <hch@lst.de> - 2017-01-08 16:20 +0100
        Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use         i_rwsem in the VFS inode instead"  change causes hang Christoph Hellwig <hch@lst.de> - 2017-01-08 16:40 +0100
          Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use    i_rwsem  in the VFS inode instead"  change causes hang Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-01-08 17:40 +0100
            Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use         i_rwsem in the VFS inode instead"  change causes hang Christoph Hellwig <hch@lst.de> - 2017-01-08 17:50 +0100
        Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use    i_rwsem  in the VFS inode instead"  change causes hang Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-01-08 16:40 +0100
    Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem  in the VFS inode instead"  change causes hang James Bottomley <James.Bottomley@HansenPartnership.com> - 2017-01-08 19:00 +0100
      Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use         i_rwsem in the VFS inode instead"  change causes hang Christoph Hellwig <hch@lst.de> - 2017-01-08 19:20 +0100
        Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem  in the VFS inode instead"  change causes hang James Bottomley <James.Bottomley@HansenPartnership.com> - 2017-01-08 20:00 +0100
          Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use         i_rwsem in the VFS inode instead"  change causes hang Christoph Hellwig <hch@lst.de> - 2017-01-08 20:20 +0100
            Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem  in the VFS inode instead"  change causes hang Al Viro <viro@ZenIV.linux.org.uk> - 2017-01-08 20:30 +0100
              Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem  in the VFS inode instead"  change causes hang Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-01-08 21:20 +0100
            Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use    i_rwsem  in the VFS inode instead"  change causes hang Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-01-08 20:50 +0100
            Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem  in the VFS inode instead"  change causes hang Jeff Layton <jlayton@redhat.com> - 2017-01-09 20:50 +0100
              Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem  in the VFS inode instead"  change causes hang Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-01-10 04:00 +0100
                Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem  in the VFS inode instead"  change causes hang Jeff Layton <jlayton@redhat.com> - 2017-01-10 17:30 +0100
          Re: xfs:  commit 6552321831dc "xfs: remove i_iolock and use i_rwsem  in the VFS inode instead"  change causes hang Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-01-08 20:20 +0100

csiph-web