[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/14] qtest/ahci: remove pcibus global
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 09/14] qtest/ahci: remove pcibus global |
Date: |
Mon, 19 Jan 2015 18:05:15 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 13/01/2015 04:34, John Snow wrote:
> Rely on the PCI Device's bus pointer instead.
> One less global to worry about.
>
> Signed-off-by: John Snow <address@hidden>
> ---
> tests/ahci-test.c | 14 ++++++--------
> 1 file changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/tests/ahci-test.c b/tests/ahci-test.c
> index c41c7d9..1981b6a 100644
> --- a/tests/ahci-test.c
> +++ b/tests/ahci-test.c
> @@ -45,7 +45,6 @@
>
> /*** Globals ***/
> static QGuestAllocator *guest_malloc;
> -static QPCIBus *pcibus;
> static char tmp_path[] = "/tmp/qtest.XXXXXX";
> static bool ahci_pedantic;
>
> @@ -100,6 +99,7 @@ static QPCIDevice *get_ahci_device(uint32_t *fingerprint)
> {
> QPCIDevice *ahci;
> uint32_t ahci_fingerprint;
> + QPCIBus *pcibus;
>
> pcibus = qpci_init_pc();
>
> @@ -123,15 +123,13 @@ static QPCIDevice *get_ahci_device(uint32_t
> *fingerprint)
> return ahci;
> }
>
> -static void free_ahci_device(QPCIDevice *ahci)
> +static void free_ahci_device(QPCIDevice *dev)
> {
> + QPCIBus *pcibus = dev ? dev->bus : NULL;
> +
> /* libqos doesn't have a function for this, so free it manually */
> - g_free(ahci);
> -
> - if (pcibus) {
> - qpci_free_pc(pcibus);
> - pcibus = NULL;
> - }
> + g_free(dev);
> + qpci_free_pc(pcibus);
> }
>
> /*** Test Setup & Teardown ***/
>
Reviewed-by: Paolo Bonzini <address@hidden>
- [Qemu-devel] [PATCH 11/14] libqos/ahci: Functional register helpers, (continued)
- [Qemu-devel] [PATCH 11/14] libqos/ahci: Functional register helpers, John Snow, 2015/01/12
- [Qemu-devel] [PATCH 08/14] qtest/ahci: finalize AHCIQState consolidation, John Snow, 2015/01/12
- [Qemu-devel] [PATCH 12/14] qtest/ahci: remove getter/setter macros, John Snow, 2015/01/12
- [Qemu-devel] [PATCH 13/14] qtest/ahci: Bookmark FB and CLB pointers, John Snow, 2015/01/12
- [Qemu-devel] [PATCH 09/14] qtest/ahci: remove pcibus global, John Snow, 2015/01/12
- Re: [Qemu-devel] [PATCH 09/14] qtest/ahci: remove pcibus global,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 14/14] libqos/ahci: create libqos/ahci.c, John Snow, 2015/01/12