[Ocfs2-devel] [PATCH 1/2] Ocfs2: Journaling i_flags and i_orphaned_slot when adding inode to orphan dir.
tristan
tristan.ye at oracle.com
Sun Mar 21 19:58:49 PDT 2010
Joel Becker wrote:
> On Fri, Mar 19, 2010 at 09:21:09AM +0800, Tristan Ye wrote:
>> Currently, some callers were missing to journal the dirty inode after
>> adding it to orphan dir.
>>
>> Now we're going to journal such modifications within the ocfs2_orphan_add()
>> itself, It's safe to do so, though some existing caller may duplicate this,
>> and it makes the logic look more straightforward anyway.
>>
>> Signed-off-by: Tristan Ye <tristan.ye at oracle.com>
>
> This patch has been added to the 'fixes' branch of ocfs2.git.
> Can you go over the callers of ocfs2_orphan_add() to see if there is any
> cleanup you can do? Redundant dirty calls where nothing else is
> dirtied, etc. That cleanup will be a separate patch that I push to the
> merge window when done.
3 callers are calling ocfs2_orphan_add() for now:
1. ocfs2_unlink()
2. ocfs2_rename()
3. ocfs2_create_inode_in_orphan()
caller #1 and #2 are also doing dirty journals, they however are not
redundant, just for i_links_count change on-disk.
while #3 caller is doing nothing dirty journal outside the
ocfs2_oprhan_add(), it's fine.
As in, we're not going to do any cleanup in the callers.
Regards,
Tristan.
>
> Joel
>
More information about the Ocfs2-devel
mailing list