Re: [PATCH v2 11/14] target/*: Remove TARGET_LONG_BITS from cpu-param.h

2025-02-03 Thread Philippe Mathieu-Daudé

On 3/2/25 04:18, Richard Henderson wrote:

This is now handled by the configs/targets/*.mak fragment.

Signed-off-by: Richard Henderson 
---
  target/alpha/cpu-param.h  | 2 --
  target/arm/cpu-param.h| 2 --
  target/avr/cpu-param.h| 1 -
  target/hexagon/cpu-param.h| 1 -
  target/hppa/cpu-param.h   | 2 --
  target/i386/cpu-param.h   | 2 --
  target/loongarch/cpu-param.h  | 1 -
  target/m68k/cpu-param.h   | 1 -
  target/microblaze/cpu-param.h | 2 --
  target/mips/cpu-param.h   | 5 -
  target/openrisc/cpu-param.h   | 1 -
  target/ppc/cpu-param.h| 2 --
  target/riscv/cpu-param.h  | 2 --
  target/rx/cpu-param.h | 1 -
  target/s390x/cpu-param.h  | 1 -
  target/sh4/cpu-param.h| 1 -
  target/sparc/cpu-param.h  | 2 --
  target/tricore/cpu-param.h| 1 -
  target/xtensa/cpu-param.h | 1 -
  19 files changed, 31 deletions(-)


Reviewed-by: Philippe Mathieu-Daudé 




Re: [PATCH v2 11/14] target/*: Remove TARGET_LONG_BITS from cpu-param.h

2025-02-03 Thread Alex Bennée
Richard Henderson  writes:

> This is now handled by the configs/targets/*.mak fragment.
>
> Signed-off-by: Richard Henderson 

Reviewed-by: Alex Bennée 

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro



Re: [PATCH v2 11/14] target/*: Remove TARGET_LONG_BITS from cpu-param.h

2025-02-03 Thread Thomas Huth

On 03/02/2025 04.18, Richard Henderson wrote:

This is now handled by the configs/targets/*.mak fragment.

Signed-off-by: Richard Henderson 
---
  target/alpha/cpu-param.h  | 2 --
  target/arm/cpu-param.h| 2 --
  target/avr/cpu-param.h| 1 -
  target/hexagon/cpu-param.h| 1 -
  target/hppa/cpu-param.h   | 2 --
  target/i386/cpu-param.h   | 2 --
  target/loongarch/cpu-param.h  | 1 -
  target/m68k/cpu-param.h   | 1 -
  target/microblaze/cpu-param.h | 2 --
  target/mips/cpu-param.h   | 5 -
  target/openrisc/cpu-param.h   | 1 -
  target/ppc/cpu-param.h| 2 --
  target/riscv/cpu-param.h  | 2 --
  target/rx/cpu-param.h | 1 -
  target/s390x/cpu-param.h  | 1 -
  target/sh4/cpu-param.h| 1 -
  target/sparc/cpu-param.h  | 2 --
  target/tricore/cpu-param.h| 1 -
  target/xtensa/cpu-param.h | 1 -
  19 files changed, 31 deletions(-)


Reviewed-by: Thomas Huth 




[PATCH v2 11/14] target/*: Remove TARGET_LONG_BITS from cpu-param.h

2025-02-02 Thread Richard Henderson
This is now handled by the configs/targets/*.mak fragment.

Signed-off-by: Richard Henderson 
---
 target/alpha/cpu-param.h  | 2 --
 target/arm/cpu-param.h| 2 --
 target/avr/cpu-param.h| 1 -
 target/hexagon/cpu-param.h| 1 -
 target/hppa/cpu-param.h   | 2 --
 target/i386/cpu-param.h   | 2 --
 target/loongarch/cpu-param.h  | 1 -
 target/m68k/cpu-param.h   | 1 -
 target/microblaze/cpu-param.h | 2 --
 target/mips/cpu-param.h   | 5 -
 target/openrisc/cpu-param.h   | 1 -
 target/ppc/cpu-param.h| 2 --
 target/riscv/cpu-param.h  | 2 --
 target/rx/cpu-param.h | 1 -
 target/s390x/cpu-param.h  | 1 -
 target/sh4/cpu-param.h| 1 -
 target/sparc/cpu-param.h  | 2 --
 target/tricore/cpu-param.h| 1 -
 target/xtensa/cpu-param.h | 1 -
 19 files changed, 31 deletions(-)

diff --git a/target/alpha/cpu-param.h b/target/alpha/cpu-param.h
index c21ddf1afd..ff06e41497 100644
--- a/target/alpha/cpu-param.h
+++ b/target/alpha/cpu-param.h
@@ -8,8 +8,6 @@
 #ifndef ALPHA_CPU_PARAM_H
 #define ALPHA_CPU_PARAM_H
 
-#define TARGET_LONG_BITS 64
-
 /* ??? EV4 has 34 phys addr bits, EV5 has 40, EV6 has 44.  */
 #define TARGET_PHYS_ADDR_SPACE_BITS  44
 
diff --git a/target/arm/cpu-param.h b/target/arm/cpu-param.h
index bed29613c8..896b35bd6d 100644
--- a/target/arm/cpu-param.h
+++ b/target/arm/cpu-param.h
@@ -9,11 +9,9 @@
 #define ARM_CPU_PARAM_H
 
 #ifdef TARGET_AARCH64
