[v2] Patch to adjust coroutine pool size adaptively

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

Hiroki Narukawa posted 1 patch 4 years ago

Diff against v1 v3 v4
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220111091950.840-1-hnarukaw@yahoo-corp.jp

Maintainers: Kevin Wolf <kwolf@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>, Hanna Reitz <hreitz@redhat.com>

There is a newer version of this series

hw/block/virtio-blk.c    |  3 +++
include/qemu/coroutine.h |  5 +++++
util/qemu-coroutine.c    | 15 +++++++++++----
3 files changed, 19 insertions(+), 4 deletions(-)

Expand all Fold all

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

Posted by Hiroki Narukawa 4 years ago

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 Hiroki Narukawa 4 years 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 4 years 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