From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Mon, 16 Jan 2006 16:28:03 +0000 (GMT) From: Hugh Dickins Subject: Re: Race in new page migration code? In-Reply-To: Message-ID: References: <20060114155517.GA30543@wotan.suse.de> <20060114181949.GA27382@wotan.suse.de> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org Return-Path: To: Christoph Lameter Cc: Andi Kleen , Nick Piggin , Andrew Morton , Linux Kernel Mailing List , Linux Memory Management List List-ID: On Mon, 16 Jan 2006, Christoph Lameter wrote: > On Mon, 16 Jan 2006, Hugh Dickins wrote: > > > > It also applies to the policy compliance check. > > > > Good point, I missed that: you've inadventently changed the behaviour > > of sys_mbind when it encounters a zero page from a disallowed node. > > Another reason to remove your PageReserved test. > > The zero page always come from node zero on IA64. I think this is more the > inadvertent fixing of a bug. The policy compliance check currently fails > if an address range contains a zero page but node zero is not contained in > the nodelist. To me it sounds more like you introduced a bug than fixed one. If MPOL_MF_STRICT and the zero page is found but not in the nodelist demanded, then it's right to refuse, I'd say. If Andi shares your view that the zero pages should be ignored, I won't argue; but we shouldn't change behaviour by mistake, without review or comment. Hugh -- 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