[libvirt] [PATCH] Fix typo s/EMULATORIN/EMULATORPIN/

Michal Privoznik mprivozn at redhat.com
Thu Oct 2 13:23:59 UTC 2014


On 26.09.2014 12:22, Daniel P. Berrange wrote:
> Fix the typo in VIR_DOMAIN_TUNABLE_CPU_EMULATORIN
>
> Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
> ---
>   include/libvirt/libvirt.h.in | 4 ++--
>   src/qemu/qemu_driver.c       | 2 +-
>   2 files changed, 3 insertions(+), 3 deletions(-)
>
> Pushed under trivial rule
>
> diff --git a/include/libvirt/libvirt.h.in b/include/libvirt/libvirt.h.in
> index 2ff5204..5217ab3 100644
> --- a/include/libvirt/libvirt.h.in
> +++ b/include/libvirt/libvirt.h.in
> @@ -5213,12 +5213,12 @@ typedef void (*virConnectDomainEventDeviceRemovedCallback)(virConnectPtr conn,
>   #define VIR_DOMAIN_TUNABLE_CPU_VCPUPIN "cputune.vcpupin%u"
>
>   /**
> - * VIR_DOMAIN_TUNABLE_CPU_EMULATORIN:
> + * VIR_DOMAIN_TUNABLE_CPU_EMULATORPIN:
>    *
>    * Macro represents formatted pinning for emulator process,
>    * as VIR_TYPED_PARAM_STRING.
>    */
> -#define VIR_DOMAIN_TUNABLE_CPU_EMULATORIN "cputune.emulatorpin"
> +#define VIR_DOMAIN_TUNABLE_CPU_EMULATORPIN "cputune.emulatorpin"
>
>   /**
>    * VIR_DOMAIN_TUNABLE_CPU_CPU_SHARES:
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 54119f5..6606154 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -4940,7 +4940,7 @@ qemuDomainPinEmulator(virDomainPtr dom,
>           str = virBitmapFormat(pcpumap);
>           if (virTypedParamsAddString(&eventParams, &eventNparams,
>                                       &eventMaxparams,
> -                                    VIR_DOMAIN_TUNABLE_CPU_EMULATORIN,
> +                                    VIR_DOMAIN_TUNABLE_CPU_EMULATORPIN,
>                                       str) < 0)
>               goto cleanup;
>
>

This makes already existing application fail to compile with new libvirt 
headers. And I think libvirt header files are part of our API. Hence I'm 
not that inclined to ACKing this one, sorry (I know you've already 
pushed it anyway).

Michal




More information about the libvir-list mailing list