bcachefs: delete duplicated code

Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>

fs/bcachefs/journal.c

@@ -55,19 +55,6 @@ static void bch2_journal_buf_init(struct journal *j)
 	buf->data->u64s	= 0;
 }
 
-static inline bool journal_entry_empty(struct jset *j)
-{
-	struct jset_entry *i;
-
-	if (j->seq != j->last_seq)
-		return false;
-
-	vstruct_for_each(j, i)
-		if (i->type || i->u64s)
-			return false;
-	return true;
-}
-
 void bch2_journal_halt(struct journal *j)
 {
 	union journal_res_state old, new;

fs/bcachefs/journal.h

@@ -231,6 +231,19 @@ static inline void bch2_journal_add_keys(struct journal *j, struct journal_res *
 			       id, 0, k, k->k.u64s);
 }
 
+static inline bool journal_entry_empty(struct jset *j)
+{
+	struct jset_entry *i;
+
+	if (j->seq != j->last_seq)
+		return false;
+
+	vstruct_for_each(j, i)
+		if (i->type == BCH_JSET_ENTRY_btree_keys && i->u64s)
+			return false;
+	return true;
+}
+
 void __bch2_journal_buf_put(struct journal *, bool);
 
 static inline void bch2_journal_buf_put(struct journal *j, unsigned idx,

fs/bcachefs/recovery.c

@@ -322,32 +322,9 @@ static int read_btree_roots(struct bch_fs *c)
 
 static bool journal_empty(struct list_head *journal)
 {
-	struct journal_replay *i;
-	struct jset_entry *entry;
-
-	if (list_empty(journal))
-		return true;
-
-	i = list_last_entry(journal, struct journal_replay, list);
-
-	if (i->j.last_seq != i->j.seq)
-		return false;
-
-	list_for_each_entry(i, journal, list) {
-		vstruct_for_each(&i->j, entry) {
-			if (entry->type == BCH_JSET_ENTRY_btree_root ||
-			    entry->type == BCH_JSET_ENTRY_usage ||
-			    entry->type == BCH_JSET_ENTRY_data_usage)
-				continue;
-
-			if (entry->type == BCH_JSET_ENTRY_btree_keys &&
-			    !entry->u64s)
-				continue;
-			return false;
-		}
-	}
-
-	return true;
+	return list_empty(journal) ||
+		journal_entry_empty(&list_last_entry(journal,
+					struct journal_replay, list)->j);
 }
 
 int bch2_fs_recovery(struct bch_fs *c)