openembedded-core.lists.openembedded.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] qemux86-64: Allow higher tunes
@ 2022-08-11 15:40 Tom Rini
  2022-08-11 15:51 ` [OE-core] " Alexander Kanavin
  0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2022-08-11 15:40 UTC (permalink / raw)
  To: openembedded-core; +Cc: Joshua Watt, Richard Purdie

Back in 0be64e54a0e6 ("qemux86: Allow higher tunes") we moved the
qemux86 machine to using the core-i7 tune file, for maximum flexibility
and to allow for enabling advanced processor features if desired or
required by various packagess, without changing the default tune. Do the
same now for qemux86-64.

Cc: Joshua Watt <JPEWhacker@gmail.com>
Cc: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Tom Rini <trini@konsulko.com>
---
 meta/conf/machine/qemux86-64.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/machine/qemux86-64.conf b/meta/conf/machine/qemux86-64.conf
index 901353499c81..864086791179 100644
--- a/meta/conf/machine/qemux86-64.conf
+++ b/meta/conf/machine/qemux86-64.conf
@@ -10,7 +10,7 @@ PREFERRED_PROVIDER_virtual/libgles3 ?= "mesa"
 
 require conf/machine/include/qemu.inc
 DEFAULTTUNE ?= "core2-64"
-require conf/machine/include/x86/tune-core2.inc
+require conf/machine/include/x86/tune-corei7.inc
 require conf/machine/include/x86/qemuboot-x86.inc
 
 UBOOT_MACHINE ?= "qemu-x86_64_defconfig"
-- 
2.25.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [OE-core] [PATCH] qemux86-64: Allow higher tunes
  2022-08-11 15:40 [PATCH] qemux86-64: Allow higher tunes Tom Rini
@ 2022-08-11 15:51 ` Alexander Kanavin
  2022-08-11 16:04   ` Tom Rini
  2022-08-11 16:42   ` Mittal, Anuj
  0 siblings, 2 replies; 6+ messages in thread
From: Alexander Kanavin @ 2022-08-11 15:51 UTC (permalink / raw)
  To: Tom Rini; +Cc: OE-core, Joshua Watt, Richard Purdie

