Received: by 10.223.185.116 with SMTP id b49csp8151196wrg; Thu, 1 Mar 2018 18:41:01 -0800 (PST) X-Google-Smtp-Source: AG47ELvmGWtVd0QoRGdUKr/+z/SeTyB9gLhKssnUiot7p6KYjS4Zj0NNckBNs0MohVThsWuqdgIJ X-Received: by 2002:a17:902:4d:: with SMTP id 71-v6mr3843016pla.341.1519958461361; Thu, 01 Mar 2018 18:41:01 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519958461; cv=none; d=google.com; s=arc-20160816; b=uumO55ibmle/c5OvWY4j7pfDvYvKaJWA6BROmNK6XV+dkake2+MUDoMITK6skhYSp/ ApQLov90oREXhiZLIJkHNh7oqLspeOpGxrsULMbxvqBLuSzPgqytMKyya/yL4Kk+Zdui fCxMRVc62A0DexC6DyrYzIvoOR85Uu+WgGmbdXu0gdoqEXJWXh3hUohYPpgemJenuVow 2lS3R7xK02HrCy2fIXmsRzAL04Vp3oPogEPnMsiWMN9gFuYEl9WZZkOlPz6iCGBGAPG7 aXhXSzcir3lvvu9SjkshxurgPWDqw+hLyecaYZNixlyKk163kOCxxqpaZf0ve8SiMOwa /TMA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-language :content-transfer-encoding:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:arc-authentication-results; bh=3SJhYY1lMtt+U0up5l2sg+kvjDbTWXtS0/gbQQA4rRs=; b=0jbQJ3x24Vkd04wo65RwWCT/sd7Tf+Nuwu3M4+NtAexEZxbBkyyHGHe4TZCMmK8yXc eAFEx6iWo9evejm+ndIjM4vRRYC67Usapfd+aX+sW0MecRhGsBUpYA8JUQFAgjAClvfm 6FmbbGSyVz8CSkgJKoTTXXkZ7eU2p2sKkMg3IRWyBtAj1yY1m1i4mQEZD6q3qSOuugZG 12ajYIvC2cadh3Y+twuJV38MfNSZBF1o7nnnmxuQnHDCr0x+oSkPaTBewksiG4FWPOQY 9G5vBtqsa0lSRbXZhuzX8j4Z/6SQbbLL02ZLN8djMN01B4ja6TvvmETzbKC9fgr3ryBx nwgQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id w22-v6si3112501pll.71.2018.03.01.18.40.46; Thu, 01 Mar 2018 18:41:01 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1164354AbeCBCha (ORCPT + 99 others); Thu, 1 Mar 2018 21:37:30 -0500 Received: from smtp2.provo.novell.com ([137.65.250.81]:55784 "EHLO smtp2.provo.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1164179AbeCBCh3 (ORCPT ); Thu, 1 Mar 2018 21:37:29 -0500 Received: from [10.67.19.114] (prv-ext-foundry1int.gns.novell.com [137.65.251.240]) by smtp2.provo.novell.com with ESMTP (TLS encrypted); Thu, 01 Mar 2018 19:37:14 -0700 Subject: Re: [Ocfs2-devel] [PATCH] Correct a comment error To: piaojun , Changwei Ge , "mfasheh@versity.com" , "jlbec@evilplan.org" Cc: "linux-kernel@vger.kernel.org" , "ocfs2-devel@oss.oracle.com" References: <20180228101720.20725-1-lchen@suse.com> <5A97F88E.4010303@huawei.com> <63ADC13FD55D6546B7DECE290D39E373F292C0FC@H3CMLB12-EX.srv.huawei-3com.com> <5A98B3BB.3040408@huawei.com> From: Larry Chen Message-ID: <40b08149-1a35-0dcc-116a-9df71b629950@suse.com> Date: Fri, 2 Mar 2018 10:37:04 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <5A98B3BB.3040408@huawei.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Changwei and Jun, Thanks for your advice. On 03/02/2018 10:15 AM, piaojun wrote: > Hi Changwei, > > On 2018/3/2 9:59, Changwei Ge wrote: >> Hi Jun, >> I think the comments for both two functions are OK. >> No need to rework them. >> As we know, ocfs2 lock name(lock id) are composed of several parts including >> block number. > I looked though the comments involved 'lockid', and found 'lockid' is a > concept in dlm level, so ocfs2 level should not be aware of it. When reading source code of ocfs2 level, you'll find that 'lockid' is a new concept that has not occurred or be explained before. It's kind of difficult to understand. Maybe 'blockno' or 'blockid' is more sensible than 'lockid' I'm willing to fix them all. Thanks Larry > thanks, > Jun >> Thanks, >> Changw2ei >> >> On 2018/3/1 20:58, piaojun wrote: >>> Hi Larry, >>> >>> There is the same mistake in ocfs2_reflink_inodes_lock(), could you help >>> fixing them all? >>> >>> thanks, >>> Jun >>> >>> On 2018/2/28 18:17, Larry Chen wrote: >>>> The function ocfs2_double_lock tries to lock the inode with lower >>>> blockid first, not lockid. >>>> >>>> Signed-off-by: Larry Chen >>>> --- >>>> fs/ocfs2/namei.c | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/fs/ocfs2/namei.c b/fs/ocfs2/namei.c >>>> index c801eddc4bf3..30d454de35a8 100644 >>>> --- a/fs/ocfs2/namei.c >>>> +++ b/fs/ocfs2/namei.c >>>> @@ -1133,7 +1133,7 @@ static int ocfs2_double_lock(struct ocfs2_super *osb, >>>> if (*bh2) >>>> *bh2 = NULL; >>>> >>>> - /* we always want to lock the one with the lower lockid first. >>>> + /* we always want to lock the one with the lower blockid first. >>>> * and if they are nested, we lock ancestor first */ >>>> if (oi1->ip_blkno != oi2->ip_blkno) { >>>> inode1_is_ancestor = ocfs2_check_if_ancestor(osb, oi2->ip_blkno, >>>> >>> _______________________________________________ >>> Ocfs2-devel mailing list >>> Ocfs2-devel@oss.oracle.com >>> https://oss.oracle.com/mailman/listinfo/ocfs2-devel >>> >> . >>