Re: [libvirt] [PATCH v3 1/5] Add a threadpool implementation

2010-12-01 Thread Hu Tao
Hi Eric, Thanks for your careful review of these patches. I'll post v4 patches tomorrow fixing all problems you pointed out. daemon/libvirtd.c already has a notion of worker threads; I'm wondering how much overlap there is between your implementation and that one. A better proof that this

Re: [libvirt] [PATCH v3 1/5] Add a threadpool implementation

2010-12-01 Thread Daniel P. Berrange
On Wed, Dec 01, 2010 at 05:32:44PM +0800, Hu Tao wrote: Hi Eric, Thanks for your careful review of these patches. I'll post v4 patches tomorrow fixing all problems you pointed out. daemon/libvirtd.c already has a notion of worker threads; I'm wondering how much overlap there is

Re: [libvirt] [PATCH v3 1/5] Add a threadpool implementation

2010-11-30 Thread Eric Blake
On 11/30/2010 12:14 AM, Hu Tao wrote: --- src/Makefile.am |1 + src/util/threadpool.c | 140 + src/util/threadpool.h | 35 3 files changed, 176 insertions(+), 0 deletions(-) create mode 100644 src/util/threadpool.c

[libvirt] [PATCH v3 1/5] Add a threadpool implementation

2010-11-29 Thread Hu Tao
--- src/Makefile.am |1 + src/util/threadpool.c | 140 + src/util/threadpool.h | 35 3 files changed, 176 insertions(+), 0 deletions(-) create mode 100644 src/util/threadpool.c create mode 100644 src/util/threadpool.h