[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject.
From: |
Anthony PERARD |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject. |
Date: |
Wed, 16 May 2012 12:15:45 +0100 |
On Wed, May 16, 2012 at 11:32 AM, Stefano Stabellini
<address@hidden> wrote:
> On Tue, 15 May 2012, Michael S. Tsirkin wrote:
>> On Tue, May 15, 2012 at 04:26:36PM +0100, Anthony PERARD wrote:
>> > This hotplug state will be used to remove a device without the guest
>> > cooperation.
>> >
>> > Signed-off-by: Anthony PERARD <address@hidden>
>>
>> This can crash guest, can't it? If you are fine with crashing guest,
>> we already let you do this:
>> - delete device
>> - reset guest
>> no need for new flags.
>
> Given that the guest is not going to crash (if it knows what it is
> doing), we could just:
>
>
> diff --git a/hw/xen_platform.c b/hw/xen_platform.c
> index a9c52a6..a1e1a33 100644
> --- a/hw/xen_platform.c
> +++ b/hw/xen_platform.c
> @@ -88,6 +88,7 @@ static void unplug_nic(PCIBus *b, PCIDevice *d)
> if (pci_get_word(d->config + PCI_CLASS_DEVICE) ==
> PCI_CLASS_NETWORK_ETHERNET) {
> qdev_unplug(&(d->qdev), NULL);
> + qdev_free(&(d->qdev));
> }
> }
>
>
> Anthony, can you confirm that this solves the problem for you?
This work until I try to hotplug a new device to the guest at wish
point I have this:
ERROR:/local/home/anthony/work/qemu/qom/object.c:389:object_delete:
assertion failed: (obj->ref == 0)
This is because there is still a pending request of the hotunplug in
the acpi piix4.
If I call qdev_free without qdev_unplug, I hit the same assert, but
rigth away. This is way something new.
--
Anthony PERARD
- Re: [Qemu-devel] [PATCH 4/4] xen: Fix PV-on-HVM, (continued)
- Re: [Qemu-devel] [PATCH 4/4] xen: Fix PV-on-HVM, Paolo Bonzini, 2012/05/16
- Re: [Qemu-devel] [PATCH 4/4] xen: Fix PV-on-HVM, Stefano Stabellini, 2012/05/16
- Re: [Qemu-devel] [PATCH 4/4] xen: Fix PV-on-HVM, Michael S. Tsirkin, 2012/05/16
- Re: [Qemu-devel] [PATCH 4/4] xen: Fix PV-on-HVM, Stefano Stabellini, 2012/05/16
- Re: [Qemu-devel] [PATCH 4/4] xen: Fix PV-on-HVM, Michael S. Tsirkin, 2012/05/16
[Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Anthony PERARD, 2012/05/15
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Stefano Stabellini, 2012/05/15
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Michael S. Tsirkin, 2012/05/15
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Stefano Stabellini, 2012/05/16
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject.,
Anthony PERARD <=
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Paolo Bonzini, 2012/05/16
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Anthony PERARD, 2012/05/16
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Michael S. Tsirkin, 2012/05/16
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Paolo Bonzini, 2012/05/16
- Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Michael S. Tsirkin, 2012/05/16
Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Anthony Liguori, 2012/05/16
Re: [Qemu-devel] [PATCH 1/4] Introduce a new hotplug state: Force eject., Michael S. Tsirkin, 2012/05/16
[Qemu-devel] [PATCH 3/4] pci: Add force_unplug callback., Anthony PERARD, 2012/05/15
Re: [Qemu-devel] [PATCH 1.1 0/4] Xen: Fix PV-on-HVM, Anthony Liguori, 2012/05/15