On Mon, Jan 23, 2017 at 07:26:58PM +0530, Nitesh Konkar wrote:
Hi Daniel,
On Mon, Jan 23, 2017 at 4:08 PM, Daniel P. Berrange <berrange@redhat.com> wrote:
On Mon, Jan 23, 2017 at 04:03:21PM +0530, Nitesh Konkar wrote:
@@ -9570,16 +9570,17 @@ qemuDomainSetPerfEvents(virDomainPtr dom, if (def) { for (i = 0; i < nparams; i++) { virTypedParameterPtr param = ¶ms[i]; - enabled = param->value.b; + state = param->value.i;
NACK, this semantic change is not backwards compatible - this will break all applications & language bindings currently using libvirt perf events.
Oh I see . I didn't realize that. Any other API/place you suggest, through which we can get this functionality in?
Why not just reset counters to zero when first enabling the perf event. If people want to reset them again later, they can just disable+enable the event to get the reset Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|