From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx104.postini.com [74.125.245.104]) by kanga.kvack.org (Postfix) with SMTP id 1EB6E6B005A for ; Mon, 17 Dec 2012 14:44:38 -0500 (EST) Received: by mail-wg0-f41.google.com with SMTP id ds1so1903411wgb.2 for ; Mon, 17 Dec 2012 11:44:36 -0800 (PST) MIME-Version: 1.0 In-Reply-To: References: <1355708488-2913-1-git-send-email-tangchen@cn.fujitsu.com> <1355708488-2913-3-git-send-email-tangchen@cn.fujitsu.com> Date: Mon, 17 Dec 2012 11:44:36 -0800 Message-ID: Subject: Re: [PATCH v2 2/2] memory-hotplug: Disable CONFIG_MOVABLE_NODE option by default. From: Linus Torvalds Content-Type: multipart/alternative; boundary=e89a8f13eaf874cb9204d1119c36 Sender: owner-linux-mm@kvack.org List-ID: To: David Rientjes Cc: Pekka Enberg , Tang Chen , linux-mm@kvack.org, wency@cn.fujitsu.com, mel@csn.ul.ie, mingo@elte.hu, akpm@linux-foundation.org, kamezawa.hiroyu@jp.fujitsu.com, laijs@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, linux-kernel@vger.kernel.org --e89a8f13eaf874cb9204d1119c36 Content-Type: text/plain; charset=ISO-8859-1 The only thing broken about it was the condign option and the lack of docs, so... Linus On Dec 17, 2012 3:19 AM, "David Rientjes" wrote: > On Mon, 17 Dec 2012, Tang Chen wrote: > > > This patch set CONFIG_MOVABLE_NODE to "default n" instead of > > "depends on BROKEN". > > > > Signed-off-by: Tang Chen > > Reviewed-by: Yasuaki Ishimatsu > > It's fine to change the default, but what's missing here is a rationale > for no longer making it depend on CONFIG_BROKEN. > --e89a8f13eaf874cb9204d1119c36 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable

The only thing broken about it was the condign option and th= e lack of docs, so...

=A0=A0=A0=A0 Linus

On Dec 17, 2012 3:19 AM, "David Rientjes&qu= ot; <rientjes@google.com> = wrote:
On Mon, 17 Dec 2012, Tang Chen wrote:

> This patch set CONFIG_MOVABLE_NODE to "default n" instead of=
> "depends on BROKEN".
>
> Signed-off-by: Tang Chen <tangchen@cn.fujitsu.com>
> Reviewed-by: Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>

It's fine to change the default, but what's missing here is a ratio= nale
for no longer making it depend on CONFIG_BROKEN.
--e89a8f13eaf874cb9204d1119c36-- -- 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