On Tue, 11 Dec 2012 19:07:48 +0000 Dietmar Maurer <diet...@proxmox.com> 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).
