Re: [PATCH 01/19] fs: new API for handling inode->i_version

2017-12-18 Thread Jeff Layton
On Sat, 2017-12-16 at 15:17 +1100, NeilBrown wrote: > On Wed, Dec 13 2017, Jeff Layton wrote: > > > On Thu, 2017-12-14 at 09:04 +1100, NeilBrown wrote: > > > On Wed, Dec 13 2017, Jeff Layton wrote: > > > > > > > +/* > > > > + * The change attribute (i_version) is mandated by NFSv4 and is mostly

Re: [PATCH 01/19] fs: new API for handling inode->i_version

2017-12-17 Thread Jeff Layton
On Sat, 2017-12-16 at 15:17 +1100, NeilBrown wrote: > On Wed, Dec 13 2017, Jeff Layton wrote: > > > On Thu, 2017-12-14 at 09:04 +1100, NeilBrown wrote: > > > On Wed, Dec 13 2017, Jeff Layton wrote: > > > > > > > +/* > > > > + * The change attribute (i_version) is mandated by NFSv4 and is mostly

Re: [PATCH 01/19] fs: new API for handling inode->i_version

2017-12-15 Thread NeilBrown
On Wed, Dec 13 2017, Jeff Layton wrote: > On Thu, 2017-12-14 at 09:04 +1100, NeilBrown wrote: >> On Wed, Dec 13 2017, Jeff Layton wrote: >> >> > +/* >> > + * The change attribute (i_version) is mandated by NFSv4 and is mostly for >> > + * knfsd, but is also used for other purposes (e.g. IMA). The

Re: [PATCH 01/19] fs: new API for handling inode->i_version

2017-12-13 Thread Jeff Layton
On Thu, 2017-12-14 at 09:04 +1100, NeilBrown wrote: > On Wed, Dec 13 2017, Jeff Layton wrote: > > > +/* > > + * The change attribute (i_version) is mandated by NFSv4 and is mostly for > > + * knfsd, but is also used for other purposes (e.g. IMA). The i_version > > must > > + * appear different to

Re: [PATCH 01/19] fs: new API for handling inode->i_version

2017-12-13 Thread NeilBrown
On Wed, Dec 13 2017, Jeff Layton wrote: > +/* > + * The change attribute (i_version) is mandated by NFSv4 and is mostly for > + * knfsd, but is also used for other purposes (e.g. IMA). The i_version must > + * appear different to observers if there was a change to the inode's data or > + * metadat

[PATCH 01/19] fs: new API for handling inode->i_version

2017-12-13 Thread Jeff Layton
From: Jeff Layton Add a documentation blob that explains what the i_version field is, how it is expected to work, and how it is currently implemented by various filesystems. We already have inode_inc_iversion. Add several other functions for manipulating and accessing the i_version counter. For