Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NAS-121765 / Improve zpl_permission performance #131

Merged
merged 1 commit into from
May 4, 2023

Conversation

anodos325
Copy link

This function can be frequently called with MAY_EXEC|MAY_NOT_BLOCK during RCU path walk. Where possible we should try not to break out of it. In this case we check whether flag ZFS_NO_EXECS_DENIED is set and check mode (similar to fastexecute check in zfs_acl.c).

@anodos325
Copy link
Author

In some rough benchmarking of ls metadata-heavy workload from some SMB clients, fast-path improved perf by around 10%. Front-loading the capability checks is not problematic here because subsequent vfs ops will end up performing zfs_zaccess checks and catch AV-quarantined files (if we should ever implement this).

@anodos325 anodos325 added the WIP Work In Progress label May 1, 2023
@anodos325 anodos325 force-pushed the improve-zpl-permission branch 3 times, most recently from 9ded555 to f4f8a1f Compare May 1, 2023 14:25
@anodos325
Copy link
Author

anodos325 commented May 1, 2023

image
Before

image
After

@anodos325 anodos325 changed the title Improve zpl_permission performance NAS-121765 / Improve zpl_permission performance May 1, 2023
@anodos325 anodos325 removed the WIP Work In Progress label May 1, 2023
This function can be frequently called with MAY_EXEC|MAY_NOT_BLOCK
during RCU path walk. Where possible we should try not to break
out of it. In this case we check whether flag ZFS_NO_EXECS_DENIED is
set and check mode (similar to fastexecute check in zfs_acl.c).

Signed-off-by: Andrew Walker <awalker@ixsystems.com>
@anodos325 anodos325 force-pushed the improve-zpl-permission branch 2 times, most recently from b71d14f to 13ab16d Compare May 4, 2023 16:32
@anodos325 anodos325 merged commit 90ee0f9 into truenas/zfs-2.1-release May 4, 2023
@anodos325 anodos325 deleted the improve-zpl-permission branch May 4, 2023 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants