Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932224Ab2F2Rqm (ORCPT ); Fri, 29 Jun 2012 13:46:42 -0400 Received: from mx1.redhat.com ([209.132.183.28]:31496 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752375Ab2F2Rql (ORCPT ); Fri, 29 Jun 2012 13:46:41 -0400 Date: Fri, 29 Jun 2012 14:46:25 -0300 From: Rafael Aquini To: Rusty Russell Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, virtualization@lists.linux-foundation.org, "Michael S. Tsirkin" , Rik van Riel , Mel Gorman , Andi Kleen , Andrew Morton , Konrad Rzeszutek Wilk Subject: Re: [PATCH v2 0/4] make balloon pages movable by compaction Message-ID: <20120629174624.GC1774@t510.redhat.com> References: <87r4syzqkn.fsf@rustcorp.com.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87r4syzqkn.fsf@rustcorp.com.au> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1969 Lines: 46 On Fri, Jun 29, 2012 at 02:01:52PM +0930, Rusty Russell wrote: > On Thu, 28 Jun 2012 18:49:38 -0300, Rafael Aquini wrote: > > This patchset follows the main idea discussed at 2012 LSFMMS section: > > "Ballooning for transparent huge pages" -- http://lwn.net/Articles/490114/ > > > > to introduce the required changes to the virtio_balloon driver, as well as > > changes to the core compaction & migration bits, in order to allow > > memory balloon pages become movable within a guest. > > > > Rafael Aquini (4): > > mm: introduce compaction and migration for virtio ballooned pages > > virtio_balloon: handle concurrent accesses to virtio_balloon struct > > elements > > virtio_balloon: introduce migration primitives to balloon pages > > mm: add vm event counters for balloon pages compaction > > > > drivers/virtio/virtio_balloon.c | 142 +++++++++++++++++++++++++++++++++++---- > > include/linux/mm.h | 16 +++++ > > include/linux/virtio_balloon.h | 6 ++ > > include/linux/vm_event_item.h | 2 + > > mm/compaction.c | 111 ++++++++++++++++++++++++------ > > mm/migrate.c | 32 ++++++++- > > mm/vmstat.c | 4 ++ > > 7 files changed, 280 insertions(+), 33 deletions(-) > > > > > > V2: address Mel Gorman's review comments > > If Mel is happy, I am happy. Seems sensible that the virtio_baloon > changes go in at the same time as the mm changes, so: > > Acked-by: Rusty Russell Thanks Rusty! I'll be respinning a v3 to address some extra suggestions Mel did, so if you have any concern that you'd like me to address, do not hesitate on letting me know! Cheers! Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/