On Fri, 2023-10-27 at 11:29 -0400, Kent Overstreet wrote: > On Fri, Oct 27, 2023 at 08:32:30AM -0400, Jeff Layton wrote: > > Hi Kent, > > > > We spoke several months ago about adding an i_version counter to > > bcachefs (primarily for serving via NFSD). I noticed that that hasn't > > been plumbed in yet, but I did notice the bi_version field in the v3 > > inode that doesn't look like it does anything useful today. Is that > > field intended to be for the i_version counter in the future? If so, > > then I can take a crack at adding it. > > > > If not, then can we carve out an extra 64 bits in the on-disk inode for > > that purpose before bcachefs is merged? > > Yeah, that was the idea, but we're probably going to want this in the > non-varint inode section - it'll be a more work, but that way direct IO > updates won't be forced to do the full unpack/repack. > > If you want to go ahead with using bi_version for now, I should be able > to handle making it more efficient :)
Sounds good! No promises on how soon I'll get to it, but I'll put it on my to-do list. Thanks! -- Jeff Layton <[email protected]>
