From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34800) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ety39-00082g-9C for qemu-devel@nongnu.org; Thu, 08 Mar 2018 11:05:48 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ety33-0001cz-Ds for qemu-devel@nongnu.org; Thu, 08 Mar 2018 11:05:47 -0500 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:37130 helo=mx1.redhat.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1ety33-0001ch-6s for qemu-devel@nongnu.org; Thu, 08 Mar 2018 11:05:41 -0500 Date: Thu, 8 Mar 2018 16:05:29 +0000 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Message-ID: <20180308160529.GA4718@redhat.com> Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= References: <20180308154016.22228-1-peter.maydell@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20180308154016.22228-1-peter.maydell@linaro.org> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH] unicore32: Mark as deprecated List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: qemu-devel@nongnu.org, patches@linaro.org, Guan Xuetao , Paolo Bonzini On Thu, Mar 08, 2018 at 03:40:16PM +0000, Peter Maydell wrote: > We intend to remove the unicore32 target code: >=20 > * It has had no changes since 2012 that were not tree-wide > maintenance/API changes/other global updates > * We dropped the linux-user unicore32 support in 2016 because of a > clash between the 'old ABI' that it was implementing and the ABI > that's actually in the upstream Linux kernel, and there have been > no moves to get this fixed so we could re-enable it, nor any > complaints when it went away > * Linux is now planning to drop unicore support: > https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1619640= .html > * there is apparently no upstream gcc support for the architecture > * nobody has ever reported a bug or problem to us about it >=20 > This commit patches the unicore32 target to warn on startup that it > is deprecated, and adds a note to our documentation's deprecation > section. >=20 > Signed-off-by: Peter Maydell > --- > I proposed deprecation of unicore32 on 27th Feb: > https://lists.gnu.org/archive/html/qemu-devel/2018-02/msg06634.html > and got several 'yes'es and no objections. This is the "changing > the codebase" part of that, ready for the 2.12 codefreeze. >=20 > We should also note this in the 2.12 Changelog. > --- > target/unicore32/cpu.c | 11 +++++++++++ > qemu-doc.texi | 5 +++++ > 2 files changed, 16 insertions(+) >=20 > diff --git a/target/unicore32/cpu.c b/target/unicore32/cpu.c > index 29d160a88d..e95fa55b8e 100644 > --- a/target/unicore32/cpu.c > +++ b/target/unicore32/cpu.c > @@ -19,6 +19,8 @@ > #include "migration/vmstate.h" > #include "exec/exec-all.h" > #include "fpu/softfloat.h" > +#include "sysemu/qtest.h" > +#include "qemu/error-report.h" > =20 > static void uc32_cpu_set_pc(CPUState *cs, vaddr value) > { > @@ -120,6 +122,15 @@ static void uc32_cpu_initfn(Object *obj) > #endif > =20 > tlb_flush(cs); > + > + /* We can't do this in class_init because the qtest_enabled > + * flag hasn't yet been initialized there. Luckily the Unicore32 > + * machines don't support SMP so the message will only appear once= . > + */ > + if (!qtest_enabled()) { > + warn_report("qemu-system-unicore32 is deprecated " > + "and will be removed in a future QEMU release."); > + } > } > =20 > static const VMStateDescription vmstate_uc32_cpu =3D { > diff --git a/qemu-doc.texi b/qemu-doc.texi > index 39e38c87ec..0e8609c11c 100644 > --- a/qemu-doc.texi > +++ b/qemu-doc.texi > @@ -2779,6 +2779,11 @@ support page sizes < 4096 any longer. > The ``xlnx-ep108'' machine has been replaced by the ``xlnx-zcu102'' ma= chine. > The ``xlnx-zcu102'' machine has the same features and capabilites in Q= EMU. > =20 > +@subsection Unicore32 targets (since 2.12.0) > + > +The ``unicore32'' CPU target and ``qemu-system-unicore32'' will be rem= oved, > +since this architecture is unmaintained. > + > @node License > @appendix License Reviewed-by: Daniel P. Berrang=C3=A9 Regards, Daniel --=20 |: https://berrange.com -o- https://www.flickr.com/photos/dberran= ge :| |: https://libvirt.org -o- https://fstop138.berrange.c= om :| |: https://entangle-photo.org -o- https://www.instagram.com/dberran= ge :|