]> www.infradead.org Git - users/jedix/linux-maple.git/commit
fs: switch f_iocb_flags and f_ra
authorChristian Brauner <brauner@kernel.org>
Thu, 22 Aug 2024 14:14:46 +0000 (16:14 +0200)
committerChristian Brauner <brauner@kernel.org>
Wed, 28 Aug 2024 11:06:12 +0000 (13:06 +0200)
commita55d1cbd1720679cfe9837bce250e397ec513989
treed21d06bb5e34e8f3f761003eb59d1a9ebd0b387f
parent1934b212615dc617ac84fc306333ab2b9fc3b04f
fs: switch f_iocb_flags and f_ra

Now that we shrank struct file by 24 bytes we still have a 4 byte hole.
If we move struct file_ra_state into the union and f_iocb_flags out of
the union we close that whole and bring down struct file to 192 bytes.
Which means struct file is 3 cachelines and we managed to shrink it by
40 bytes this cycle.

I've tried to audit all codepaths that use f_ra and none of them seem to
rely on it in file->f_op->release() and never have since commit
1da177e4c3f4 ("Linux-2.6.12-rc2").

Link: https://lore.kernel.org/r/20240823-luftdicht-berappen-d69a2166a0db@brauner
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Reviewed-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Christian Brauner <brauner@kernel.org>
include/linux/fs.h