1. Patchew
  2. QEMU
  3. [RFC PATCH-for-10.1 0/4] tcg: Move TCG_GUEST_DEFAULT_MO -> TCGCPUOps::guest_default_memory_order
  4. View patch

[PATCH-for-10.1 2/4] tcg: Simplify tcg_req_mo() macro

Philippe Mathieu-Daudé posted 4 patches 1 week, 3 days ago
There is a newer version of this series
[PATCH-for-10.1 2/4] tcg: Simplify tcg_req_mo() macro
Posted by Philippe Mathieu-Daudé 1 week, 3 days ago 
Now that TCG_GUEST_DEFAULT_MO is always defined,
simplify the tcg_req_mo() macro.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 accel/tcg/internal-target.h | 9 +--------
 accel/tcg/tcg-all.c         | 3 ---
 2 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/accel/tcg/internal-target.h b/accel/tcg/internal-target.h
index 2cdf11c905e..1cb35dba99e 100644
--- a/accel/tcg/internal-target.h
+++ b/accel/tcg/internal-target.h
@@ -50,17 +50,10 @@ G_NORETURN void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr);
  * memory ordering vs the host memory ordering.  A non-zero
  * result indicates that some barrier is required.
  *
- * If TCG_GUEST_DEFAULT_MO is not defined, assume that the
- * guest requires strict ordering.
- *
  * This is a macro so that it's constant even without optimization.
  */
-#ifdef TCG_GUEST_DEFAULT_MO
-# define tcg_req_mo(type) \
+#define tcg_req_mo(type) \
     ((type) & TCG_GUEST_DEFAULT_MO & ~TCG_TARGET_DEFAULT_MO)
