[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Fix for qemu crash on assertion error when addi
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH] Fix for qemu crash on assertion error when adding PCI passthru device. |
Date: |
Tue, 12 Jun 2012 11:26:41 +0300 |
On Tue, Jun 12, 2012 at 04:31:20AM +0000, Ma, Stephen B. wrote:
> diff --git a/hw/qdev.c b/hw/qdev.c
> index 6a8f6bd..d2dc28b 100644
> --- a/hw/qdev.c
> +++ b/hw/qdev.c
> @@ -139,7 +139,7 @@ DeviceState *qdev_try_create(BusState *bus, const char
> *type)
> /* Initialize a device. Device properties should be set before calling
> this function. IRQs and MMIO regions should be connected/mapped after
> calling this function.
> - On failure, destroy the device and return negative value.
> + On failure, return a negative value.
> Return 0 on success. */
> int qdev_init(DeviceState *dev)
> {
Yes, I agree. qdev_init did now allocate the device so
it should not free it.
> @@ -150,7 +150,6 @@ int qdev_init(DeviceState *dev)
>
> rc = dc->init(dev);
> if (rc < 0) {
> - qdev_free(dev);
> return rc;
> }
>
Another thing we need to fix is unparent the device in
qdev_free.
--
MST