diff options
author | Brian Foster <bfoster@redhat.com> | 2012-12-03 15:50:39 -0500 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2012-12-17 02:14:38 -0500 |
commit | c36ced27f2bae003ddafe0d5a408424ba57af243 (patch) | |
tree | 19610bdc90f8787e01eaf83ea1e50c4e6720e6e7 /xlators/cluster/afr/src/afr-transaction.c | |
parent | 3e255fb10f404dbc0d3add7164c95b0721231312 (diff) |
afr: make flush non-transactional
Flush is historically a transaction to ensure all previous writes
were complete. This is no longer required as write-behind has
learned to make flush a barrier operation (re: conversation w/
Avati).
Flush taking a full file lock causes VMs running on afr volumes
to stall when a migration occurs and self-heal is in progress.
Make afr_flush() a non-transactional operation.
BUG: 874045
Change-Id: Ie287b79e7f300df88aca6030e2d80311772746bf
Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-on: https://code.engineering.redhat.com/gerrit/1912
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Tested-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/cluster/afr/src/afr-transaction.c')
-rw-r--r-- | xlators/cluster/afr/src/afr-transaction.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/xlators/cluster/afr/src/afr-transaction.c b/xlators/cluster/afr/src/afr-transaction.c index 33b87780dda..b29e11fafc4 100644 --- a/xlators/cluster/afr/src/afr-transaction.c +++ b/xlators/cluster/afr/src/afr-transaction.c @@ -1285,9 +1285,6 @@ void afr_delayed_changelog_post_op (xlator_t *this, call_frame_t *frame, fd_t *fd); void -afr_delayed_changelog_wake_up (xlator_t *this, fd_t *fd); - -void afr_delayed_changelog_wake_up_cbk (void *data) { fd_t *fd = NULL; |