qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Thomas Huth <thuth@redhat.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
	qemu-devel@nongnu.org, "Paolo Bonzini" <pbonzini@redhat.com>,
	"Markus Armbruster" <armbru@redhat.com>,
	"Alex Bennée" <alex.bennee@linaro.org>,
	qemu-arm@nongnu.org, "Maxim Levitsky" <mlevitsk@redhat.com>,
	libvir-list@redhat.com,
	"Richard Henderson" <richard.henderson@linaro.org>,
	xen-devel@lists.xenproject.org,
	"Reinoud Zandijk" <reinoud@netbsd.org>
Subject: Re: [PATCH v2 4/6] docs/about/deprecated: Deprecate the qemu-system-arm binary
Date: Fri, 3 Mar 2023 11:39:09 +0000	[thread overview]
Message-ID: <ZAHcXUvAKFHeJiM7@redhat.com> (raw)
In-Reply-To: <4e6512e4-b159-cbe2-5f74-89fab25208e6@redhat.com>

On Fri, Mar 03, 2023 at 12:31:42PM +0100, Thomas Huth wrote:
> On 03/03/2023 12.16, Peter Maydell wrote:
> > On Thu, 2 Mar 2023 at 16:31, Thomas Huth <thuth@redhat.com> wrote:
> > > 
> > > qemu-system-aarch64 is a proper superset of qemu-system-arm,
> > > and the latter was mainly still required for 32-bit KVM support.
> > > But this 32-bit KVM arm support has been dropped in the Linux
> > > kernel a couple of years ago already, so we don't really need
> > > qemu-system-arm anymore, thus deprecated it now.
> > > 
> > > Signed-off-by: Thomas Huth <thuth@redhat.com>
> > > ---
> > >   docs/about/deprecated.rst | 10 ++++++++++
> > >   1 file changed, 10 insertions(+)
> > > 
> > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > > index a30aa8dfdf..21ce70b5c9 100644
> > > --- a/docs/about/deprecated.rst
> > > +++ b/docs/about/deprecated.rst
> > > @@ -45,6 +45,16 @@ run 32-bit guests by selecting a 32-bit CPU model, including KVM support
> > >   on x86_64 hosts. Thus users are recommended to reconfigure their systems
> > >   to use the ``qemu-system-x86_64`` binary instead.
> > > 
> > > +``qemu-system-arm`` binary (since 8.0)
> > > +''''''''''''''''''''''''''''''''''''''
> > > +
> > > +``qemu-system-aarch64`` is a proper superset of ``qemu-system-arm``.
> > 
> > I think this is not quite true -- at the moment if you want
> > "every feature we implement, 32-bit" the only way to get
> > that is 'qemu-system-arm -cpu max'. The '-cpu max' on
> > qemu-system-aarch64 is 64-bit, and we don't implement for TCG
> > the "-cpu max,aarch64=off" syntax that we do for KVM that would
> > let the user say "no 64-bit support".
> 
> Ok ... so what does that mean now? ... can we continue with this patch, e.g.
> after rephrasing the text a little bit, or do we need to implement "-cpu
> max,aarch64=off" for TCG first?

I think we need to have a way to request the max 32-bit CPU before we
deprecate, because deprecation has to tell people what they should use
instead.

For qemu-system-i686 -cpu max, I guess we have lm=off to hide the 64-bit
support, so that's OK from QEMU POV, but will need libvirt enhancement
as I don't think we've taken that into account.


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:[~2023-03-03 11:39 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-02 16:31 [PATCH v2 0/6] Deprecate support for 32-bit x86 and arm hosts Thomas Huth
2023-03-02 16:31 ` [PATCH v2 1/6] docs/about/deprecated: Deprecate the qemu-system-i386 binary Thomas Huth
2023-03-02 18:05   ` Daniel P. Berrangé
2023-03-03  0:49     ` Wilfred Mallawa
2023-03-02 16:31 ` [PATCH v2 2/6] docs/about/deprecated: Deprecate 32-bit x86 hosts Thomas Huth
2023-03-02 18:05   ` Daniel P. Berrangé
2023-03-03  0:52   ` Wilfred Mallawa
2023-03-02 16:31 ` [PATCH v2 3/6] gitlab-ci.d/crossbuilds: Drop the i386 jobs Thomas Huth
2023-03-02 18:07   ` Daniel P. Berrangé
2023-03-03  0:53   ` Wilfred Mallawa
2023-03-02 16:31 ` [PATCH v2 4/6] docs/about/deprecated: Deprecate the qemu-system-arm binary Thomas Huth
2023-03-02 18:07   ` Daniel P. Berrangé
2023-03-02 22:16   ` Philippe Mathieu-Daudé
2023-03-03  7:36     ` Thomas Huth
2023-03-03  0:55   ` Wilfred Mallawa
2023-03-03 11:16   ` Peter Maydell
2023-03-03 11:31     ` Thomas Huth
2023-03-03 11:39       ` Daniel P. Berrangé [this message]
2023-03-02 16:31 ` [PATCH v2 5/6] docs/about/deprecated: Deprecate 32-bit arm hosts Thomas Huth
2023-03-02 18:09   ` Daniel P. Berrangé
2023-03-03  0:57   ` Wilfred Mallawa
2023-03-02 16:31 ` [PATCH v2 6/6] gitlab-ci.d/crossbuilds: Drop the 32-bit arm system emulation jobs Thomas Huth
2023-03-02 18:10   ` Daniel P. Berrangé
2023-03-02 21:58   ` Philippe Mathieu-Daudé
2023-03-03  0:57   ` Wilfred Mallawa
2023-03-02 22:07 ` [PATCH v2 0/6] Deprecate support for 32-bit x86 and arm hosts Philippe Mathieu-Daudé
2023-03-03  9:48   ` Thomas Huth
2023-03-03 11:09     ` John Paul Adrian Glaubitz
2023-03-03 11:22       ` Thomas Huth
2023-03-03 11:57         ` John Paul Adrian Glaubitz
2023-03-03 11:24       ` Peter Maydell

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=ZAHcXUvAKFHeJiM7@redhat.com \
    --to=berrange@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=armbru@redhat.com \
    --cc=libvir-list@redhat.com \
    --cc=mlevitsk@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=reinoud@netbsd.org \
    --cc=richard.henderson@linaro.org \
    --cc=thuth@redhat.com \
    --cc=xen-devel@lists.xenproject.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).