Merge patch series "two nits for path lookup"
authorChristian Brauner <brauner@kernel.org>
Thu, 17 Apr 2025 09:03:52 +0000 (11:03 +0200)
committerChristian Brauner <brauner@kernel.org>
Mon, 21 Apr 2025 08:27:59 +0000 (10:27 +0200)
commitc4044870ae2cb28d11ea771db165edb9e1a60702
treebb3e1417fd6fbfeb255e93faa29784efdff821fe
parent79beea2db0431536d79fc5d321225fb42f955466
parent4ef4ac360101f8bb11b6486ce60cd60ca015be8c
Merge patch series "two nits for path lookup"

Mateusz Guzik <mjguzik@gmail.com> says:

Since path looku is being looked at, two extra nits from me:

1. some trivial jump avoidance in inode_permission()

2. but more importantly avoiding a memory access which is most likely a
cache miss when descending into devcgroup_inode_permission()

the file seems to have no maintainer fwiw

anyhow I'm confident the way forward is to add IOP_FAST_MAY_EXEC (or
similar) to elide inode_permission() in the common case to begin with.
There are quite a few branches which straight up don't need execute.
On top of that btrfs has a permission hook only to check for MAY_WRITE,
which in case of path lookup is not set. With the above flag the call
will be avoided.

* patches from https://lore.kernel.org/20250416221626.2710239-1-mjguzik@gmail.com:
  device_cgroup: avoid access to ->i_rdev in the common case in devcgroup_inode_permission()
  fs: touch up predicts in inode_permission()

Link: https://lore.kernel.org/20250416221626.2710239-1-mjguzik@gmail.com
Signed-off-by: Christian Brauner <brauner@kernel.org>