[PATCH] Modify virCPUarmCompare to perform compare actions

[Zhenyu Zheng]

Hi,

Thanks alot for the review and feedback. As for host-passthrough cases, I
have some other understandings,
if I understand correctly, what you mean is that if a vm uses
host-passthrough, it can migrate to any other
host, since it asks for host-passthrough. In my point of view, I think in
real cases, there are few different kinds
of ARM datacenter CPUs on the market, and there CPU capabilities are
different, so one might create a vm on
hostA with feature1 and feature2 because it uses host-passthrough and hostA
has these features. Now in your
definition(if I understand correctly) of host-passthrough and the current
code(returns identical directly), this vm
can be migrated to hostB with only feature1, since there are no
limitations. If one has some important application
that depends on feature2, the app will break as feature2 is not available
on hostB. Considering this, I proposed
to add basic checks to compare CPU to limit the migration to only the same
CPU models. And once the capability
of ARM driver is enhanced in QEMU or other related projects, we can make
the compare API better.

And yes, the code referenced X86 and S390 driver, I have modified them to
be workable with ARM and tested
the functions, I was also thinking that in the future there might be
possibility that we can compare cpu features
so I kept the data compare case.

Thanks again for the feedback.

Zhenyu



On Tue, Sep 1, 2020 at 10:50 PM Jiri Denemark <jdenemar at redhat.com> wrote:

> On Fri, Aug 21, 2020 at 10:20:15 +0800, Zhenyu Zheng wrote:
> > Modify virCPUarmCompare in cpu_arm.c to perform
> > actual compare actions. Compare host cpu vendor
> > and model info with guest cpu as initial implementation,
> > as most ARM clouds uses host-passthrogh mode.
>
> In addition to the low-level coding issues found by Daniel Henrique
> Barboza, I'd like to ask some high level questions...
>
> What is the point in making this patch (except for copying the logic
> from x86 CPU driver, which mostly does not fit ARM world very well)?
>
> As you say, most ARM clouds use host-passthrough, so why would CPU
> comparison be needed at all? Host-passthrough CPU is by definition
> compatible with any host CPU as it asks for the host CPU itself.
>
> Also IIRC the CPU model names as advertised by libvirt in host
> capabilities are only useful for identifying the host CPU, but they
> cannot be directly pass to QEMU. As such, you can't use these models
> when defining a CPU in a domain XML.
>
> That said, I doubt this patch is useful at all.
>
> Jirka
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200903/862dcf30/attachment-0001.htm>