qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [RFC PATCH 0/4] target/ppc: code motion to compile translate_init
@ 2021-04-23 19:18 Bruno Larsen (billionai)
  2021-04-23 19:18 ` [RFC PATCH 1/4] target/ppc: move opcode table logic to translate.c Bruno Larsen (billionai)
                   ` (4 more replies)
  0 siblings, 5 replies; 17+ messages in thread
From: Bruno Larsen (billionai) @ 2021-04-23 19:18 UTC (permalink / raw)
  To: qemu-devel
  Cc: farosas, luis.pires, lucas.araujo, fernando.valle, qemu-ppc,
	Bruno Larsen (billionai),
	matheus.ferst, david

The current patch series aims to isolate common code from translation-related
code. This isolation is required to disable TCG at build time, and the
final system still work.

This patch series is still WIP, so comments are welcome

Bruno Larsen (billionai) (4):
  target/ppc: move opcode table logic to translate.c
  target/ppc: isolated SPR read/write callbacks
  target/ppc: Move SPR generation to separate file
  target/ppc: isolated cpu init from translation logic

 .../ppc/{translate_init.c.inc => cpu_init.c}  | 5414 +----------------
 target/ppc/internal.h                         |  114 +
 target/ppc/meson.build                        |    2 +
 target/ppc/spr_common.c                       | 2943 +++++++++
 target/ppc/spr_tcg.c.inc                      | 1002 +++
 target/ppc/spr_tcg.h                          |  132 +
 target/ppc/translate.c                        |  446 +-
 7 files changed, 4863 insertions(+), 5190 deletions(-)
 rename target/ppc/{translate_init.c.inc => cpu_init.c} (50%)
 create mode 100644 target/ppc/spr_common.c
 create mode 100644 target/ppc/spr_tcg.c.inc
 create mode 100644 target/ppc/spr_tcg.h

-- 
2.17.1



^ permalink raw reply	[flat|nested] 17+ messages in thread
* RE: [RFC PATCH 2/4] target/ppc: isolated SPR read/write callbacks
@ 2021-04-27 12:58 Bruno Piazera Larsen
  0 siblings, 0 replies; 17+ messages in thread
From: Bruno Piazera Larsen @ 2021-04-27 12:58 UTC (permalink / raw)
  To: David Gibson
  Cc: farosas, qemu-devel, Lucas Mateus Martins Araujo e Castro,
	Fernando Eckhardt Valle, qemu-ppc, Matheus Kowalczuk Ferst,
	Luis Fernando Fujita Pires

[-- Attachment #1: Type: text/plain, Size: 1464 bytes --]

> The 2 fscr functions below aren't readers and writers for the FSCR.
> Instead they're used by instructions related to facilities the FSCR
> can enable and disable - this generates the code to check the FSCR and
> generate an exception if the units are disabled.
>
> That doesn't mean they don't belong here, but it does mean

I think something got lost here... anyway, I just checked that these functions
and they're only used in spr_tcg.c.inc at this point, so I think I'll remove it
from the .h and make them static. And mention it in the commit message
as well

I could also put it in translate.c, but whoever needs to refactor this code next
would have as big a headache as we're having now.


Bruno Piazera Larsen

Instituto de Pesquisas ELDORADO<http://clickemailmkt.eldorado.org.br/ls/click?upn=UPoxpeIcHnAcbUZyo7TTaswyiVb1TXP3jEbQqiiJKKGsxOn8hBEs5ZsMLQfXkKuKXZ7MVDg0ij9eG8HV4TXI75dBzDiNGLxQ8Xx5PzCVNt6TpGrzBbU-2Biu0o69X5ce-2FW-2FOk1uUipuK0fZnWXJEgbRw-3D-3DJY4T_wWk-2BG6VvNBoa1YzxYjhCdFS9IfANIaBzDSklR1NyyrKOI1wj0P-2BdBFcuO4FnHcsA1MyHu0ly1Yt3oDMp7KKdJPM68iKuI2jiRH5v4B0d8wf3chU3qy5n5iXWnW1QjSaNFHOgELzxaP-2FnesTeBgJ5dFkjH4f279sVQpOtyjw5xAqj34M6pgNRAxVvuXif4IWDcVzXg1FzfYlEfkKzr9vvpA3Hg8kitwMtlU3zwbQUBCgL30fQoJPcRPMGKyOY8RmoAlXNqTJYDYIvqmfnI7KLUvw6vKB5R-2B5q1FJRAzX7H-2BmF0NnDET6jMLuIqtCcVIch>

Departamento Computação Embarcada

Analista de Software Trainee

Aviso Legal - Disclaimer<https://www.eldorado.org.br/disclaimer.html>

[-- Attachment #2: Type: text/html, Size: 4443 bytes --]

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

end of thread, other threads:[~2021-04-27 13:04 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-23 19:18 [RFC PATCH 0/4] target/ppc: code motion to compile translate_init Bruno Larsen (billionai)
2021-04-23 19:18 ` [RFC PATCH 1/4] target/ppc: move opcode table logic to translate.c Bruno Larsen (billionai)
2021-04-26 19:15   ` Fabiano Rosas
2021-04-26 19:29     ` Bruno Piazera Larsen
2021-04-27  3:16       ` david
2021-04-23 19:18 ` [RFC PATCH 2/4] target/ppc: isolated SPR read/write callbacks Bruno Larsen (billionai)
2021-04-26 20:06   ` Fabiano Rosas
2021-04-26 20:38     ` Bruno Piazera Larsen
2021-04-27  3:28   ` David Gibson
2021-04-23 19:18 ` [RFC PATCH 3/4] target/ppc: Move SPR generation to separate file Bruno Larsen (billionai)
2021-04-26 21:08   ` Fabiano Rosas
2021-04-27  3:35     ` David Gibson
2021-04-23 19:18 ` [RFC PATCH 4/4] target/ppc: isolated cpu init from translation logic Bruno Larsen (billionai)
2021-04-27  3:38   ` David Gibson
2021-04-23 23:43 ` [RFC PATCH 0/4] target/ppc: code motion to compile translate_init Richard Henderson
2021-04-26 11:51   ` Bruno Piazera Larsen
2021-04-27 12:58 [RFC PATCH 2/4] target/ppc: isolated SPR read/write callbacks Bruno Piazera Larsen

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