[libvirt] [PATCH 1/1] qemu: build failed due to unused variables

Osier Yang jyang at redhat.com
Wed Jul 13 12:44:53 UTC 2011


于 2011年07月13日 18:11, Peter Krempa 写道:
> While compiling on F15 build crashed (probably because of new GCC).
> ---
>   AUTHORS                |    1 +
>   src/qemu/qemu_driver.c |    7 ++++---
>   2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/AUTHORS b/AUTHORS
> index 4b16a2c..636fa63 100644
> --- a/AUTHORS
> +++ b/AUTHORS
> @@ -185,6 +185,7 @@ Patches have also been contributed by:
>     Michael Santos<michael.santos at gmail.com>
>     Alex Jia<ajia at redhat.com>
>     Oskari Saarenmaa<os at ohmu.fi>
> +  Peter Krempa<pkrempa at redhat.com>
>
>     [....send patches to get your name here....]
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 0a6b48e..5c9c7bc 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -1498,7 +1498,9 @@ static int qemuDomainReboot(virDomainPtr dom, unsigned int flags) {
>       struct qemud_driver *driver = dom->conn->privateData;
>       virDomainObjPtr vm;
>       int ret = -1;
> +#if HAVE_YAJL
>       qemuDomainObjPrivatePtr priv;
> +#endif
>
>       virCheckFlags(0, -1);
>
> @@ -1513,9 +1515,10 @@ static int qemuDomainReboot(virDomainPtr dom, unsigned int flags) {
>                           _("no domain with matching uuid '%s'"), uuidstr);
>           goto cleanup;
>       }
> -    priv = vm->privateData;
>
>   #if HAVE_YAJL
> +    priv = vm->privateData;
> +
>       if (qemuCapsGet(priv->qemuCaps, QEMU_CAPS_MONITOR_JSON)) {
>           if (qemuDomainObjBeginJob(driver, vm, QEMU_JOB_MODIFY)<  0)
>               goto cleanup;
> @@ -2598,7 +2601,6 @@ static int qemudDomainCoreDump(virDomainPtr dom,
>       int resume = 0, paused = 0;
>       int ret = -1;
>       virDomainEventPtr event = NULL;
> -    qemuDomainObjPrivatePtr priv;
>
>       virCheckFlags(VIR_DUMP_LIVE | VIR_DUMP_CRASH, -1);
>
> @@ -2612,7 +2614,6 @@ static int qemudDomainCoreDump(virDomainPtr dom,
>                           _("no domain with matching uuid '%s'"), uuidstr);
>           goto cleanup;
>       }
> -    priv = vm->privateData;
>
>       if (qemuDomainObjBeginAsyncJobWithDriver(driver, vm,
>                                                QEMU_ASYNC_JOB_DUMP)<  0)

I meet the same problem on FC15, ACK

Osier




More information about the libvir-list mailing list