diff options
| author | Darrick J. Wong <djwong@kernel.org> | 2024-07-02 20:22:41 +0200 |
|---|---|---|
| committer | Darrick J. Wong <djwong@kernel.org> | 2024-07-02 20:36:58 +0200 |
| commit | b11b11e3b7a72606cfef527255a9467537bcaaa5 (patch) | |
| tree | ccbb97f18e0a5eb3d6f42851cdfa60a95b632ca0 /fs/xfs/xfs_symlink.c | |
| parent | xfs: hoist xfs_{bump,drop}link to libxfs (diff) | |
| download | linux-b11b11e3b7a72606cfef527255a9467537bcaaa5.tar.xz linux-b11b11e3b7a72606cfef527255a9467537bcaaa5.zip | |
xfs: separate the icreate logic around INIT_XATTRS
INIT_XATTRS is overloaded here -- it's set during the creat process when
we think that we're immediately going to set some ACL xattrs to save
time. However, it's also used by the parent pointers code to enable the
attr fork in preparation to receive ppptr xattrs. This results in
xfs_has_parent() branches scattered around the codebase to turn on
INIT_XATTRS.
Linkable files are created far more commonly than unlinkable temporary
files or directory tree roots, so we should centralize this logic in
xfs_inode_init. For the three callers that don't want parent pointers
(online repiar tempfiles, unlinkable tempfiles, rootdir creation) we
provide an UNLINKABLE flag to skip attr fork initialization.
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'fs/xfs/xfs_symlink.c')
| -rw-r--r-- | fs/xfs/xfs_symlink.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/xfs/xfs_symlink.c b/fs/xfs/xfs_symlink.c index 6ff736e5c4e7..e471369f6b63 100644 --- a/fs/xfs/xfs_symlink.c +++ b/fs/xfs/xfs_symlink.c @@ -115,9 +115,6 @@ xfs_symlink( if (xfs_is_shutdown(mp)) return -EIO; - if (xfs_has_parent(mp)) - args.flags |= XFS_ICREATE_INIT_XATTRS; - /* * Check component lengths of the target path name. */ |