[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 15/15] e1000: using new interface--unmap to unpl
From: |
liu ping fan |
Subject: |
Re: [Qemu-devel] [PATCH 15/15] e1000: using new interface--unmap to unplug |
Date: |
Thu, 9 Aug 2012 15:28:19 +0800 |
On Wed, Aug 8, 2012 at 5:56 PM, Paolo Bonzini <address@hidden> wrote:
> Il 08/08/2012 08:25, Liu Ping Fan ha scritto:
>>
>> +static void
>> +pci_e1000_unmap(PCIDevice *p)
>> +{
>> + /* DO NOT FREE anything!until refcnt=0 */
>> + /* isolate from memory view */
>> +}
>
> At least you need to call the superclass method.
>
Refer to 0013-hotplug-introduce-qdev_unplug_complete-to-remove-dev.patch,
we have the following sequence
qdev_unmap->pci_unmap_device->pci_e1000_unmap. So pci_e1000_unmap
need not to do anything.
Regards,
pingfan
> Paolo
>
>> static int
>> pci_e1000_uninit(PCIDevice *dev)
>> {
>> @@ -1275,6 +1282,7 @@ static void e1000_class_init(ObjectClass *klass, void
>> *data)
>> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
>>
>> k->init = pci_e1000_init;
>> + k->unmap = pci_e1000_unmap;
>> k->exit = pci_e1000_uninit;
>> k->romfile = "pxe-e1000.rom";
>> k->vendor_id = PCI_VENDOR_ID_INTEL;
>
>
- Re: [Qemu-devel] [PATCH 09/15] memory: prepare flatview and radix-tree for rcu style access, (continued)
[Qemu-devel] [PATCH 15/15] e1000: using new interface--unmap to unplug, Liu Ping Fan, 2012/08/08
[Qemu-devel] [PATCH 11/15] lock: introduce global lock for device tree, Liu Ping Fan, 2012/08/08
Re: [Qemu-devel] [PATCH 11/15] lock: introduce global lock for device tree, Avi Kivity, 2012/08/08
[Qemu-devel] [PATCH 12/15] qdev: using devtree lock to protect device's accessing, Liu Ping Fan, 2012/08/08