From mboxrd@z Thu Jan 1 00:00:00 1970 From: Trond Myklebust Date: Tue, 10 Feb 2015 13:42:36 +0000 Subject: Re: [patch] pnfs: delete an unintended goto Message-Id: <1423575756.4226.0.camel@primarydata.com> List-Id: References: <20150210080322.GA8762@mwanda> In-Reply-To: <20150210080322.GA8762@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter Cc: Anna Schumaker , linux-nfs@vger.kernel.org, kernel-janitors@vger.kernel.org On Tue, 2015-02-10 at 11:03 +0300, Dan Carpenter wrote: > There was an extra goto here where it shouldn't be, because of a merge > error. > > Fixes: e2c63e091e29 ('Merge branch 'flexfiles'') > Signed-off-by: Dan Carpenter > > diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c > index 107b321..4f802b02 100644 > --- a/fs/nfs/pnfs.c > +++ b/fs/nfs/pnfs.c > @@ -1078,7 +1078,6 @@ bool pnfs_roc(struct inode *ino) > goto out_noroc; > } > > - goto out_noroc; > pnfs_clear_retry_layoutget(lo); > list_for_each_entry_safe(lseg, tmp, &lo->plh_segs, pls_list) > if (test_bit(NFS_LSEG_ROC, &lseg->pls_flags)) { Thanks Dan! Applied to linux-next. Cheers Trond -- Trond Myklebust Linux NFS client maintainer, PrimaryData trond.myklebust@primarydata.com