From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:43190) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SWw6x-0008Tb-3q for qemu-devel@nongnu.org; Tue, 22 May 2012 16:55:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SWw6v-0006tM-Fk for qemu-devel@nongnu.org; Tue, 22 May 2012 16:55:18 -0400 Received: from mx1.redhat.com ([209.132.183.28]:23591) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SWw6v-0006sz-78 for qemu-devel@nongnu.org; Tue, 22 May 2012 16:55:17 -0400 Date: Tue, 22 May 2012 17:55:30 -0300 From: Luiz Capitulino Message-ID: <20120522175530.544db206@doriath.home> In-Reply-To: <1337619593-25823-1-git-send-email-berrange@redhat.com> References: <1337619593-25823-1-git-send-email-berrange@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/3] Event notifications for balloon driver List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Daniel P. Berrange" Cc: Amit Shah , qemu-devel@nongnu.org, Anthony Liguori , Markus Armbruster On Mon, 21 May 2012 17:59:50 +0100 "Daniel P. Berrange" wrote: > This series is a followup to 2 previously posted patches > > * BALLOON_CHANGE QMP event: > http://lists.nongnu.org/archive/html/qemu-devel/2012-05/msg02215.html > > * query-events QMP command: > http://lists.nongnu.org/archive/html/qemu-devel/2012-05/msg02255.html > > Changes since v1: > > - Use a static array of strings for QMP event ID -> string conversion > - Add BALLOON_CHANGE to qmp-events.txt > > There is also a new patch in this series, which introduces the ability > todo simple rate limiting of stateless monitor events. With the ballooning > of a 1.8 GB guest, down to 0.9 GB this reduced the number of events > emitted from ~50 down to just 4, spread across a 4 second time window. How would that be with a 1TB guest? One way of solving this would be to move the policy to the mngt app. that is, we could have a qmp-event-set-rate-limit command that could be allowed to be run while in negotiation mode (ie. before qmp_capabilities is executed). But I'm honestly not sure if rate limit is the best solution for this problem... How can several events spread in several seconds be useful to libvirt? IMO, the best would be to have a way to know when the balloon driver is done servicing a balloon request. Amit, is this possible?