On Tue, Mar 18, 2025 at 10:31:29PM -0400, Theodore Ts'o wrote:
On Tue, Mar 18, 2025 at 01:42:31PM +0530, Ojaswin Mujoo wrote:Hi Ted, Baokun,
Yes, that's what I meant; sorry for the not catching this before ISo this is something we need to do if the journal is actived, and ifI believe you mean inode == sbi->s_journal->j_inode here right?
it's active, then sbi->s_journal will be non-NULL, and so we can just
check to see if inode == sbi->s_journal instead. This will simplify
sent my reply.
Cheers,
- Ted
I got some time to revisit this. Seems like checking against
s_journal->j_inode is not enough. This is because both
ext4_check_blockref() and check_block_validity() can be called even
before journal->j_inode is set:
ext4_open_inode_journal
ext4_get_journal_inode
__ext4_iget
ext4_ind_check_inode
ext4_check_blockref /* j_inode not set */
journal = jbd2_journal_init_inode
bmap
ext4_bmap
iomap_bmap
ext4_iomap_begin
ext4_map_blocks
check_block_validity
journal->j_inode = inode
Now, I think in this case the best solution might be to use the extra
field like we do in this patch but set EXT4_SB(sb)->s_journal_ino
sufficiently early.
Thoughts?