Re: [PATCH] docs: Correct the default thread-pool-size
On Wed, Apr 13, 2022 at 12:20:54PM +0800, Liu Yiding wrote: > Refer to 26ec190964 virtiofsd: Do not use a thread pool by default > > Signed-off-by: Liu Yiding > --- > docs/tools/virtiofsd.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Hi, Sorry it took so long to apply this! Thanks, applied to my block-next tree: https://gitlab.com/stefanha/qemu/commits/block-next Stefan signature.asc Description: PGP signature
Re: [PATCH] docs: Correct the default thread-pool-size
Gently ping On 4/27/22 9:12 AM, liuyd.f...@fujitsu.com wrote: > [+cc qemu-trivial] > > On 4/14/22 8:19 PM, Vivek Goyal wrote: >> On Wed, Apr 13, 2022 at 12:20:54PM +0800, Liu Yiding wrote: >>> Refer to 26ec190964 virtiofsd: Do not use a thread pool by default >>> >>> Signed-off-by: Liu Yiding >> Looks good. Our default used to be --thread-pool-size=64. But we changed >> it to using no thread pool because on lower end of workloads it performed >> better. When multiple threads are doing parallel I/O then, thread pool >> helps. So people who want to do lots of parallel I/O should manually >> enable thread pool. >> >> Acked-by: Vivek Goyal >> >> Vivek >>> --- >>>docs/tools/virtiofsd.rst | 2 +- >>>1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/docs/tools/virtiofsd.rst b/docs/tools/virtiofsd.rst >>> index 0c0560203c..33fed08c6f 100644 >>> --- a/docs/tools/virtiofsd.rst >>> +++ b/docs/tools/virtiofsd.rst >>> @@ -127,7 +127,7 @@ Options >>>.. option:: --thread-pool-size=NUM >>> >>> Restrict the number of worker threads per request queue to NUM. The >>> default >>> - is 64. >>> + is 0. >>> >>>.. option:: --cache=none|auto|always >>> >>> -- >>> 2.31.1 >>> >>> >>> >>> -- Best Regards. Yiding Liu
Re: [PATCH] docs: Correct the default thread-pool-size
[+cc qemu-trivial] On 4/14/22 8:19 PM, Vivek Goyal wrote: > On Wed, Apr 13, 2022 at 12:20:54PM +0800, Liu Yiding wrote: >> Refer to 26ec190964 virtiofsd: Do not use a thread pool by default >> >> Signed-off-by: Liu Yiding > Looks good. Our default used to be --thread-pool-size=64. But we changed > it to using no thread pool because on lower end of workloads it performed > better. When multiple threads are doing parallel I/O then, thread pool > helps. So people who want to do lots of parallel I/O should manually > enable thread pool. > > Acked-by: Vivek Goyal > > Vivek >> --- >> docs/tools/virtiofsd.rst | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/docs/tools/virtiofsd.rst b/docs/tools/virtiofsd.rst >> index 0c0560203c..33fed08c6f 100644 >> --- a/docs/tools/virtiofsd.rst >> +++ b/docs/tools/virtiofsd.rst >> @@ -127,7 +127,7 @@ Options >> .. option:: --thread-pool-size=NUM >> >> Restrict the number of worker threads per request queue to NUM. The >> default >> - is 64. >> + is 0. >> >> .. option:: --cache=none|auto|always >> >> -- >> 2.31.1 >> >> >> >> -- Best Regards. Yiding Liu
Re: [PATCH] docs: Correct the default thread-pool-size
Hi, Stefan Please help review it. I'm sorry that I forgot to add you to the recipient 😅 On 4/14/22 8:19 PM, Vivek Goyal wrote: > On Wed, Apr 13, 2022 at 12:20:54PM +0800, Liu Yiding wrote: >> Refer to 26ec190964 virtiofsd: Do not use a thread pool by default >> >> Signed-off-by: Liu Yiding > Looks good. Our default used to be --thread-pool-size=64. But we changed > it to using no thread pool because on lower end of workloads it performed > better. When multiple threads are doing parallel I/O then, thread pool > helps. So people who want to do lots of parallel I/O should manually > enable thread pool. > > Acked-by: Vivek Goyal > > Vivek >> --- >> docs/tools/virtiofsd.rst | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/docs/tools/virtiofsd.rst b/docs/tools/virtiofsd.rst >> index 0c0560203c..33fed08c6f 100644 >> --- a/docs/tools/virtiofsd.rst >> +++ b/docs/tools/virtiofsd.rst >> @@ -127,7 +127,7 @@ Options >> .. option:: --thread-pool-size=NUM >> >> Restrict the number of worker threads per request queue to NUM. The >> default >> - is 64. >> + is 0. >> >> .. option:: --cache=none|auto|always >> >> -- >> 2.31.1 >> >> >> >> -- Best Regards. Yiding Liu
Re: [PATCH] docs: Correct the default thread-pool-size
On Wed, Apr 13, 2022 at 12:20:54PM +0800, Liu Yiding wrote: > Refer to 26ec190964 virtiofsd: Do not use a thread pool by default > > Signed-off-by: Liu Yiding Looks good. Our default used to be --thread-pool-size=64. But we changed it to using no thread pool because on lower end of workloads it performed better. When multiple threads are doing parallel I/O then, thread pool helps. So people who want to do lots of parallel I/O should manually enable thread pool. Acked-by: Vivek Goyal Vivek > --- > docs/tools/virtiofsd.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/docs/tools/virtiofsd.rst b/docs/tools/virtiofsd.rst > index 0c0560203c..33fed08c6f 100644 > --- a/docs/tools/virtiofsd.rst > +++ b/docs/tools/virtiofsd.rst > @@ -127,7 +127,7 @@ Options > .. option:: --thread-pool-size=NUM > >Restrict the number of worker threads per request queue to NUM. The > default > - is 64. > + is 0. > > .. option:: --cache=none|auto|always > > -- > 2.31.1 > > > >
Re: [PATCH] docs: Correct the default thread-pool-size
[+cc dgilb...@redhat.com stefa...@redhat.com] On 4/14/22 1:05 PM, liuyd.f...@fujitsu.com wrote: > [+cc vgo...@redhat.com] > > On 4/13/22 12:20 PM, Liu Yiding wrote: >> Refer to 26ec190964 virtiofsd: Do not use a thread pool by default >> >> Signed-off-by: Liu Yiding >> --- >>docs/tools/virtiofsd.rst | 2 +- >>1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/docs/tools/virtiofsd.rst b/docs/tools/virtiofsd.rst >> index 0c0560203c..33fed08c6f 100644 >> --- a/docs/tools/virtiofsd.rst >> +++ b/docs/tools/virtiofsd.rst >> @@ -127,7 +127,7 @@ Options >>.. option:: --thread-pool-size=NUM >> >> Restrict the number of worker threads per request queue to NUM. The >> default >> - is 64. >> + is 0. >> >>.. option:: --cache=none|auto|always >> -- Best Regards. Yiding Liu
Re: [PATCH] docs: Correct the default thread-pool-size
[+cc vgo...@redhat.com] On 4/13/22 12:20 PM, Liu Yiding wrote: > Refer to 26ec190964 virtiofsd: Do not use a thread pool by default > > Signed-off-by: Liu Yiding > --- > docs/tools/virtiofsd.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/docs/tools/virtiofsd.rst b/docs/tools/virtiofsd.rst > index 0c0560203c..33fed08c6f 100644 > --- a/docs/tools/virtiofsd.rst > +++ b/docs/tools/virtiofsd.rst > @@ -127,7 +127,7 @@ Options > .. option:: --thread-pool-size=NUM > > Restrict the number of worker threads per request queue to NUM. The > default > - is 64. > + is 0. > > .. option:: --cache=none|auto|always > -- Best Regards. Yiding Liu