From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail138.messagelabs.com (mail138.messagelabs.com [216.82.249.35]) by kanga.kvack.org (Postfix) with SMTP id 7B9576B006A for ; Fri, 21 Aug 2009 15:24:22 -0400 (EDT) Date: Fri, 21 Aug 2009 20:23:51 +0100 (BST) From: Hugh Dickins Subject: Re: [PATCH mmotm] ksm: antidote to MADV_MERGEABLE HWPOISON In-Reply-To: <20090821184112.GB18623@basil.fritz.box> Message-ID: References: <20090821184112.GB18623@basil.fritz.box> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org To: Andi Kleen Cc: Andrew Morton , Wu Fengguang , Izik Eidus , Chris Wright , Andrea Arcangeli , Michael Kerrisk , Richard Henderson , Ivan Kokshaysky , Ralf Baechle , Kyle McMartin , Helge Deller , Chris Zankel , Rik van RIel , Balbir Singh , KAMEZAWA Hiroyuki , Avi Kivity , Nick Piggin , linux-kernel@vger.kernel.org, linux-mm@kvack.org List-ID: On Fri, 21 Aug 2009, Andi Kleen wrote: > On Fri, Aug 21, 2009 at 07:30:15PM +0100, Hugh Dickins wrote: > > linux-next is now sporting MADV_HWPOISON at 12, which would have a very > > nasty effect on KSM if you had CONFIG_MEMORY_FAILURE=y with CONFIG_KSM=y. > > Shift MADV_MERGEABLE and MADV_UNMERGEABLE down two - two to reduce the > > confusion if old and new userspace and kernel are mismatched. > > > > Personally I'd prefer the MADV_HWPOISON testing feature to shift; but > > linux-next comes first in the mmotm lineup, and I can't be sure that > > madvise KSM already has more users than there are HWPOISON testers: > > so unless Andi is happy to shift MADV_HWPOISON, mmotm needs this. > > Thanks for catching. > > Shifting is fine, but I would prefer then if it was to some > value that is not reused (like 100) so that I can probe for it > in the test programs. Thanks a lot for conceding so generously, Andi. Okay, let's forget that first patch I posted: here's its replacement, fix to linux-next.patch in mmotm, but you'll put into your tree to be picked up by linux-next in due course? Thanks again. (patch ends up with the 100 line in between the 11 line and the 12 line, but doesn't matter, it'll be easier to tidy that up with another patch afterwards.) [PATCH] hwpoison: shift MADV_HWPOISON to 100 mm assigns KSM's MADV_MERGEABLE to 12: shift MADV_HWPOISON from 12 to 100, out of the way so that poisoning test programs can safely probe for it. Signed-off-by: Hugh Dickins --- include/asm-generic/mman-common.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) --- next/include/asm-generic/mman-common.h 2009-08-21 19:51:15.000000000 +0100 +++ mmotm/include/asm-generic/mman-common.h 2009-08-21 19:53:21.000000000 +0100 @@ -34,7 +34,8 @@ #define MADV_REMOVE 9 /* remove these pages & resources */ #define MADV_DONTFORK 10 /* don't inherit across fork */ #define MADV_DOFORK 11 /* do inherit across fork */ -#define MADV_HWPOISON 12 /* poison a page for testing */ + +#define MADV_HWPOISON 100 /* poison a page for testing */ /* compatibility flags */ #define MAP_FILE 0 -- 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