]> git.ipfire.org Git - thirdparty/xfsprogs-dev.git/commit
xfs: separate out initial attr_set states
authorDave Chinner <dchinner@redhat.com>
Wed, 22 Jun 2022 19:28:52 +0000 (14:28 -0500)
committerEric Sandeen <sandeen@sandeen.net>
Wed, 22 Jun 2022 19:28:52 +0000 (14:28 -0500)
commitcb787289c04ccd6c24e66d283b359af6b9ee0479
tree1c4f5337585d07358078b68919fe9636e7c0b30f
parent08b9530a875828647814e8d45972ca48fcb7240d
xfs: separate out initial attr_set states

Source kernel commit: e0c41089b998f5a54dabd7a34ab24108e192d2ee

We current use XFS_DAS_UNINIT for several steps in the attr_set
state machine. We use it for setting shortform xattrs, converting
from shortform to leaf, leaf add, leaf-to-node and leaf add. All of
these things are essentially known before we start the state machine
iterating, so we really should separate them out:

XFS_DAS_SF_ADD:
        - tries to do a shortform add
        - on success -> done
        - on ENOSPC converts to leaf, -> XFS_DAS_LEAF_ADD
        - on error, dies.

XFS_DAS_LEAF_ADD:
        - tries to do leaf add
        - on success:
                - inline attr -> done
                - remote xattr || REPLACE -> XFS_DAS_FOUND_LBLK
        - on ENOSPC converts to node, -> XFS_DAS_NODE_ADD
        - on error, dies

XFS_DAS_NODE_ADD:
        - tries to do node add
        - on success:
                - inline attr -> done
                - remote xattr || REPLACE -> XFS_DAS_FOUND_NBLK
        - on error, dies

This makes it easier to understand how the state machine starts
up and sets us up on the path to further state machine
simplifications.

This also converts the DAS state tracepoints to use strings rather
than numbers, as converting between enums and numbers requires
manual counting rather than just reading the name.

This also introduces a XFS_DAS_DONE state so that we can trace
successful operation completions easily.

Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Allison Henderson<allison.henderson@oracle.com>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
libxfs/xfs_attr.c
libxfs/xfs_attr.h
libxfs/xfs_defer.c