From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pd0-f171.google.com (mail-pd0-f171.google.com [209.85.192.171]) by kanga.kvack.org (Postfix) with ESMTP id 9BA096B0037 for ; Fri, 12 Sep 2014 20:06:18 -0400 (EDT) Received: by mail-pd0-f171.google.com with SMTP id p10so2269781pdj.2 for ; Fri, 12 Sep 2014 17:06:18 -0700 (PDT) Received: from mail.linuxfoundation.org (mail.linuxfoundation.org. [140.211.169.12]) by mx.google.com with ESMTPS id rt2si10786964pbc.18.2014.09.12.17.06.17 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 12 Sep 2014 17:06:17 -0700 (PDT) Date: Fri, 12 Sep 2014 17:06:16 -0700 From: Andrew Morton Subject: Re: [PATCH v2 6/6] mm/balloon_compaction: general cleanup Message-Id: <20140912170616.cb4c832a09cc2b221453ad32@linux-foundation.org> In-Reply-To: <20140912170404.f14663cc823691cab36bf793@linux-foundation.org> References: <20140830163834.29066.98205.stgit@zurg> <20140830164127.29066.99498.stgit@zurg> <20140912170404.f14663cc823691cab36bf793@linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Konstantin Khlebnikov , Konstantin Khlebnikov , Rafael Aquini , Linux Kernel Mailing List , linux-mm@kvack.org, Andrey Ryabinin , Sasha Levin On Fri, 12 Sep 2014 17:04:04 -0700 Andrew Morton wrote: > On Sat, 30 Aug 2014 20:41:27 +0400 Konstantin Khlebnikov wrote: > > > * move special branch for balloon migraion into migrate_pages > > * remove special mapping for balloon and its flag AS_BALLOON_MAP > > * embed struct balloon_dev_info into struct virtio_balloon > > * cleanup balloon_page_dequeue, kill balloon_page_free > > Not sure what's going on here - your include/linux/balloon_compaction.h > seems significantly different from mine. OK, I worked it out. > I think I'll just drop this patch - it's quite inconvenient to have a > large "general cleanup" coming after a stack of significant functional > changes. It makes review, debug, fix, merge and reversion harder. > Let's worry about it later. But I'm still thinking we should defer this one? -- 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