Quoting Ming Lei ([email protected]):
> Looks the upstreamed commit mentioned in #5 includes most of the patch we are 
> using, and that is great
> really, :-)
> 
> But maybe below part is needed too:
> 
> diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
> index e964c75..c947e2e 100644
> --- a/src/qemu/qemu_domain.c
> +++ b/src/qemu/qemu_domain.c
> @@ -793,10 +794,14 @@ qemuDomainDefaultNetModel(const virDomainDef *def)
>          def->os.arch == VIR_ARCH_S390X)
>          return "virtio";
>  
> -    if (def->os.arch == VIR_ARCH_ARMV7L) {
> +    if (def->os.arch == VIR_ARCH_ARMV7L ||
> +     def->os.arch == VIR_ARCH_AARCH64) {
>          if (STREQ(def->os.machine, "versatilepb"))
>              return "smc91c111";
>  
> +        if (strstr(def->os.machine, "virt"))
> +            return "virtio";
> +
>          /* Incomplete. vexpress (and a few others) use this, but not all
>           * arm boards */
>          return "lan9118";

Can you create a single patch with what remains of what you need?  Also,
if you look at the changelog for the upstreamed patch they had to do
some work to the testcase to be accepted - does what remains of your
patch need any testcase tweaking?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1265425

Title:
  Doesn't support arm64

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1265425/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to