From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wu Fengguang Subject: Re: [PATCH] NFS: return -EAGAIN when skipped commit in nfs_commit_unstable_pages() Date: Tue, 14 Jun 2011 10:30:42 +0800 Message-ID: <20110614023042.GA5174@localhost> References: <20110610140459.GA8021@localhost> <1307993886.5569.3.camel@lade.trondhjem.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Christoph Hellwig , "linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , LKML To: Trond Myklebust Return-path: Content-Disposition: inline In-Reply-To: <1307993886.5569.3.camel-SyLVLa/KEI9HwK5hSS5vWB2eb7JE58TQ@public.gmane.org> Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-fsdevel.vger.kernel.org > > @@ -1521,7 +1521,7 @@ static int nfs_commit_unstable_pages(str > > { > > struct nfs_inode *nfsi = NFS_I(inode); > > int flags = FLUSH_SYNC; > > - int ret = 0; > > + int ret = -EAGAIN; > > > > if (wbc->sync_mode == WB_SYNC_NONE) { > > /* Don't commit yet if this is a non-blocking flush and there > > Looks good, but since this is a cleanup, I'm queueing it for 3.1 rather > than pushing it in the regression-fixing window. OK, thanks! Thanks, Fengguang -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html