Re: fscrypt API cleanups v3
Eric Biggers <[email protected]> Mon, 9 Mar 2026 13:43:09 -0700
| Newsgroups | org.kernel.vger.linux-fscrypt,net.sourceforge.lists.linux-f2fs-devel,org.kernel.vger.linux-ext4,org.kernel.vger.linux-fsdevel |
|---|---|
| Message-ID | <20260309204309.GB2048@quark> |
On Tue, Mar 03, 2026 at 02:35:07PM -0800, Eric Biggers wrote: > On Mon, Mar 02, 2026 at 06:18:05AM -0800, Christoph Hellwig wrote: > > Hi all, > > > > this series cleans up various fscrypt APIs to pass logical offsets in > > and lengths in bytes, and on-disk sectors as 512-byte sector units, > > like most of the VFS and block code. > > > > Changes since v2: > > - use the local bio variable in io_submit_init_bio > > - use folio instead of io_folio (and actually test the noinline mode, > > which should have cought this for the last round) > > - add an extra IS_ENABLED(CONFIG_FS_ENCRYPTION) to safeguard > > against potentially stupid compilers > > - document the byte length needs to be a multiple of the block > > size > > - case to u64 when passing the byte length > > - move a hunk to an earlier patch > > Changes since v1: > > - remove all buffer_head helpers, and do that before the API cleanups > > to simplify the series > > - fix a bisection hazard > > - spelling fixes in the commit logs > > - use "file position" to describe the byte offset into an inode > > - add another small ext4 cleanup at the end > > This looks good now. I'll plan to apply this to fscrypt.git#for-next in > a bit. Other reviews and acks appreciated. Applied to https://git.kernel.org/pub/scm/fs/fscrypt/linux.git/log/?h=for-next - Eric