Re: [PATCH] Add a page cache-backed balloon device driver.

2012-06-27 Thread Frank Swiderski
On Wed, Jun 27, 2012 at 9:06 AM, Michael S. Tsirkin wrote: > On Wed, Jun 27, 2012 at 08:48:55AM -0700, Frank Swiderski wrote: >> On Tue, Jun 26, 2012 at 7:56 PM, Rusty Russell wrote: >> > On Wed, 27 Jun 2012 00:41:06 +0300, "Michael S. Tsirkin" >> > wrote

Re: [PATCH] Add a page cache-backed balloon device driver.

2012-06-27 Thread Frank Swiderski
On Tue, Jun 26, 2012 at 7:56 PM, Rusty Russell wrote: > On Wed, 27 Jun 2012 00:41:06 +0300, "Michael S. Tsirkin" > wrote: >> On Tue, Jun 26, 2012 at 01:32:58PM -0700, Frank Swiderski wrote: >> > This implementation of a virtio balloon driver uses the page cache

Re: [PATCH] Add a page cache-backed balloon device driver.

2012-06-26 Thread Frank Swiderski
On Tue, Jun 26, 2012 at 2:45 PM, Rik van Riel wrote: > On 06/26/2012 05:31 PM, Frank Swiderski wrote: >> >> On Tue, Jun 26, 2012 at 1:40 PM, Rik van Riel  wrote: > > >>> The code looks good to me, my only worry is the >>> code duplication. We now have 5 ball

Re: [PATCH] Add a page cache-backed balloon device driver.

2012-06-26 Thread Frank Swiderski
On Tue, Jun 26, 2012 at 2:47 PM, Michael S. Tsirkin wrote: > On Tue, Jun 26, 2012 at 02:31:26PM -0700, Frank Swiderski wrote: >> On Tue, Jun 26, 2012 at 1:40 PM, Rik van Riel wrote: >> > On 06/26/2012 04:32 PM, Frank Swiderski wrote: >> >> >> >> This

Re: [PATCH] Add a page cache-backed balloon device driver.

2012-06-26 Thread Frank Swiderski
On Tue, Jun 26, 2012 at 1:40 PM, Rik van Riel wrote: > On 06/26/2012 04:32 PM, Frank Swiderski wrote: >> >> This implementation of a virtio balloon driver uses the page cache to >> "store" pages that have been released to the host.  The communication >> (outs

[PATCH] Add a page cache-backed balloon device driver.

2012-06-26 Thread Frank Swiderski
ll be mapped in, allowing automatic (and fast, + * compared to requiring a host notification via a virtio queue to get memory + * back) reclaim. + * + * Copyright 2008 Rusty Russell IBM Corporation + * Copyright 2011 Frank Swiderski Google Inc + * + * This program is free software; you can redistrib