[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+55aFwrwYmWFXWpPeg-keKukW0=dwvmUBuN4NKA=JcseiUX3g@mail.gmail.com>
Date: Tue, 8 Apr 2014 10:01:39 -0700
From: Linus Torvalds <torvalds@...ux-foundation.org>
To: Mel Gorman <mgorman@...e.de>
Cc: "H. Peter Anvin" <hpa@...or.com>, Linux-X86 <x86@...nel.org>,
Cyrill Gorcunov <gorcunov@...il.com>,
Ingo Molnar <mingo@...nel.org>,
Steven Noonan <steven@...inklabs.net>,
Rik van Riel <riel@...hat.com>,
David Vrabel <david.vrabel@...rix.com>,
Andrew Morton <akpm@...ux-foundation.org>,
Peter Zijlstra <peterz@...radead.org>,
Andrea Arcangeli <aarcange@...hat.com>,
Dave Hansen <dave.hansen@...el.com>,
Srikar Dronamraju <srikar@...ux.vnet.ibm.com>,
Linux-MM <linux-mm@...ck.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [RFC PATCH 0/5] Use an alternative to _PAGE_PROTNONE for
_PAGE_NUMA v2
On Tue, Apr 8, 2014 at 9:46 AM, Mel Gorman <mgorman@...e.de> wrote:
>
> If you are ok with leaving _PAGE_NUMA as _PAGE_PROTNONE
NO I AM NOT!
Dammit, this feature is f*cking brain-damaged.
My complaint has been (and continues to be):
- either it is 100% the same as PROTNONE, in which case thjat
_PAGE_NUMA bit had better go away, and you just use the protnone
helpers!
- if it's not the same as PROTNONE, then it damn well needs a different bit.
You can't have it both ways. You guys tried. The Xen case shows that
trying to distinguish the two DOES NOT WORK. But even apart from the
Xen case, it was just a confusing hell.
Like Yoda said: "Either they are the same or they are not. There is no 'try'".
So pick one solution. Don't try to pick the mixed-up half-way case
that is a disaster and makes no sense.
Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists