From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-px0-f174.google.com ([209.85.212.174]:51470 "EHLO mail-px0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751674Ab0HTNXU (ORCPT ); Fri, 20 Aug 2010 09:23:20 -0400 Date: Fri, 20 Aug 2010 21:23:09 +0800 From: Wu Fengguang To: Trond Myklebust Cc: Jeff Layton , Christoph Hellwig , linux-nfs@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Subject: Re: why are WB_SYNC_NONE COMMITs being done with FLUSH_SYNC set ? Message-ID: <20100820132309.GB20126@localhost> References: <20100819101525.076831ad@barsoom.rdu.redhat.com> <20100819143710.GA4752@infradead.org> <1282229905.6199.19.camel@heimdal.trondhjem.org> <20100819151618.5f769dc9@tlielax.poochiereds.net> <1282246999.7799.66.camel@heimdal.trondhjem.org> Content-Type: text/plain; charset=us-ascii In-Reply-To: <1282246999.7799.66.camel@heimdal.trondhjem.org> Sender: linux-nfs-owner@vger.kernel.org List-ID: MIME-Version: 1.0 > > Here's a lightly tested patch that turns the check for the two flags > > into a check for WB_SYNC_NONE. It seems to do the right thing, but I > > don't have a clear testcase for it. Does this look reasonable? > > Looks fine to me. I'll queue it up for the post-2.6.36 merge window... Trond, I just created a patch that removes the wbc->nonblocking definition and all its references except NFS. So there will be merge dependencies. What should we do? To push both patches to Andrew's -mm tree? Thanks, Fengguang From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wu Fengguang Subject: Re: why are WB_SYNC_NONE COMMITs being done with FLUSH_SYNC set ? Date: Fri, 20 Aug 2010 21:23:09 +0800 Message-ID: <20100820132309.GB20126@localhost> References: <20100819101525.076831ad@barsoom.rdu.redhat.com> <20100819143710.GA4752@infradead.org> <1282229905.6199.19.camel@heimdal.trondhjem.org> <20100819151618.5f769dc9@tlielax.poochiereds.net> <1282246999.7799.66.camel@heimdal.trondhjem.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jeff Layton , Christoph Hellwig , linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org To: Trond Myklebust Return-path: Content-Disposition: inline In-Reply-To: <1282246999.7799.66.camel-rJ7iovZKK19ZJLDQqaL3InhyD016LWXt@public.gmane.org> Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-fsdevel.vger.kernel.org > > Here's a lightly tested patch that turns the check for the two flags > > into a check for WB_SYNC_NONE. It seems to do the right thing, but I > > don't have a clear testcase for it. Does this look reasonable? > > Looks fine to me. I'll queue it up for the post-2.6.36 merge window... Trond, I just created a patch that removes the wbc->nonblocking definition and all its references except NFS. So there will be merge dependencies. What should we do? To push both patches to Andrew's -mm tree? 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 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail143.messagelabs.com (mail143.messagelabs.com [216.82.254.35]) by kanga.kvack.org (Postfix) with SMTP id A7BB76B0331 for ; Fri, 20 Aug 2010 09:23:21 -0400 (EDT) Received: by pxi5 with SMTP id 5so1354699pxi.14 for ; Fri, 20 Aug 2010 06:23:20 -0700 (PDT) Date: Fri, 20 Aug 2010 21:23:09 +0800 From: Wu Fengguang Subject: Re: why are WB_SYNC_NONE COMMITs being done with FLUSH_SYNC set ? Message-ID: <20100820132309.GB20126@localhost> References: <20100819101525.076831ad@barsoom.rdu.redhat.com> <20100819143710.GA4752@infradead.org> <1282229905.6199.19.camel@heimdal.trondhjem.org> <20100819151618.5f769dc9@tlielax.poochiereds.net> <1282246999.7799.66.camel@heimdal.trondhjem.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1282246999.7799.66.camel@heimdal.trondhjem.org> Sender: owner-linux-mm@kvack.org To: Trond Myklebust Cc: Jeff Layton , Christoph Hellwig , linux-nfs@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org List-ID: > > Here's a lightly tested patch that turns the check for the two flags > > into a check for WB_SYNC_NONE. It seems to do the right thing, but I > > don't have a clear testcase for it. Does this look reasonable? > > Looks fine to me. I'll queue it up for the post-2.6.36 merge window... Trond, I just created a patch that removes the wbc->nonblocking definition and all its references except NFS. So there will be merge dependencies. What should we do? To push both patches to Andrew's -mm tree? Thanks, Fengguang -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org