Re: [libvirt] [PATCH 7/7] remote: Use virDomainEventState helpers

2011-05-13 Thread Cole Robinson
On 05/13/2011 05:26 AM, Daniel P. Berrange wrote: > On Thu, May 12, 2011 at 01:14:31PM -0400, Cole Robinson wrote: >> One functionality change here is that we no longer force enable the event >> timeout for every queued event, only enable it for the first event after >> the queue has been flushed.

Re: [libvirt] [PATCH 7/7] remote: Use virDomainEventState helpers

2011-05-13 Thread Daniel P. Berrange
On Thu, May 12, 2011 at 01:14:31PM -0400, Cole Robinson wrote: > One functionality change here is that we no longer force enable the event > timeout for every queued event, only enable it for the first event after > the queue has been flushed. This is how other drivers have already done it, > and I

[libvirt] [PATCH 7/7] remote: Use virDomainEventState helpers

2011-05-12 Thread Cole Robinson
One functionality change here is that we no longer force enable the event timeout for every queued event, only enable it for the first event after the queue has been flushed. This is how other drivers have already done it, and I haven't encountered problems in practice. v3: Adjust for new virD