This is less useful than it may seem, and merely moves the supported
CPU from circa-2006 core 2 to circa-2008 core i7. You won't get AVX or
anything newer, which requires adding these instructions to qemu
usermode (a project I'd like to do if I had the time).

Alex

On Thu, 11 Aug 2022 at 17:40, Tom Rini <trini@konsulko.com> wrote:
>
> Back in 0be64e54a0e6 ("qemux86: Allow higher tunes") we moved the
> qemux86 machine to using the core-i7 tune file, for maximum flexibility
> and to allow for enabling advanced processor features if desired or
> required by various packagess, without changing the default tune. Do the
> same now for qemux86-64.
>
> Cc: Joshua Watt <JPEWhacker@gmail.com>
> Cc: Richard Purdie <richard.purdie@linuxfoundation.org>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  meta/conf/machine/qemux86-64.conf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/conf/machine/qemux86-64.conf b/meta/conf/machine/qemux86-64.conf
> index 901353499c81..864086791179 100644
> --- a/meta/conf/machine/qemux86-64.conf
> +++ b/meta/conf/machine/qemux86-64.conf
> @@ -10,7 +10,7 @@ PREFERRED_PROVIDER_virtual/libgles3 ?= "mesa"
>
>  require conf/machine/include/qemu.inc
>  DEFAULTTUNE ?= "core2-64"
> -require conf/machine/include/x86/tune-core2.inc
> +require conf/machine/include/x86/tune-corei7.inc
>  require conf/machine/include/x86/qemuboot-x86.inc
>
>  UBOOT_MACHINE ?= "qemu-x86_64_defconfig"
> --
> 2.25.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#169248): https://lists.openembedded.org/g/openembedded-core/message/169248
> Mute This Topic: https://lists.openembedded.org/mt/92961093/1686489
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [OE-core] [PATCH] qemux86-64: Allow higher tunes
  2022-08-11 15:51 ` [OE-core] " Alexander Kanavin
@ 2022-08-11 16:04   ` Tom Rini
  2022-08-11 16:07     ` Alexander Kanavin
  2022-08-11 16:42   ` Mittal, Anuj
  1 sibling, 1 reply; 6+ messages in thread
From: Tom Rini @ 2022-08-11 16:04 UTC (permalink / raw)
  To: Alexander Kanavin; +Cc: OE-core, Joshua Watt, Richard Purdie

On Thu, Aug 11, 2022 at 05:51:52PM +0200, Alexander Kanavin wrote:

> This is less useful than it may seem, and merely moves the supported
> CPU from circa-2006 core 2 to circa-2008 core i7. You won't get AVX or
> anything newer, which requires adding these instructions to qemu
> usermode (a project I'd like to do if I had the time).

True but it's also the case that tune-corei7.conf is the newest one in
oe-core, yes? I had previously peeked in meta-intel which has a
skylake tune, but bringing that in is something I'd leave to Intel/WR
folks to take care of.

-- 
Tom


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [OE-core] [PATCH] qemux86-64: Allow higher tunes
  2022-08-11 16:04   ` Tom Rini
@ 2022-08-11 16:07     ` Alexander Kanavin
  0 siblings, 0 replies; 6+ messages in thread
From: Alexander Kanavin @ 2022-08-11 16:07 UTC (permalink / raw)
  To: Tom Rini; +Cc: OE-core, Joshua Watt, Richard Purdie

On Thu, 11 Aug 2022 at 18:04, Tom Rini <trini@konsulko.com> wrote:
> > This is less useful than it may seem, and merely moves the supported
> > CPU from circa-2006 core 2 to circa-2008 core i7. You won't get AVX or
> > anything newer, which requires adding these instructions to qemu
> > usermode (a project I'd like to do if I had the time).
>
> True but it's also the case that tune-corei7.conf is the newest one in
> oe-core, yes? I had previously peeked in meta-intel which has a
> skylake tune, but bringing that in is something I'd leave to Intel/WR
> folks to take care of.

Yes, from making things consistent perspective this change is fine.

The skylake tune in meta-intel gives its users two less than great
choices: disable qemu usermode, or disable everything that qemu
doesn't support (AVX and up).

Alex


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [OE-core] [PATCH] qemux86-64: Allow higher tunes
  2022-08-11 15:51 ` [OE-core] " Alexander Kanavin
  2022-08-11 16:04   ` Tom Rini
@ 2022-08-11 16:42   ` Mittal, Anuj
  2022-12-09 22:23     ` Alexander Kanavin
  1 sibling, 1 reply; 6+ messages in thread
From: Mittal, Anuj @ 2022-08-11 16:42 UTC (permalink / raw)
  To: trini, alex.kanavin; +Cc: openembedded-core, richard.purdie, JPEWhacker

On Thu, 2022-08-11 at 17:51 +0200, Alexander Kanavin wrote:
> You won't get AVX or
> anything newer, which requires adding these instructions to qemu
> usermode (a project I'd like to do if I had the time).

https://lore.kernel.org/all/20220418173904.3746036-1-paul@nowt.org/

I didn't notice any v3 or an update on the bug.

Thanks,

Anuj

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [OE-core] [PATCH] qemux86-64: Allow higher tunes
  2022-08-11 16:42   ` Mittal, Anuj
@ 2022-12-09 22:23     ` Alexander Kanavin
  0 siblings, 0 replies; 6+ messages in thread
From: Alexander Kanavin @ 2022-12-09 22:23 UTC (permalink / raw)
  To: Mittal, Anuj; +Cc: trini, openembedded-core, richard.purdie, JPEWhacker

On Thu, 11 Aug 2022 at 18:42, Mittal, Anuj <anuj.mittal@intel.com> wrote:
>
> On Thu, 2022-08-11 at 17:51 +0200, Alexander Kanavin wrote:
> > You won't get AVX or
> > anything newer, which requires adding these instructions to qemu
> > usermode (a project I'd like to do if I had the time).
>
> https://lore.kernel.org/all/20220418173904.3746036-1-paul@nowt.org/
>
> I didn't notice any v3 or an update on the bug.

This is now merged in qemu:
https://lists.nongnu.org/archive/html/qemu-devel/2022-10/msg02253.html

7.2 will be out shortly, so then we can look into adding skylake tunes
and perhaps moving qemux86_64 to them. Making skylake a fully
supported default in meta-intel would be nice too.

Alex


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-12-09 22:23 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-11 15:40 [PATCH] qemux86-64: Allow higher tunes Tom Rini
2022-08-11 15:51 ` [OE-core] " Alexander Kanavin
2022-08-11 16:04   ` Tom Rini
2022-08-11 16:07     ` Alexander Kanavin
2022-08-11 16:42   ` Mittal, Anuj
2022-12-09 22:23     ` Alexander Kanavin

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).