Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755082Ab1CYUuY (ORCPT ); Fri, 25 Mar 2011 16:50:24 -0400 Received: from e8.ny.us.ibm.com ([32.97.182.138]:54676 "EHLO e8.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754515Ab1CYUuV (ORCPT ); Fri, 25 Mar 2011 16:50:21 -0400 Message-ID: <4D8D0009.8030806@linux.vnet.ibm.com> Date: Fri, 25 Mar 2011 13:50:17 -0700 From: Venkateswararao Jujjuri User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.2.15) Gecko/20110303 Thunderbird/3.1.9 MIME-Version: 1.0 To: "Aneesh Kumar K.V" CC: v9fs-developer@lists.sourceforge.net, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [V9fs-developer] [PATCH 1/5] fs/9p: Fix revalidate to return correct value References: <1301052651-21440-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> In-Reply-To: <1301052651-21440-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Content-Scanned: Fidelis XPS MAILER Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1006 Lines: 30 On 03/25/2011 04:30 AM, Aneesh Kumar K.V wrote: > revalidate should return> 0 on success. > > Signed-off-by: Aneesh Kumar K.V Reviewed-by: Venkateswararao Jujjuri > --- > fs/9p/vfs_dentry.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/9p/vfs_dentry.c b/fs/9p/vfs_dentry.c > index b6a3b9f..59c5ddc 100644 > --- a/fs/9p/vfs_dentry.c > +++ b/fs/9p/vfs_dentry.c > @@ -126,7 +126,7 @@ static int v9fs_lookup_revalidate(struct dentry *dentry, struct nameidata *nd) > retval = v9fs_refresh_inode_dotl(fid, inode); > else > retval = v9fs_refresh_inode(fid, inode); > - if (retval<= 0) > + if (retval< 0) > return retval; > } > out_valid: -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/