[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race |
Date: |
Mon, 14 Jan 2019 16:15:25 +0000 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Mon, Jan 14, 2019 at 03:40:39PM +0100, Alberto Garcia wrote:
> On Mon 14 Jan 2019 02:35:53 PM CET, Stefan Hajnoczi wrote:
> > On Fri, Jan 11, 2019 at 03:14:08PM +0100, Alberto Garcia wrote:
> >> On Fri 11 Jan 2019 02:24:16 PM CET, Kevin Wolf wrote:
> >> >> >> The following QMP command leads to a crash when iothreads are used:
> >> >> >>
> >> >> >> { 'execute': 'device_del', 'arguments': {'id': 'data'} }
> >> >> >
> >> >> > How did you reproduce this? Do you have a test case?
> >> >>
> >> >> Ok, I finally reproduced it, the patch looks good to me.
> >> >
> >> > Can it be turned into a qemu-iotests case or is it too complicated for
> >> > that?
> >>
> >> I can reproduce the problem reliably with this:
> >>
> >> { "execute": "qmp_capabilities" }
> >> { "execute": "blockdev-add",
> >> "arguments": {"driver": "null-co", "node-name": "hd0"}}
> >> { "execute": "object-add",
> >> "arguments": {"qom-type": "iothread", "id": "iothread0"}}
> >> { "execute": "device_add",
> >> "arguments": {"id": "scsi0", "driver": "virtio-scsi-pci",
> >> "iothread": "iothread0"}}
> >> { "execute": "device_add",
> >> "arguments": {"id": "scsi-hd0", "driver": "scsi-hd", "drive": "hd0"}}
> >> { "execute": "block_set_io_throttle",
> >> "arguments": {"id": "scsi-hd0", "bps": 0, "bps_rd": 0, "bps_wr": 0,
> >> "iops": 1000, "iops_rd": 0, "iops_wr": 0}}
> >> { "execute": "device_del", "arguments": {"id": "scsi-hd0"}}
> >>
> >> But this doesn't crash if I put it in an iotest.
> >
> > I've been able to reproduce this in an iotest, please see v2 of this
> > series.
>
> That iotest doesn't crash for me :-?
Does my iotest pass for you?
On my machine the diff fails because signal 11 killed QEMU (and this
prints a warning to the test output).
Stefan
signature.asc
Description: PGP signature
- [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Stefan Hajnoczi, 2019/01/09
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Alberto Garcia, 2019/01/09
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Alberto Garcia, 2019/01/09
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Alberto Garcia, 2019/01/11
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Kevin Wolf, 2019/01/11
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Alberto Garcia, 2019/01/11
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Stefan Hajnoczi, 2019/01/14
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Alberto Garcia, 2019/01/14
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Alberto Garcia, 2019/01/14
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Stefan Hajnoczi, 2019/01/14
- Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Alberto Garcia, 2019/01/14
- Re: [Qemu-devel] [Qemu-block] [PATCH] throttle-groups: fix restart coroutine iothread race, Stefan Hajnoczi, 2019/01/15
- Re: [Qemu-devel] [Qemu-block] [PATCH] throttle-groups: fix restart coroutine iothread race, Alberto Garcia, 2019/01/15
Re: [Qemu-devel] [PATCH] throttle-groups: fix restart coroutine iothread race, Paolo Bonzini, 2019/01/11