tcg/tci: Add TCG_TARGET_DEFAULT_MO

[Qemu-devel] [PATCH] tcg/tci: Add TCG_TARGET_DEFAULT_MO

Richard Henderson posted 1 patch 6 years, 7 months ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170907175430.2720-1-richard.henderson@linaro.org

Test checkpatch passed

Test docker passed

Test s390x passed

tcg/tci/tcg-target.h | 5 +++++
1 file changed, 5 insertions(+)

Expand all Fold all

[Qemu-devel] [PATCH] tcg/tci: Add TCG_TARGET_DEFAULT_MO

Posted by Richard Henderson 6 years, 7 months ago

Missed being added as part of 71650df7b0ee.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 tcg/tci/tcg-target.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/tcg/tci/tcg-target.h b/tcg/tci/tcg-target.h
index 06963288dc..8df628a319 100644
--- a/tcg/tci/tcg-target.h
+++ b/tcg/tci/tcg-target.h
@@ -192,4 +192,9 @@ static inline void flush_icache_range(uintptr_t start, uintptr_t stop)
 {
 }
 
+/* We could notice __i386__ or __s390x__ and reduce the barriers depending
+   on the host.  But if you want performance, you use the normal backend.
+   We prefer consistency across hosts on this.  */
+#define TCG_TARGET_DEFAULT_MO  (0)
+
 #endif /* TCG_TARGET_H */
-- 
2.13.5

Re: [Qemu-devel] [PATCH] tcg/tci: Add TCG_TARGET_DEFAULT_MO

Posted by Peter Maydell 6 years, 7 months ago

On 7 September 2017 at 18:54, Richard Henderson
<richard.henderson@linaro.org> wrote:
> Missed being added as part of 71650df7b0ee.
>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>  tcg/tci/tcg-target.h | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/tcg/tci/tcg-target.h b/tcg/tci/tcg-target.h
> index 06963288dc..8df628a319 100644
> --- a/tcg/tci/tcg-target.h
> +++ b/tcg/tci/tcg-target.h
> @@ -192,4 +192,9 @@ static inline void flush_icache_range(uintptr_t start, uintptr_t stop)
>  {
>  }
>
> +/* We could notice __i386__ or __s390x__ and reduce the barriers depending
> +   on the host.  But if you want performance, you use the normal backend.
> +   We prefer consistency across hosts on this.  */
> +#define TCG_TARGET_DEFAULT_MO  (0)
> +
>  #endif /* TCG_TARGET_H */

Applied to master as a travis build fix, thanks.

-- PMM