qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] vl: introduce vm_shutdown()


From: Fam Zheng
Subject: Re: [Qemu-devel] [PATCH] vl: introduce vm_shutdown()
Date: Fri, 23 Feb 2018 16:20:44 +0800
User-agent: Mutt/1.9.2 (2017-12-15)

On Tue, 02/20 13:10, Stefan Hajnoczi wrote:
> 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().

Does this patch fix the issue completely? IIUC virtio_scsi_handle_cmd can
already be entered at the time of main thread calling virtio_scsi_clear_aio(),
so this race condition still exists:

  main thread                           iothread
-----------------------------------------------------------------------------
  vm_shutdown
    ...
      virtio_bus_stop_ioeventfd
        virtio_scsi_dataplane_stop
                                        aio_poll()
                                          ...
                                            virtio_scsi_data_plane_handle_cmd()
          aio_context_acquire(s->ctx)
                                              virtio_scsi_acquire(s).enter
          virtio_scsi_clear_aio()
          aio_context_release(s->ctx)
                                              virtio_scsi_acquire(s).return
                                              virtio_scsi_handle_cmd_vq()
                                                ...
                                                  virtqueue_pop()

Is it possible that the above virtqueue_pop() still returns one element that was
queued before vm_shutdown() was called?

If so I think we additionally need to an "s->ioeventfd_stopped" flag set in
virtio_scsi_stop_ioeventfd() and check it in
virtio_scsi_data_plane_handle_cmd().

Fam



reply via email to

[Prev in Thread] Current Thread [Next in Thread]