Re: [Buildroot] [PATCH 4/6] configs/qemu: add qemu_arc_defconfig

From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Waldemar Brodkorb <mail@waldemar-brodkorb.de>
Cc: Romain Naour <romain.naour@gmail.com>,
	ARC Maintainers <arc-buildroot@synopsys.com>,
	Giulio Benetti <giulio.benetti@benettiengineering.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 4/6] configs/qemu: add qemu_arc_defconfig
Date: Mon, 1 Jan 2024 21:18:26 +0100	[thread overview]
Message-ID: <ZZMeEjKs7cVLRZwb@landeda> (raw)
In-Reply-To: <ZZHjRSzIb7WUnhUo@waldemar-brodkorb.de>

Waldemar, All,

On 2023-12-31 22:55 +0100, Waldemar Brodkorb spake thusly:
> Yann E. MORIN wrote,
> > On 2023-12-31 18:25 +0100, Waldemar Brodkorb spake thusly:
> > > Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
[--SNIP--]
> > > +You need Qemu fork from here:
> > > +https://github.com/foss-for-synopsys-dwc-arc-processors/qemu
> > So, we are dropping all the arc specific versions for the toolchain
> > components, but we eventually introduce a defconfig that relies on an
> > arc specific version of qemu, that we do not even package in Buildroot,
> > so that users can't even enable host-qemu in Buildroot.
> > That is a bit sad...
> > What is the rationale for introducing this defconfig, then?
> Qemu is more widely known then the nsimdrv.

OK, fair enough.

Still, Qemu with ARC support is not sidely available (e.h .it's not ins
distros), and we do not have a way to easily build it from source,
leaving to the user the duty to find how to do get such a Qemu.

And since you point to the source tree (which is nice), it implies one
has to build from source on their own, which we know is a bit tedious...

> And I think when ARCv3
> support is added to Buildroot

*When* is the key word here. I know Alexey has said he was looking into
it, but IIRC the topic has been pending since last August, so I would
like to decorelate the two topics.

> it can be used to test it, too.

That does not change the fact that it is a forked version, not readily
available, that we do not build in Buildroot, and that the user has to
manually build from source on their own, which is still problematic.

> I thought it would be nice to have instead of only rely on the nsim
> defconfig we have now.

That I agree on, but I don;t think it is a good-enough reason to be
using a non-upstream Qemu.

> Can we keep it? What others say?

"Mom, dad, it's a cute doggy/kitten/duck/poney/hedghog! Can we keep it?"

Let's see what other maintainers think, of course.

And in any case, that does not diminish the fact that the commit log
should contain a proper rationale.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot