Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751914AbdHCR3S (ORCPT ); Thu, 3 Aug 2017 13:29:18 -0400 Received: from casper.infradead.org ([85.118.1.10]:40322 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751158AbdHCR3R (ORCPT ); Thu, 3 Aug 2017 13:29:17 -0400 Date: Thu, 3 Aug 2017 18:29:11 +0100 (BST) From: James Simmons To: NeilBrown cc: Greg KH , Oleg Drokin , Linux Kernel Mailing List , Lustre Development List Subject: Re: [lustre-devel] [PATCH] staging: lustre: fix minor typos in comments In-Reply-To: <87379aveek.fsf@notabene.neil.brown.name> Message-ID: References: <150041997277.20736.17112251996623587423.stgit@noble> <150042040735.20736.6997246449635538567.stgit@noble> <20170719082955.GA30088@kroah.com> <87379aveek.fsf@notabene.neil.brown.name> User-Agent: Alpine 2.20 (LFD 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170803_182911_339679_3E6430B3 X-CRM114-Status: GOOD ( 19.08 ) X-Spam-Score: -1.9 (-) X-Spam-Report: SpamAssassin version 3.4.1 on casper.infradead.org summary: Content analysis details: (-1.9 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 NO_RELAYS Informational: message was not relayed via SMTP -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2236 Lines: 50 > Fix minor typos in comments. Acked-by: James Simmons > Signed-off-by: NeilBrown > --- > drivers/staging/lustre/lustre/llite/namei.c | 2 +- > drivers/staging/lustre/lustre/mdc/mdc_locks.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/lustre/lustre/llite/namei.c b/drivers/staging/lustre/lustre/llite/namei.c > index 8ed24ec1255d..7a1133661a65 100644 > --- a/drivers/staging/lustre/lustre/llite/namei.c > +++ b/drivers/staging/lustre/lustre/llite/namei.c > @@ -488,7 +488,7 @@ static int ll_lookup_it_finish(struct ptlrpc_request *request, > *de = alias; > > if (!it_disposition(it, DISP_LOOKUP_NEG)) { > - /* we have lookup look - unhide dentry */ > + /* We have the "lookup" lock, so unhide dentry */ > if (bits & MDS_INODELOCK_LOOKUP) > d_lustre_revalidate(*de); > } else if (!it_disposition(it, DISP_OPEN_CREATE)) { > diff --git a/drivers/staging/lustre/lustre/mdc/mdc_locks.c b/drivers/staging/lustre/lustre/mdc/mdc_locks.c > index 3eb66cea65db..ae97c6f1aeb0 100644 > --- a/drivers/staging/lustre/lustre/mdc/mdc_locks.c > +++ b/drivers/staging/lustre/lustre/mdc/mdc_locks.c > @@ -1030,7 +1030,7 @@ int mdc_revalidate_lock(struct obd_export *exp, struct lookup_intent *it, > * If we're performing a creation, that means that unless the creation > * failed with EEXIST, we should fake up a negative dentry. > * > - * For everything else, we want to lookup to succeed. > + * For everything else, we want the lookup to succeed. > * > * One additional note: if CREATE or OPEN succeeded, we add an extra > * reference to the request because we need to keep it around until > @@ -1040,7 +1040,7 @@ int mdc_revalidate_lock(struct obd_export *exp, struct lookup_intent *it, > * exactly what it_status refers to. > * > * If DISP_OPEN_OPEN is set, then it_status refers to the open() call, > - * otherwise if DISP_OPEN_CREATE is set, then it status is the > + * otherwise if DISP_OPEN_CREATE is set, then it_status is the > * creation failure mode. In either case, one of DISP_LOOKUP_NEG or > * DISP_LOOKUP_POS will be set, indicating whether the child lookup > * was successful. > -- > 2.12.2 > >