-# define TARGET_LONG_BITS 64
 # define TARGET_PHYS_ADDR_SPACE_BITS  52
 # define TARGET_VIRT_ADDR_SPACE_BITS  52
 #else
-# define TARGET_LONG_BITS 32
 # define TARGET_PHYS_ADDR_SPACE_BITS  40
 # define TARGET_VIRT_ADDR_SPACE_BITS  32
 #endif
diff --git a/target/avr/cpu-param.h b/target/avr/cpu-param.h
index 93c2f470d0..81f3f49ee1 100644
--- a/target/avr/cpu-param.h
+++ b/target/avr/cpu-param.h
@@ -21,7 +21,6 @@
 #ifndef AVR_CPU_PARAM_H
 #define AVR_CPU_PARAM_H
 
-#define TARGET_LONG_BITS 32
 /*
  * TARGET_PAGE_BITS cannot be more than 8 bits because
  * 1.  all IO registers occupy [0x .. 0x00ff] address range, and they
diff --git a/target/hexagon/cpu-param.h b/target/hexagon/cpu-param.h
index 71b4a9b83e..45ee7b4640 100644
--- a/target/hexagon/cpu-param.h
+++ b/target/hexagon/cpu-param.h
@@ -19,7 +19,6 @@
 #define HEXAGON_CPU_PARAM_H
 
 #define TARGET_PAGE_BITS 16 /* 64K pages */
-#define TARGET_LONG_BITS 32
 
 #define TARGET_PHYS_ADDR_SPACE_BITS 36
 #define TARGET_VIRT_ADDR_SPACE_BITS 32
diff --git a/target/hppa/cpu-param.h b/target/hppa/cpu-param.h
index ef3200f0f3..7ed6b5741e 100644
--- a/target/hppa/cpu-param.h
+++ b/target/hppa/cpu-param.h
@@ -8,8 +8,6 @@
 #ifndef HPPA_CPU_PARAM_H
 #define HPPA_CPU_PARAM_H
 
-#define TARGET_LONG_BITS  64
-
 #if defined(CONFIG_USER_ONLY) && defined(TARGET_ABI32)
 # define TARGET_PHYS_ADDR_SPACE_BITS  32
 # define TARGET_VIRT_ADDR_SPACE_BITS  32
diff --git a/target/i386/cpu-param.h b/target/i386/cpu-param.h
index 8c75abe141..b0e884c5d7 100644
--- a/target/i386/cpu-param.h
+++ b/target/i386/cpu-param.h
@@ -9,7 +9,6 @@
 #define I386_CPU_PARAM_H
 
 #ifdef TARGET_X86_64
-# define TARGET_LONG_BITS 64
 # define TARGET_PHYS_ADDR_SPACE_BITS  52
 /*
  * ??? This is really 48 bits, sign-extended, but the only thing
@@ -18,7 +17,6 @@
  */
 # define TARGET_VIRT_ADDR_SPACE_BITS  47
 #else
-# define TARGET_LONG_BITS 32
 # define TARGET_PHYS_ADDR_SPACE_BITS  36
 # define TARGET_VIRT_ADDR_SPACE_BITS  32
 #endif
diff --git a/target/loongarch/cpu-param.h b/target/loongarch/cpu-param.h
index db5ad1c69f..52437946e5 100644
--- a/target/loongarch/cpu-param.h
+++ b/target/loongarch/cpu-param.h
@@ -8,7 +8,6 @@
 #ifndef LOONGARCH_CPU_PARAM_H
 #define LOONGARCH_CPU_PARAM_H
 
-#define TARGET_LONG_BITS 64
 #define TARGET_PHYS_ADDR_SPACE_BITS 48
 #define TARGET_VIRT_ADDR_SPACE_BITS 48
 
diff --git a/target/m68k/cpu-param.h b/target/m68k/cpu-param.h
index 5bbe623ba7..7afbf6d302 100644
--- a/target/m68k/cpu-param.h
+++ b/target/m68k/cpu-param.h
@@ -8,7 +8,6 @@
 #ifndef M68K_CPU_PARAM_H
 #define M68K_CPU_PARAM_H
 
-#define TARGET_LONG_BITS 32
 /*
  * Coldfire Linux uses 8k pages
  * and m68k linux uses 4k pages
diff --git a/target/microblaze/cpu-param.h b/target/microblaze/cpu-param.h
index 00efb509e3..c866ec6c14 100644
--- a/target/microblaze/cpu-param.h
+++ b/target/microblaze/cpu-param.h
@@ -17,11 +17,9 @@
  * of address space.
  */
 #ifdef CONFIG_USER_ONLY
-#define TARGET_LONG_BITS 32
 #define TARGET_PHYS_ADDR_SPACE_BITS 32
 #define TARGET_VIRT_ADDR_SPACE_BITS 32
 #else
-#define TARGET_LONG_BITS 64
 #define TARGET_PHYS_ADDR_SPACE_BITS 64
 #define TARGET_VIRT_ADDR_SPACE_BITS 64
 #endif
diff --git a/target/mips/cpu-param.h b/target/mips/cpu-param.h
index f3a37e2dbe..11b3ac0ac6 100644
--- a/target/mips/cpu-param.h
+++ b/target/mips/cpu-param.h
@@ -7,11 +7,6 @@
 #ifndef MIPS_CPU_PARAM_H
 #define MIPS_CPU_PARAM_H
 
-#ifdef TARGET_MIPS64
-# define TARGET_LONG_BITS 64
-#else
-# define TARGET_LONG_BITS 32
-#endif
 #ifdef TA