Re: [PATCH] ocfs2: kill osb->system_file_mutex lock
From: Tetsuo Handa
Date: Fri May 15 2026 - 11:55:32 EST
On 2026/05/16 0:35, Heming Zhao wrote:
> Hi,
>
> The logic here is incorrect. The purpose of the refcount is to track how many
> consumers are using the inode.
The igrab(inode) for the first time is for getting a refcount for the slot, isn't it?
The igrab(inode) for the subsequent times is for getting a refcount which is supposed to be dropped by the caller, isn't it?
_ocfs2_get_system_file_inode() always gets a refcount which is supposed to be dropped by the caller when it succeeds, doesn't it?
So why do you want one more refcount?