zil: Some micro-optimizations.

Should not cause functional changes.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by:  Alexander Motin <mav@FreeBSD.org>
Sponsored by:   iXsystems, Inc.
Closes #14854
This commit is contained in:
Alexander Motin 2023-05-12 12:14:29 -04:00 committed by GitHub
parent da211a4a33
commit 895e03135e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 23 additions and 52 deletions

View File

@ -146,9 +146,6 @@ static uint64_t zil_slog_bulk = 768 * 1024;
static kmem_cache_t *zil_lwb_cache; static kmem_cache_t *zil_lwb_cache;
static kmem_cache_t *zil_zcw_cache; static kmem_cache_t *zil_zcw_cache;
#define LWB_EMPTY(lwb) ((BP_GET_LSIZE(&lwb->lwb_blk) - \
sizeof (zil_chain_t)) == (lwb->lwb_sz - lwb->lwb_nused))
static int static int
zil_bp_compare(const void *x1, const void *x2) zil_bp_compare(const void *x1, const void *x2)
{ {
@ -769,11 +766,6 @@ zil_alloc_lwb(zilog_t *zilog, blkptr_t *bp, boolean_t slog, uint64_t txg,
list_insert_tail(&zilog->zl_lwb_list, lwb); list_insert_tail(&zilog->zl_lwb_list, lwb);
mutex_exit(&zilog->zl_lock); mutex_exit(&zilog->zl_lock);
ASSERT(!MUTEX_HELD(&lwb->lwb_vdev_lock));
ASSERT(avl_is_empty(&lwb->lwb_vdev_tree));
VERIFY(list_is_empty(&lwb->lwb_waiters));
VERIFY(list_is_empty(&lwb->lwb_itxs));
return (lwb); return (lwb);
} }
@ -782,8 +774,8 @@ zil_free_lwb(zilog_t *zilog, lwb_t *lwb)
{ {
ASSERT(MUTEX_HELD(&zilog->zl_lock)); ASSERT(MUTEX_HELD(&zilog->zl_lock));
ASSERT(!MUTEX_HELD(&lwb->lwb_vdev_lock)); ASSERT(!MUTEX_HELD(&lwb->lwb_vdev_lock));
VERIFY(list_is_empty(&lwb->lwb_waiters)); ASSERT(list_is_empty(&lwb->lwb_waiters));
VERIFY(list_is_empty(&lwb->lwb_itxs)); ASSERT(list_is_empty(&lwb->lwb_itxs));
ASSERT(avl_is_empty(&lwb->lwb_vdev_tree)); ASSERT(avl_is_empty(&lwb->lwb_vdev_tree));
ASSERT3P(lwb->lwb_write_zio, ==, NULL); ASSERT3P(lwb->lwb_write_zio, ==, NULL);
ASSERT3P(lwb->lwb_root_zio, ==, NULL); ASSERT3P(lwb->lwb_root_zio, ==, NULL);
@ -1026,12 +1018,10 @@ zil_destroy(zilog_t *zilog, boolean_t keep_first)
if (!list_is_empty(&zilog->zl_lwb_list)) { if (!list_is_empty(&zilog->zl_lwb_list)) {
ASSERT(zh->zh_claim_txg == 0); ASSERT(zh->zh_claim_txg == 0);
VERIFY(!keep_first); VERIFY(!keep_first);
while ((lwb = list_head(&zilog->zl_lwb_list)) != NULL) { while ((lwb = list_remove_head(&zilog->zl_lwb_list)) != NULL) {
if (lwb->lwb_fastwrite) if (lwb->lwb_fastwrite)
metaslab_fastwrite_unmark(zilog->zl_spa, metaslab_fastwrite_unmark(zilog->zl_spa,
&lwb->lwb_blk); &lwb->lwb_blk);
list_remove(&zilog->zl_lwb_list, lwb);
if (lwb->lwb_buf != NULL) if (lwb->lwb_buf != NULL)
zio_buf_free(lwb->lwb_buf, lwb->lwb_sz); zio_buf_free(lwb->lwb_buf, lwb->lwb_sz);
zio_free(zilog->zl_spa, txg, &lwb->lwb_blk); zio_free(zilog->zl_spa, txg, &lwb->lwb_blk);
@ -1387,6 +1377,7 @@ zil_lwb_flush_vdevs_done(zio_t *zio)
spa_config_exit(zilog->zl_spa, SCL_STATE, lwb); spa_config_exit(zilog->zl_spa, SCL_STATE, lwb);
zio_buf_free(lwb->lwb_buf, lwb->lwb_sz); zio_buf_free(lwb->lwb_buf, lwb->lwb_sz);
hrtime_t t = gethrtime() - lwb->lwb_issued_timestamp;
mutex_enter(&zilog->zl_lock); mutex_enter(&zilog->zl_lock);
@ -1399,9 +1390,7 @@ zil_lwb_flush_vdevs_done(zio_t *zio)
*/ */
lwb->lwb_buf = NULL; lwb->lwb_buf = NULL;
ASSERT3U(lwb->lwb_issued_timestamp, >, 0); zilog->zl_last_lwb_latency = (zilog->zl_last_lwb_latency * 7 + t) / 8;
zilog->zl_last_lwb_latency = (zilog->zl_last_lwb_latency * 3 +
gethrtime() - lwb->lwb_issued_timestamp) / 4;
lwb->lwb_root_zio = NULL; lwb->lwb_root_zio = NULL;
@ -1418,17 +1407,12 @@ zil_lwb_flush_vdevs_done(zio_t *zio)
zilog->zl_commit_lr_seq = zilog->zl_lr_seq; zilog->zl_commit_lr_seq = zilog->zl_lr_seq;
} }
while ((itx = list_head(&lwb->lwb_itxs)) != NULL) { while ((itx = list_remove_head(&lwb->lwb_itxs)) != NULL)
list_remove(&lwb->lwb_itxs, itx);
zil_itx_destroy(itx); zil_itx_destroy(itx);
}
while ((zcw = list_head(&lwb->lwb_waiters)) != NULL) { while ((zcw = list_remove_head(&lwb->lwb_waiters)) != NULL) {
mutex_enter(&zcw->zcw_lock); mutex_enter(&zcw->zcw_lock);
ASSERT(list_link_active(&zcw->zcw_node));
list_remove(&lwb->lwb_waiters, zcw);
ASSERT3P(zcw->zcw_lwb, ==, lwb); ASSERT3P(zcw->zcw_lwb, ==, lwb);
zcw->zcw_lwb = NULL; zcw->zcw_lwb = NULL;
/* /*
@ -1581,7 +1565,7 @@ zil_lwb_write_done(zio_t *zio)
* write and/or fsync activity, as it has the potential to * write and/or fsync activity, as it has the potential to
* coalesce multiple flush commands to a vdev into one. * coalesce multiple flush commands to a vdev into one.
*/ */
if (list_head(&lwb->lwb_waiters) == NULL && nlwb != NULL) { if (list_is_empty(&lwb->lwb_waiters) && nlwb != NULL) {
zil_lwb_flush_defer(lwb, nlwb); zil_lwb_flush_defer(lwb, nlwb);
ASSERT(avl_is_empty(&lwb->lwb_vdev_tree)); ASSERT(avl_is_empty(&lwb->lwb_vdev_tree));
return; return;
@ -1589,7 +1573,7 @@ zil_lwb_write_done(zio_t *zio)
while ((zv = avl_destroy_nodes(t, &cookie)) != NULL) { while ((zv = avl_destroy_nodes(t, &cookie)) != NULL) {
vdev_t *vd = vdev_lookup_top(spa, zv->zv_vdev); vdev_t *vd = vdev_lookup_top(spa, zv->zv_vdev);
if (vd != NULL) { if (vd != NULL && !vd->vdev_nowritecache) {
/* /*
* The "ZIO_FLAG_DONT_PROPAGATE" is currently * The "ZIO_FLAG_DONT_PROPAGATE" is currently
* always used within "zio_flush". This means, * always used within "zio_flush". This means,
@ -1980,8 +1964,6 @@ zil_lwb_commit(zilog_t *zilog, itx_t *itx, lwb_t *lwb)
zilog->zl_cur_used += (reclen + dlen); zilog->zl_cur_used += (reclen + dlen);
txg = lrc->lrc_txg; txg = lrc->lrc_txg;
ASSERT3U(zilog->zl_cur_used, <, UINT64_MAX - (reclen + dlen));
cont: cont:
/* /*
* If this record won't fit in the current log block, start a new one. * If this record won't fit in the current log block, start a new one.
@ -1997,7 +1979,6 @@ cont:
if (lwb == NULL) if (lwb == NULL)
return (NULL); return (NULL);
zil_lwb_write_open(zilog, lwb); zil_lwb_write_open(zilog, lwb);
ASSERT(LWB_EMPTY(lwb));
lwb_sp = lwb->lwb_sz - lwb->lwb_nused; lwb_sp = lwb->lwb_sz - lwb->lwb_nused;
/* /*
@ -2184,7 +2165,7 @@ zil_itxg_clean(void *arg)
itx_async_node_t *ian; itx_async_node_t *ian;
list = &itxs->i_sync_list; list = &itxs->i_sync_list;
while ((itx = list_head(list)) != NULL) { while ((itx = list_remove_head(list)) != NULL) {
/* /*
* In the general case, commit itxs will not be found * In the general case, commit itxs will not be found
* here, as they'll be committed to an lwb via * here, as they'll be committed to an lwb via
@ -2207,7 +2188,6 @@ zil_itxg_clean(void *arg)
if (itx->itx_lr.lrc_txtype == TX_COMMIT) if (itx->itx_lr.lrc_txtype == TX_COMMIT)
zil_commit_waiter_skip(itx->itx_private); zil_commit_waiter_skip(itx->itx_private);
list_remove(list, itx);
zil_itx_destroy(itx); zil_itx_destroy(itx);
} }
@ -2215,8 +2195,7 @@ zil_itxg_clean(void *arg)
t = &itxs->i_async_tree; t = &itxs->i_async_tree;
while ((ian = avl_destroy_nodes(t, &cookie)) != NULL) { while ((ian = avl_destroy_nodes(t, &cookie)) != NULL) {
list = &ian->ia_list; list = &ian->ia_list;
while ((itx = list_head(list)) != NULL) { while ((itx = list_remove_head(list)) != NULL) {
list_remove(list, itx);
/* commit itxs should never be on the async lists. */ /* commit itxs should never be on the async lists. */
ASSERT3U(itx->itx_lr.lrc_txtype, !=, TX_COMMIT); ASSERT3U(itx->itx_lr.lrc_txtype, !=, TX_COMMIT);
zil_itx_destroy(itx); zil_itx_destroy(itx);
@ -2277,8 +2256,7 @@ zil_remove_async(zilog_t *zilog, uint64_t oid)
list_move_tail(&clean_list, &ian->ia_list); list_move_tail(&clean_list, &ian->ia_list);
mutex_exit(&itxg->itxg_lock); mutex_exit(&itxg->itxg_lock);
} }
while ((itx = list_head(&clean_list)) != NULL) { while ((itx = list_remove_head(&clean_list)) != NULL) {
list_remove(&clean_list, itx);
/* commit itxs should never be on the async lists. */ /* commit itxs should never be on the async lists. */
ASSERT3U(itx->itx_lr.lrc_txtype, !=, TX_COMMIT); ASSERT3U(itx->itx_lr.lrc_txtype, !=, TX_COMMIT);
zil_itx_destroy(itx); zil_itx_destroy(itx);
@ -2580,7 +2558,7 @@ zil_commit_writer_stall(zilog_t *zilog)
*/ */
ASSERT(MUTEX_HELD(&zilog->zl_issuer_lock)); ASSERT(MUTEX_HELD(&zilog->zl_issuer_lock));
txg_wait_synced(zilog->zl_dmu_pool, 0); txg_wait_synced(zilog->zl_dmu_pool, 0);
ASSERT3P(list_tail(&zilog->zl_lwb_list), ==, NULL); ASSERT(list_is_empty(&zilog->zl_lwb_list));
} }
/* /*
@ -2605,7 +2583,7 @@ zil_process_commit_list(zilog_t *zilog)
* Return if there's nothing to commit before we dirty the fs by * Return if there's nothing to commit before we dirty the fs by
* calling zil_create(). * calling zil_create().
*/ */
if (list_head(&zilog->zl_itx_commit_list) == NULL) if (list_is_empty(&zilog->zl_itx_commit_list))
return; return;
list_create(&nolwb_itxs, sizeof (itx_t), offsetof(itx_t, itx_node)); list_create(&nolwb_itxs, sizeof (itx_t), offsetof(itx_t, itx_node));
@ -2629,7 +2607,7 @@ zil_process_commit_list(zilog_t *zilog)
plwb->lwb_state == LWB_STATE_FLUSH_DONE); plwb->lwb_state == LWB_STATE_FLUSH_DONE);
} }
while ((itx = list_head(&zilog->zl_itx_commit_list)) != NULL) { while ((itx = list_remove_head(&zilog->zl_itx_commit_list)) != NULL) {
lr_t *lrc = &itx->itx_lr; lr_t *lrc = &itx->itx_lr;
uint64_t txg = lrc->lrc_txg; uint64_t txg = lrc->lrc_txg;
@ -2643,8 +2621,6 @@ zil_process_commit_list(zilog_t *zilog)
zilog_t *, zilog, itx_t *, itx); zilog_t *, zilog, itx_t *, itx);
} }
list_remove(&zilog->zl_itx_commit_list, itx);
boolean_t synced = txg <= spa_last_synced_txg(spa); boolean_t synced = txg <= spa_last_synced_txg(spa);
boolean_t frozen = txg > spa_freeze_txg(spa); boolean_t frozen = txg > spa_freeze_txg(spa);
@ -2730,20 +2706,16 @@ zil_process_commit_list(zilog_t *zilog)
* normal. * normal.
*/ */
zil_commit_waiter_t *zcw; zil_commit_waiter_t *zcw;
while ((zcw = list_head(&nolwb_waiters)) != NULL) { while ((zcw = list_remove_head(&nolwb_waiters)) != NULL)
zil_commit_waiter_skip(zcw); zil_commit_waiter_skip(zcw);
list_remove(&nolwb_waiters, zcw);
}
/* /*
* And finally, we have to destroy the itx's that * And finally, we have to destroy the itx's that
* couldn't be committed to an lwb; this will also call * couldn't be committed to an lwb; this will also call
* the itx's callback if one exists for the itx. * the itx's callback if one exists for the itx.
*/ */
while ((itx = list_head(&nolwb_itxs)) != NULL) { while ((itx = list_remove_head(&nolwb_itxs)) != NULL)
list_remove(&nolwb_itxs, itx);
zil_itx_destroy(itx); zil_itx_destroy(itx);
}
} else { } else {
ASSERT(list_is_empty(&nolwb_waiters)); ASSERT(list_is_empty(&nolwb_waiters));
ASSERT3P(lwb, !=, NULL); ASSERT3P(lwb, !=, NULL);
@ -2951,7 +2923,7 @@ zil_commit_waiter_timeout(zilog_t *zilog, zil_commit_waiter_t *zcw)
*/ */
lwb_t *nlwb = zil_lwb_write_issue(zilog, lwb); lwb_t *nlwb = zil_lwb_write_issue(zilog, lwb);
IMPLY(nlwb != NULL, lwb->lwb_state != LWB_STATE_OPENED); ASSERT3S(lwb->lwb_state, !=, LWB_STATE_OPENED);
/* /*
* Since the lwb's zio hadn't been issued by the time this thread * Since the lwb's zio hadn't been issued by the time this thread
@ -3429,7 +3401,7 @@ zil_sync(zilog_t *zilog, dmu_tx_t *tx)
blkptr_t blk = zh->zh_log; blkptr_t blk = zh->zh_log;
dsl_dataset_t *ds = dmu_objset_ds(zilog->zl_os); dsl_dataset_t *ds = dmu_objset_ds(zilog->zl_os);
ASSERT(list_head(&zilog->zl_lwb_list) == NULL); ASSERT(list_is_empty(&zilog->zl_lwb_list));
memset(zh, 0, sizeof (zil_header_t)); memset(zh, 0, sizeof (zil_header_t));
memset(zilog->zl_replayed_seq, 0, memset(zilog->zl_replayed_seq, 0,
@ -3473,7 +3445,7 @@ zil_sync(zilog_t *zilog, dmu_tx_t *tx)
* out the zil_header blkptr so that we don't end * out the zil_header blkptr so that we don't end
* up freeing the same block twice. * up freeing the same block twice.
*/ */
if (list_head(&zilog->zl_lwb_list) == NULL) if (list_is_empty(&zilog->zl_lwb_list))
BP_ZERO(&zh->zh_log); BP_ZERO(&zh->zh_log);
} }
@ -3674,7 +3646,7 @@ zil_close(zilog_t *zilog)
if (!dmu_objset_is_snapshot(zilog->zl_os)) { if (!dmu_objset_is_snapshot(zilog->zl_os)) {
zil_commit(zilog, 0); zil_commit(zilog, 0);
} else { } else {
ASSERT3P(list_tail(&zilog->zl_lwb_list), ==, NULL); ASSERT(list_is_empty(&zilog->zl_lwb_list));
ASSERT0(zilog->zl_dirty_max_txg); ASSERT0(zilog->zl_dirty_max_txg);
ASSERT3B(zilog_is_dirty(zilog), ==, B_FALSE); ASSERT3B(zilog_is_dirty(zilog), ==, B_FALSE);
} }
@ -3716,15 +3688,14 @@ zil_close(zilog_t *zilog)
* We should have only one lwb left on the list; remove it now. * We should have only one lwb left on the list; remove it now.
*/ */
mutex_enter(&zilog->zl_lock); mutex_enter(&zilog->zl_lock);
lwb = list_head(&zilog->zl_lwb_list); lwb = list_remove_head(&zilog->zl_lwb_list);
if (lwb != NULL) { if (lwb != NULL) {
ASSERT3P(lwb, ==, list_tail(&zilog->zl_lwb_list)); ASSERT(list_is_empty(&zilog->zl_lwb_list));
ASSERT3S(lwb->lwb_state, !=, LWB_STATE_ISSUED); ASSERT3S(lwb->lwb_state, !=, LWB_STATE_ISSUED);
if (lwb->lwb_fastwrite) if (lwb->lwb_fastwrite)
metaslab_fastwrite_unmark(zilog->zl_spa, &lwb->lwb_blk); metaslab_fastwrite_unmark(zilog->zl_spa, &lwb->lwb_blk);
list_remove(&zilog->zl_lwb_list, lwb);
zio_buf_free(lwb->lwb_buf, lwb->lwb_sz); zio_buf_free(lwb->lwb_buf, lwb->lwb_sz);
zil_free_lwb(zilog, lwb); zil_free_lwb(zilog, lwb);
} }