From: "J. Bruce Fields" Subject: Re: warning in ext4_journal_start_sb on filesystem freeze Date: Tue, 1 Apr 2014 14:40:03 -0400 Message-ID: <20140401184003.GA20903@fieldses.org> References: <1697998867.143517.1385454051031.JavaMail.zimbra@rapitasystems.com> <20131126125826.GA4503@quack.suse.cz> <622177618.727.1393062606061.JavaMail.zimbra@rapitasystems.com> <20140224095525.GA20532@quack.suse.cz> <20140224154532.GB11992@fieldses.org> <20140225102126.GB1669@quack.suse.cz> <20140304164306.GC12805@fieldses.org> <20140304190442.GE12805@fieldses.org> <20140310133451.GA17807@infradead.org> <20140310195709.GH28006@fieldses.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jan Kara , Matthew Rahtz , linux-ext4@vger.kernel.org, linux-nfs@vger.kernel.org To: Christoph Hellwig Return-path: Received: from fieldses.org ([174.143.236.118]:37888 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751254AbaDASkL (ORCPT ); Tue, 1 Apr 2014 14:40:11 -0400 Content-Disposition: inline In-Reply-To: <20140310195709.GH28006@fieldses.org> Sender: linux-ext4-owner@vger.kernel.org List-ID: On Mon, Mar 10, 2014 at 03:57:09PM -0400, J. Bruce Fields wrote: > On Mon, Mar 10, 2014 at 06:34:51AM -0700, Christoph Hellwig wrote: > > On Tue, Mar 04, 2014 at 02:04:42PM -0500, J. Bruce Fields wrote: > > > diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c > > > index 6d7be3f..eea5ad1 100644 > > > --- a/fs/nfsd/vfs.c > > > +++ b/fs/nfsd/vfs.c > > > @@ -404,6 +404,7 @@ nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap, > > > umode_t ftype = 0; > > > __be32 err; > > > int host_err; > > > + bool get_write_count; > > > int size_change = 0; > > > > > > if (iap->ia_valid & (ATTR_ATIME | ATTR_MTIME | ATTR_SIZE)) > > > @@ -411,10 +412,18 @@ nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap, > > > if (iap->ia_valid & ATTR_SIZE) > > > ftype = S_IFREG; > > > > > > + /* Callers that do fh_verify should do the fh_want_write: */ > > > + get_write_count = !fhp->fh_dentry; > > > > Eww, this is nasty. Given that there are only 6 callers of nfsd_setattr > > in total, and only half of these might cause size changes I'd rather > > deal with this properly, e.g. by taking both the fh_verify into the > > callers. > > Maybe so. Gah, I found clearing out my invoice that a) I'd forgotten this, b) I'd already committed and pushed out the patch. And I'd rather leave the fix as is and the cleanup to be done later. But it's not OK to just drop review like that and if you think it warrants reverting or rebasing I can do that. --b.