[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/4] qtest: fix "device_del" out-of-order events
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH 0/4] qtest: fix "device_del" out-of-order events |
Date: |
Wed, 13 Sep 2017 18:42:35 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Wed, Sep 13, 2017 at 12:35:17PM +0200, Thomas Huth wrote:
[...]
> >> As far as I know, some test currently use QMP in a bad way, for example
> >> usb_test_hotplug() only checks for the DEVICE_DELETED at the end, but
> >> forgets to read back the final return value. That return value is then
> >> presented to the next part of the code that uses QMP instead ... it
> >> currently only works more or less by accident, but as soon as you try to
> >> add new code inbetween, it certainly will fail.
> >> ==> We really got to clean this up (either with my patch or your patch
> >> series).
> >
> > Agree.
> >
> > I think your patch is nicer on the interface (as you have mentioned in
> > the other reply), I can try to review it later.
> >
> > However it seems that your patch didn't really solve the problem I
> > encountered (mis-ordered message arrivals). It would be good if you
> > want to solve it together, or I can draft patch upon yours.
>
> True, I'll try to respin my patch, including the ideas from your patch...
That would be great. I've left some comments there already. Thanks!
--
Peter Xu
- [Qemu-devel] [PATCH 0/4] qtest: fix "device_del" out-of-order events, Peter Xu, 2017/09/13
- [Qemu-devel] [PATCH 1/4] libqtest: add qmp_device_del(), Peter Xu, 2017/09/13
- [Qemu-devel] [PATCH 2/4] tests: use qmp_device_del() where proper, Peter Xu, 2017/09/13
- [Qemu-devel] [PATCH 3/4] libqtest: add qmp_device_add(), Peter Xu, 2017/09/13
- [Qemu-devel] [PATCH 4/4] tests: use qmp_device_add() where proper, Peter Xu, 2017/09/13
- Re: [Qemu-devel] [PATCH 0/4] qtest: fix "device_del" out-of-order events, Thomas Huth, 2017/09/13