From: Christoph Hellwig Subject: Re: Why doesn't zap_pte_range() call page_mkwrite() Date: Tue, 8 Sep 2009 22:21:02 -0400 Message-ID: <20090909022102.GA28318@infradead.org> References: <1240510668.11148.40.camel@heimdal.trondhjem.org> <1240519320.5602.9.camel@heimdal.trondhjem.org> <20090424104137.GA7601@sgi.com> <1240592448.4946.35.camel@heimdal.trondhjem.org> <20090425051028.GC10088@wotan.suse.de> <20090908153007.GB2513@think> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii To: Chris Mason , Nick Piggin , Trond Myklebust , Miklos Szeredi , holt@sgi.com, linux-nfs-fy+rA21nqHI@public.gmane.org Return-path: Received: from bombadil.infradead.org ([18.85.46.34]:43423 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751284AbZIICVK (ORCPT ); Tue, 8 Sep 2009 22:21:10 -0400 In-Reply-To: <20090908153007.GB2513@think> Sender: linux-nfs-owner@vger.kernel.org List-ID: On Tue, Sep 08, 2009 at 11:30:07AM -0400, Chris Mason wrote: > Sorry for digging up an old thread, but is there any reason we can't > just use page_mkwrite here? I'd love to get rid of the btrfs code to > detect places that use set_page_dirty without a page_mkwrite. It's not just btrfs, it's also a complete pain in the a** for XFS and probably every filesystems using ->page_mkwrite for dirty page tracking.