From: Vyacheslav Dubeyko <slava@xxxxxxxxxxx> Subject: [PATCH v3 11/15] hfsplus: implement functionality of journal log wrapping The journal log is a circular buffer. Thereby, journal log's content can be splitted by journal end on two parts in some situations. When reading or writing the journal buffer, the I/O operation must stop at the end of the journal buffer and resume (wrap around) immediately following the journal header. This patch implements functionality of wrapping of journal log for such situation. Namely, it means setting of "start" field of journal header by journal header size value. Signed-off-by: Vyacheslav Dubeyko <slava@xxxxxxxxxxx> CC: Al Viro <viro@xxxxxxxxxxxxxxxxxx> CC: Christoph Hellwig <hch@xxxxxxxxxxxxx> CC: Hin-Tak Leung <htl10@xxxxxxxxxxxxxxxxxxxxx> --- fs/hfsplus/journal.c | 42 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 42 insertions(+) diff --git a/fs/hfsplus/journal.c b/fs/hfsplus/journal.c index 4562187..c40ef77 100644 --- a/fs/hfsplus/journal.c +++ b/fs/hfsplus/journal.c @@ -304,6 +304,36 @@ static inline bool need_to_wrap_journal(struct hfsplus_journal_header *jh, return (cur_off + req_size) > JOURNAL_SIZE(jh); } +static inline void __hfsplus_wrap_journal(struct super_block *sb) +{ + struct hfsplus_journal *jnl = HFSPLUS_SB(sb)->jnl; + + jnl->jh->start = jnl->jh->jhdr_size; +} + +/* Return rest bytes of binfo or blhdr from the journal begin */ +static inline u32 hfsplus_wrap_journal(struct super_block *sb, + u64 cur_off, u32 req_size) +{ + struct hfsplus_journal *jnl = HFSPLUS_SB(sb)->jnl; + struct hfsplus_journal_header *jh = jnl->jh; + u32 used_bytes; + u32 rest_bytes; + + hfs_dbg(JOURNAL, "cur_off %llu, req_size %u, JOURNAL_SIZE(jh) %llu\n", + cur_off, req_size, JOURNAL_SIZE(jh)); + + BUG_ON(cur_off > JOURNAL_SIZE(jh)); + BUG_ON((cur_off + req_size) < JOURNAL_SIZE(jh)); + + used_bytes = JOURNAL_SIZE(jh) - cur_off; + rest_bytes = req_size - used_bytes; + + __hfsplus_wrap_journal(sb); + + return rest_bytes; +} + static void hfsplus_deinit_block_list_desc(struct hfsplus_blist_desc *desc) { kfree(desc->blist_buf); @@ -557,6 +587,9 @@ static int hfsplus_replay_journal(struct super_block *sb) struct hfsplus_block_info *binfo; u32 i; + if (TR_START(jh) == JOURNAL_SIZE(jh)) + __hfsplus_wrap_journal(sb); + blhdr = hfsplus_get_blhdr(sb, JOURNAL_OFF_TO_SEC(sb), &desc); if (!blhdr) { err = -EIO; @@ -572,6 +605,15 @@ static int hfsplus_replay_journal(struct super_block *sb) last_seq_num = TR_SEQ_NUM(blhdr); + if (need_to_wrap_journal(jh, TR_START(jh), BLHDR_SIZE(jh))) { + u32 rest; + + rest = hfsplus_wrap_journal(sb, TR_START(jh), + BLHDR_SIZE(jh)); + le64_add_cpu(&jh->start, rest); + } else + le64_add_cpu(&jh->start, BLHDR_SIZE(jh)); + /* Check transaction */ for (i = 1; i < TR_BLOCKS(blhdr); i++) { binfo = hfsplus_get_binfo(sb, i, &desc); -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html