On Tue, Nov 17, 2020 at 02:41:31PM -0500, Vivek Goyal wrote: > This is V3 of the patch. A minor change since V2 is to reverse the list > before executing requests. We are prepending elements to the list and that > means when we start processing requests, we are processing these in > the reverse order. Though we don't guarantee any ordering but it does > not hurt to process requests in same order as received as much as > possible. > > Right now we create a thread pool and main thread hands over the request > to thread in thread pool to process. Number of threads in thread pool > can be managed by option --thread-pool-size. > > In tests we have noted that many of the workloads are getting better > performance if we don't use a thread pool at all and process all > the requests in the context of a thread receiving the request. > > Hence give user an option to be able to run virtiofsd without using > a thread pool. > > To implement this, I have used existing option --thread-pool-size. This > option defines how many maximum threads can be in the thread pool. > Thread pool size zero freezes thead pool. I can't see why will one > start virtiofsd with a frozen thread pool (hence frozen file system). > So I am redefining --thread-pool-size=0 to mean, don't use a thread pool. > Instead process the request in the context of thread receiving request > from the queue. > > Signed-off-by: Vivek Goyal <[email protected]> > --- > tools/virtiofsd/fuse_virtio.c | 37 ++++++++++++++++++++++++++--------- > 1 file changed, 28 insertions(+), 9 deletions(-)
Reviewed-by: Stefan Hajnoczi <[email protected]>
signature.asc
Description: PGP signature
