All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Phil Dennis-Jordan <lists@philjordan.eu>
Cc: "Philippe Mathieu-Daudé" <philmd@linaro.org>,
	qemu-devel@nongnu.org, "Thomas Huth" <thuth@redhat.com>,
	"Daniel Henrique Barboza" <dbarboza@ventanamicro.com>,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Paolo Bonzini" <pbonzini@redhat.com>
Subject: Re: [PATCH] system: Try hardware accelerators (KVM, HVF) before software one (TCG)
Date: Mon, 6 Jan 2025 18:43:07 +0000	[thread overview]
Message-ID: <Z3wkO0GP-VxfEcc0@redhat.com> (raw)
In-Reply-To: <CAGCz3vtSUD-8pG7GVUAjx0ydOKAh-YxvWDUpcECef7ch7dbeGg@mail.gmail.com>

On Sat, Jan 04, 2025 at 12:28:14PM +0100, Phil Dennis-Jordan wrote:
> On Fri, 3 Jan 2025 at 16:16, Daniel P. Berrangé <berrange@redhat.com> wrote:
> 
> > On Fri, Jan 03, 2025 at 04:05:58PM +0100, Philippe Mathieu-Daudé wrote:
> > > As Daniel suggested [*]:
> > >
> > > > We should consider to rank HVF above TCG, on the basis
> > > > that HW acceleration is faster and should provide a
> > > > host<->guest security boundary that we don't claim for TCG
> > >
> > > [*] https://lore.kernel.org/qemu-devel/Z07YASl2Pd3CPtjE@redhat.com/
> >
> > Note, my statement above was on the basis that HVF passes all our
> > functional tests, thus indicating a decent level of confidence
> > in the correctness of the HVF impl.
> >
> > If anyone knows any show stopper problems with HVF that would
> > justify blocking its promotion ahead of TCG.... say now.
> >
> 
> I don't know about showstoppers, but:
> 
> 1. As far as I'm aware there are/were problems with the virtual IOMMU
> devices in HVF. It's been a while (~half a year?) since I tried them, but I
> had problems getting guests booted with intel_iommu etc.

I think that vIOMMU is niche enough that we can merely consider it
a nice-to-fix bug, and not block promoting HVF.

> 2. I think there might also be a few remaining edge cases where the x86
> instruction emulation on fault/trap is incomplete. Most notably, MMIO using
> SSE/AVX/etc. instructions will, I think, fail. In practice this is a fairly
> obscure use case - I'm not aware of any guest OS that actually performs
> MMIO using these instructions. I have a patch kicking around that adds
> support for missing 64-bit variants of common scalar arithmetic
> instructions with memory operands. I can dig that up and post it - do we
> have a good way of adding tests for this kind of thing?

Not sure how best to test this, other than finding a guest OS that
exhibits this ? Others probably have better suggestions...

> 3. As far as I'm aware, there's no CI happening on HVF? Or has that
> changed? macOS is notoriously a pain in the rear in terms of CI thanks to
> its licensing, and the big cloud CI platforms tend to run it in a VM which
> in turn typically doesn't support nested HVF. I've been working on an
> on-prem solution to provisioning bare-metal Macs to run clean-slate OS
> images for CI. This has been a side project though and I haven't had the
> resources to focus on that project to see it through. It might be possible
> to do this in the cloud on Amazon's EC2 Mac Minis as well, but those aren't
> exactly cheap.

The only CI we have is running under Cirrus CI which uses VMs on
real mac aarch64 hardware, but I don't think we can test HVF there.

Mostly we rely on regular contributors periodically running tests
and reporting on problems. This is not ideal, but also not a blocker
for enabling HVF - it just means macOS isn't a full tier 1 platform
for us.

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|



  reply	other threads:[~2025-01-06 18:43 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-03 15:05 [PATCH] system: Try hardware accelerators (KVM, HVF) before software one (TCG) Philippe Mathieu-Daudé
2025-01-03 15:15 ` Daniel P. Berrangé
2025-01-03 17:16   ` Philippe Mathieu-Daudé
2025-01-03 17:48     ` Peter Xu
2025-01-04 11:28   ` Phil Dennis-Jordan
2025-01-06 18:43     ` Daniel P. Berrangé [this message]
2025-01-08  8:36       ` Phil Dennis-Jordan
2025-01-06 17:33 ` Richard Henderson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Z3wkO0GP-VxfEcc0@redhat.com \
    --to=berrange@redhat.com \
    --cc=dbarboza@ventanamicro.com \
    --cc=lists@philjordan.eu \
    --cc=pbonzini@redhat.com \
    --cc=philmd@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.