From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-vc0-f179.google.com (mail-vc0-f179.google.com [209.85.220.179]) by kanga.kvack.org (Postfix) with ESMTP id A00446B006E for ; Fri, 3 Oct 2014 14:31:32 -0400 (EDT) Received: by mail-vc0-f179.google.com with SMTP id im17so1018598vcb.24 for ; Fri, 03 Oct 2014 11:31:32 -0700 (PDT) Received: from mail-vc0-x236.google.com (mail-vc0-x236.google.com [2607:f8b0:400c:c03::236]) by mx.google.com with ESMTPS id sn8si4803275vdc.88.2014.10.03.11.31.31 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 03 Oct 2014 11:31:31 -0700 (PDT) Received: by mail-vc0-f182.google.com with SMTP id la4so1069527vcb.27 for ; Fri, 03 Oct 2014 11:31:31 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <1412356087-16115-11-git-send-email-aarcange@redhat.com> References: <1412356087-16115-1-git-send-email-aarcange@redhat.com> <1412356087-16115-11-git-send-email-aarcange@redhat.com> Date: Fri, 3 Oct 2014 11:31:31 -0700 Message-ID: Subject: Re: [PATCH 10/17] mm: rmap preparation for remap_anon_pages From: Linus Torvalds Content-Type: text/plain; charset=UTF-8 Sender: owner-linux-mm@kvack.org List-ID: To: Andrea Arcangeli Cc: qemu-devel@nongnu.org, KVM list , Linux Kernel Mailing List , linux-mm , Linux API , Andres Lagar-Cavilla , Dave Hansen , Paolo Bonzini , Rik van Riel , Mel Gorman , Andy Lutomirski , Andrew Morton , Sasha Levin , Hugh Dickins , Peter Feiner , "\\Dr. David Alan Gilbert\\" , Christopher Covington , Johannes Weiner , Android Kernel Team , Robert Love , Dmitry Adamushko , Neil Brown , Mike Hommey , Taras Glek , Jan Kara , KOSAKI Motohiro , Michel Lespinasse , Minchan Kim , Keith Packard , "Huangpeng (Peter)" , Isaku Yamahata , Anthony Liguori , Stefan Hajnoczi , Wenchao Xia , Andrew Jones , Juan Quintela On Fri, Oct 3, 2014 at 10:08 AM, Andrea Arcangeli wrote: > > Overall this looks a fairly small change to the rmap code, notably > less intrusive than the nonlinear vmas created by remap_file_pages. Considering that remap_file_pages() was an unmitigated disaster, and -mm has a patch to remove it entirely, I'm not at all convinced this is a good argument. We thought remap_file_pages() was a good idea, and it really really really wasn't. Almost nobody used it, why would the anonymous page case be any different? Linus -- 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