[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/39] msix: split msix_free from msix_uninit
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 11/39] msix: split msix_free from msix_uninit |
Date: |
Wed, 05 Jun 2013 00:40:00 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 |
Il 05/06/2013 00:03, Michael S. Tsirkin ha scritto:
>> > + if (dev->msix_table || dev->msix_pba || dev->msix_entry_used) {
>> > + msix_free(dev);
>> > + }
>> > +
>> > dev->msix_table = g_malloc0(table_size);
>> > dev->msix_pba = g_malloc0(pba_size);
>> > dev->msix_entry_used = g_malloc0(nentries * sizeof
>> > *dev->msix_entry_used);
> Wow msix_init calls msix_free, and not on error path?
> What's going on here?
I wasn't too sure that you could get here only with NULL
msix_table/pba/entry_used and wanted to protect against leaks. I'll
change it to an assertion.
>> > @@ -359,16 +363,26 @@ void msix_uninit(PCIDevice *dev, MemoryRegion
>> > *table_bar, MemoryRegion *pba_bar)
>> > msix_free_irq_entries(dev);
>> > dev->msix_entries_nr = 0;
>> > memory_region_del_subregion(pba_bar, &dev->msix_pba_mmio);
>> > - memory_region_destroy(&dev->msix_pba_mmio);
>> > - g_free(dev->msix_pba);
>> > - dev->msix_pba = NULL;
>> > memory_region_del_subregion(table_bar, &dev->msix_table_mmio);
>> > - memory_region_destroy(&dev->msix_table_mmio);
>> > - g_free(dev->msix_table);
>> > + dev->cap_present &= ~QEMU_PCI_CAP_MSIX;
>> > +}
>> > +
>> > +void msix_free(PCIDevice *dev)
>> > +{
>> > + if (dev->msix_pba) {
>> > + memory_region_destroy(&dev->msix_pba_mmio);
>> > + g_free(dev->msix_pba);
>> > + }
>> > + dev->msix_pba = NULL;
>> > +
>> > + if (dev->msix_table) {
>> > + memory_region_destroy(&dev->msix_table_mmio);
>> > + g_free(dev->msix_table);
>> > + }
>> > dev->msix_table = NULL;
>> > +
>> > g_free(dev->msix_entry_used);
>> > dev->msix_entry_used = NULL;
>> > - dev->cap_present &= ~QEMU_PCI_CAP_MSIX;
>> > }
>> >
>> > void msix_uninit_exclusive_bar(PCIDevice *dev)
> As long as we had init and uninit, it was mostly
> self-documenting.
> Now, there are two cleanup functions, so please add documentation.
Yes, will do.
Paolo
- Re: [Qemu-devel] [PATCH 03/39] pci: split exit and finalize, (continued)
- [Qemu-devel] [PATCH 04/39] ac97: use instance_finalize instead of exit, Paolo Bonzini, 2013/06/04
- [Qemu-devel] [PATCH 05/39] es1370: use instance_finalize instead of exit, Paolo Bonzini, 2013/06/04
- [Qemu-devel] [PATCH 06/39] hda: split exit and instance_finalize, Paolo Bonzini, 2013/06/04
- [Qemu-devel] [PATCH 07/39] serial: split exit and instance_finalize, Paolo Bonzini, 2013/06/04
- [Qemu-devel] [PATCH 08/39] tpci200: use instance_finalize instead of exit, Paolo Bonzini, 2013/06/04
- [Qemu-devel] [PATCH 09/39] pci-assign: split exit and instance_finalize, Paolo Bonzini, 2013/06/04
- [Qemu-devel] [PATCH 10/39] ahci: split exit and instance_finalize, Paolo Bonzini, 2013/06/04
- [Qemu-devel] [PATCH 11/39] msix: split msix_free from msix_uninit, Paolo Bonzini, 2013/06/04
[Qemu-devel] [PATCH 12/39] cmd646: use instance_finalize instead of exit, Paolo Bonzini, 2013/06/04
[Qemu-devel] [PATCH 13/39] ide/piix: use instance_finalize instead of exit, Paolo Bonzini, 2013/06/04
[Qemu-devel] [PATCH 14/39] ide/via: use instance_finalize instead of exit, Paolo Bonzini, 2013/06/04
[Qemu-devel] [PATCH 15/39] ivshmem: split exit and instance_finalize, Paolo Bonzini, 2013/06/04
[Qemu-devel] [PATCH 16/39] pci-testdev: use instance_finalize instead of exit, Paolo Bonzini, 2013/06/04
[Qemu-devel] [PATCH 17/39] vfio: split exit and instance_finalize, Paolo Bonzini, 2013/06/04