diff options
author | lidongyang <[email protected]> | 2017-12-23 05:19:51 +1100 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2017-12-22 10:19:51 -0800 |
commit | 823d48bfb182137c53b9432498f1f0564eaa8bfc (patch) | |
tree | 35f32420b30cdaad0fea38ec0ccc2dd0abbe52a4 /include/sys/dmu_tx.h | |
parent | 44b61ea506212c287333e03d2cf8933216810800 (diff) |
Call commit callbacks from the tail of the list
Our zfs backed Lustre MDT had soft lockups while under heavy metadata
workloads while handling transaction callbacks from osd_zfs.
The problem is zfs is not taking advantage of the fast path in
Lustre's trans callback handling, where Lustre will skip the calls
to ptlrpc_commit_replies() when it already saw a higher transaction
number.
This patch corrects this, it also has a positive impact on metadata
performance on Lustre with osd_zfs, plus some cleanup in the headers.
A similar issue for ext4/ldiskfs is described on:
https://jira.hpdd.intel.com/browse/LU-6527
Reviewed-by: Olaf Faaland <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Li Dongyang <[email protected]>
Closes #6986
Diffstat (limited to 'include/sys/dmu_tx.h')
-rw-r--r-- | include/sys/dmu_tx.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/include/sys/dmu_tx.h b/include/sys/dmu_tx.h index f16e1e858..d82a79310 100644 --- a/include/sys/dmu_tx.h +++ b/include/sys/dmu_tx.h @@ -145,10 +145,6 @@ uint64_t dmu_tx_get_txg(dmu_tx_t *tx); struct dsl_pool *dmu_tx_pool(dmu_tx_t *tx); void dmu_tx_wait(dmu_tx_t *tx); -void dmu_tx_callback_register(dmu_tx_t *tx, dmu_tx_callback_func_t *dcb_func, - void *dcb_data); -void dmu_tx_do_callbacks(list_t *cb_list, int error); - /* * These routines are defined in dmu_spa.h, and are called by the SPA. */ |