Displaying 2 results from an estimated 2 matches for "populate_pool".
2020 Aug 07
2
Re: [PATCH nbdkit 2/2] vddk: Relax thread model to PARALLEL and implement a disk handle pool.
...nection/handle. This may not matter if
the library initializes all fields of the struct, but it is safer
to zero the memory.
2. We create the handlers in request threads, so most likely not in
the same thread.
This may be ok, but risky.
I moved the creation of all connections and handles to populate_pool(), called
from vddk_open(). This may create more connections and handles than needed
if the client does not use multiple inflight requests, but this is not
important now.
The code compiles but I cannot test it. Does it make any difference?
Nir
>
> Rich.
>
> --
> Richard Jones, Vi...
2020 Aug 06
3
Re: [PATCH nbdkit 2/2] vddk: Relax thread model to PARALLEL and implement a disk handle pool.
On Thu, Aug 6, 2020, 16:16 Richard W.M. Jones <rjones@redhat.com> wrote:
> The pool is only used for readonly connections, since writable
> connections usually take a lock on the server side and therefore you
> cannot open more than one.
> ---
> plugins/vddk/nbdkit-vddk-plugin.pod | 7 +
> plugins/vddk/vddk.c | 201 ++++++++++++++++++++++------
> 2