All of lore.kernel.org
 help / color / mirror / Atom feed
* [GIT PULL] file_remove_privs() fix for v6.0-rc3
@ 2022-08-23 11:10 Christian Brauner
  2022-08-24  2:45 ` pr-tracker-bot
  2022-08-24  7:14   ` Christian Brauner
  0 siblings, 2 replies; 6+ messages in thread
From: Christian Brauner @ 2022-08-23 11:10 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: Christian Brauner, linux-fsdevel, linux-kernel, Jens Axboe

Hey Linus,

/* Summary */
As part of Stefan's and Jens' work to add async buffered write support to xfs
we refactored file_remove_privs() and added __file_remove_privs() to avoid
calling __remove_privs() when IOCB_NOWAIT is passed.

While debugging a recent performance regression report I found that during
review we missed that faf99b563558 ("fs: add __remove_file_privs() with flags
parameter") accidently changed behavior when dentry_needs_remove_privs()
returns zero. Before the commit it would still call inode_has_no_xattr()
setting the S_NOSEC bit and thereby avoiding even calling into
dentry_needs_remove_privs() the next time this function is called. After that
commit inode_has_no_xattr() would only be called if __remove_privs() had to be
called. Restore the old behavior. This is likely the cause of the performance
regression.

/* Testing */
All patches are based on v6.0-rc1 and have been sitting in linux-next. No build
failures or warnings were observed and fstests, selftests, and LTP have seen no
regressions.

/* Conflicts */
At the time of creating this PR no merge conflicts were reported from
linux-next and no merge conflicts showed up doing a test-merge with current
mainline.

The following changes since commit 568035b01cfb107af8d2e4bd2fb9aea22cf5b868:

  Linux 6.0-rc1 (2022-08-14 15:50:18 -0700)

are available in the Git repository at:

  ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/vfs/idmapping.git tags/fs.fixes.v6.0-rc3

for you to fetch changes up to 41191cf6bf565f4139046d7be68ec30c290af92d:

  fs: __file_remove_privs(): restore call to inode_has_no_xattr() (2022-08-18 09:39:33 +0200)

Please consider pulling these changes from the signed fs.fixes.v6.0-rc3 tag.

Thanks!
Christian

----------------------------------------------------------------
fs.fixes.v6.0-rc3

----------------------------------------------------------------
Stefan Roesch (1):
      fs: __file_remove_privs(): restore call to inode_has_no_xattr()

 fs/inode.c | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

^ permalink raw reply	[flat|nested] 6+ messages in thread
[parent not found: <20220824015126.GA86725@inn2.lkp.intel.com>]

end of thread, other threads:[~2022-08-24  7:14 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-08-23 11:10 [GIT PULL] file_remove_privs() fix for v6.0-rc3 Christian Brauner
2022-08-24  2:45 ` pr-tracker-bot
2022-08-24  7:14 ` Christian Brauner
2022-08-24  7:14   ` Christian Brauner
     [not found] <20220824015126.GA86725@inn2.lkp.intel.com>
2022-08-24  2:00 ` [fs] 41191cf6bf: aim7.jobs-per-min 20.1% improvement kernel test robot
2022-08-24  2:00   ` kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.