Skip to content

Commit c89e999

Browse files
robnspauka
authored andcommitted
dnode: remove dn_dirtyctx_firstset
Old debug param, not used for anything. Sponsored-by: Klara, Inc. Sponsored-by: Wasabi Technology, Inc. Reviewed-by: Brian Behlendorf <[email protected]> Reviewed-by: Robert Evans <[email protected]> Reviewed-by: Adam Moss <[email protected]> Reviewed-by: Alexander Motin <[email protected]> Signed-off-by: Rob Norris <[email protected]> Closes openzfs#16297 Closes openzfs#17652 Closes openzfs#17658
1 parent e099147 commit c89e999

File tree

3 files changed

+0
-9
lines changed

3 files changed

+0
-9
lines changed

include/sys/dmu_impl.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,6 @@ extern "C" {
173173
* dn_notxholds
174174
* dn_nodnholds
175175
* dn_dirtyctx
176-
* dn_dirtyctx_firstset
177176
* (dn_phys copy fields?)
178177
* (dn_phys contents?)
179178
* held from:

include/sys/dnode.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,6 @@ struct dnode {
344344
kcondvar_t dn_notxholds;
345345
kcondvar_t dn_nodnholds;
346346
enum dnode_dirtycontext dn_dirtyctx;
347-
const void *dn_dirtyctx_firstset; /* dbg: contents meaningless */
348347

349348
/* protected by own devices */
350349
zfs_refcount_t dn_tx_holds;

module/zfs/dnode.c

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,6 @@ dnode_cons(void *arg, void *unused, int kmflag)
174174
dn->dn_free_txg = 0;
175175
dn->dn_assigned_txg = 0;
176176
dn->dn_dirtyctx = 0;
177-
dn->dn_dirtyctx_firstset = NULL;
178177
dn->dn_dirtycnt = 0;
179178
dn->dn_bonus = NULL;
180179
dn->dn_have_spill = B_FALSE;
@@ -230,7 +229,6 @@ dnode_dest(void *arg, void *unused)
230229
ASSERT0(dn->dn_free_txg);
231230
ASSERT0(dn->dn_assigned_txg);
232231
ASSERT0(dn->dn_dirtyctx);
233-
ASSERT0P(dn->dn_dirtyctx_firstset);
234232
ASSERT0(dn->dn_dirtycnt);
235233
ASSERT0P(dn->dn_bonus);
236234
ASSERT(!dn->dn_have_spill);
@@ -695,7 +693,6 @@ dnode_destroy(dnode_t *dn)
695693
dn->dn_dirtycnt = 0;
696694

697695
dn->dn_dirtyctx = 0;
698-
dn->dn_dirtyctx_firstset = NULL;
699696
if (dn->dn_bonus != NULL) {
700697
mutex_enter(&dn->dn_bonus->db_mtx);
701698
dbuf_destroy(dn->dn_bonus);
@@ -803,7 +800,6 @@ dnode_allocate(dnode_t *dn, dmu_object_type_t ot, int blocksize, int ibs,
803800
dn->dn_dirtyctx = 0;
804801

805802
dn->dn_free_txg = 0;
806-
dn->dn_dirtyctx_firstset = NULL;
807803
dn->dn_dirtycnt = 0;
808804

809805
dn->dn_allocated_txg = tx->tx_txg;
@@ -956,7 +952,6 @@ dnode_move_impl(dnode_t *odn, dnode_t *ndn)
956952
ndn->dn_free_txg = odn->dn_free_txg;
957953
ndn->dn_assigned_txg = odn->dn_assigned_txg;
958954
ndn->dn_dirtyctx = odn->dn_dirtyctx;
959-
ndn->dn_dirtyctx_firstset = odn->dn_dirtyctx_firstset;
960955
ndn->dn_dirtycnt = odn->dn_dirtycnt;
961956
ASSERT0(zfs_refcount_count(&odn->dn_tx_holds));
962957
zfs_refcount_transfer(&ndn->dn_holds, &odn->dn_holds);
@@ -1021,7 +1016,6 @@ dnode_move_impl(dnode_t *odn, dnode_t *ndn)
10211016
odn->dn_free_txg = 0;
10221017
odn->dn_assigned_txg = 0;
10231018
odn->dn_dirtyctx = 0;
1024-
odn->dn_dirtyctx_firstset = NULL;
10251019
odn->dn_dirtycnt = 0;
10261020
odn->dn_have_spill = B_FALSE;
10271021
odn->dn_zio = NULL;
@@ -2238,7 +2232,6 @@ dnode_set_dirtyctx(dnode_t *dn, dmu_tx_t *tx, const void *tag)
22382232
dn->dn_dirtyctx = DN_DIRTY_SYNC;
22392233
else
22402234
dn->dn_dirtyctx = DN_DIRTY_OPEN;
2241-
dn->dn_dirtyctx_firstset = tag;
22422235
}
22432236
if (ds != NULL) {
22442237
rrw_exit(&ds->ds_bp_rwlock, tag);

0 commit comments

Comments
 (0)