RE: [PATCH 0/1 v2] Patch to adjust coroutine pool size adaptively

Hiroki Narukawa posted 1 patch 2 years, 2 months ago
Only 0 patches received!
RE: [PATCH 0/1 v2] Patch to adjust coroutine pool size adaptively
Posted by Hiroki Narukawa 2 years, 2 months ago
ping, how is the status of this patch?

Link for this patch v2 on patchew is this one: https://patchew.org/QEMU/20220111091950.840-1-hnarukaw@yahoo-corp.jp/
The last message on patch v1 is this one: https://lore.kernel.org/qemu-devel/TYCPR01MB8357E8D13D661265CDBB442C80519@TYCPR01MB8357.jpnprd01.prod.outlook.com/T/#u

The difference from v1 is use of qatomic_read().


> Resending patch with fixing atomic access method to use qatomic_read().
> 
> We encountered random disk IO performance drop since qemu-5.0.0, and this
> patch fixes it.
> 
> Commit message in c740ad92 implied to adjust coroutine pool size adaptively, so I
> tried to implement this.
> 
> Could you review this patch?
> 
> 
> Hiroki Narukawa (1):
>   util: adjust coroutine pool size to virtio block queue
> 
>  hw/block/virtio-blk.c    |  3 +++
>  include/qemu/coroutine.h |  5 +++++
>  util/qemu-coroutine.c    | 15 +++++++++++----
>  3 files changed, 19 insertions(+), 4 deletions(-)
> 
> --
> 2.17.1


Re: [PATCH 0/1 v2] Patch to adjust coroutine pool size adaptively
Posted by Stefan Hajnoczi 2 years, 2 months ago
On Mon, Jan 24, 2022 at 10:01:30AM +0000, Hiroki Narukawa wrote:
> ping, how is the status of this patch?
> 
> Link for this patch v2 on patchew is this one: https://patchew.org/QEMU/20220111091950.840-1-hnarukaw@yahoo-corp.jp/
> The last message on patch v1 is this one: https://lore.kernel.org/qemu-devel/TYCPR01MB8357E8D13D661265CDBB442C80519@TYCPR01MB8357.jpnprd01.prod.outlook.com/T/#u
> 
> The difference from v1 is use of qatomic_read().

Hi,
Sorry for the slow response. I will review this tomorrow (Thursday)!

Stefan