From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755687AbZHUSlR (ORCPT ); Fri, 21 Aug 2009 14:41:17 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753103AbZHUSlR (ORCPT ); Fri, 21 Aug 2009 14:41:17 -0400 Received: from one.firstfloor.org ([213.235.205.2]:40631 "EHLO one.firstfloor.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753841AbZHUSlQ (ORCPT ); Fri, 21 Aug 2009 14:41:16 -0400 Date: Fri, 21 Aug 2009 20:41:12 +0200 From: Andi Kleen To: Hugh Dickins Cc: Andrew Morton , Andi Kleen , 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 Subject: Re: [PATCH mmotm] ksm: antidote to MADV_MERGEABLE HWPOISON Message-ID: <20090821184112.GB18623@basil.fritz.box> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. -Andi