[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 11/11] hw/misc/vmcoreinfo: Convert reset handler to DeviceRes
From: |
Peter Maydell |
Subject: |
Re: [PATCH 11/11] hw/misc/vmcoreinfo: Convert reset handler to DeviceReset |
Date: |
Wed, 9 Oct 2019 14:51:30 +0100 |
On Thu, 26 Sep 2019 at 17:02, Philippe Mathieu-Daudé <address@hidden> wrote:
>
> On 9/26/19 5:17 PM, Philippe Mathieu-Daudé wrote:
> > Convert the reset handler into a proper Device reset method.
>
> Marc-André noticed this one is incorrect, because while being QDEV it is
> not connected to a QBUS.
>
> Maybe we can add a Device::unconnected property, and when set, the
> parent realize() calls 'qemu_register_reset(dev->reset, dev);'?
> This might look the same, but at least Devices implementations could
> stop to use this function...
I'm not in favour of ad-hoc attempts to patch the
problem with some devices not being reset like this.
I'd rather we figured out a general solution to the design
problem (which isn't easy, but on the other hand the
set of workarounds we currently have isn't too awful
to deal with).
thanks
-- PMM