public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Dmitry Torokhov <dtor@vmware.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"pv-drivers@vmware.com" <pv-drivers@vmware.com>
Subject: Re: [PATCH] VMware balloon: stop locking pages when hypervisor tells us enough
Date: Tue, 7 Dec 2010 17:15:08 -0800	[thread overview]
Message-ID: <201012071715.08384.dtor@vmware.com> (raw)
In-Reply-To: <20101207171023.d28e20ee.akpm@linux-foundation.org>

On Tuesday, December 07, 2010 05:10:23 pm Andrew Morton wrote:
> On Fri, 3 Dec 2010 08:52:26 -0800
> 
> Dmitry Torokhov <dtor@vmware.com> wrote:
> > When hypervisor decides to decrease target balloon size while the
> > balloon driver tries to lock pages hypervisor may respond with
> > VMW_BALLOON_PPN_NOTNEEDED. Use this data and immediately stop reserving
> > pages and wait for the next update cycle to fetch new target instead
> > of continuing trying to lock pages until size of refused list grows
> > above VMW_BALLOON_MAX_REFUSED (16) pages.
> 
> OK, but ... why?  What is the externally visible effect of this change?

As a result the driver stops bothering hypervisor with its attempts to
lock more pages that are not needed anymore. Most likely next order from
hypervisor will be to reduce ballon size anyway.

It is a small optimization.

Thanks,

Dmitry

      reply	other threads:[~2010-12-08  1:15 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-03 16:52 [PATCH] VMware balloon: stop locking pages when hypervisor tells us enough Dmitry Torokhov
2010-12-08  1:10 ` Andrew Morton
2010-12-08  1:15   ` Dmitry Torokhov [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201012071715.08384.dtor@vmware.com \
    --to=dtor@vmware.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pv-drivers@vmware.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox