diff options
author | Brian Foster <bfoster@redhat.com> | 2018-09-29 13:41:58 +1000 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2018-09-29 13:41:58 +1000 |
commit | d5a2e2893da0d62c3888c91ae2da798adc17a9b9 (patch) | |
tree | db2dcdff07f1ca24be625bb8b1e727b8d8f02547 /fs/xfs/xfs_trace.h | |
parent | ae29478766f4c8e16edca6fe1e25d73c47991ebe (diff) | |
download | op-kernel-dev-d5a2e2893da0d62c3888c91ae2da798adc17a9b9.zip op-kernel-dev-d5a2e2893da0d62c3888c91ae2da798adc17a9b9.tar.gz |
xfs: remove last of unnecessary xfs_defer_cancel() callers
Now that deferred operations are completely managed via
transactions, it's no longer necessary to cancel the dfops in error
paths that already cancel the associated transaction. There are a
few such calls lingering throughout the codebase.
Remove all remaining unnecessary calls to xfs_defer_cancel(). This
leaves xfs_defer_cancel() calls in two places. The first is the call
in the transaction cancel path itself, which facilitates this patch.
The second is made via the xfs_defer_finish() error path to provide
consistent error semantics with transaction commit. For example,
xfs_trans_commit() expects an xfs_defer_finish() failure to clean up
the dfops structure before it returns.
Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs/xfs_trace.h')
0 files changed, 0 insertions, 0 deletions