[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 13/15] tcg: disable tcg in CPUX86State struct
From: |
Zhong Yang |
Subject: |
Re: [Qemu-devel] [PATCH 13/15] tcg: disable tcg in CPUX86State struct |
Date: |
Thu, 22 Jun 2017 17:32:01 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Jun 21, 2017 at 03:24:29PM -0700, Richard Henderson wrote:
> On 06/21/2017 03:19 AM, Yang Zhong wrote:
> >--- a/target/i386/cpu.h
> >+++ b/target/i386/cpu.h
> >@@ -52,7 +52,9 @@
> > #include "exec/cpu-defs.h"
> >+#ifdef CONFIG_TCG
> > #include "fpu/softfloat.h"
> >+#endif
> > #define R_EAX 0
> > #define R_ECX 1
> >@@ -1130,8 +1132,9 @@ typedef struct CPUX86State {
> > /* Fields up to this point are cleared by a CPU reset */
> > struct {} end_reset_fields;
> >-
> >+#ifdef CONFIG_TCG
> > CPU_COMMON
> >+#endif
>
> This is at the wrong level. If !CONFIG_TCG, then CPU_COMMON should be empty.
>
>
> r~
Hello Richard,
In fact, i can remove this CONFIG_TCG for CPU_COMMON in the CPUX86State struct
because there is one change in the same patch ID.
diff --git a/include/exec/cpu-defs.h b/include/exec/cpu-defs.h index
5f4e303..bc8e7f8 100644
--- a/include/exec/cpu-defs.h
+++ b/include/exec/cpu-defs.h
@@ -25,7 +25,9 @@
#include "qemu/host-utils.h"
#include "qemu/queue.h"
+#ifdef CONFIG_TCG
#include "tcg-target.h"
+#endif
#ifndef CONFIG_USER_ONLY
#include "exec/hwaddr.h"
#endif
@@ -54,7 +56,7 @@ typedef uint64_t target_ulong; #error TARGET_LONG_SIZE
undefined #endif
-#if !defined(CONFIG_USER_ONLY)
+#if !defined(CONFIG_USER_ONLY) && defined(CONFIG_TCG)
/* use a fully associative victim tlb of 8 entries */ #define CPU_VTLB_SIZE 8
if tcg is disabled, CPU_COMMON_TLB is NULL.
#define CPU_COMMON \
/* soft mmu support */ \
CPU_COMMON_TLB \
So the CPU_COMMON should be NULL if the tcg is disabled.
Yang
- [Qemu-devel] [PATCH 09/15] tcg: add the tcg-stub.c file into accel/stubs/, (continued)
- [Qemu-devel] [PATCH 12/15] tcg: remove inline definition of flush_icache_range(), Yang Zhong, 2017/06/21
- [Qemu-devel] [PATCH 13/15] tcg: disable tcg in CPUX86State struct, Yang Zhong, 2017/06/21
- [Qemu-devel] [PATCH 15/15] tcg: add the CONFIG_TCG into Makefiles, Yang Zhong, 2017/06/21
- [Qemu-devel] [PATCH 14/15] tcg: add the CONFIG_TCG for header, Yang Zhong, 2017/06/21
- Re: [Qemu-devel] [PATCH 00/15] add disable-tcg option for x86 build, no-reply, 2017/06/21
- Re: [Qemu-devel] [PATCH 00/15] add disable-tcg option for x86 build, Paolo Bonzini, 2017/06/21