From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from digeo-nav01.digeo.com (digeo-nav01.digeo.com [192.168.1.233]) by packet.digeo.com (8.9.3+Sun/8.9.3) with SMTP id LAA27500 for ; Fri, 13 Dec 2002 11:58:56 -0800 (PST) Message-ID: <3DFA3BFC.525D1B3@digeo.com> Date: Fri, 13 Dec 2002 11:58:52 -0800 From: Andrew Morton MIME-Version: 1.0 Subject: Re: 2.5.50-mm2 References: <3DF453C8.18B24E66@digeo.com> <20021213175526.C2581@sgi.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org Return-Path: To: Christoph Hellwig , "linux-xfs@oss.sgi.com" Cc: lkml , linux-mm@kvack.org List-ID: Christoph Hellwig wrote: > > On Mon, Dec 09, 2002 at 12:26:48AM -0800, Andrew Morton wrote: > > +remove-PF_SYNC.patch > > > > remove the current->flags:PF_SYNC abomination. Adds a `sync' arg to > > all writepage implementations to tell them whether they are being > > called for memory cleansing or for data integrity. > > Any chance you could pass down a struct writeback_control instead of > just the sync flag? XFS always used ->writepage similar to the > ->vm_writeback in older kernel releases because writing out more > than one page of delalloc space is really needed to be efficient and > this would allow us to get a few more hints about the VM's intentions. Yup, no probs. It would be good to measure how often that codepath actually gets invoked during testing and use. It's typically quite rare. It should be just MAP_SHARED stuff, although there are probably some highmem-related scenarii in which it will happen. I'll add a writeback_control.for_reclaim boolean so we don't have to play games with PF_MEMALLOC to reverse engineer the calling context. If XFS is going to writearound extra pages in ->writepage() then it would be best to set PG_reclaim (if wbc->for_reclaim) so end_page_writeback() will rotate them. -- 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/