lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 9 Sep 2022 16:34:14 -0400 From: "John Stoffel" <john@...ffel.org> To: Jeff Layton <jlayton@...nel.org> Cc: "J. Bruce Fields" <bfields@...ldses.org>, Theodore Ts'o <tytso@....edu>, Jan Kara <jack@...e.cz>, NeilBrown <neilb@...e.de>, adilger.kernel@...ger.ca, djwong@...nel.org, david@...morbit.com, trondmy@...merspace.com, viro@...iv.linux.org.uk, zohar@...ux.ibm.com, xiubli@...hat.com, chuck.lever@...cle.com, lczerner@...hat.com, brauner@...nel.org, fweimer@...hat.com, linux-man@...r.kernel.org, linux-api@...r.kernel.org, linux-btrfs@...r.kernel.org, linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org, ceph-devel@...r.kernel.org, linux-ext4@...r.kernel.org, linux-nfs@...r.kernel.org, linux-xfs@...r.kernel.org Subject: Re: [man-pages RFC PATCH v4] statx, inode: document the new STATX_INO_VERSION field >>>>> "Jeff" == Jeff Layton <jlayton@...nel.org> writes: > On Thu, 2022-09-08 at 14:22 -0400, J. Bruce Fields wrote: >> On Thu, Sep 08, 2022 at 01:40:11PM -0400, Jeff Layton wrote: >> > Yeah, ok. That does make some sense. So we would mix this into the >> > i_version instead of the ctime when it was available. Preferably, we'd >> > mix that in when we store the i_version rather than adding it afterward. >> > >> > Ted, how would we access this? Maybe we could just add a new (generic) >> > super_block field for this that ext4 (and other filesystems) could >> > populate at mount time? >> >> Couldn't the filesystem just return an ino_version that already includes >> it? >> > Yes. That's simple if we want to just fold it in during getattr. If we > want to fold that into the values stored on disk, then I'm a little less > clear on how that will work. I wonder if this series should also include some updates to the various xfstests to hopefully document in code what this statx() call will do in various situations. Or at least document how to test it in some manner? Especially since it's layers on top of layers to make this work. My assumption is that if the underlying filesystem doesn't support the new values, it just returns 0 or c_time? John
Powered by blists - more mailing lists