[Ocfs2-devel] [PATCH] ocfs2/dlm: fix misuse of list_move_tail() in dlm_run_purge_list()
Xue jiufei
xuejiufei at huawei.com
Sun Jun 15 18:20:49 PDT 2014
On 2014/6/14 5:16, Mark Fasheh wrote:
> On Fri, Jun 13, 2014 at 03:26:29PM +0800, Xue jiufei wrote:
>> When a lockres in purge list but is still in use, it should be moved to
>> the tail of purge list. dlm_thread will continue to check next lockres
>> in purge list. However, code list_move_tail(&dlm->purge_list,
>> &lockres->purge) will do *no* movements, so dlm_thread will purge the
>> same lockres in this loop again and again. If it is in use for a long
>> time, other lockres will not be processed.
>>
>> Signed-off-by: Yiwen Jiang <jiangyiwen at huawei.com>
>> Signed-off-by: joyce.xue <xuejiufei at huawei.com>
>> ---
>> fs/ocfs2/dlm/dlmthread.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/fs/ocfs2/dlm/dlmthread.c b/fs/ocfs2/dlm/dlmthread.c
>> index 9db869d..ce017de 100644
>> --- a/fs/ocfs2/dlm/dlmthread.c
>> +++ b/fs/ocfs2/dlm/dlmthread.c
>> @@ -264,7 +264,7 @@ static void dlm_run_purge_list(struct dlm_ctxt *dlm,
>> "used %d, state %d\n", dlm->name,
>> lockres->lockname.len, lockres->lockname.name,
>> !unused, lockres->state);
>> - list_move_tail(&dlm->purge_list, &lockres->purge);
>> + list_move_tail(&lockres->purge, dlm->purge_list);
>
> Erf, good catch!
>
> But this wants to be:
>
> list_move_tail(&lockres->purge, &dlm->purge_list);
> ^^^^
> Notice the extra '&'
>
> correct?
> --Mark
>
Yes, my mistake. I have already sent the version two.
Would you please review it? thank you.
> --
> Mark Fasheh
>
More information about the Ocfs2-devel
mailing list