From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Thu, 19 Jan 2006 19:41:45 -0200 From: Marcelo Tosatti Subject: Re: [patch 3/3] mm: PageActive no testset Message-ID: <20060119214145.GA5115@dmt.cnet> References: <20060118024106.10241.69438.sendpatchset@linux.site> <20060118024139.10241.73020.sendpatchset@linux.site> <20060118141346.GB7048@dmt.cnet> <20060119145008.GA20126@wotan.suse.de> <20060119165222.GC4418@dmt.cnet> <20060119200226.GA1756@wotan.suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060119200226.GA1756@wotan.suse.de> Sender: owner-linux-mm@kvack.org Return-Path: To: Nick Piggin Cc: Linux Memory Management , Linux Kernel Mailing List , Hugh Dickins , Andrew Morton , Andrea Arcangeli , Linus Torvalds , David Miller List-ID: On Thu, Jan 19, 2006 at 09:02:26PM +0100, Nick Piggin wrote: > On Thu, Jan 19, 2006 at 02:52:22PM -0200, Marcelo Tosatti wrote: > > On Thu, Jan 19, 2006 at 03:50:08PM +0100, Nick Piggin wrote: > > > > > The test-set / test-clear operations also kind of imply that it is > > > being used for locking or without other synchronisation (usually). > > > > Non-atomic versions such as __ClearPageLRU()/__ClearPageActive() are > > not usable, though. > > > > Correct. Although I was able to use them in a couple of other places > in a subsequent patch in the series. I trust you don't see a problem > with those usages? Indeed, sorry. Would you mind adding a comment that page->flags must be accessed atomically otherwise and that __ versions are special as to when the page cannot be referenced anymore? (its really not obvious) Also this comments on top of page-flags.h could be updated * During disk I/O, PG_locked is used. This bit is set before I/O and * reset when I/O completes. page_waitqueue(page) is a wait queue of all tasks * waiting for the I/O on this page to complete. s/PG_locked/PG_writeback/ * Note that the referenced bit, the page->lru list_head and the active, * inactive_dirty and inactive_clean lists are protected by the * zone->lru_lock, and *NOT* by the usual PG_locked bit! inactive_dirty and inactive_clean do not exist anymore -- 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