[v1] vl: flush all task from rcu queue before exiting

[PATCH v1 0/2] vl: flush all task from rcu queue before exiting

Denis Plotnikov posted 2 patches 4 years, 2 months ago

Diff against v0 v1 v1
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211115094143.157399-1-den-plotnikov@yandex-team.ru

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Markus Armbruster <armbru@redhat.com>

There is a newer version of this series

include/qemu/rcu.h |  1 +
monitor/monitor.c  |  6 ++++++
softmmu/runstate.c |  4 +++-
util/rcu.c         | 12 ++++++++++++
4 files changed, 22 insertions(+), 1 deletion(-)

Expand all Fold all

[PATCH v1 0/2] vl: flush all task from rcu queue before exiting

Posted by Denis Plotnikov 4 years, 2 months ago

v1 -> v0:
 * move monitor cleanup to the very end of qemu cleanup [Paolo]

The goal is to notify management layer about device destruction on qemu shutdown.
Without this series DEVICE_DELETED event may not be sent because of stuck tasks
in the rcu thread. The rcu tasks may stuck on qemu shutdown because the rcu
not always have enough time to run them. 


Denis Plotnikov (2):
  monitor: move monitor destruction to the very end of qemu cleanup
  vl: flush all task from rcu queue before exiting

 include/qemu/rcu.h |  1 +
 monitor/monitor.c  |  6 ++++++
 softmmu/runstate.c |  4 +++-
 util/rcu.c         | 12 ++++++++++++
 4 files changed, 22 insertions(+), 1 deletion(-)

-- 
2.25.1

[Ping] [PATCH v1 0/2] vl: flush all task from rcu queue before exiting

Posted by Denis Plotnikov 4 years, 2 months ago

Ping!

On 15.11.2021 12:41, Denis Plotnikov wrote:
> v1 -> v0:
>   * move monitor cleanup to the very end of qemu cleanup [Paolo]
>
> The goal is to notify management layer about device destruction on qemu shutdown.
> Without this series DEVICE_DELETED event may not be sent because of stuck tasks
> in the rcu thread. The rcu tasks may stuck on qemu shutdown because the rcu
> not always have enough time to run them.
>
>
> Denis Plotnikov (2):
>    monitor: move monitor destruction to the very end of qemu cleanup
>    vl: flush all task from rcu queue before exiting
>
>   include/qemu/rcu.h |  1 +
>   monitor/monitor.c  |  6 ++++++
>   softmmu/runstate.c |  4 +++-
>   util/rcu.c         | 12 ++++++++++++
>   4 files changed, 22 insertions(+), 1 deletion(-)
>

[PING][Ping] [PATCH v1 0/2] vl: flush all task from rcu queue before exiting

Posted by Denis Plotnikov 4 years, 2 months ago

ping ping

On 19.11.2021 12:42, Denis Plotnikov wrote:
>
> Ping!
>
> On 15.11.2021 12:41, Denis Plotnikov wrote:
>> v1 -> v0:
>>   * move monitor cleanup to the very end of qemu cleanup [Paolo]
>>
>> The goal is to notify management layer about device destruction on qemu shutdown.
>> Without this series DEVICE_DELETED event may not be sent because of stuck tasks
>> in the rcu thread. The rcu tasks may stuck on qemu shutdown because the rcu
>> not always have enough time to run them.
>>
>>
>> Denis Plotnikov (2):
>>    monitor: move monitor destruction to the very end of qemu cleanup
>>    vl: flush all task from rcu queue before exiting
>>
>>   include/qemu/rcu.h |  1 +
>>   monitor/monitor.c  |  6 ++++++
>>   softmmu/runstate.c |  4 +++-
>>   util/rcu.c         | 12 ++++++++++++
>>   4 files changed, 22 insertions(+), 1 deletion(-)
>>