-#else
-# define tcg_req_mo(type) ((type) & ~TCG_TARGET_DEFAULT_MO)
-#endif
 
 /**
  * cpu_req_mo:
diff --git a/accel/tcg/tcg-all.c b/accel/tcg/tcg-all.c
index c1a30b01219..cb632cc8cc7 100644
--- a/accel/tcg/tcg-all.c
+++ b/accel/tcg/tcg-all.c
@@ -77,9 +77,6 @@ static bool default_mttcg_enabled(void)
         return false;
     }
 #ifdef TARGET_SUPPORTS_MTTCG
-# ifndef TCG_GUEST_DEFAULT_MO
-#  error "TARGET_SUPPORTS_MTTCG without TCG_GUEST_DEFAULT_MO"
-# endif
     return true;
 #else
     return false;
-- 
2.47.1
Re: [PATCH-for-10.1 2/4] tcg: Simplify tcg_req_mo() macro
Posted by Pierrick Bouvier 1 week, 3 days ago 
On 3/21/25 05:57, Philippe Mathieu-Daudé wrote:
> Now that TCG_GUEST_DEFAULT_MO is always defined,
> simplify the tcg_req_mo() macro.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   accel/tcg/internal-target.h | 9 +--------
>   accel/tcg/tcg-all.c         | 3 ---
>   2 files changed, 1 insertion(+), 11 deletions(-)
> 
> diff --git a/accel/tcg/internal-target.h b/accel/tcg/internal-target.h
> index 2cdf11c905e..1cb35dba99e 100644
> --- a/accel/tcg/internal-target.h
> +++ b/accel/tcg/internal-target.h
> @@ -50,17 +50,10 @@ G_NORETURN void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr);
>    * memory ordering vs the host memory ordering.  A non-zero
>    * result indicates that some barrier is required.
>    *
> - * If TCG_GUEST_DEFAULT_MO is not defined, assume that the
> - * guest requires strict ordering.
> - *
>    * This is a macro so that it's constant even without optimization.
>    */
> -#ifdef TCG_GUEST_DEFAULT_MO
> -# define tcg_req_mo(type) \
> +#define tcg_req_mo(type) \
>       ((type) & TCG_GUEST_DEFAULT_MO & ~TCG_TARGET_DEFAULT_MO)
> -#else
> -# define tcg_req_mo(type) ((type) & ~TCG_TARGET_DEFAULT_MO)
> -#endif
>   
>   /**
>    * cpu_req_mo:
> diff --git a/accel/tcg/tcg-all.c b/accel/tcg/tcg-all.c
> index c1a30b01219..cb632cc8cc7 100644
> --- a/accel/tcg/tcg-all.c
> +++ b/accel/tcg/tcg-all.c
> @@ -77,9 +77,6 @@ static bool default_mttcg_enabled(void)
>           return false;
>       }
>   #ifdef TARGET_SUPPORTS_MTTCG
> -# ifndef TCG_GUEST_DEFAULT_MO
> -#  error "TARGET_SUPPORTS_MTTCG without TCG_GUEST_DEFAULT_MO"
> -# endif
>       return true;
>   #else
>       return false;

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Re: [PATCH-for-10.1 2/4] tcg: Simplify tcg_req_mo() macro
Posted by Richard Henderson 1 week, 3 days ago 
On 3/21/25 05:57, Philippe Mathieu-Daudé wrote:
> Now that TCG_GUEST_DEFAULT_MO is always defined,
> simplify the tcg_req_mo() macro.
> 
> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
> ---
>   accel/tcg/internal-target.h | 9 +--------
>   accel/tcg/tcg-all.c         | 3 ---
>   2 files changed, 1 insertion(+), 11 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~
Re: [PATCH-for-10.1 2/4] tcg: Simplify tcg_req_mo() macro
Posted by Anton Johansson via 1 week, 3 days ago 
On 21/03/25, Philippe Mathieu-Daudé wrote:
> Now that TCG_GUEST_DEFAULT_MO is always defined,
> simplify the tcg_req_mo() macro.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>  accel/tcg/internal-target.h | 9 +--------
>  accel/tcg/tcg-all.c         | 3 ---
>  2 files changed, 1 insertion(+), 11 deletions(-)
> 
> diff --git a/accel/tcg/internal-target.h b/accel/tcg/internal-target.h
> index 2cdf11c905e..1cb35dba99e 100644
> --- a/accel/tcg/internal-target.h
> +++ b/accel/tcg/internal-target.h
> @@ -50,17 +50,10 @@ G_NORETURN void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr);
>   * memory ordering vs the host memory ordering.  A non-zero
>   * result indicates that some barrier is required.
>   *
> - * If TCG_GUEST_DEFAULT_MO is not defined, assume that the
> - * guest requires strict ordering.
> - *
>   * This is a macro so that it's constant even without optimization.
>   */
> -#ifdef TCG_GUEST_DEFAULT_MO
> -# define tcg_req_mo(type) \
> +#define tcg_req_mo(type) \
>      ((type) & TCG_GUEST_DEFAULT_MO & ~TCG_TARGET_DEFAULT_MO)
> -#else
> -# define tcg_req_mo(type) ((type) & ~TCG_TARGET_DEFAULT_MO)
> -#endif
>  
>  /**
>   * cpu_req_mo:
> diff --git a/accel/tcg/tcg-all.c b/accel/tcg/tcg-all.c
> index c1a30b01219..cb632cc8cc7 100644
> --- a/accel/tcg/tcg-all.c
> +++ b/accel/tcg/tcg-all.c
> @@ -77,9 +77,6 @@ static bool default_mttcg_enabled(void)
>          return false;
>      }
>  #ifdef TARGET_SUPPORTS_MTTCG
> -# ifndef TCG_GUEST_DEFAULT_MO
> -#  error "TARGET_SUPPORTS_MTTCG without TCG_GUEST_DEFAULT_MO"
> -# endif
>      return true;
>  #else
>      return false;
> -- 
> 2.47.1
> 

Reviewed-by: Anton Johansson <anjo@rev.ng>

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.