On Friday 30 June 2006 18:59, Hisashi Hifumi wrote: > Hi. > > > This is correct, excpet you need to put the update_inode_transaction call > > inside reiserfs_write_lock/unlock. > > Again, I remade my patch. Thanks, this one looks good. > I have a question. > I understand update_inode_transaction must be inside reiserfs_write_lock. > Is update_inode_transaction needed to be inside journal_begin/end ? reiserfs_write_lock is enough, do_journal_end has reiserfs_write_lock while it is updating j_current_jl and j_trans_id. We just need to make sure the transid and the jl pointer are consistent with each other. -chris - 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