[libvirt] [PATCH] gic: Introduce VIR_GIC_VERSION_DEFAULT alias
Andrea Bolognani
abologna at redhat.com
Fri Feb 19 17:33:12 UTC 2016
On Fri, 2016-02-19 at 11:05 -0500, John Ferlan wrote:
>
> On 02/18/2016 10:28 AM, Andrea Bolognani wrote:
> > GIC v2 is the default, but checking against that specific version when
> > we want to know whether the default has been selected is potentially
> > error prone; using an alias instead makes it safer.
> > ---
> > src/qemu/qemu_command.c | 7 +++----
> > src/qemu/qemu_domain.c | 4 ++--
> > src/util/virgic.h | 3 +++
> > 3 files changed, 8 insertions(+), 6 deletions(-)
> >
>
> This works, but so does I think changing VIR_GIC_VERSION_2 in virgic.h
> to VIR_GIC_VERSION_DEFAULT
>
> ACK either way, but I think changing _2 to _DEFAULT would be better...
That would hide the fact that v2 is the default version, though.
I've pushed it as is, thanks for the review.
Cheers.
--
Andrea Bolognani
Software Engineer - Virtualization Team
More information about the libvir-list
mailing list