mbox series

[0/2] monitor: Shutdown fixes

Message ID 20210212172028.288825-1-kwolf@redhat.com (mailing list archive)
Headers show
Series monitor: Shutdown fixes | expand

Message

Kevin Wolf Feb. 12, 2021, 5:20 p.m. UTC 
This fixes the bug(s) that Markus reported here:
https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg07719.html

Kevin Wolf (2):
  monitor: Fix assertion failure on shutdown
  monitor/qmp: Stop processing requests when shutdown is requested

 monitor/monitor.c | 25 +++++++++++++++----------
 monitor/qmp.c     |  5 +++++
 2 files changed, 20 insertions(+), 10 deletions(-)

Comments

Markus Armbruster Feb. 15, 2021, 12:16 p.m. UTC | #1 
Kevin Wolf <kwolf@redhat.com> writes:

> This fixes the bug(s) that Markus reported here:
> https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg07719.html

Tested-by: Markus Armbruster <armbru@redhat.com>
Markus Armbruster Feb. 15, 2021, 12:44 p.m. UTC | #2 
Kevin Wolf <kwolf@redhat.com> writes:

> This fixes the bug(s) that Markus reported here:
> https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg07719.html

Mostly thanks to the helpful commit messages and code comments:
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Vladimir Sementsov-Ogievskiy June 29, 2021, 9:14 a.m. UTC | #3 
12.02.2021 20:20, Kevin Wolf wrote:
> This fixes the bug(s) that Markus reported here:
> https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg07719.html
> 
> Kevin Wolf (2):
>    monitor: Fix assertion failure on shutdown
>    monitor/qmp: Stop processing requests when shutdown is requested
> 
>   monitor/monitor.c | 25 +++++++++++++++----------
>   monitor/qmp.c     |  5 +++++
>   2 files changed, 20 insertions(+), 10 deletions(-)
> 


Hi!

Now we faced this bug after rebasing Virtuozzo qemu package onto qemu-kvm-5.2.0-16.module+el8.4.0+10806+b7d97207.src.rpm

So, I'm backporting these patches.

This probably should go to stable and/or to further Rhel packages.
Markus Armbruster July 6, 2021, 3:32 p.m. UTC | #4 
Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> writes:

> 12.02.2021 20:20, Kevin Wolf wrote:
>> This fixes the bug(s) that Markus reported here:
>> https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg07719.html
>> Kevin Wolf (2):
>>    monitor: Fix assertion failure on shutdown
>>    monitor/qmp: Stop processing requests when shutdown is requested
>>   monitor/monitor.c | 25 +++++++++++++++----------
>>   monitor/qmp.c     |  5 +++++
>>   2 files changed, 20 insertions(+), 10 deletions(-)
>> 
>
>
> Hi!
>
> Now we faced this bug after rebasing Virtuozzo qemu package onto qemu-kvm-5.2.0-16.module+el8.4.0+10806+b7d97207.src.rpm
>
> So, I'm backporting these patches.
>
> This probably should go to stable and/or to further Rhel packages.

Cc'ing qemu-stable, so it gets considered.