Re: [Qemu-devel] [PATCH 0/3] re-enable balloon stats

[Luiz Capitulino]

On Tue, 11 Dec 2012 19:07:48 +0000
Dietmar Maurer <address@hidden> wrote:

> > I could move the check for the stats feature bit from the function that
> > enables polling to the timer callback. This would solve Dietmar's use-case,
> 
> that would be great.
> 
> > but it would silently fail if the feature is never negotiated (as you said
> > above).
> 
> Why don't we raise the error when we query the values?

Hmm, that's a good idea.

Only small nit is that, today old stats will remain available to be queried
even if you disable the timer. If we do what you suggest, old stats won't
be available if the module is unloaded (well, I *guess* the guest will unset
the feature bit on module removal).