[libvirt] [PATCH] qemu: Enable 'host-passthrough' cpu mode for aarch64

Eric Blake eblake at redhat.com
Fri Jan 24 17:57:41 UTC 2014


On 01/23/2014 08:19 AM, Oleg Strikov wrote:
> This patch allows libvirt user to specify 'host-passthrough'
> cpu mode while using qemu/kvm backend on aarch64.
> It uses 'host' as a CPU model name instead of some other stub
> (correct CPU detection is not implemented yet) to allow libvirt
> user to specify 'host-model' cpu mode as well.
> 
> Signed-off-by: Oleg Strikov <oleg.strikov at canonical.com>
> ---
>  src/cpu/cpu_aarch64.c |   30 ++++++++++++++++++++++++++----
>  1 file changed, 26 insertions(+), 4 deletions(-)

I'm not sure about the technical aspect of the patch, but for the style
aspect:

> 
>  static int
> -AArch64Decode(virCPUDefPtr cpu ATTRIBUTE_UNUSED,
> +AArch64Decode(virCPUDefPtr cpu,
>            const virCPUData *data ATTRIBUTE_UNUSED,
>            const char **models ATTRIBUTE_UNUSED,
>            unsigned int nmodels ATTRIBUTE_UNUSED,
>            const char *preferred ATTRIBUTE_UNUSED,
>            unsigned int flags)

As long as you are touching this, can you fix up the botched
indentation?  It should look like:

AArch64Decode(virCPUDefPtr cpu,
              const virCPUData *data ATTRIBUTE_UNUSED,

with secondary lines under the first byte after (.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140124/65540021/attachment-0001.sig>


More information about the libvir-list mailing list