Re: [libvirt] PATCH: 18/25: Dynamic thread workers pool

[Jim Meyering <jim meyering net>]

"Daniel P. Berrange" <berrange 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;