[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PULL 7/7] vl: introduce vm_shutdown()
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PULL 7/7] vl: introduce vm_shutdown() |
Date: |
Fri, 16 Mar 2018 15:52:00 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 03/12/2018 08:05 PM, John Snow wrote:
>
>
> On 03/09/2018 08:19 AM, Stefan Hajnoczi wrote:
>> Commit 00d09fdbbae5f7864ce754913efc84c12fdf9f1a ("vl: pause vcpus before
>> stopping iothreads") and commit dce8921b2baaf95974af8176406881872067adfa
>> ("iothread: Stop threads before main() quits") tried to work around the
>> fact that emulation was still active during termination by stopping
>> iothreads. They suffer from race conditions:
>> 1. virtio_scsi_handle_cmd_vq() racing with iothread_stop_all() hits the
>> virtio_scsi_ctx_check() assertion failure because the BDS AioContext
>> has been modified by iothread_stop_all().
>> 2. Guest vq kick racing with main loop termination leaves a readable
>> ioeventfd that is handled by the next aio_poll() when external
>> clients are enabled again, resulting in unwanted emulation activity.
>>
>> This patch obsoletes those commits by fully disabling emulation activity
>> when vcpus are stopped.
>>
>> Use the new vm_shutdown() function instead of pause_all_vcpus() so that
>> vm change state handlers are invoked too. Virtio devices will now stop
>> their ioeventfds, preventing further emulation activity after vm_stop().
>>
>> Note that vm_stop(RUN_STATE_SHUTDOWN) cannot be used because it emits a
>> QMP STOP event that may affect existing clients.
>>
>> It is no longer necessary to call replay_disable_events() directly since
>> vm_shutdown() does so already.
>>
>> Drop iothread_stop_all() since it is no longer used.
>>
>> Cc: Fam Zheng <address@hidden>
>> Cc: Kevin Wolf <address@hidden>
>> Signed-off-by: Stefan Hajnoczi <address@hidden>
>> Reviewed-by: Fam Zheng <address@hidden>
>> Acked-by: Paolo Bonzini <address@hidden>
>> Message-id: address@hidden
>> Signed-off-by: Stefan Hajnoczi <address@hidden>
Stefan, I see the same iotest regression that was reported by John Snow also
on s390. I can confirm that it works with this patch reverted.
Christian
- [Qemu-block] [PULL 0/7] Block patches, Stefan Hajnoczi, 2018/03/09
- [Qemu-block] [PULL 1/7] block: Fix qemu crash when using scsi-block, Stefan Hajnoczi, 2018/03/09
- [Qemu-block] [PULL 2/7] README: Fix typo 'git-publish', Stefan Hajnoczi, 2018/03/09
- [Qemu-block] [PULL 4/7] block: add aio_wait_bh_oneshot(), Stefan Hajnoczi, 2018/03/09
- [Qemu-block] [PULL 3/7] virtio-blk: dataplane: Don't batch notifications if EVENT_IDX is present, Stefan Hajnoczi, 2018/03/09
- [Qemu-block] [PULL 6/7] virtio-scsi: fix race between .ioeventfd_stop() and vq handler, Stefan Hajnoczi, 2018/03/09
- [Qemu-block] [PULL 7/7] vl: introduce vm_shutdown(), Stefan Hajnoczi, 2018/03/09
- [Qemu-block] [PULL 5/7] virtio-blk: fix race between .ioeventfd_stop() and vq handler, Stefan Hajnoczi, 2018/03/09
- Re: [Qemu-block] [PULL 0/7] Block patches, Peter Maydell, 2018/03/09