diff options
author | Cyril Plisko <[email protected]> | 2014-01-15 11:26:12 +0200 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2014-01-23 13:31:51 -0800 |
commit | 01b738f457f2a406fb6b4b264fb7a947b9b9989b (patch) | |
tree | 6b31f68856959d4d499f365114ffdad79c172aeb /include/sys/spa.h | |
parent | 478d64fdaeb89c8f029f3dd1969447317eedaa6e (diff) |
Call gethrtime() only once per new txg creation
When transitioning current open TXG into QUIESCE state and opening
a new one txg_quiesce() calls gethrtime():
- to mark the birth time of the new TXG
- to record the SPA txg history kstat
- implicitely inside spa_txg_history_add()
These timestamps are practically the same, so that the first one
can be used instead of the other two. The only visible difference
is that inside spa_txg_history_add() the time spent in kmem_zalloc()
will be counted towards the opened TXG.
Since at this point the new TXG already exists (tx->tx_open_txg
has been already incremented) it is actually a correct accounting.
In any case this extra work is only happening when spa_txg_history
kstat is activated (i.e. zfs_txg_history > 0) and doesn't affect
the normal processing in any way.
Signed-off-by: Cyril Plisko <[email protected]>
Issue #2075
Diffstat (limited to 'include/sys/spa.h')
-rw-r--r-- | include/sys/spa.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/sys/spa.h b/include/sys/spa.h index 0079c18fe..e7cbff806 100644 --- a/include/sys/spa.h +++ b/include/sys/spa.h @@ -566,7 +566,7 @@ extern void spa_stats_init(spa_t *spa); extern void spa_stats_destroy(spa_t *spa); extern void spa_read_history_add(spa_t *spa, const zbookmark_t *zb, uint32_t aflags); -extern void spa_txg_history_add(spa_t *spa, uint64_t txg); +extern void spa_txg_history_add(spa_t *spa, uint64_t txg, hrtime_t birth_time); extern int spa_txg_history_set(spa_t *spa, uint64_t txg, txg_state_t completed_state, hrtime_t completed_time); extern int spa_txg_history_set_io(spa_t *spa, uint64_t txg, uint64_t nread, |