[libvirt] PATCH: 18/25: Dynamic thread workers pool
Jim Meyering
jim at meyering.net
Tue Jan 20 10:21:03 UTC 2009
"Daniel P. Berrange" <berrange at redhat.com> wrote:
> The libvirtd.conf file has three parameters
>
> max_clients
> min_workers
> max_workers
>
> When the daemon starts up it spawns min_workers threads. It
> accepts connections from upto max_clients. I never implemented
> the logic to auto-spawn more threads upto max_workers though.
...
Other than the request to rename "active",
and the signedness nits below, this looks fine.
ACK
> diff --git a/qemud/qemud.h b/qemud/qemud.h
...
> struct qemud_server {
> virMutex lock;
> virCond job;
>
> int nworkers;
> - pthread_t *workers;
> + int nactiveworkers;
> + struct qemud_worker *workers;
It'd be nice to make all of these "n*" members unsigned,
assuming that they should never go negative.
> int nsockets;
> struct qemud_socket *sockets;
> int nclients;
More information about the libvir-list
mailing list