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-126719 / Add O_RESOLVE_NO_SYMLINKS #268

Open
wants to merge 1 commit into
base: truenas/13.1-stable
Choose a base branch
from

Conversation

anodos325
Copy link
Contributor

This is required for Samba 4.17 and higher. Internally samba uses the openat2 flag RESOLVE_NO_SYMLINKS as an optimization to create a fast-path for opening files that helps mitigate some of Samba 4.13+ metadata performance regression due to symlink safety checks. This commit adds an open flag equivalent to MNT_NOSYMLFOLLOW mount flag.

This is required for Samba 4.17 and higher. Internally
samba uses the openat2 flag RESOLVE_NO_SYMLINKS as an
optimization to create a fast-path for opening files that
helps mitigate some of Samba 4.13+ metadata performance
regression due to symlink safety checks. This commit
adds an open flag equivalent to MNT_NOSYMLFOLLOW mount
flag.
@anodos325 anodos325 requested a review from amotin November 27, 2023 12:28
@anodos325 anodos325 force-pushed the add-resolve-no-symlinks-13.1 branch from b4aeb91 to 32574e3 Compare January 9, 2024 17:01
@anodos325 anodos325 changed the title Add O_RESOLVE_NO_SYMLINKS NAS-126719 / Add O_RESOLVE_NO_SYMLINKS Jan 9, 2024
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.

1 participant