diff options
| author | Kent Overstreet <kent.overstreet@linux.dev> | 2022-10-30 22:21:11 -0400 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-03-13 11:34:41 -0400 |
| commit | 4494cd1542e9b3ef6865959dc882d702b85f7626 (patch) | |
| tree | 160dc21db06185688232a1da10bad163c3a42ff7 | |
| parent | dad8d8dc7221871b16be68cab770e7fc425c5fe4 (diff) | |
bcachefs: Kill BCH_WRITE_JOURNAL_SEQ_PTR
Dead code, delete.
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
| -rw-r--r-- | fs/bcachefs/data_update.c | 2 | ||||
| -rw-r--r-- | fs/bcachefs/io.c | 4 | ||||
| -rw-r--r-- | fs/bcachefs/io.h | 8 | ||||
| -rw-r--r-- | fs/bcachefs/io_types.h | 9 |
4 files changed, 4 insertions, 19 deletions
diff --git a/fs/bcachefs/data_update.c b/fs/bcachefs/data_update.c index 04b84e10c25c..f340c6234486 100644 --- a/fs/bcachefs/data_update.c +++ b/fs/bcachefs/data_update.c @@ -227,7 +227,7 @@ int bch2_data_update_index_update(struct bch_write_op *op) bch2_trans_update(&trans, &iter, insert, BTREE_UPDATE_INTERNAL_SNAPSHOT_NODE) ?: bch2_trans_commit(&trans, &op->res, - op_journal_seq(op), + &op->journal_seq, BTREE_INSERT_NOFAIL| m->data_opts.btree_insert_flags); if (!ret) { diff --git a/fs/bcachefs/io.c b/fs/bcachefs/io.c index 32039aa7b534..a4537f410cc4 100644 --- a/fs/bcachefs/io.c +++ b/fs/bcachefs/io.c @@ -508,7 +508,7 @@ static int bch2_write_index_default(struct bch_write_op *op) BTREE_ITER_SLOTS|BTREE_ITER_INTENT); ret = bch2_extent_update(&trans, inum, &iter, sk.k, - &op->res, op_journal_seq(op), + &op->res, &op->journal_seq, op->new_i_size, &op->i_sectors_delta, op->flags & BCH_WRITE_CHECK_ENOSPC); bch2_trans_iter_exit(&trans, &iter); @@ -777,7 +777,7 @@ unlock: __bch2_write(op); } else if (!op->error && (op->flags & BCH_WRITE_FLUSH)) { bch2_journal_flush_seq_async(&op->c->journal, - *op_journal_seq(op), + op->journal_seq, &op->cl); continue_at(&op->cl, bch2_write_done, index_update_wq(op)); } else { diff --git a/fs/bcachefs/io.h b/fs/bcachefs/io.h index 5ab9ecb632d1..ac1711481f6b 100644 --- a/fs/bcachefs/io.h +++ b/fs/bcachefs/io.h @@ -40,7 +40,6 @@ enum bch_write_flags { __BCH_WRITE_FROM_INTERNAL, __BCH_WRITE_CHECK_ENOSPC, __BCH_WRITE_MOVE, - __BCH_WRITE_JOURNAL_SEQ_PTR, __BCH_WRITE_IN_WORKER, __BCH_WRITE_DONE, __BCH_WRITE_IO_ERROR, @@ -59,17 +58,10 @@ enum bch_write_flags { #define BCH_WRITE_MOVE (1U << __BCH_WRITE_MOVE) /* Internal: */ -#define BCH_WRITE_JOURNAL_SEQ_PTR (1U << __BCH_WRITE_JOURNAL_SEQ_PTR) #define BCH_WRITE_IN_WORKER (1U << __BCH_WRITE_IN_WORKER) #define BCH_WRITE_DONE (1U << __BCH_WRITE_DONE) #define BCH_WRITE_IO_ERROR (1U << __BCH_WRITE_IO_ERROR) -static inline u64 *op_journal_seq(struct bch_write_op *op) -{ - return (op->flags & BCH_WRITE_JOURNAL_SEQ_PTR) - ? op->journal_seq_p : &op->journal_seq; -} - static inline struct workqueue_struct *index_update_wq(struct bch_write_op *op) { return op->alloc_reserve == RESERVE_movinggc diff --git a/fs/bcachefs/io_types.h b/fs/bcachefs/io_types.h index 10fbe352a182..ca65f2c52c1c 100644 --- a/fs/bcachefs/io_types.h +++ b/fs/bcachefs/io_types.h @@ -140,14 +140,7 @@ struct bch_write_op { struct open_buckets open_buckets; - /* - * If caller wants to flush but hasn't passed us a journal_seq ptr, we - * still need to stash the journal_seq somewhere: - */ - union { - u64 *journal_seq_p; - u64 journal_seq; - }; + u64 journal_seq; u64 new_i_size; s64 i_sectors_delta; |