[libvirt] [PATCH v2 1/2] timer impl

Eric Blake eblake at redhat.com
Wed Dec 15 15:24:44 UTC 2010


On 12/15/2010 08:20 AM, Eric Blake wrote:
> On 12/14/2010 07:34 PM, Wen Congyang wrote:
> 
> In addition to Hu's comments, and the fact that you are probably going
> to revise the exposed interface anyways, here's some additional points.

One other point - how does this relate to the timeouts already
implemented in places like daemon/event.c or src/util/event.c?  Are
those implementations already sufficient for your needs without having
to write a new implementation?  Or conversely, should your patch series
be lengthened into rewriting those interfaces to take advantage of your
new implementation in order to ease maintenance by focusing all timeout
code into a single reusable interface?  In other words, I'm still
seeking a bit more justification for this patch.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20101215/e74cd000/attachment-0001.sig>


More information about the libvir-list mailing list