1 | The following changes since commit 2ecfc0657afa5d29a373271b342f704a1a3c6737: | 1 | Pulling together some cleanups, fixes, and prepatory tci stuff. |
---|---|---|---|
2 | Most of this has been reviewed, but not all. | ||
2 | 3 | ||
3 | Merge remote-tracking branch 'remotes/armbru/tags/pull-misc-2020-12-10' into staging (2020-12-10 17:01:05 +0000) | 4 | Those lacking review: |
4 | 5 | ||
5 | are available in the Git repository at: | 6 | 01-tcg-aarch64-Fix-constant-subtraction-in-tcg_out_adds.patch |
7 | 02-tcg-aarch64-Fix-I3617_CMLE0.patch | ||
8 | 03-tcg-aarch64-Fix-generation-of-scalar-vector-operatio.patch | ||
9 | 04-tcg-tci-Use-exec-cpu_ldst.h-interfaces.patch | ||
10 | 06-tcg-Manage-splitwx-in-tc_ptr_to_region_tree-by-hand.patch | ||
11 | 23-accel-tcg-rename-tb_lookup__cpu_state-and-hoist-stat.patch | ||
12 | 24-accel-tcg-move-CF_CLUSTER-calculation-to-curr_cflags.patch | ||
13 | 25-accel-tcg-drop-the-use-of-CF_HASH_MASK-and-rename-pa.patch | ||
14 | 26-include-exec-lightly-re-arrange-TranslationBlock.patch | ||
15 | 27-accel-tcg-Precompute-curr_cflags-into-cpu-tcg_cflags.patch | ||
6 | 16 | ||
7 | https://gitlab.com/rth7680/qemu.git tags/pull-tcg-20201210 | 17 | Alex, the last patch is a re-write and extension of one that |
18 | you did review. | ||
8 | 19 | ||
9 | for you to fetch changes up to 9e2658d62ebc23efe7df43fc0e306f129510d874: | ||
10 | 20 | ||
11 | accel/tcg: rename tcg-cpus functions to match module name (2020-12-10 17:44:10 -0600) | 21 | r~ |
12 | 22 | ||
13 | ---------------------------------------------------------------- | ||
14 | Split CpusAccel for tcg variants | ||
15 | 23 | ||
16 | ---------------------------------------------------------------- | 24 | Alex Bennée (4): |
17 | Claudio Fontana (3): | 25 | accel/tcg: rename tb_lookup__cpu_state and hoist state extraction |
18 | accel/tcg: split CpusAccel into three TCG variants | 26 | accel/tcg: move CF_CLUSTER calculation to curr_cflags |
19 | accel/tcg: split tcg_start_vcpu_thread | 27 | accel/tcg: drop the use of CF_HASH_MASK and rename params |
20 | accel/tcg: rename tcg-cpus functions to match module name | 28 | include/exec: lightly re-arrange TranslationBlock |
21 | 29 | ||
22 | accel/tcg/tcg-cpus-icount.h | 17 ++ | 30 | Richard Henderson (23): |
23 | accel/tcg/tcg-cpus-rr.h | 21 ++ | 31 | tcg/aarch64: Fix constant subtraction in tcg_out_addsub2 |
24 | accel/tcg/tcg-cpus.h | 12 +- | 32 | tcg/aarch64: Fix I3617_CMLE0 |
25 | accel/tcg/tcg-all.c | 13 +- | 33 | tcg/aarch64: Fix generation of "scalar" vector operations |
26 | accel/tcg/tcg-cpus-icount.c | 147 +++++++++++++ | 34 | tcg/tci: Use exec/cpu_ldst.h interfaces |
27 | accel/tcg/tcg-cpus-mttcg.c | 140 ++++++++++++ | 35 | tcg: Split out tcg_raise_tb_overflow |
28 | accel/tcg/tcg-cpus-rr.c | 305 ++++++++++++++++++++++++++ | 36 | tcg: Manage splitwx in tc_ptr_to_region_tree by hand |
29 | accel/tcg/tcg-cpus.c | 506 +------------------------------------------- | 37 | tcg/tci: Merge identical cases in generation (arithmetic opcodes) |
30 | softmmu/icount.c | 2 +- | 38 | tcg/tci: Merge identical cases in generation (exchange opcodes) |
31 | accel/tcg/meson.build | 9 +- | 39 | tcg/tci: Merge identical cases in generation (deposit opcode) |
32 | 10 files changed, 670 insertions(+), 502 deletions(-) | 40 | tcg/tci: Merge identical cases in generation (conditional opcodes) |
33 | create mode 100644 accel/tcg/tcg-cpus-icount.h | 41 | tcg/tci: Merge identical cases in generation (load/store opcodes) |
34 | create mode 100644 accel/tcg/tcg-cpus-rr.h | 42 | tcg/tci: Remove tci_read_r8 |
35 | create mode 100644 accel/tcg/tcg-cpus-icount.c | 43 | tcg/tci: Remove tci_read_r8s |
36 | create mode 100644 accel/tcg/tcg-cpus-mttcg.c | 44 | tcg/tci: Remove tci_read_r16 |
37 | create mode 100644 accel/tcg/tcg-cpus-rr.c | 45 | tcg/tci: Remove tci_read_r16s |
46 | tcg/tci: Remove tci_read_r32 | ||
47 | tcg/tci: Remove tci_read_r32s | ||
48 | tcg/tci: Reduce use of tci_read_r64 | ||
49 | tcg/tci: Merge basic arithmetic operations | ||
50 | tcg/tci: Merge extension operations | ||
51 | tcg/tci: Merge bswap operations | ||
52 | tcg/tci: Merge mov, not and neg operations | ||
53 | accel/tcg: Precompute curr_cflags into cpu->tcg_cflags | ||
38 | 54 | ||
55 | accel/tcg/tcg-accel-ops.h | 1 + | ||
56 | include/exec/exec-all.h | 19 +- | ||
57 | include/exec/tb-lookup.h | 26 +- | ||
58 | include/hw/core/cpu.h | 2 + | ||
59 | accel/tcg/cpu-exec.c | 34 ++- | ||
60 | accel/tcg/tcg-accel-ops-mttcg.c | 3 +- | ||
61 | accel/tcg/tcg-accel-ops-rr.c | 2 +- | ||
62 | accel/tcg/tcg-accel-ops.c | 8 + | ||
63 | accel/tcg/tcg-runtime.c | 6 +- | ||
64 | accel/tcg/translate-all.c | 18 +- | ||
65 | linux-user/main.c | 1 + | ||
66 | linux-user/sh4/signal.c | 8 +- | ||
67 | linux-user/syscall.c | 18 +- | ||
68 | softmmu/physmem.c | 2 +- | ||
69 | tcg/tcg.c | 29 +- | ||
70 | tcg/tci.c | 526 ++++++++++---------------------- | ||
71 | tcg/aarch64/tcg-target.c.inc | 229 +++++++++++--- | ||
72 | tcg/tci/tcg-target.c.inc | 204 +++++-------- | ||
73 | 18 files changed, 526 insertions(+), 610 deletions(-) | ||
74 | |||
75 | -- | ||
76 | 2.25.1 | ||
77 | |||
78 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | An hppa guest executing | ||
1 | 2 | ||
3 | 0x000000000000e05c: ldil L%10000,r4 | ||
4 | 0x000000000000e060: ldo 0(r4),r4 | ||
5 | 0x000000000000e064: sub r3,r4,sp | ||
6 | |||
7 | produces | ||
8 | |||
9 | ---- 000000000000e064 000000000000e068 | ||
10 | sub2_i32 tmp0,tmp4,r3,$0x1,$0x10000,$0x0 | ||
11 | |||
12 | after folding and constant propagation. Then we hit | ||
13 | |||
14 | tcg-target.c.inc:640: tcg_out_insn_3401: Assertion `aimm <= 0xfff' failed. | ||
15 | |||
16 | because aimm is in fact -16, but unsigned. | ||
17 | |||
18 | The ((bl < 0) ^ sub) condition which negates bl is incorrect and will | ||
19 | always lead to this abort. If the constant is positive, sub will make | ||
20 | it negative; if the constant is negative, sub will keep it negative. | ||
21 | |||
22 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
23 | --- | ||
24 | tcg/aarch64/tcg-target.c.inc | 16 +++++++++------- | ||
25 | 1 file changed, 9 insertions(+), 7 deletions(-) | ||
26 | |||
27 | diff --git a/tcg/aarch64/tcg-target.c.inc b/tcg/aarch64/tcg-target.c.inc | ||
28 | index XXXXXXX..XXXXXXX 100644 | ||
29 | --- a/tcg/aarch64/tcg-target.c.inc | ||
30 | +++ b/tcg/aarch64/tcg-target.c.inc | ||
31 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_addsubi(TCGContext *s, int ext, TCGReg rd, | ||
32 | } | ||
33 | } | ||
34 | |||
35 | -static inline void tcg_out_addsub2(TCGContext *s, TCGType ext, TCGReg rl, | ||
36 | - TCGReg rh, TCGReg al, TCGReg ah, | ||
37 | - tcg_target_long bl, tcg_target_long bh, | ||
38 | - bool const_bl, bool const_bh, bool sub) | ||
39 | +static void tcg_out_addsub2(TCGContext *s, TCGType ext, TCGReg rl, | ||
40 | + TCGReg rh, TCGReg al, TCGReg ah, | ||
41 | + tcg_target_long bl, tcg_target_long bh, | ||
42 | + bool const_bl, bool const_bh, bool sub) | ||
43 | { | ||
44 | TCGReg orig_rl = rl; | ||
45 | AArch64Insn insn; | ||
46 | @@ -XXX,XX +XXX,XX @@ static inline void tcg_out_addsub2(TCGContext *s, TCGType ext, TCGReg rl, | ||
47 | } | ||
48 | |||
49 | if (const_bl) { | ||
50 | - insn = I3401_ADDSI; | ||
51 | - if ((bl < 0) ^ sub) { | ||
52 | - insn = I3401_SUBSI; | ||
53 | + if (bl < 0) { | ||
54 | bl = -bl; | ||
55 | + insn = sub ? I3401_ADDSI : I3401_SUBSI; | ||
56 | + } else { | ||
57 | + insn = sub ? I3401_SUBSI : I3401_ADDSI; | ||
58 | } | ||
59 | + | ||
60 | if (unlikely(al == TCG_REG_XZR)) { | ||
61 | /* ??? We want to allow al to be zero for the benefit of | ||
62 | negation via subtraction. However, that leaves open the | ||
63 | -- | ||
64 | 2.25.1 | ||
65 | |||
66 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Fix a typo in the encodeing of the cmle (zero) instruction. | ||
1 | 2 | ||
3 | Fixes: 14e4c1e2355 ("tcg/aarch64: Add vector operations") | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | --- | ||
6 | tcg/aarch64/tcg-target.c.inc | 2 +- | ||
7 | 1 file changed, 1 insertion(+), 1 deletion(-) | ||
8 | |||
9 | diff --git a/tcg/aarch64/tcg-target.c.inc b/tcg/aarch64/tcg-target.c.inc | ||
10 | index XXXXXXX..XXXXXXX 100644 | ||
11 | --- a/tcg/aarch64/tcg-target.c.inc | ||
12 | +++ b/tcg/aarch64/tcg-target.c.inc | ||
13 | @@ -XXX,XX +XXX,XX @@ typedef enum { | ||
14 | I3617_CMEQ0 = 0x0e209800, | ||
15 | I3617_CMLT0 = 0x0e20a800, | ||
16 | I3617_CMGE0 = 0x2e208800, | ||
17 | - I3617_CMLE0 = 0x2e20a800, | ||
18 | + I3617_CMLE0 = 0x2e209800, | ||
19 | I3617_NOT = 0x2e205800, | ||
20 | I3617_ABS = 0x0e20b800, | ||
21 | I3617_NEG = 0x2e20b800, | ||
22 | -- | ||
23 | 2.25.1 | ||
24 | |||
25 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | For some vector operations, "1D" is not a valid type, and there | ||
2 | are separate instructions for the 64-bit scalar operation. | ||
1 | 3 | ||
4 | Tested-by: Stefan Weil <sw@weilnetz.de> | ||
5 | Buglink: https://bugs.launchpad.net/qemu/+bug/1916112 | ||
6 | Fixes: 14e4c1e2355 ("tcg/aarch64: Add vector operations") | ||
7 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
8 | --- | ||
9 | tcg/aarch64/tcg-target.c.inc | 211 ++++++++++++++++++++++++++++++----- | ||
10 | 1 file changed, 181 insertions(+), 30 deletions(-) | ||
11 | |||
12 | diff --git a/tcg/aarch64/tcg-target.c.inc b/tcg/aarch64/tcg-target.c.inc | ||
13 | index XXXXXXX..XXXXXXX 100644 | ||
14 | --- a/tcg/aarch64/tcg-target.c.inc | ||
15 | +++ b/tcg/aarch64/tcg-target.c.inc | ||
16 | @@ -XXX,XX +XXX,XX @@ typedef enum { | ||
17 | I3606_BIC = 0x2f001400, | ||
18 | I3606_ORR = 0x0f001400, | ||
19 | |||
20 | + /* AdvSIMD scalar shift by immediate */ | ||
21 | + I3609_SSHR = 0x5f000400, | ||
22 | + I3609_SSRA = 0x5f001400, | ||
23 | + I3609_SHL = 0x5f005400, | ||
24 | + I3609_USHR = 0x7f000400, | ||
25 | + I3609_USRA = 0x7f001400, | ||
26 | + I3609_SLI = 0x7f005400, | ||
27 | + | ||
28 | + /* AdvSIMD scalar three same */ | ||
29 | + I3611_SQADD = 0x5e200c00, | ||
30 | + I3611_SQSUB = 0x5e202c00, | ||
31 | + I3611_CMGT = 0x5e203400, | ||
32 | + I3611_CMGE = 0x5e203c00, | ||
33 | + I3611_SSHL = 0x5e204400, | ||
34 | + I3611_ADD = 0x5e208400, | ||
35 | + I3611_CMTST = 0x5e208c00, | ||
36 | + I3611_UQADD = 0x7e200c00, | ||
37 | + I3611_UQSUB = 0x7e202c00, | ||
38 | + I3611_CMHI = 0x7e203400, | ||
39 | + I3611_CMHS = 0x7e203c00, | ||
40 | + I3611_USHL = 0x7e204400, | ||
41 | + I3611_SUB = 0x7e208400, | ||
42 | + I3611_CMEQ = 0x7e208c00, | ||
43 | + | ||
44 | + /* AdvSIMD scalar two-reg misc */ | ||
45 | + I3612_CMGT0 = 0x5e208800, | ||
46 | + I3612_CMEQ0 = 0x5e209800, | ||
47 | + I3612_CMLT0 = 0x5e20a800, | ||
48 | + I3612_ABS = 0x5e20b800, | ||
49 | + I3612_CMGE0 = 0x7e208800, | ||
50 | + I3612_CMLE0 = 0x7e209800, | ||
51 | + I3612_NEG = 0x7e20b800, | ||
52 | + | ||
53 | /* AdvSIMD shift by immediate */ | ||
54 | I3614_SSHR = 0x0f000400, | ||
55 | I3614_SSRA = 0x0f001400, | ||
56 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_insn_3606(TCGContext *s, AArch64Insn insn, bool q, | ||
57 | | (imm8 & 0xe0) << (16 - 5) | (imm8 & 0x1f) << 5); | ||
58 | } | ||
59 | |||
60 | +static void tcg_out_insn_3609(TCGContext *s, AArch64Insn insn, | ||
61 | + TCGReg rd, TCGReg rn, unsigned immhb) | ||
62 | +{ | ||
63 | + tcg_out32(s, insn | immhb << 16 | (rn & 0x1f) << 5 | (rd & 0x1f)); | ||
64 | +} | ||
65 | + | ||
66 | +static void tcg_out_insn_3611(TCGContext *s, AArch64Insn insn, | ||
67 | + unsigned size, TCGReg rd, TCGReg rn, TCGReg rm) | ||
68 | +{ | ||
69 | + tcg_out32(s, insn | (size << 22) | (rm & 0x1f) << 16 | ||
70 | + | (rn & 0x1f) << 5 | (rd & 0x1f)); | ||
71 | +} | ||
72 | + | ||
73 | +static void tcg_out_insn_3612(TCGContext *s, AArch64Insn insn, | ||
74 | + unsigned size, TCGReg rd, TCGReg rn) | ||
75 | +{ | ||
76 | + tcg_out32(s, insn | (size << 22) | (rn & 0x1f) << 5 | (rd & 0x1f)); | ||
77 | +} | ||
78 | + | ||
79 | static void tcg_out_insn_3614(TCGContext *s, AArch64Insn insn, bool q, | ||
80 | TCGReg rd, TCGReg rn, unsigned immhb) | ||
81 | { | ||
82 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_vec_op(TCGContext *s, TCGOpcode opc, | ||
83 | unsigned vecl, unsigned vece, | ||
84 | const TCGArg *args, const int *const_args) | ||
85 | { | ||
86 | - static const AArch64Insn cmp_insn[16] = { | ||
87 | + static const AArch64Insn cmp_vec_insn[16] = { | ||
88 | [TCG_COND_EQ] = I3616_CMEQ, | ||
89 | [TCG_COND_GT] = I3616_CMGT, | ||
90 | [TCG_COND_GE] = I3616_CMGE, | ||
91 | [TCG_COND_GTU] = I3616_CMHI, | ||
92 | [TCG_COND_GEU] = I3616_CMHS, | ||
93 | }; | ||
94 | - static const AArch64Insn cmp0_insn[16] = { | ||
95 | + static const AArch64Insn cmp_scalar_insn[16] = { | ||
96 | + [TCG_COND_EQ] = I3611_CMEQ, | ||
97 | + [TCG_COND_GT] = I3611_CMGT, | ||
98 | + [TCG_COND_GE] = I3611_CMGE, | ||
99 | + [TCG_COND_GTU] = I3611_CMHI, | ||
100 | + [TCG_COND_GEU] = I3611_CMHS, | ||
101 | + }; | ||
102 | + static const AArch64Insn cmp0_vec_insn[16] = { | ||
103 | [TCG_COND_EQ] = I3617_CMEQ0, | ||
104 | [TCG_COND_GT] = I3617_CMGT0, | ||
105 | [TCG_COND_GE] = I3617_CMGE0, | ||
106 | [TCG_COND_LT] = I3617_CMLT0, | ||
107 | [TCG_COND_LE] = I3617_CMLE0, | ||
108 | }; | ||
109 | + static const AArch64Insn cmp0_scalar_insn[16] = { | ||
110 | + [TCG_COND_EQ] = I3612_CMEQ0, | ||
111 | + [TCG_COND_GT] = I3612_CMGT0, | ||
112 | + [TCG_COND_GE] = I3612_CMGE0, | ||
113 | + [TCG_COND_LT] = I3612_CMLT0, | ||
114 | + [TCG_COND_LE] = I3612_CMLE0, | ||
115 | + }; | ||
116 | |||
117 | TCGType type = vecl + TCG_TYPE_V64; | ||
118 | unsigned is_q = vecl; | ||
119 | + bool is_scalar = !is_q && vece == MO_64; | ||
120 | TCGArg a0, a1, a2, a3; | ||
121 | int cmode, imm8; | ||
122 | |||
123 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_vec_op(TCGContext *s, TCGOpcode opc, | ||
124 | tcg_out_dupm_vec(s, type, vece, a0, a1, a2); | ||
125 | break; | ||
126 | case INDEX_op_add_vec: | ||
127 | - tcg_out_insn(s, 3616, ADD, is_q, vece, a0, a1, a2); | ||
128 | + if (is_scalar) { | ||
129 | + tcg_out_insn(s, 3611, ADD, vece, a0, a1, a2); | ||
130 | + } else { | ||
131 | + tcg_out_insn(s, 3616, ADD, is_q, vece, a0, a1, a2); | ||
132 | + } | ||
133 | break; | ||
134 | case INDEX_op_sub_vec: | ||
135 | - tcg_out_insn(s, 3616, SUB, is_q, vece, a0, a1, a2); | ||
136 | + if (is_scalar) { | ||
137 | + tcg_out_insn(s, 3611, SUB, vece, a0, a1, a2); | ||
138 | + } else { | ||
139 | + tcg_out_insn(s, 3616, SUB, is_q, vece, a0, a1, a2); | ||
140 | + } | ||
141 | break; | ||
142 | case INDEX_op_mul_vec: | ||
143 | tcg_out_insn(s, 3616, MUL, is_q, vece, a0, a1, a2); | ||
144 | break; | ||
145 | case INDEX_op_neg_vec: | ||
146 | - tcg_out_insn(s, 3617, NEG, is_q, vece, a0, a1); | ||
147 | + if (is_scalar) { | ||
148 | + tcg_out_insn(s, 3612, NEG, vece, a0, a1); | ||
149 | + } else { | ||
150 | + tcg_out_insn(s, 3617, NEG, is_q, vece, a0, a1); | ||
151 | + } | ||
152 | break; | ||
153 | case INDEX_op_abs_vec: | ||
154 | - tcg_out_insn(s, 3617, ABS, is_q, vece, a0, a1); | ||
155 | + if (is_scalar) { | ||
156 | + tcg_out_insn(s, 3612, ABS, vece, a0, a1); | ||
157 | + } else { | ||
158 | + tcg_out_insn(s, 3617, ABS, is_q, vece, a0, a1); | ||
159 | + } | ||
160 | break; | ||
161 | case INDEX_op_and_vec: | ||
162 | if (const_args[2]) { | ||
163 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_vec_op(TCGContext *s, TCGOpcode opc, | ||
164 | tcg_out_insn(s, 3616, EOR, is_q, 0, a0, a1, a2); | ||
165 | break; | ||
166 | case INDEX_op_ssadd_vec: | ||
167 | - tcg_out_insn(s, 3616, SQADD, is_q, vece, a0, a1, a2); | ||
168 | + if (is_scalar) { | ||
169 | + tcg_out_insn(s, 3611, SQADD, vece, a0, a1, a2); | ||
170 | + } else { | ||
171 | + tcg_out_insn(s, 3616, SQADD, is_q, vece, a0, a1, a2); | ||
172 | + } | ||
173 | break; | ||
174 | case INDEX_op_sssub_vec: | ||
175 | - tcg_out_insn(s, 3616, SQSUB, is_q, vece, a0, a1, a2); | ||
176 | + if (is_scalar) { | ||
177 | + tcg_out_insn(s, 3611, SQSUB, vece, a0, a1, a2); | ||
178 | + } else { | ||
179 | + tcg_out_insn(s, 3616, SQSUB, is_q, vece, a0, a1, a2); | ||
180 | + } | ||
181 | break; | ||
182 | case INDEX_op_usadd_vec: | ||
183 | - tcg_out_insn(s, 3616, UQADD, is_q, vece, a0, a1, a2); | ||
184 | + if (is_scalar) { | ||
185 | + tcg_out_insn(s, 3611, UQADD, vece, a0, a1, a2); | ||
186 | + } else { | ||
187 | + tcg_out_insn(s, 3616, UQADD, is_q, vece, a0, a1, a2); | ||
188 | + } | ||
189 | break; | ||
190 | case INDEX_op_ussub_vec: | ||
191 | - tcg_out_insn(s, 3616, UQSUB, is_q, vece, a0, a1, a2); | ||
192 | + if (is_scalar) { | ||
193 | + tcg_out_insn(s, 3611, UQSUB, vece, a0, a1, a2); | ||
194 | + } else { | ||
195 | + tcg_out_insn(s, 3616, UQSUB, is_q, vece, a0, a1, a2); | ||
196 | + } | ||
197 | break; | ||
198 | case INDEX_op_smax_vec: | ||
199 | tcg_out_insn(s, 3616, SMAX, is_q, vece, a0, a1, a2); | ||
200 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_vec_op(TCGContext *s, TCGOpcode opc, | ||
201 | tcg_out_insn(s, 3617, NOT, is_q, 0, a0, a1); | ||
202 | break; | ||
203 | case INDEX_op_shli_vec: | ||
204 | - tcg_out_insn(s, 3614, SHL, is_q, a0, a1, a2 + (8 << vece)); | ||
205 | + if (is_scalar) { | ||
206 | + tcg_out_insn(s, 3609, SHL, a0, a1, a2 + (8 << vece)); | ||
207 | + } else { | ||
208 | + tcg_out_insn(s, 3614, SHL, is_q, a0, a1, a2 + (8 << vece)); | ||
209 | + } | ||
210 | break; | ||
211 | case INDEX_op_shri_vec: | ||
212 | - tcg_out_insn(s, 3614, USHR, is_q, a0, a1, (16 << vece) - a2); | ||
213 | + if (is_scalar) { | ||
214 | + tcg_out_insn(s, 3609, USHR, a0, a1, (16 << vece) - a2); | ||
215 | + } else { | ||
216 | + tcg_out_insn(s, 3614, USHR, is_q, a0, a1, (16 << vece) - a2); | ||
217 | + } | ||
218 | break; | ||
219 | case INDEX_op_sari_vec: | ||
220 | - tcg_out_insn(s, 3614, SSHR, is_q, a0, a1, (16 << vece) - a2); | ||
221 | + if (is_scalar) { | ||
222 | + tcg_out_insn(s, 3609, SSHR, a0, a1, (16 << vece) - a2); | ||
223 | + } else { | ||
224 | + tcg_out_insn(s, 3614, SSHR, is_q, a0, a1, (16 << vece) - a2); | ||
225 | + } | ||
226 | break; | ||
227 | case INDEX_op_aa64_sli_vec: | ||
228 | - tcg_out_insn(s, 3614, SLI, is_q, a0, a2, args[3] + (8 << vece)); | ||
229 | + if (is_scalar) { | ||
230 | + tcg_out_insn(s, 3609, SLI, a0, a2, args[3] + (8 << vece)); | ||
231 | + } else { | ||
232 | + tcg_out_insn(s, 3614, SLI, is_q, a0, a2, args[3] + (8 << vece)); | ||
233 | + } | ||
234 | break; | ||
235 | case INDEX_op_shlv_vec: | ||
236 | - tcg_out_insn(s, 3616, USHL, is_q, vece, a0, a1, a2); | ||
237 | + if (is_scalar) { | ||
238 | + tcg_out_insn(s, 3611, USHL, vece, a0, a1, a2); | ||
239 | + } else { | ||
240 | + tcg_out_insn(s, 3616, USHL, is_q, vece, a0, a1, a2); | ||
241 | + } | ||
242 | break; | ||
243 | case INDEX_op_aa64_sshl_vec: | ||
244 | - tcg_out_insn(s, 3616, SSHL, is_q, vece, a0, a1, a2); | ||
245 | + if (is_scalar) { | ||
246 | + tcg_out_insn(s, 3611, SSHL, vece, a0, a1, a2); | ||
247 | + } else { | ||
248 | + tcg_out_insn(s, 3616, SSHL, is_q, vece, a0, a1, a2); | ||
249 | + } | ||
250 | break; | ||
251 | case INDEX_op_cmp_vec: | ||
252 | { | ||
253 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_vec_op(TCGContext *s, TCGOpcode opc, | ||
254 | |||
255 | if (cond == TCG_COND_NE) { | ||
256 | if (const_args[2]) { | ||
257 | - tcg_out_insn(s, 3616, CMTST, is_q, vece, a0, a1, a1); | ||
258 | + if (is_scalar) { | ||
259 | + tcg_out_insn(s, 3611, CMTST, vece, a0, a1, a1); | ||
260 | + } else { | ||
261 | + tcg_out_insn(s, 3616, CMTST, is_q, vece, a0, a1, a1); | ||
262 | + } | ||
263 | } else { | ||
264 | - tcg_out_insn(s, 3616, CMEQ, is_q, vece, a0, a1, a2); | ||
265 | + if (is_scalar) { | ||
266 | + tcg_out_insn(s, 3611, CMEQ, vece, a0, a1, a2); | ||
267 | + } else { | ||
268 | + tcg_out_insn(s, 3616, CMEQ, is_q, vece, a0, a1, a2); | ||
269 | + } | ||
270 | tcg_out_insn(s, 3617, NOT, is_q, 0, a0, a0); | ||
271 | } | ||
272 | } else { | ||
273 | if (const_args[2]) { | ||
274 | - insn = cmp0_insn[cond]; | ||
275 | - if (insn) { | ||
276 | - tcg_out_insn_3617(s, insn, is_q, vece, a0, a1); | ||
277 | - break; | ||
278 | + if (is_scalar) { | ||
279 | + insn = cmp0_scalar_insn[cond]; | ||
280 | + if (insn) { | ||
281 | + tcg_out_insn_3612(s, insn, vece, a0, a1); | ||
282 | + break; | ||
283 | + } | ||
284 | + } else { | ||
285 | + insn = cmp0_vec_insn[cond]; | ||
286 | + if (insn) { | ||
287 | + tcg_out_insn_3617(s, insn, is_q, vece, a0, a1); | ||
288 | + break; | ||
289 | + } | ||
290 | } | ||
291 | tcg_out_dupi_vec(s, type, MO_8, TCG_VEC_TMP, 0); | ||
292 | a2 = TCG_VEC_TMP; | ||
293 | } | ||
294 | - insn = cmp_insn[cond]; | ||
295 | - if (insn == 0) { | ||
296 | - TCGArg t; | ||
297 | - t = a1, a1 = a2, a2 = t; | ||
298 | - cond = tcg_swap_cond(cond); | ||
299 | - insn = cmp_insn[cond]; | ||
300 | - tcg_debug_assert(insn != 0); | ||
301 | + if (is_scalar) { | ||
302 | + insn = cmp_scalar_insn[cond]; | ||
303 | + if (insn == 0) { | ||
304 | + TCGArg t; | ||
305 | + t = a1, a1 = a2, a2 = t; | ||
306 | + cond = tcg_swap_cond(cond); | ||
307 | + insn = cmp_scalar_insn[cond]; | ||
308 | + tcg_debug_assert(insn != 0); | ||
309 | + } | ||
310 | + tcg_out_insn_3611(s, insn, vece, a0, a1, a2); | ||
311 | + } else { | ||
312 | + insn = cmp_vec_insn[cond]; | ||
313 | + if (insn == 0) { | ||
314 | + TCGArg t; | ||
315 | + t = a1, a1 = a2, a2 = t; | ||
316 | + cond = tcg_swap_cond(cond); | ||
317 | + insn = cmp_vec_insn[cond]; | ||
318 | + tcg_debug_assert(insn != 0); | ||
319 | + } | ||
320 | + tcg_out_insn_3616(s, insn, is_q, vece, a0, a1, a2); | ||
321 | } | ||
322 | - tcg_out_insn_3616(s, insn, is_q, vece, a0, a1, a2); | ||
323 | } | ||
324 | } | ||
325 | break; | ||
326 | -- | ||
327 | 2.25.1 | ||
328 | |||
329 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use the provided cpu_ldst.h interfaces. This fixes the build vs | ||
2 | the unconverted uses of g2h(), adds missed memory trace events, | ||
3 | and correctly recognizes when a SIGSEGV belongs to the guest via | ||
4 | set_helper_retaddr(). | ||
1 | 5 | ||
6 | Fixes: 3e8f1628e864 | ||
7 | Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
8 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
9 | --- | ||
10 | tcg/tci.c | 73 +++++++++++++++++++++---------------------------------- | ||
11 | 1 file changed, 28 insertions(+), 45 deletions(-) | ||
12 | |||
13 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
14 | index XXXXXXX..XXXXXXX 100644 | ||
15 | --- a/tcg/tci.c | ||
16 | +++ b/tcg/tci.c | ||
17 | @@ -XXX,XX +XXX,XX @@ static bool tci_compare64(uint64_t u0, uint64_t u1, TCGCond condition) | ||
18 | return result; | ||
19 | } | ||
20 | |||
21 | -#ifdef CONFIG_SOFTMMU | ||
22 | -# define qemu_ld_ub \ | ||
23 | - helper_ret_ldub_mmu(env, taddr, oi, (uintptr_t)tb_ptr) | ||
24 | -# define qemu_ld_leuw \ | ||
25 | - helper_le_lduw_mmu(env, taddr, oi, (uintptr_t)tb_ptr) | ||
26 | -# define qemu_ld_leul \ | ||
27 | - helper_le_ldul_mmu(env, taddr, oi, (uintptr_t)tb_ptr) | ||
28 | -# define qemu_ld_leq \ | ||
29 | - helper_le_ldq_mmu(env, taddr, oi, (uintptr_t)tb_ptr) | ||
30 | -# define qemu_ld_beuw \ | ||
31 | - helper_be_lduw_mmu(env, taddr, oi, (uintptr_t)tb_ptr) | ||
32 | -# define qemu_ld_beul \ | ||
33 | - helper_be_ldul_mmu(env, taddr, oi, (uintptr_t)tb_ptr) | ||
34 | -# define qemu_ld_beq \ | ||
35 | - helper_be_ldq_mmu(env, taddr, oi, (uintptr_t)tb_ptr) | ||
36 | -# define qemu_st_b(X) \ | ||
37 | - helper_ret_stb_mmu(env, taddr, X, oi, (uintptr_t)tb_ptr) | ||
38 | -# define qemu_st_lew(X) \ | ||
39 | - helper_le_stw_mmu(env, taddr, X, oi, (uintptr_t)tb_ptr) | ||
40 | -# define qemu_st_lel(X) \ | ||
41 | - helper_le_stl_mmu(env, taddr, X, oi, (uintptr_t)tb_ptr) | ||
42 | -# define qemu_st_leq(X) \ | ||
43 | - helper_le_stq_mmu(env, taddr, X, oi, (uintptr_t)tb_ptr) | ||
44 | -# define qemu_st_bew(X) \ | ||
45 | - helper_be_stw_mmu(env, taddr, X, oi, (uintptr_t)tb_ptr) | ||
46 | -# define qemu_st_bel(X) \ | ||
47 | - helper_be_stl_mmu(env, taddr, X, oi, (uintptr_t)tb_ptr) | ||
48 | -# define qemu_st_beq(X) \ | ||
49 | - helper_be_stq_mmu(env, taddr, X, oi, (uintptr_t)tb_ptr) | ||
50 | -#else | ||
51 | -# define qemu_ld_ub ldub_p(g2h(taddr)) | ||
52 | -# define qemu_ld_leuw lduw_le_p(g2h(taddr)) | ||
53 | -# define qemu_ld_leul (uint32_t)ldl_le_p(g2h(taddr)) | ||
54 | -# define qemu_ld_leq ldq_le_p(g2h(taddr)) | ||
55 | -# define qemu_ld_beuw lduw_be_p(g2h(taddr)) | ||
56 | -# define qemu_ld_beul (uint32_t)ldl_be_p(g2h(taddr)) | ||
57 | -# define qemu_ld_beq ldq_be_p(g2h(taddr)) | ||
58 | -# define qemu_st_b(X) stb_p(g2h(taddr), X) | ||
59 | -# define qemu_st_lew(X) stw_le_p(g2h(taddr), X) | ||
60 | -# define qemu_st_lel(X) stl_le_p(g2h(taddr), X) | ||
61 | -# define qemu_st_leq(X) stq_le_p(g2h(taddr), X) | ||
62 | -# define qemu_st_bew(X) stw_be_p(g2h(taddr), X) | ||
63 | -# define qemu_st_bel(X) stl_be_p(g2h(taddr), X) | ||
64 | -# define qemu_st_beq(X) stq_be_p(g2h(taddr), X) | ||
65 | -#endif | ||
66 | +#define qemu_ld_ub \ | ||
67 | + cpu_ldub_mmuidx_ra(env, taddr, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
68 | +#define qemu_ld_leuw \ | ||
69 | + cpu_lduw_le_mmuidx_ra(env, taddr, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
70 | +#define qemu_ld_leul \ | ||
71 | + cpu_ldl_le_mmuidx_ra(env, taddr, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
72 | +#define qemu_ld_leq \ | ||
73 | + cpu_ldq_le_mmuidx_ra(env, taddr, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
74 | +#define qemu_ld_beuw \ | ||
75 | + cpu_lduw_be_mmuidx_ra(env, taddr, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
76 | +#define qemu_ld_beul \ | ||
77 | + cpu_ldl_be_mmuidx_ra(env, taddr, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
78 | +#define qemu_ld_beq \ | ||
79 | + cpu_ldq_be_mmuidx_ra(env, taddr, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
80 | +#define qemu_st_b(X) \ | ||
81 | + cpu_stb_mmuidx_ra(env, taddr, X, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
82 | +#define qemu_st_lew(X) \ | ||
83 | + cpu_stw_le_mmuidx_ra(env, taddr, X, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
84 | +#define qemu_st_lel(X) \ | ||
85 | + cpu_stl_le_mmuidx_ra(env, taddr, X, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
86 | +#define qemu_st_leq(X) \ | ||
87 | + cpu_stq_le_mmuidx_ra(env, taddr, X, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
88 | +#define qemu_st_bew(X) \ | ||
89 | + cpu_stw_be_mmuidx_ra(env, taddr, X, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
90 | +#define qemu_st_bel(X) \ | ||
91 | + cpu_stl_be_mmuidx_ra(env, taddr, X, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
92 | +#define qemu_st_beq(X) \ | ||
93 | + cpu_stq_be_mmuidx_ra(env, taddr, X, get_mmuidx(oi), (uintptr_t)tb_ptr) | ||
94 | |||
95 | #if TCG_TARGET_REG_BITS == 64 | ||
96 | # define CASE_32_64(x) \ | ||
97 | -- | ||
98 | 2.25.1 | ||
99 | |||
100 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Allow other places in tcg to restart with a smaller tb. | ||
1 | 2 | ||
3 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | --- | ||
6 | tcg/tcg.c | 9 +++++++-- | ||
7 | 1 file changed, 7 insertions(+), 2 deletions(-) | ||
8 | |||
9 | diff --git a/tcg/tcg.c b/tcg/tcg.c | ||
10 | index XXXXXXX..XXXXXXX 100644 | ||
11 | --- a/tcg/tcg.c | ||
12 | +++ b/tcg/tcg.c | ||
13 | @@ -XXX,XX +XXX,XX @@ static void set_jmp_reset_offset(TCGContext *s, int which) | ||
14 | s->tb_jmp_reset_offset[which] = tcg_current_code_size(s); | ||
15 | } | ||
16 | |||
17 | +/* Signal overflow, starting over with fewer guest insns. */ | ||
18 | +static void QEMU_NORETURN tcg_raise_tb_overflow(TCGContext *s) | ||
19 | +{ | ||
20 | + siglongjmp(s->jmp_trans, -2); | ||
21 | +} | ||
22 | + | ||
23 | #define C_PFX1(P, A) P##A | ||
24 | #define C_PFX2(P, A, B) P##A##_##B | ||
25 | #define C_PFX3(P, A, B, C) P##A##_##B##_##C | ||
26 | @@ -XXX,XX +XXX,XX @@ static TCGTemp *tcg_temp_alloc(TCGContext *s) | ||
27 | int n = s->nb_temps++; | ||
28 | |||
29 | if (n >= TCG_MAX_TEMPS) { | ||
30 | - /* Signal overflow, starting over with fewer guest insns. */ | ||
31 | - siglongjmp(s->jmp_trans, -2); | ||
32 | + tcg_raise_tb_overflow(s); | ||
33 | } | ||
34 | return memset(&s->temps[n], 0, sizeof(TCGTemp)); | ||
35 | } | ||
36 | -- | ||
37 | 2.25.1 | ||
38 | |||
39 | diff view generated by jsdifflib |
1 | From: Claudio Fontana <cfontana@suse.de> | 1 | The use in tcg_tb_lookup is given a random pc that comes from the pc |
---|---|---|---|
2 | of a signal handler. Do not assert that the pointer is already within | ||
3 | the code gen buffer at all, much less the writable mirror of it. | ||
2 | 4 | ||
3 | split up the CpusAccel tcg_cpus into three TCG variants: | 5 | Fixes: db0c51a3803 |
4 | |||
5 | tcg_cpus_rr (single threaded, round robin cpus) | ||
6 | tcg_cpus_icount (same as rr, but with instruction counting enabled) | ||
7 | tcg_cpus_mttcg (multi-threaded cpus) | ||
8 | |||
9 | Suggested-by: Richard Henderson <richard.henderson@linaro.org> | ||
10 | Signed-off-by: Claudio Fontana <cfontana@suse.de> | ||
11 | Reviewed-by: Richard Henderson <richard.henderson@linaro.org> | ||
12 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
13 | Message-Id: <20201015143217.29337-2-cfontana@suse.de> | ||
14 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | 6 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> |
15 | --- | 7 | --- |
16 | accel/tcg/tcg-cpus-icount.h | 17 ++ | 8 | tcg/tcg.c | 20 ++++++++++++++++++-- |
17 | accel/tcg/tcg-cpus-mttcg.h | 21 ++ | 9 | 1 file changed, 18 insertions(+), 2 deletions(-) |
18 | accel/tcg/tcg-cpus-rr.h | 20 ++ | ||
19 | accel/tcg/tcg-cpus.h | 13 +- | ||
20 | accel/tcg/tcg-all.c | 8 +- | ||
21 | accel/tcg/tcg-cpus-icount.c | 147 +++++++++++ | ||
22 | accel/tcg/tcg-cpus-mttcg.c | 117 +++++++++ | ||
23 | accel/tcg/tcg-cpus-rr.c | 270 ++++++++++++++++++++ | ||
24 | accel/tcg/tcg-cpus.c | 484 ++---------------------------------- | ||
25 | softmmu/icount.c | 2 +- | ||
26 | accel/tcg/meson.build | 9 +- | ||
27 | 11 files changed, 646 insertions(+), 462 deletions(-) | ||
28 | create mode 100644 accel/tcg/tcg-cpus-icount.h | ||
29 | create mode 100644 accel/tcg/tcg-cpus-mttcg.h | ||
30 | create mode 100644 accel/tcg/tcg-cpus-rr.h | ||
31 | create mode 100644 accel/tcg/tcg-cpus-icount.c | ||
32 | create mode 100644 accel/tcg/tcg-cpus-mttcg.c | ||
33 | create mode 100644 accel/tcg/tcg-cpus-rr.c | ||
34 | 10 | ||
35 | diff --git a/accel/tcg/tcg-cpus-icount.h b/accel/tcg/tcg-cpus-icount.h | 11 | diff --git a/tcg/tcg.c b/tcg/tcg.c |
36 | new file mode 100644 | ||
37 | index XXXXXXX..XXXXXXX | ||
38 | --- /dev/null | ||
39 | +++ b/accel/tcg/tcg-cpus-icount.h | ||
40 | @@ -XXX,XX +XXX,XX @@ | ||
41 | +/* | ||
42 | + * QEMU TCG Single Threaded vCPUs implementation using instruction counting | ||
43 | + * | ||
44 | + * Copyright 2020 SUSE LLC | ||
45 | + * | ||
46 | + * This work is licensed under the terms of the GNU GPL, version 2 or later. | ||
47 | + * See the COPYING file in the top-level directory. | ||
48 | + */ | ||
49 | + | ||
50 | +#ifndef TCG_CPUS_ICOUNT_H | ||
51 | +#define TCG_CPUS_ICOUNT_H | ||
52 | + | ||
53 | +void handle_icount_deadline(void); | ||
54 | +void prepare_icount_for_run(CPUState *cpu); | ||
55 | +void process_icount_data(CPUState *cpu); | ||
56 | + | ||
57 | +#endif /* TCG_CPUS_ICOUNT_H */ | ||
58 | diff --git a/accel/tcg/tcg-cpus-mttcg.h b/accel/tcg/tcg-cpus-mttcg.h | ||
59 | new file mode 100644 | ||
60 | index XXXXXXX..XXXXXXX | ||
61 | --- /dev/null | ||
62 | +++ b/accel/tcg/tcg-cpus-mttcg.h | ||
63 | @@ -XXX,XX +XXX,XX @@ | ||
64 | +/* | ||
65 | + * QEMU TCG Multi Threaded vCPUs implementation | ||
66 | + * | ||
67 | + * Copyright 2020 SUSE LLC | ||
68 | + * | ||
69 | + * This work is licensed under the terms of the GNU GPL, version 2 or later. | ||
70 | + * See the COPYING file in the top-level directory. | ||
71 | + */ | ||
72 | + | ||
73 | +#ifndef TCG_CPUS_MTTCG_H | ||
74 | +#define TCG_CPUS_MTTCG_H | ||
75 | + | ||
76 | +/* | ||
77 | + * In the multi-threaded case each vCPU has its own thread. The TLS | ||
78 | + * variable current_cpu can be used deep in the code to find the | ||
79 | + * current CPUState for a given thread. | ||
80 | + */ | ||
81 | + | ||
82 | +void *tcg_cpu_thread_fn(void *arg); | ||
83 | + | ||
84 | +#endif /* TCG_CPUS_MTTCG_H */ | ||
85 | diff --git a/accel/tcg/tcg-cpus-rr.h b/accel/tcg/tcg-cpus-rr.h | ||
86 | new file mode 100644 | ||
87 | index XXXXXXX..XXXXXXX | ||
88 | --- /dev/null | ||
89 | +++ b/accel/tcg/tcg-cpus-rr.h | ||
90 | @@ -XXX,XX +XXX,XX @@ | ||
91 | +/* | ||
92 | + * QEMU TCG Single Threaded vCPUs implementation | ||
93 | + * | ||
94 | + * Copyright 2020 SUSE LLC | ||
95 | + * | ||
96 | + * This work is licensed under the terms of the GNU GPL, version 2 or later. | ||
97 | + * See the COPYING file in the top-level directory. | ||
98 | + */ | ||
99 | + | ||
100 | +#ifndef TCG_CPUS_RR_H | ||
101 | +#define TCG_CPUS_RR_H | ||
102 | + | ||
103 | +#define TCG_KICK_PERIOD (NANOSECONDS_PER_SECOND / 10) | ||
104 | + | ||
105 | +/* Kick all RR vCPUs. */ | ||
106 | +void qemu_cpu_kick_rr_cpus(CPUState *unused); | ||
107 | + | ||
108 | +void *tcg_rr_cpu_thread_fn(void *arg); | ||
109 | + | ||
110 | +#endif /* TCG_CPUS_RR_H */ | ||
111 | diff --git a/accel/tcg/tcg-cpus.h b/accel/tcg/tcg-cpus.h | ||
112 | index XXXXXXX..XXXXXXX 100644 | 12 | index XXXXXXX..XXXXXXX 100644 |
113 | --- a/accel/tcg/tcg-cpus.h | 13 | --- a/tcg/tcg.c |
114 | +++ b/accel/tcg/tcg-cpus.h | 14 | +++ b/tcg/tcg.c |
115 | @@ -XXX,XX +XXX,XX @@ | 15 | @@ -XXX,XX +XXX,XX @@ static void tcg_region_trees_init(void) |
116 | /* | 16 | } |
117 | - * Accelerator CPUS Interface | ||
118 | + * QEMU TCG vCPU common functionality | ||
119 | + * | ||
120 | + * Functionality common to all TCG vcpu variants: mttcg, rr and icount. | ||
121 | * | ||
122 | * Copyright 2020 SUSE LLC | ||
123 | * | ||
124 | @@ -XXX,XX +XXX,XX @@ | ||
125 | |||
126 | #include "sysemu/cpus.h" | ||
127 | |||
128 | -extern const CpusAccel tcg_cpus; | ||
129 | +extern const CpusAccel tcg_cpus_mttcg; | ||
130 | +extern const CpusAccel tcg_cpus_icount; | ||
131 | +extern const CpusAccel tcg_cpus_rr; | ||
132 | + | ||
133 | +void tcg_start_vcpu_thread(CPUState *cpu); | ||
134 | +void qemu_tcg_destroy_vcpu(CPUState *cpu); | ||
135 | +int tcg_cpu_exec(CPUState *cpu); | ||
136 | +void tcg_handle_interrupt(CPUState *cpu, int mask); | ||
137 | |||
138 | #endif /* TCG_CPUS_H */ | ||
139 | diff --git a/accel/tcg/tcg-all.c b/accel/tcg/tcg-all.c | ||
140 | index XXXXXXX..XXXXXXX 100644 | ||
141 | --- a/accel/tcg/tcg-all.c | ||
142 | +++ b/accel/tcg/tcg-all.c | ||
143 | @@ -XXX,XX +XXX,XX @@ static int tcg_init(MachineState *ms) | ||
144 | |||
145 | tcg_exec_init(s->tb_size * 1024 * 1024); | ||
146 | mttcg_enabled = s->mttcg_enabled; | ||
147 | - cpus_register_accel(&tcg_cpus); | ||
148 | |||
149 | + if (mttcg_enabled) { | ||
150 | + cpus_register_accel(&tcg_cpus_mttcg); | ||
151 | + } else if (icount_enabled()) { | ||
152 | + cpus_register_accel(&tcg_cpus_icount); | ||
153 | + } else { | ||
154 | + cpus_register_accel(&tcg_cpus_rr); | ||
155 | + } | ||
156 | return 0; | ||
157 | } | 17 | } |
158 | 18 | ||
159 | diff --git a/accel/tcg/tcg-cpus-icount.c b/accel/tcg/tcg-cpus-icount.c | 19 | -static struct tcg_region_tree *tc_ptr_to_region_tree(const void *cp) |
160 | new file mode 100644 | 20 | +static struct tcg_region_tree *tc_ptr_to_region_tree(const void *p) |
161 | index XXXXXXX..XXXXXXX | 21 | { |
162 | --- /dev/null | 22 | - void *p = tcg_splitwx_to_rw(cp); |
163 | +++ b/accel/tcg/tcg-cpus-icount.c | 23 | size_t region_idx; |
164 | @@ -XXX,XX +XXX,XX @@ | 24 | |
165 | +/* | ||
166 | + * QEMU TCG Single Threaded vCPUs implementation using instruction counting | ||
167 | + * | ||
168 | + * Copyright (c) 2003-2008 Fabrice Bellard | ||
169 | + * Copyright (c) 2014 Red Hat Inc. | ||
170 | + * | ||
171 | + * Permission is hereby granted, free of charge, to any person obtaining a copy | ||
172 | + * of this software and associated documentation files (the "Software"), to deal | ||
173 | + * in the Software without restriction, including without limitation the rights | ||
174 | + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell | ||
175 | + * copies of the Software, and to permit persons to whom the Software is | ||
176 | + * furnished to do so, subject to the following conditions: | ||
177 | + * | ||
178 | + * The above copyright notice and this permission notice shall be included in | ||
179 | + * all copies or substantial portions of the Software. | ||
180 | + * | ||
181 | + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR | ||
182 | + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, | ||
183 | + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL | ||
184 | + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER | ||
185 | + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, | ||
186 | + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN | ||
187 | + * THE SOFTWARE. | ||
188 | + */ | ||
189 | + | ||
190 | +#include "qemu/osdep.h" | ||
191 | +#include "qemu-common.h" | ||
192 | +#include "sysemu/tcg.h" | ||
193 | +#include "sysemu/replay.h" | ||
194 | +#include "qemu/main-loop.h" | ||
195 | +#include "qemu/guest-random.h" | ||
196 | +#include "exec/exec-all.h" | ||
197 | +#include "hw/boards.h" | ||
198 | + | ||
199 | +#include "tcg-cpus.h" | ||
200 | +#include "tcg-cpus-icount.h" | ||
201 | +#include "tcg-cpus-rr.h" | ||
202 | + | ||
203 | +static int64_t tcg_get_icount_limit(void) | ||
204 | +{ | ||
205 | + int64_t deadline; | ||
206 | + | ||
207 | + if (replay_mode != REPLAY_MODE_PLAY) { | ||
208 | + /* | ||
209 | + * Include all the timers, because they may need an attention. | ||
210 | + * Too long CPU execution may create unnecessary delay in UI. | ||
211 | + */ | ||
212 | + deadline = qemu_clock_deadline_ns_all(QEMU_CLOCK_VIRTUAL, | ||
213 | + QEMU_TIMER_ATTR_ALL); | ||
214 | + /* Check realtime timers, because they help with input processing */ | ||
215 | + deadline = qemu_soonest_timeout(deadline, | ||
216 | + qemu_clock_deadline_ns_all(QEMU_CLOCK_REALTIME, | ||
217 | + QEMU_TIMER_ATTR_ALL)); | ||
218 | + | ||
219 | + /* | ||
220 | + * Maintain prior (possibly buggy) behaviour where if no deadline | ||
221 | + * was set (as there is no QEMU_CLOCK_VIRTUAL timer) or it is more than | ||
222 | + * INT32_MAX nanoseconds ahead, we still use INT32_MAX | ||
223 | + * nanoseconds. | ||
224 | + */ | ||
225 | + if ((deadline < 0) || (deadline > INT32_MAX)) { | ||
226 | + deadline = INT32_MAX; | ||
227 | + } | ||
228 | + | ||
229 | + return icount_round(deadline); | ||
230 | + } else { | ||
231 | + return replay_get_instructions(); | ||
232 | + } | ||
233 | +} | ||
234 | + | ||
235 | +static void notify_aio_contexts(void) | ||
236 | +{ | ||
237 | + /* Wake up other AioContexts. */ | ||
238 | + qemu_clock_notify(QEMU_CLOCK_VIRTUAL); | ||
239 | + qemu_clock_run_timers(QEMU_CLOCK_VIRTUAL); | ||
240 | +} | ||
241 | + | ||
242 | +void handle_icount_deadline(void) | ||
243 | +{ | ||
244 | + assert(qemu_in_vcpu_thread()); | ||
245 | + int64_t deadline = qemu_clock_deadline_ns_all(QEMU_CLOCK_VIRTUAL, | ||
246 | + QEMU_TIMER_ATTR_ALL); | ||
247 | + | ||
248 | + if (deadline == 0) { | ||
249 | + notify_aio_contexts(); | ||
250 | + } | ||
251 | +} | ||
252 | + | ||
253 | +void prepare_icount_for_run(CPUState *cpu) | ||
254 | +{ | ||
255 | + int insns_left; | ||
256 | + | ||
257 | + /* | 25 | + /* |
258 | + * These should always be cleared by process_icount_data after | 26 | + * Like tcg_splitwx_to_rw, with no assert. The pc may come from |
259 | + * each vCPU execution. However u16.high can be raised | 27 | + * a signal handler over which the caller has no control. |
260 | + * asynchronously by cpu_exit/cpu_interrupt/tcg_handle_interrupt | ||
261 | + */ | 28 | + */ |
262 | + g_assert(cpu_neg(cpu)->icount_decr.u16.low == 0); | 29 | + if (!in_code_gen_buffer(p)) { |
263 | + g_assert(cpu->icount_extra == 0); | 30 | + p -= tcg_splitwx_diff; |
264 | + | 31 | + if (!in_code_gen_buffer(p)) { |
265 | + cpu->icount_budget = tcg_get_icount_limit(); | 32 | + return NULL; |
266 | + insns_left = MIN(0xffff, cpu->icount_budget); | ||
267 | + cpu_neg(cpu)->icount_decr.u16.low = insns_left; | ||
268 | + cpu->icount_extra = cpu->icount_budget - insns_left; | ||
269 | + | ||
270 | + replay_mutex_lock(); | ||
271 | + | ||
272 | + if (cpu->icount_budget == 0 && replay_has_checkpoint()) { | ||
273 | + notify_aio_contexts(); | ||
274 | + } | ||
275 | +} | ||
276 | + | ||
277 | +void process_icount_data(CPUState *cpu) | ||
278 | +{ | ||
279 | + /* Account for executed instructions */ | ||
280 | + icount_update(cpu); | ||
281 | + | ||
282 | + /* Reset the counters */ | ||
283 | + cpu_neg(cpu)->icount_decr.u16.low = 0; | ||
284 | + cpu->icount_extra = 0; | ||
285 | + cpu->icount_budget = 0; | ||
286 | + | ||
287 | + replay_account_executed_instructions(); | ||
288 | + | ||
289 | + replay_mutex_unlock(); | ||
290 | +} | ||
291 | + | ||
292 | +static void icount_handle_interrupt(CPUState *cpu, int mask) | ||
293 | +{ | ||
294 | + int old_mask = cpu->interrupt_request; | ||
295 | + | ||
296 | + tcg_handle_interrupt(cpu, mask); | ||
297 | + if (qemu_cpu_is_self(cpu) && | ||
298 | + !cpu->can_do_io | ||
299 | + && (mask & ~old_mask) != 0) { | ||
300 | + cpu_abort(cpu, "Raised interrupt while not in I/O function"); | ||
301 | + } | ||
302 | +} | ||
303 | + | ||
304 | +const CpusAccel tcg_cpus_icount = { | ||
305 | + .create_vcpu_thread = tcg_start_vcpu_thread, | ||
306 | + .kick_vcpu_thread = qemu_cpu_kick_rr_cpus, | ||
307 | + | ||
308 | + .handle_interrupt = icount_handle_interrupt, | ||
309 | + .get_virtual_clock = icount_get, | ||
310 | + .get_elapsed_ticks = icount_get, | ||
311 | +}; | ||
312 | diff --git a/accel/tcg/tcg-cpus-mttcg.c b/accel/tcg/tcg-cpus-mttcg.c | ||
313 | new file mode 100644 | ||
314 | index XXXXXXX..XXXXXXX | ||
315 | --- /dev/null | ||
316 | +++ b/accel/tcg/tcg-cpus-mttcg.c | ||
317 | @@ -XXX,XX +XXX,XX @@ | ||
318 | +/* | ||
319 | + * QEMU TCG Multi Threaded vCPUs implementation | ||
320 | + * | ||
321 | + * Copyright (c) 2003-2008 Fabrice Bellard | ||
322 | + * Copyright (c) 2014 Red Hat Inc. | ||
323 | + * | ||
324 | + * Permission is hereby granted, free of charge, to any person obtaining a copy | ||
325 | + * of this software and associated documentation files (the "Software"), to deal | ||
326 | + * in the Software without restriction, including without limitation the rights | ||
327 | + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell | ||
328 | + * copies of the Software, and to permit persons to whom the Software is | ||
329 | + * furnished to do so, subject to the following conditions: | ||
330 | + * | ||
331 | + * The above copyright notice and this permission notice shall be included in | ||
332 | + * all copies or substantial portions of the Software. | ||
333 | + * | ||
334 | + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR | ||
335 | + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, | ||
336 | + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL | ||
337 | + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER | ||
338 | + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, | ||
339 | + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN | ||
340 | + * THE SOFTWARE. | ||
341 | + */ | ||
342 | + | ||
343 | +#include "qemu/osdep.h" | ||
344 | +#include "qemu-common.h" | ||
345 | +#include "sysemu/tcg.h" | ||
346 | +#include "sysemu/replay.h" | ||
347 | +#include "qemu/main-loop.h" | ||
348 | +#include "qemu/guest-random.h" | ||
349 | +#include "exec/exec-all.h" | ||
350 | +#include "hw/boards.h" | ||
351 | + | ||
352 | +#include "tcg-cpus.h" | ||
353 | +#include "tcg-cpus-mttcg.h" | ||
354 | + | ||
355 | +/* | ||
356 | + * In the multi-threaded case each vCPU has its own thread. The TLS | ||
357 | + * variable current_cpu can be used deep in the code to find the | ||
358 | + * current CPUState for a given thread. | ||
359 | + */ | ||
360 | + | ||
361 | +void *tcg_cpu_thread_fn(void *arg) | ||
362 | +{ | ||
363 | + CPUState *cpu = arg; | ||
364 | + | ||
365 | + assert(tcg_enabled()); | ||
366 | + g_assert(!icount_enabled()); | ||
367 | + | ||
368 | + rcu_register_thread(); | ||
369 | + tcg_register_thread(); | ||
370 | + | ||
371 | + qemu_mutex_lock_iothread(); | ||
372 | + qemu_thread_get_self(cpu->thread); | ||
373 | + | ||
374 | + cpu->thread_id = qemu_get_thread_id(); | ||
375 | + cpu->can_do_io = 1; | ||
376 | + current_cpu = cpu; | ||
377 | + cpu_thread_signal_created(cpu); | ||
378 | + qemu_guest_random_seed_thread_part2(cpu->random_seed); | ||
379 | + | ||
380 | + /* process any pending work */ | ||
381 | + cpu->exit_request = 1; | ||
382 | + | ||
383 | + do { | ||
384 | + if (cpu_can_run(cpu)) { | ||
385 | + int r; | ||
386 | + qemu_mutex_unlock_iothread(); | ||
387 | + r = tcg_cpu_exec(cpu); | ||
388 | + qemu_mutex_lock_iothread(); | ||
389 | + switch (r) { | ||
390 | + case EXCP_DEBUG: | ||
391 | + cpu_handle_guest_debug(cpu); | ||
392 | + break; | ||
393 | + case EXCP_HALTED: | ||
394 | + /* | ||
395 | + * during start-up the vCPU is reset and the thread is | ||
396 | + * kicked several times. If we don't ensure we go back | ||
397 | + * to sleep in the halted state we won't cleanly | ||
398 | + * start-up when the vCPU is enabled. | ||
399 | + * | ||
400 | + * cpu->halted should ensure we sleep in wait_io_event | ||
401 | + */ | ||
402 | + g_assert(cpu->halted); | ||
403 | + break; | ||
404 | + case EXCP_ATOMIC: | ||
405 | + qemu_mutex_unlock_iothread(); | ||
406 | + cpu_exec_step_atomic(cpu); | ||
407 | + qemu_mutex_lock_iothread(); | ||
408 | + default: | ||
409 | + /* Ignore everything else? */ | ||
410 | + break; | ||
411 | + } | ||
412 | + } | ||
413 | + | ||
414 | + qatomic_mb_set(&cpu->exit_request, 0); | ||
415 | + qemu_wait_io_event(cpu); | ||
416 | + } while (!cpu->unplug || cpu_can_run(cpu)); | ||
417 | + | ||
418 | + qemu_tcg_destroy_vcpu(cpu); | ||
419 | + qemu_mutex_unlock_iothread(); | ||
420 | + rcu_unregister_thread(); | ||
421 | + return NULL; | ||
422 | +} | ||
423 | + | ||
424 | +static void mttcg_kick_vcpu_thread(CPUState *cpu) | ||
425 | +{ | ||
426 | + cpu_exit(cpu); | ||
427 | +} | ||
428 | + | ||
429 | +const CpusAccel tcg_cpus_mttcg = { | ||
430 | + .create_vcpu_thread = tcg_start_vcpu_thread, | ||
431 | + .kick_vcpu_thread = mttcg_kick_vcpu_thread, | ||
432 | + | ||
433 | + .handle_interrupt = tcg_handle_interrupt, | ||
434 | +}; | ||
435 | diff --git a/accel/tcg/tcg-cpus-rr.c b/accel/tcg/tcg-cpus-rr.c | ||
436 | new file mode 100644 | ||
437 | index XXXXXXX..XXXXXXX | ||
438 | --- /dev/null | ||
439 | +++ b/accel/tcg/tcg-cpus-rr.c | ||
440 | @@ -XXX,XX +XXX,XX @@ | ||
441 | +/* | ||
442 | + * QEMU TCG Single Threaded vCPUs implementation | ||
443 | + * | ||
444 | + * Copyright (c) 2003-2008 Fabrice Bellard | ||
445 | + * Copyright (c) 2014 Red Hat Inc. | ||
446 | + * | ||
447 | + * Permission is hereby granted, free of charge, to any person obtaining a copy | ||
448 | + * of this software and associated documentation files (the "Software"), to deal | ||
449 | + * in the Software without restriction, including without limitation the rights | ||
450 | + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell | ||
451 | + * copies of the Software, and to permit persons to whom the Software is | ||
452 | + * furnished to do so, subject to the following conditions: | ||
453 | + * | ||
454 | + * The above copyright notice and this permission notice shall be included in | ||
455 | + * all copies or substantial portions of the Software. | ||
456 | + * | ||
457 | + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR | ||
458 | + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, | ||
459 | + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL | ||
460 | + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER | ||
461 | + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, | ||
462 | + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN | ||
463 | + * THE SOFTWARE. | ||
464 | + */ | ||
465 | + | ||
466 | +#include "qemu/osdep.h" | ||
467 | +#include "qemu-common.h" | ||
468 | +#include "sysemu/tcg.h" | ||
469 | +#include "sysemu/replay.h" | ||
470 | +#include "qemu/main-loop.h" | ||
471 | +#include "qemu/guest-random.h" | ||
472 | +#include "exec/exec-all.h" | ||
473 | +#include "hw/boards.h" | ||
474 | + | ||
475 | +#include "tcg-cpus.h" | ||
476 | +#include "tcg-cpus-rr.h" | ||
477 | +#include "tcg-cpus-icount.h" | ||
478 | + | ||
479 | +/* Kick all RR vCPUs */ | ||
480 | +void qemu_cpu_kick_rr_cpus(CPUState *unused) | ||
481 | +{ | ||
482 | + CPUState *cpu; | ||
483 | + | ||
484 | + CPU_FOREACH(cpu) { | ||
485 | + cpu_exit(cpu); | ||
486 | + }; | ||
487 | +} | ||
488 | + | ||
489 | +/* | ||
490 | + * TCG vCPU kick timer | ||
491 | + * | ||
492 | + * The kick timer is responsible for moving single threaded vCPU | ||
493 | + * emulation on to the next vCPU. If more than one vCPU is running a | ||
494 | + * timer event with force a cpu->exit so the next vCPU can get | ||
495 | + * scheduled. | ||
496 | + * | ||
497 | + * The timer is removed if all vCPUs are idle and restarted again once | ||
498 | + * idleness is complete. | ||
499 | + */ | ||
500 | + | ||
501 | +static QEMUTimer *tcg_kick_vcpu_timer; | ||
502 | +static CPUState *tcg_current_rr_cpu; | ||
503 | + | ||
504 | +#define TCG_KICK_PERIOD (NANOSECONDS_PER_SECOND / 10) | ||
505 | + | ||
506 | +static inline int64_t qemu_tcg_next_kick(void) | ||
507 | +{ | ||
508 | + return qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + TCG_KICK_PERIOD; | ||
509 | +} | ||
510 | + | ||
511 | +/* Kick the currently round-robin scheduled vCPU to next */ | ||
512 | +static void qemu_cpu_kick_rr_next_cpu(void) | ||
513 | +{ | ||
514 | + CPUState *cpu; | ||
515 | + do { | ||
516 | + cpu = qatomic_mb_read(&tcg_current_rr_cpu); | ||
517 | + if (cpu) { | ||
518 | + cpu_exit(cpu); | ||
519 | + } | ||
520 | + } while (cpu != qatomic_mb_read(&tcg_current_rr_cpu)); | ||
521 | +} | ||
522 | + | ||
523 | +static void kick_tcg_thread(void *opaque) | ||
524 | +{ | ||
525 | + timer_mod(tcg_kick_vcpu_timer, qemu_tcg_next_kick()); | ||
526 | + qemu_cpu_kick_rr_next_cpu(); | ||
527 | +} | ||
528 | + | ||
529 | +static void start_tcg_kick_timer(void) | ||
530 | +{ | ||
531 | + if (!tcg_kick_vcpu_timer && CPU_NEXT(first_cpu)) { | ||
532 | + tcg_kick_vcpu_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL, | ||
533 | + kick_tcg_thread, NULL); | ||
534 | + } | ||
535 | + if (tcg_kick_vcpu_timer && !timer_pending(tcg_kick_vcpu_timer)) { | ||
536 | + timer_mod(tcg_kick_vcpu_timer, qemu_tcg_next_kick()); | ||
537 | + } | ||
538 | +} | ||
539 | + | ||
540 | +static void stop_tcg_kick_timer(void) | ||
541 | +{ | ||
542 | + if (tcg_kick_vcpu_timer && timer_pending(tcg_kick_vcpu_timer)) { | ||
543 | + timer_del(tcg_kick_vcpu_timer); | ||
544 | + } | ||
545 | +} | ||
546 | + | ||
547 | +static void qemu_tcg_rr_wait_io_event(void) | ||
548 | +{ | ||
549 | + CPUState *cpu; | ||
550 | + | ||
551 | + while (all_cpu_threads_idle()) { | ||
552 | + stop_tcg_kick_timer(); | ||
553 | + qemu_cond_wait_iothread(first_cpu->halt_cond); | ||
554 | + } | ||
555 | + | ||
556 | + start_tcg_kick_timer(); | ||
557 | + | ||
558 | + CPU_FOREACH(cpu) { | ||
559 | + qemu_wait_io_event_common(cpu); | ||
560 | + } | ||
561 | +} | ||
562 | + | ||
563 | +/* | ||
564 | + * Destroy any remaining vCPUs which have been unplugged and have | ||
565 | + * finished running | ||
566 | + */ | ||
567 | +static void deal_with_unplugged_cpus(void) | ||
568 | +{ | ||
569 | + CPUState *cpu; | ||
570 | + | ||
571 | + CPU_FOREACH(cpu) { | ||
572 | + if (cpu->unplug && !cpu_can_run(cpu)) { | ||
573 | + qemu_tcg_destroy_vcpu(cpu); | ||
574 | + break; | ||
575 | + } | ||
576 | + } | ||
577 | +} | ||
578 | + | ||
579 | +/* | ||
580 | + * In the single-threaded case each vCPU is simulated in turn. If | ||
581 | + * there is more than a single vCPU we create a simple timer to kick | ||
582 | + * the vCPU and ensure we don't get stuck in a tight loop in one vCPU. | ||
583 | + * This is done explicitly rather than relying on side-effects | ||
584 | + * elsewhere. | ||
585 | + */ | ||
586 | + | ||
587 | +void *tcg_rr_cpu_thread_fn(void *arg) | ||
588 | +{ | ||
589 | + CPUState *cpu = arg; | ||
590 | + | ||
591 | + assert(tcg_enabled()); | ||
592 | + rcu_register_thread(); | ||
593 | + tcg_register_thread(); | ||
594 | + | ||
595 | + qemu_mutex_lock_iothread(); | ||
596 | + qemu_thread_get_self(cpu->thread); | ||
597 | + | ||
598 | + cpu->thread_id = qemu_get_thread_id(); | ||
599 | + cpu->can_do_io = 1; | ||
600 | + cpu_thread_signal_created(cpu); | ||
601 | + qemu_guest_random_seed_thread_part2(cpu->random_seed); | ||
602 | + | ||
603 | + /* wait for initial kick-off after machine start */ | ||
604 | + while (first_cpu->stopped) { | ||
605 | + qemu_cond_wait_iothread(first_cpu->halt_cond); | ||
606 | + | ||
607 | + /* process any pending work */ | ||
608 | + CPU_FOREACH(cpu) { | ||
609 | + current_cpu = cpu; | ||
610 | + qemu_wait_io_event_common(cpu); | ||
611 | + } | 33 | + } |
612 | + } | 34 | + } |
613 | + | 35 | + |
614 | + start_tcg_kick_timer(); | 36 | if (p < region.start_aligned) { |
615 | + | 37 | region_idx = 0; |
616 | + cpu = first_cpu; | 38 | } else { |
617 | + | 39 | @@ -XXX,XX +XXX,XX @@ void tcg_tb_insert(TranslationBlock *tb) |
618 | + /* process any pending work */ | 40 | { |
619 | + cpu->exit_request = 1; | 41 | struct tcg_region_tree *rt = tc_ptr_to_region_tree(tb->tc.ptr); |
620 | + | 42 | |
621 | + while (1) { | 43 | + g_assert(rt != NULL); |
622 | + qemu_mutex_unlock_iothread(); | 44 | qemu_mutex_lock(&rt->lock); |
623 | + replay_mutex_lock(); | 45 | g_tree_insert(rt->tree, &tb->tc, tb); |
624 | + qemu_mutex_lock_iothread(); | 46 | qemu_mutex_unlock(&rt->lock); |
625 | + | 47 | @@ -XXX,XX +XXX,XX @@ void tcg_tb_remove(TranslationBlock *tb) |
626 | + if (icount_enabled()) { | 48 | { |
627 | + /* Account partial waits to QEMU_CLOCK_VIRTUAL. */ | 49 | struct tcg_region_tree *rt = tc_ptr_to_region_tree(tb->tc.ptr); |
628 | + icount_account_warp_timer(); | 50 | |
629 | + /* | 51 | + g_assert(rt != NULL); |
630 | + * Run the timers here. This is much more efficient than | 52 | qemu_mutex_lock(&rt->lock); |
631 | + * waking up the I/O thread and waiting for completion. | 53 | g_tree_remove(rt->tree, &tb->tc); |
632 | + */ | 54 | qemu_mutex_unlock(&rt->lock); |
633 | + handle_icount_deadline(); | 55 | @@ -XXX,XX +XXX,XX @@ TranslationBlock *tcg_tb_lookup(uintptr_t tc_ptr) |
634 | + } | 56 | TranslationBlock *tb; |
635 | + | 57 | struct tb_tc s = { .ptr = (void *)tc_ptr }; |
636 | + replay_mutex_unlock(); | 58 | |
637 | + | 59 | + if (rt == NULL) { |
638 | + if (!cpu) { | 60 | + return NULL; |
639 | + cpu = first_cpu; | ||
640 | + } | ||
641 | + | ||
642 | + while (cpu && cpu_work_list_empty(cpu) && !cpu->exit_request) { | ||
643 | + | ||
644 | + qatomic_mb_set(&tcg_current_rr_cpu, cpu); | ||
645 | + current_cpu = cpu; | ||
646 | + | ||
647 | + qemu_clock_enable(QEMU_CLOCK_VIRTUAL, | ||
648 | + (cpu->singlestep_enabled & SSTEP_NOTIMER) == 0); | ||
649 | + | ||
650 | + if (cpu_can_run(cpu)) { | ||
651 | + int r; | ||
652 | + | ||
653 | + qemu_mutex_unlock_iothread(); | ||
654 | + if (icount_enabled()) { | ||
655 | + prepare_icount_for_run(cpu); | ||
656 | + } | ||
657 | + r = tcg_cpu_exec(cpu); | ||
658 | + if (icount_enabled()) { | ||
659 | + process_icount_data(cpu); | ||
660 | + } | ||
661 | + qemu_mutex_lock_iothread(); | ||
662 | + | ||
663 | + if (r == EXCP_DEBUG) { | ||
664 | + cpu_handle_guest_debug(cpu); | ||
665 | + break; | ||
666 | + } else if (r == EXCP_ATOMIC) { | ||
667 | + qemu_mutex_unlock_iothread(); | ||
668 | + cpu_exec_step_atomic(cpu); | ||
669 | + qemu_mutex_lock_iothread(); | ||
670 | + break; | ||
671 | + } | ||
672 | + } else if (cpu->stop) { | ||
673 | + if (cpu->unplug) { | ||
674 | + cpu = CPU_NEXT(cpu); | ||
675 | + } | ||
676 | + break; | ||
677 | + } | ||
678 | + | ||
679 | + cpu = CPU_NEXT(cpu); | ||
680 | + } /* while (cpu && !cpu->exit_request).. */ | ||
681 | + | ||
682 | + /* Does not need qatomic_mb_set because a spurious wakeup is okay. */ | ||
683 | + qatomic_set(&tcg_current_rr_cpu, NULL); | ||
684 | + | ||
685 | + if (cpu && cpu->exit_request) { | ||
686 | + qatomic_mb_set(&cpu->exit_request, 0); | ||
687 | + } | ||
688 | + | ||
689 | + if (icount_enabled() && all_cpu_threads_idle()) { | ||
690 | + /* | ||
691 | + * When all cpus are sleeping (e.g in WFI), to avoid a deadlock | ||
692 | + * in the main_loop, wake it up in order to start the warp timer. | ||
693 | + */ | ||
694 | + qemu_notify_event(); | ||
695 | + } | ||
696 | + | ||
697 | + qemu_tcg_rr_wait_io_event(); | ||
698 | + deal_with_unplugged_cpus(); | ||
699 | + } | 61 | + } |
700 | + | 62 | + |
701 | + rcu_unregister_thread(); | 63 | qemu_mutex_lock(&rt->lock); |
702 | + return NULL; | 64 | tb = g_tree_lookup(rt->tree, &s); |
703 | +} | 65 | qemu_mutex_unlock(&rt->lock); |
704 | + | ||
705 | +const CpusAccel tcg_cpus_rr = { | ||
706 | + .create_vcpu_thread = tcg_start_vcpu_thread, | ||
707 | + .kick_vcpu_thread = qemu_cpu_kick_rr_cpus, | ||
708 | + | ||
709 | + .handle_interrupt = tcg_handle_interrupt, | ||
710 | +}; | ||
711 | diff --git a/accel/tcg/tcg-cpus.c b/accel/tcg/tcg-cpus.c | ||
712 | index XXXXXXX..XXXXXXX 100644 | ||
713 | --- a/accel/tcg/tcg-cpus.c | ||
714 | +++ b/accel/tcg/tcg-cpus.c | ||
715 | @@ -XXX,XX +XXX,XX @@ | ||
716 | /* | ||
717 | - * QEMU System Emulator | ||
718 | + * QEMU TCG vCPU common functionality | ||
719 | + * | ||
720 | + * Functionality common to all TCG vCPU variants: mttcg, rr and icount. | ||
721 | * | ||
722 | * Copyright (c) 2003-2008 Fabrice Bellard | ||
723 | * Copyright (c) 2014 Red Hat Inc. | ||
724 | @@ -XXX,XX +XXX,XX @@ | ||
725 | #include "hw/boards.h" | ||
726 | |||
727 | #include "tcg-cpus.h" | ||
728 | +#include "tcg-cpus-mttcg.h" | ||
729 | +#include "tcg-cpus-rr.h" | ||
730 | |||
731 | -/* Kick all RR vCPUs */ | ||
732 | -static void qemu_cpu_kick_rr_cpus(void) | ||
733 | -{ | ||
734 | - CPUState *cpu; | ||
735 | +/* common functionality among all TCG variants */ | ||
736 | |||
737 | - CPU_FOREACH(cpu) { | ||
738 | - cpu_exit(cpu); | ||
739 | - }; | ||
740 | -} | ||
741 | - | ||
742 | -static void tcg_kick_vcpu_thread(CPUState *cpu) | ||
743 | -{ | ||
744 | - if (qemu_tcg_mttcg_enabled()) { | ||
745 | - cpu_exit(cpu); | ||
746 | - } else { | ||
747 | - qemu_cpu_kick_rr_cpus(); | ||
748 | - } | ||
749 | -} | ||
750 | - | ||
751 | -/* | ||
752 | - * TCG vCPU kick timer | ||
753 | - * | ||
754 | - * The kick timer is responsible for moving single threaded vCPU | ||
755 | - * emulation on to the next vCPU. If more than one vCPU is running a | ||
756 | - * timer event with force a cpu->exit so the next vCPU can get | ||
757 | - * scheduled. | ||
758 | - * | ||
759 | - * The timer is removed if all vCPUs are idle and restarted again once | ||
760 | - * idleness is complete. | ||
761 | - */ | ||
762 | - | ||
763 | -static QEMUTimer *tcg_kick_vcpu_timer; | ||
764 | -static CPUState *tcg_current_rr_cpu; | ||
765 | - | ||
766 | -#define TCG_KICK_PERIOD (NANOSECONDS_PER_SECOND / 10) | ||
767 | - | ||
768 | -static inline int64_t qemu_tcg_next_kick(void) | ||
769 | -{ | ||
770 | - return qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + TCG_KICK_PERIOD; | ||
771 | -} | ||
772 | - | ||
773 | -/* Kick the currently round-robin scheduled vCPU to next */ | ||
774 | -static void qemu_cpu_kick_rr_next_cpu(void) | ||
775 | -{ | ||
776 | - CPUState *cpu; | ||
777 | - do { | ||
778 | - cpu = qatomic_mb_read(&tcg_current_rr_cpu); | ||
779 | - if (cpu) { | ||
780 | - cpu_exit(cpu); | ||
781 | - } | ||
782 | - } while (cpu != qatomic_mb_read(&tcg_current_rr_cpu)); | ||
783 | -} | ||
784 | - | ||
785 | -static void kick_tcg_thread(void *opaque) | ||
786 | -{ | ||
787 | - timer_mod(tcg_kick_vcpu_timer, qemu_tcg_next_kick()); | ||
788 | - qemu_cpu_kick_rr_next_cpu(); | ||
789 | -} | ||
790 | - | ||
791 | -static void start_tcg_kick_timer(void) | ||
792 | -{ | ||
793 | - assert(!mttcg_enabled); | ||
794 | - if (!tcg_kick_vcpu_timer && CPU_NEXT(first_cpu)) { | ||
795 | - tcg_kick_vcpu_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL, | ||
796 | - kick_tcg_thread, NULL); | ||
797 | - } | ||
798 | - if (tcg_kick_vcpu_timer && !timer_pending(tcg_kick_vcpu_timer)) { | ||
799 | - timer_mod(tcg_kick_vcpu_timer, qemu_tcg_next_kick()); | ||
800 | - } | ||
801 | -} | ||
802 | - | ||
803 | -static void stop_tcg_kick_timer(void) | ||
804 | -{ | ||
805 | - assert(!mttcg_enabled); | ||
806 | - if (tcg_kick_vcpu_timer && timer_pending(tcg_kick_vcpu_timer)) { | ||
807 | - timer_del(tcg_kick_vcpu_timer); | ||
808 | - } | ||
809 | -} | ||
810 | - | ||
811 | -static void qemu_tcg_destroy_vcpu(CPUState *cpu) | ||
812 | -{ | ||
813 | -} | ||
814 | - | ||
815 | -static void qemu_tcg_rr_wait_io_event(void) | ||
816 | -{ | ||
817 | - CPUState *cpu; | ||
818 | - | ||
819 | - while (all_cpu_threads_idle()) { | ||
820 | - stop_tcg_kick_timer(); | ||
821 | - qemu_cond_wait_iothread(first_cpu->halt_cond); | ||
822 | - } | ||
823 | - | ||
824 | - start_tcg_kick_timer(); | ||
825 | - | ||
826 | - CPU_FOREACH(cpu) { | ||
827 | - qemu_wait_io_event_common(cpu); | ||
828 | - } | ||
829 | -} | ||
830 | - | ||
831 | -static int64_t tcg_get_icount_limit(void) | ||
832 | -{ | ||
833 | - int64_t deadline; | ||
834 | - | ||
835 | - if (replay_mode != REPLAY_MODE_PLAY) { | ||
836 | - /* | ||
837 | - * Include all the timers, because they may need an attention. | ||
838 | - * Too long CPU execution may create unnecessary delay in UI. | ||
839 | - */ | ||
840 | - deadline = qemu_clock_deadline_ns_all(QEMU_CLOCK_VIRTUAL, | ||
841 | - QEMU_TIMER_ATTR_ALL); | ||
842 | - /* Check realtime timers, because they help with input processing */ | ||
843 | - deadline = qemu_soonest_timeout(deadline, | ||
844 | - qemu_clock_deadline_ns_all(QEMU_CLOCK_REALTIME, | ||
845 | - QEMU_TIMER_ATTR_ALL)); | ||
846 | - | ||
847 | - /* | ||
848 | - * Maintain prior (possibly buggy) behaviour where if no deadline | ||
849 | - * was set (as there is no QEMU_CLOCK_VIRTUAL timer) or it is more than | ||
850 | - * INT32_MAX nanoseconds ahead, we still use INT32_MAX | ||
851 | - * nanoseconds. | ||
852 | - */ | ||
853 | - if ((deadline < 0) || (deadline > INT32_MAX)) { | ||
854 | - deadline = INT32_MAX; | ||
855 | - } | ||
856 | - | ||
857 | - return icount_round(deadline); | ||
858 | - } else { | ||
859 | - return replay_get_instructions(); | ||
860 | - } | ||
861 | -} | ||
862 | - | ||
863 | -static void notify_aio_contexts(void) | ||
864 | -{ | ||
865 | - /* Wake up other AioContexts. */ | ||
866 | - qemu_clock_notify(QEMU_CLOCK_VIRTUAL); | ||
867 | - qemu_clock_run_timers(QEMU_CLOCK_VIRTUAL); | ||
868 | -} | ||
869 | - | ||
870 | -static void handle_icount_deadline(void) | ||
871 | -{ | ||
872 | - assert(qemu_in_vcpu_thread()); | ||
873 | - if (icount_enabled()) { | ||
874 | - int64_t deadline = qemu_clock_deadline_ns_all(QEMU_CLOCK_VIRTUAL, | ||
875 | - QEMU_TIMER_ATTR_ALL); | ||
876 | - | ||
877 | - if (deadline == 0) { | ||
878 | - notify_aio_contexts(); | ||
879 | - } | ||
880 | - } | ||
881 | -} | ||
882 | - | ||
883 | -static void prepare_icount_for_run(CPUState *cpu) | ||
884 | -{ | ||
885 | - if (icount_enabled()) { | ||
886 | - int insns_left; | ||
887 | - | ||
888 | - /* | ||
889 | - * These should always be cleared by process_icount_data after | ||
890 | - * each vCPU execution. However u16.high can be raised | ||
891 | - * asynchronously by cpu_exit/cpu_interrupt/tcg_handle_interrupt | ||
892 | - */ | ||
893 | - g_assert(cpu_neg(cpu)->icount_decr.u16.low == 0); | ||
894 | - g_assert(cpu->icount_extra == 0); | ||
895 | - | ||
896 | - cpu->icount_budget = tcg_get_icount_limit(); | ||
897 | - insns_left = MIN(0xffff, cpu->icount_budget); | ||
898 | - cpu_neg(cpu)->icount_decr.u16.low = insns_left; | ||
899 | - cpu->icount_extra = cpu->icount_budget - insns_left; | ||
900 | - | ||
901 | - replay_mutex_lock(); | ||
902 | - | ||
903 | - if (cpu->icount_budget == 0 && replay_has_checkpoint()) { | ||
904 | - notify_aio_contexts(); | ||
905 | - } | ||
906 | - } | ||
907 | -} | ||
908 | - | ||
909 | -static void process_icount_data(CPUState *cpu) | ||
910 | -{ | ||
911 | - if (icount_enabled()) { | ||
912 | - /* Account for executed instructions */ | ||
913 | - icount_update(cpu); | ||
914 | - | ||
915 | - /* Reset the counters */ | ||
916 | - cpu_neg(cpu)->icount_decr.u16.low = 0; | ||
917 | - cpu->icount_extra = 0; | ||
918 | - cpu->icount_budget = 0; | ||
919 | - | ||
920 | - replay_account_executed_instructions(); | ||
921 | - | ||
922 | - replay_mutex_unlock(); | ||
923 | - } | ||
924 | -} | ||
925 | - | ||
926 | -static int tcg_cpu_exec(CPUState *cpu) | ||
927 | -{ | ||
928 | - int ret; | ||
929 | -#ifdef CONFIG_PROFILER | ||
930 | - int64_t ti; | ||
931 | -#endif | ||
932 | - | ||
933 | - assert(tcg_enabled()); | ||
934 | -#ifdef CONFIG_PROFILER | ||
935 | - ti = profile_getclock(); | ||
936 | -#endif | ||
937 | - cpu_exec_start(cpu); | ||
938 | - ret = cpu_exec(cpu); | ||
939 | - cpu_exec_end(cpu); | ||
940 | -#ifdef CONFIG_PROFILER | ||
941 | - qatomic_set(&tcg_ctx->prof.cpu_exec_time, | ||
942 | - tcg_ctx->prof.cpu_exec_time + profile_getclock() - ti); | ||
943 | -#endif | ||
944 | - return ret; | ||
945 | -} | ||
946 | - | ||
947 | -/* | ||
948 | - * Destroy any remaining vCPUs which have been unplugged and have | ||
949 | - * finished running | ||
950 | - */ | ||
951 | -static void deal_with_unplugged_cpus(void) | ||
952 | -{ | ||
953 | - CPUState *cpu; | ||
954 | - | ||
955 | - CPU_FOREACH(cpu) { | ||
956 | - if (cpu->unplug && !cpu_can_run(cpu)) { | ||
957 | - qemu_tcg_destroy_vcpu(cpu); | ||
958 | - cpu_thread_signal_destroyed(cpu); | ||
959 | - break; | ||
960 | - } | ||
961 | - } | ||
962 | -} | ||
963 | - | ||
964 | -/* | ||
965 | - * Single-threaded TCG | ||
966 | - * | ||
967 | - * In the single-threaded case each vCPU is simulated in turn. If | ||
968 | - * there is more than a single vCPU we create a simple timer to kick | ||
969 | - * the vCPU and ensure we don't get stuck in a tight loop in one vCPU. | ||
970 | - * This is done explicitly rather than relying on side-effects | ||
971 | - * elsewhere. | ||
972 | - */ | ||
973 | - | ||
974 | -static void *tcg_rr_cpu_thread_fn(void *arg) | ||
975 | -{ | ||
976 | - CPUState *cpu = arg; | ||
977 | - | ||
978 | - assert(tcg_enabled()); | ||
979 | - rcu_register_thread(); | ||
980 | - tcg_register_thread(); | ||
981 | - | ||
982 | - qemu_mutex_lock_iothread(); | ||
983 | - qemu_thread_get_self(cpu->thread); | ||
984 | - | ||
985 | - cpu->thread_id = qemu_get_thread_id(); | ||
986 | - cpu->can_do_io = 1; | ||
987 | - cpu_thread_signal_created(cpu); | ||
988 | - qemu_guest_random_seed_thread_part2(cpu->random_seed); | ||
989 | - | ||
990 | - /* wait for initial kick-off after machine start */ | ||
991 | - while (first_cpu->stopped) { | ||
992 | - qemu_cond_wait_iothread(first_cpu->halt_cond); | ||
993 | - | ||
994 | - /* process any pending work */ | ||
995 | - CPU_FOREACH(cpu) { | ||
996 | - current_cpu = cpu; | ||
997 | - qemu_wait_io_event_common(cpu); | ||
998 | - } | ||
999 | - } | ||
1000 | - | ||
1001 | - start_tcg_kick_timer(); | ||
1002 | - | ||
1003 | - cpu = first_cpu; | ||
1004 | - | ||
1005 | - /* process any pending work */ | ||
1006 | - cpu->exit_request = 1; | ||
1007 | - | ||
1008 | - while (1) { | ||
1009 | - qemu_mutex_unlock_iothread(); | ||
1010 | - replay_mutex_lock(); | ||
1011 | - qemu_mutex_lock_iothread(); | ||
1012 | - /* Account partial waits to QEMU_CLOCK_VIRTUAL. */ | ||
1013 | - icount_account_warp_timer(); | ||
1014 | - | ||
1015 | - /* | ||
1016 | - * Run the timers here. This is much more efficient than | ||
1017 | - * waking up the I/O thread and waiting for completion. | ||
1018 | - */ | ||
1019 | - handle_icount_deadline(); | ||
1020 | - | ||
1021 | - replay_mutex_unlock(); | ||
1022 | - | ||
1023 | - if (!cpu) { | ||
1024 | - cpu = first_cpu; | ||
1025 | - } | ||
1026 | - | ||
1027 | - while (cpu && cpu_work_list_empty(cpu) && !cpu->exit_request) { | ||
1028 | - | ||
1029 | - qatomic_mb_set(&tcg_current_rr_cpu, cpu); | ||
1030 | - current_cpu = cpu; | ||
1031 | - | ||
1032 | - qemu_clock_enable(QEMU_CLOCK_VIRTUAL, | ||
1033 | - (cpu->singlestep_enabled & SSTEP_NOTIMER) == 0); | ||
1034 | - | ||
1035 | - if (cpu_can_run(cpu)) { | ||
1036 | - int r; | ||
1037 | - | ||
1038 | - qemu_mutex_unlock_iothread(); | ||
1039 | - prepare_icount_for_run(cpu); | ||
1040 | - | ||
1041 | - r = tcg_cpu_exec(cpu); | ||
1042 | - | ||
1043 | - process_icount_data(cpu); | ||
1044 | - qemu_mutex_lock_iothread(); | ||
1045 | - | ||
1046 | - if (r == EXCP_DEBUG) { | ||
1047 | - cpu_handle_guest_debug(cpu); | ||
1048 | - break; | ||
1049 | - } else if (r == EXCP_ATOMIC) { | ||
1050 | - qemu_mutex_unlock_iothread(); | ||
1051 | - cpu_exec_step_atomic(cpu); | ||
1052 | - qemu_mutex_lock_iothread(); | ||
1053 | - break; | ||
1054 | - } | ||
1055 | - } else if (cpu->stop) { | ||
1056 | - if (cpu->unplug) { | ||
1057 | - cpu = CPU_NEXT(cpu); | ||
1058 | - } | ||
1059 | - break; | ||
1060 | - } | ||
1061 | - | ||
1062 | - cpu = CPU_NEXT(cpu); | ||
1063 | - } /* while (cpu && !cpu->exit_request).. */ | ||
1064 | - | ||
1065 | - /* Does not need qatomic_mb_set because a spurious wakeup is okay. */ | ||
1066 | - qatomic_set(&tcg_current_rr_cpu, NULL); | ||
1067 | - | ||
1068 | - if (cpu && cpu->exit_request) { | ||
1069 | - qatomic_mb_set(&cpu->exit_request, 0); | ||
1070 | - } | ||
1071 | - | ||
1072 | - if (icount_enabled() && all_cpu_threads_idle()) { | ||
1073 | - /* | ||
1074 | - * When all cpus are sleeping (e.g in WFI), to avoid a deadlock | ||
1075 | - * in the main_loop, wake it up in order to start the warp timer. | ||
1076 | - */ | ||
1077 | - qemu_notify_event(); | ||
1078 | - } | ||
1079 | - | ||
1080 | - qemu_tcg_rr_wait_io_event(); | ||
1081 | - deal_with_unplugged_cpus(); | ||
1082 | - } | ||
1083 | - | ||
1084 | - rcu_unregister_thread(); | ||
1085 | - return NULL; | ||
1086 | -} | ||
1087 | - | ||
1088 | -/* | ||
1089 | - * Multi-threaded TCG | ||
1090 | - * | ||
1091 | - * In the multi-threaded case each vCPU has its own thread. The TLS | ||
1092 | - * variable current_cpu can be used deep in the code to find the | ||
1093 | - * current CPUState for a given thread. | ||
1094 | - */ | ||
1095 | - | ||
1096 | -static void *tcg_cpu_thread_fn(void *arg) | ||
1097 | -{ | ||
1098 | - CPUState *cpu = arg; | ||
1099 | - | ||
1100 | - assert(tcg_enabled()); | ||
1101 | - g_assert(!icount_enabled()); | ||
1102 | - | ||
1103 | - rcu_register_thread(); | ||
1104 | - tcg_register_thread(); | ||
1105 | - | ||
1106 | - qemu_mutex_lock_iothread(); | ||
1107 | - qemu_thread_get_self(cpu->thread); | ||
1108 | - | ||
1109 | - cpu->thread_id = qemu_get_thread_id(); | ||
1110 | - cpu->can_do_io = 1; | ||
1111 | - current_cpu = cpu; | ||
1112 | - cpu_thread_signal_created(cpu); | ||
1113 | - qemu_guest_random_seed_thread_part2(cpu->random_seed); | ||
1114 | - | ||
1115 | - /* process any pending work */ | ||
1116 | - cpu->exit_request = 1; | ||
1117 | - | ||
1118 | - do { | ||
1119 | - if (cpu_can_run(cpu)) { | ||
1120 | - int r; | ||
1121 | - qemu_mutex_unlock_iothread(); | ||
1122 | - r = tcg_cpu_exec(cpu); | ||
1123 | - qemu_mutex_lock_iothread(); | ||
1124 | - switch (r) { | ||
1125 | - case EXCP_DEBUG: | ||
1126 | - cpu_handle_guest_debug(cpu); | ||
1127 | - break; | ||
1128 | - case EXCP_HALTED: | ||
1129 | - /* | ||
1130 | - * during start-up the vCPU is reset and the thread is | ||
1131 | - * kicked several times. If we don't ensure we go back | ||
1132 | - * to sleep in the halted state we won't cleanly | ||
1133 | - * start-up when the vCPU is enabled. | ||
1134 | - * | ||
1135 | - * cpu->halted should ensure we sleep in wait_io_event | ||
1136 | - */ | ||
1137 | - g_assert(cpu->halted); | ||
1138 | - break; | ||
1139 | - case EXCP_ATOMIC: | ||
1140 | - qemu_mutex_unlock_iothread(); | ||
1141 | - cpu_exec_step_atomic(cpu); | ||
1142 | - qemu_mutex_lock_iothread(); | ||
1143 | - default: | ||
1144 | - /* Ignore everything else? */ | ||
1145 | - break; | ||
1146 | - } | ||
1147 | - } | ||
1148 | - | ||
1149 | - qatomic_mb_set(&cpu->exit_request, 0); | ||
1150 | - qemu_wait_io_event(cpu); | ||
1151 | - } while (!cpu->unplug || cpu_can_run(cpu)); | ||
1152 | - | ||
1153 | - qemu_tcg_destroy_vcpu(cpu); | ||
1154 | - cpu_thread_signal_destroyed(cpu); | ||
1155 | - qemu_mutex_unlock_iothread(); | ||
1156 | - rcu_unregister_thread(); | ||
1157 | - return NULL; | ||
1158 | -} | ||
1159 | - | ||
1160 | -static void tcg_start_vcpu_thread(CPUState *cpu) | ||
1161 | +void tcg_start_vcpu_thread(CPUState *cpu) | ||
1162 | { | ||
1163 | char thread_name[VCPU_THREAD_NAME_SIZE]; | ||
1164 | static QemuCond *single_tcg_halt_cond; | ||
1165 | @@ -XXX,XX +XXX,XX @@ static void tcg_start_vcpu_thread(CPUState *cpu) | ||
1166 | } | ||
1167 | } | ||
1168 | |||
1169 | -static int64_t tcg_get_virtual_clock(void) | ||
1170 | +void qemu_tcg_destroy_vcpu(CPUState *cpu) | ||
1171 | { | ||
1172 | - if (icount_enabled()) { | ||
1173 | - return icount_get(); | ||
1174 | - } | ||
1175 | - return cpu_get_clock(); | ||
1176 | + cpu_thread_signal_destroyed(cpu); | ||
1177 | } | ||
1178 | |||
1179 | -static int64_t tcg_get_elapsed_ticks(void) | ||
1180 | +int tcg_cpu_exec(CPUState *cpu) | ||
1181 | { | ||
1182 | - if (icount_enabled()) { | ||
1183 | - return icount_get(); | ||
1184 | - } | ||
1185 | - return cpu_get_ticks(); | ||
1186 | + int ret; | ||
1187 | +#ifdef CONFIG_PROFILER | ||
1188 | + int64_t ti; | ||
1189 | +#endif | ||
1190 | + assert(tcg_enabled()); | ||
1191 | +#ifdef CONFIG_PROFILER | ||
1192 | + ti = profile_getclock(); | ||
1193 | +#endif | ||
1194 | + cpu_exec_start(cpu); | ||
1195 | + ret = cpu_exec(cpu); | ||
1196 | + cpu_exec_end(cpu); | ||
1197 | +#ifdef CONFIG_PROFILER | ||
1198 | + qatomic_set(&tcg_ctx->prof.cpu_exec_time, | ||
1199 | + tcg_ctx->prof.cpu_exec_time + profile_getclock() - ti); | ||
1200 | +#endif | ||
1201 | + return ret; | ||
1202 | } | ||
1203 | |||
1204 | /* mask must never be zero, except for A20 change call */ | ||
1205 | -static void tcg_handle_interrupt(CPUState *cpu, int mask) | ||
1206 | +void tcg_handle_interrupt(CPUState *cpu, int mask) | ||
1207 | { | ||
1208 | - int old_mask; | ||
1209 | g_assert(qemu_mutex_iothread_locked()); | ||
1210 | |||
1211 | - old_mask = cpu->interrupt_request; | ||
1212 | cpu->interrupt_request |= mask; | ||
1213 | |||
1214 | /* | ||
1215 | @@ -XXX,XX +XXX,XX @@ static void tcg_handle_interrupt(CPUState *cpu, int mask) | ||
1216 | qemu_cpu_kick(cpu); | ||
1217 | } else { | ||
1218 | qatomic_set(&cpu_neg(cpu)->icount_decr.u16.high, -1); | ||
1219 | - if (icount_enabled() && | ||
1220 | - !cpu->can_do_io | ||
1221 | - && (mask & ~old_mask) != 0) { | ||
1222 | - cpu_abort(cpu, "Raised interrupt while not in I/O function"); | ||
1223 | - } | ||
1224 | } | ||
1225 | } | ||
1226 | - | ||
1227 | -const CpusAccel tcg_cpus = { | ||
1228 | - .create_vcpu_thread = tcg_start_vcpu_thread, | ||
1229 | - .kick_vcpu_thread = tcg_kick_vcpu_thread, | ||
1230 | - | ||
1231 | - .handle_interrupt = tcg_handle_interrupt, | ||
1232 | - | ||
1233 | - .get_virtual_clock = tcg_get_virtual_clock, | ||
1234 | - .get_elapsed_ticks = tcg_get_elapsed_ticks, | ||
1235 | -}; | ||
1236 | diff --git a/softmmu/icount.c b/softmmu/icount.c | ||
1237 | index XXXXXXX..XXXXXXX 100644 | ||
1238 | --- a/softmmu/icount.c | ||
1239 | +++ b/softmmu/icount.c | ||
1240 | @@ -XXX,XX +XXX,XX @@ void icount_start_warp_timer(void) | ||
1241 | |||
1242 | void icount_account_warp_timer(void) | ||
1243 | { | ||
1244 | - if (!icount_enabled() || !icount_sleep) { | ||
1245 | + if (!icount_sleep) { | ||
1246 | return; | ||
1247 | } | ||
1248 | |||
1249 | diff --git a/accel/tcg/meson.build b/accel/tcg/meson.build | ||
1250 | index XXXXXXX..XXXXXXX 100644 | ||
1251 | --- a/accel/tcg/meson.build | ||
1252 | +++ b/accel/tcg/meson.build | ||
1253 | @@ -XXX,XX +XXX,XX @@ tcg_ss.add(when: 'CONFIG_SOFTMMU', if_false: files('user-exec-stub.c')) | ||
1254 | tcg_ss.add(when: 'CONFIG_PLUGIN', if_true: [files('plugin-gen.c'), libdl]) | ||
1255 | specific_ss.add_all(when: 'CONFIG_TCG', if_true: tcg_ss) | ||
1256 | |||
1257 | -specific_ss.add(when: ['CONFIG_SOFTMMU', 'CONFIG_TCG'], if_true: files('tcg-all.c', 'cputlb.c', 'tcg-cpus.c')) | ||
1258 | +specific_ss.add(when: ['CONFIG_SOFTMMU', 'CONFIG_TCG'], if_true: files( | ||
1259 | + 'tcg-all.c', | ||
1260 | + 'cputlb.c', | ||
1261 | + 'tcg-cpus.c', | ||
1262 | + 'tcg-cpus-mttcg.c', | ||
1263 | + 'tcg-cpus-icount.c', | ||
1264 | + 'tcg-cpus-rr.c' | ||
1265 | +)) | ||
1266 | -- | 66 | -- |
1267 | 2.25.1 | 67 | 2.25.1 |
1268 | 68 | ||
1269 | 69 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use CASE_32_64 and CASE_64 to reduce ifdefs and merge | ||
2 | cases that are identical between 32-bit and 64-bit hosts. | ||
1 | 3 | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
6 | Message-Id: <20210217202036.1724901-5-richard.henderson@linaro.org> | ||
7 | [PMD: Split patch as 1/5] | ||
8 | Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
9 | Message-Id: <20210218232840.1760806-2-f4bug@amsat.org> | ||
10 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
11 | --- | ||
12 | tcg/tci/tcg-target.c.inc | 85 +++++++++++++++++----------------------- | ||
13 | 1 file changed, 37 insertions(+), 48 deletions(-) | ||
14 | |||
15 | diff --git a/tcg/tci/tcg-target.c.inc b/tcg/tci/tcg-target.c.inc | ||
16 | index XXXXXXX..XXXXXXX 100644 | ||
17 | --- a/tcg/tci/tcg-target.c.inc | ||
18 | +++ b/tcg/tci/tcg-target.c.inc | ||
19 | @@ -XXX,XX +XXX,XX @@ static inline void tcg_out_call(TCGContext *s, const tcg_insn_unit *arg) | ||
20 | old_code_ptr[1] = s->code_ptr - old_code_ptr; | ||
21 | } | ||
22 | |||
23 | +#if TCG_TARGET_REG_BITS == 64 | ||
24 | +# define CASE_32_64(x) \ | ||
25 | + case glue(glue(INDEX_op_, x), _i64): \ | ||
26 | + case glue(glue(INDEX_op_, x), _i32): | ||
27 | +# define CASE_64(x) \ | ||
28 | + case glue(glue(INDEX_op_, x), _i64): | ||
29 | +#else | ||
30 | +# define CASE_32_64(x) \ | ||
31 | + case glue(glue(INDEX_op_, x), _i32): | ||
32 | +# define CASE_64(x) | ||
33 | +#endif | ||
34 | + | ||
35 | static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
36 | const int *const_args) | ||
37 | { | ||
38 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
39 | case INDEX_op_exit_tb: | ||
40 | tcg_out64(s, args[0]); | ||
41 | break; | ||
42 | + | ||
43 | case INDEX_op_goto_tb: | ||
44 | if (s->tb_jmp_insn_offset) { | ||
45 | /* Direct jump method. */ | ||
46 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
47 | tcg_debug_assert(args[2] == (int32_t)args[2]); | ||
48 | tcg_out32(s, args[2]); | ||
49 | break; | ||
50 | - case INDEX_op_add_i32: | ||
51 | - case INDEX_op_sub_i32: | ||
52 | - case INDEX_op_mul_i32: | ||
53 | - case INDEX_op_and_i32: | ||
54 | - case INDEX_op_andc_i32: /* Optional (TCG_TARGET_HAS_andc_i32). */ | ||
55 | - case INDEX_op_eqv_i32: /* Optional (TCG_TARGET_HAS_eqv_i32). */ | ||
56 | - case INDEX_op_nand_i32: /* Optional (TCG_TARGET_HAS_nand_i32). */ | ||
57 | - case INDEX_op_nor_i32: /* Optional (TCG_TARGET_HAS_nor_i32). */ | ||
58 | - case INDEX_op_or_i32: | ||
59 | - case INDEX_op_orc_i32: /* Optional (TCG_TARGET_HAS_orc_i32). */ | ||
60 | - case INDEX_op_xor_i32: | ||
61 | - case INDEX_op_shl_i32: | ||
62 | - case INDEX_op_shr_i32: | ||
63 | - case INDEX_op_sar_i32: | ||
64 | - case INDEX_op_rotl_i32: /* Optional (TCG_TARGET_HAS_rot_i32). */ | ||
65 | - case INDEX_op_rotr_i32: /* Optional (TCG_TARGET_HAS_rot_i32). */ | ||
66 | + | ||
67 | + CASE_32_64(add) | ||
68 | + CASE_32_64(sub) | ||
69 | + CASE_32_64(mul) | ||
70 | + CASE_32_64(and) | ||
71 | + CASE_32_64(or) | ||
72 | + CASE_32_64(xor) | ||
73 | + CASE_32_64(andc) /* Optional (TCG_TARGET_HAS_andc_*). */ | ||
74 | + CASE_32_64(orc) /* Optional (TCG_TARGET_HAS_orc_*). */ | ||
75 | + CASE_32_64(eqv) /* Optional (TCG_TARGET_HAS_eqv_*). */ | ||
76 | + CASE_32_64(nand) /* Optional (TCG_TARGET_HAS_nand_*). */ | ||
77 | + CASE_32_64(nor) /* Optional (TCG_TARGET_HAS_nor_*). */ | ||
78 | + CASE_32_64(shl) | ||
79 | + CASE_32_64(shr) | ||
80 | + CASE_32_64(sar) | ||
81 | + CASE_32_64(rotl) /* Optional (TCG_TARGET_HAS_rot_*). */ | ||
82 | + CASE_32_64(rotr) /* Optional (TCG_TARGET_HAS_rot_*). */ | ||
83 | + CASE_32_64(div) /* Optional (TCG_TARGET_HAS_div_*). */ | ||
84 | + CASE_32_64(divu) /* Optional (TCG_TARGET_HAS_div_*). */ | ||
85 | + CASE_32_64(rem) /* Optional (TCG_TARGET_HAS_div_*). */ | ||
86 | + CASE_32_64(remu) /* Optional (TCG_TARGET_HAS_div_*). */ | ||
87 | tcg_out_r(s, args[0]); | ||
88 | tcg_out_r(s, args[1]); | ||
89 | tcg_out_r(s, args[2]); | ||
90 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
91 | break; | ||
92 | |||
93 | #if TCG_TARGET_REG_BITS == 64 | ||
94 | - case INDEX_op_add_i64: | ||
95 | - case INDEX_op_sub_i64: | ||
96 | - case INDEX_op_mul_i64: | ||
97 | - case INDEX_op_and_i64: | ||
98 | - case INDEX_op_andc_i64: /* Optional (TCG_TARGET_HAS_andc_i64). */ | ||
99 | - case INDEX_op_eqv_i64: /* Optional (TCG_TARGET_HAS_eqv_i64). */ | ||
100 | - case INDEX_op_nand_i64: /* Optional (TCG_TARGET_HAS_nand_i64). */ | ||
101 | - case INDEX_op_nor_i64: /* Optional (TCG_TARGET_HAS_nor_i64). */ | ||
102 | - case INDEX_op_or_i64: | ||
103 | - case INDEX_op_orc_i64: /* Optional (TCG_TARGET_HAS_orc_i64). */ | ||
104 | - case INDEX_op_xor_i64: | ||
105 | - case INDEX_op_shl_i64: | ||
106 | - case INDEX_op_shr_i64: | ||
107 | - case INDEX_op_sar_i64: | ||
108 | - case INDEX_op_rotl_i64: /* Optional (TCG_TARGET_HAS_rot_i64). */ | ||
109 | - case INDEX_op_rotr_i64: /* Optional (TCG_TARGET_HAS_rot_i64). */ | ||
110 | - case INDEX_op_div_i64: /* Optional (TCG_TARGET_HAS_div_i64). */ | ||
111 | - case INDEX_op_divu_i64: /* Optional (TCG_TARGET_HAS_div_i64). */ | ||
112 | - case INDEX_op_rem_i64: /* Optional (TCG_TARGET_HAS_div_i64). */ | ||
113 | - case INDEX_op_remu_i64: /* Optional (TCG_TARGET_HAS_div_i64). */ | ||
114 | - tcg_out_r(s, args[0]); | ||
115 | - tcg_out_r(s, args[1]); | ||
116 | - tcg_out_r(s, args[2]); | ||
117 | - break; | ||
118 | case INDEX_op_deposit_i64: /* Optional (TCG_TARGET_HAS_deposit_i64). */ | ||
119 | tcg_out_r(s, args[0]); | ||
120 | tcg_out_r(s, args[1]); | ||
121 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
122 | tcg_out_r(s, args[0]); | ||
123 | tcg_out_r(s, args[1]); | ||
124 | break; | ||
125 | - case INDEX_op_div_i32: /* Optional (TCG_TARGET_HAS_div_i32). */ | ||
126 | - case INDEX_op_divu_i32: /* Optional (TCG_TARGET_HAS_div_i32). */ | ||
127 | - case INDEX_op_rem_i32: /* Optional (TCG_TARGET_HAS_div_i32). */ | ||
128 | - case INDEX_op_remu_i32: /* Optional (TCG_TARGET_HAS_div_i32). */ | ||
129 | - tcg_out_r(s, args[0]); | ||
130 | - tcg_out_r(s, args[1]); | ||
131 | - tcg_out_r(s, args[2]); | ||
132 | - break; | ||
133 | + | ||
134 | #if TCG_TARGET_REG_BITS == 32 | ||
135 | case INDEX_op_add2_i32: | ||
136 | case INDEX_op_sub2_i32: | ||
137 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
138 | } | ||
139 | tcg_out_i(s, *args++); | ||
140 | break; | ||
141 | + | ||
142 | case INDEX_op_mb: | ||
143 | break; | ||
144 | + | ||
145 | case INDEX_op_mov_i32: /* Always emitted via tcg_out_mov. */ | ||
146 | case INDEX_op_mov_i64: | ||
147 | case INDEX_op_call: /* Always emitted via tcg_out_call. */ | ||
148 | -- | ||
149 | 2.25.1 | ||
150 | |||
151 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use CASE_32_64 and CASE_64 to reduce ifdefs and merge | ||
2 | cases that are identical between 32-bit and 64-bit hosts. | ||
1 | 3 | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
6 | Message-Id: <20210217202036.1724901-5-richard.henderson@linaro.org> | ||
7 | [PMD: Split patch as 2/5] | ||
8 | Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
9 | Message-Id: <20210218232840.1760806-3-f4bug@amsat.org> | ||
10 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
11 | --- | ||
12 | tcg/tci/tcg-target.c.inc | 35 ++++++++++++++--------------------- | ||
13 | 1 file changed, 14 insertions(+), 21 deletions(-) | ||
14 | |||
15 | diff --git a/tcg/tci/tcg-target.c.inc b/tcg/tci/tcg-target.c.inc | ||
16 | index XXXXXXX..XXXXXXX 100644 | ||
17 | --- a/tcg/tci/tcg-target.c.inc | ||
18 | +++ b/tcg/tci/tcg-target.c.inc | ||
19 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
20 | tcg_out8(s, args[2]); /* condition */ | ||
21 | tci_out_label(s, arg_label(args[3])); | ||
22 | break; | ||
23 | - case INDEX_op_bswap16_i64: /* Optional (TCG_TARGET_HAS_bswap16_i64). */ | ||
24 | - case INDEX_op_bswap32_i64: /* Optional (TCG_TARGET_HAS_bswap32_i64). */ | ||
25 | - case INDEX_op_bswap64_i64: /* Optional (TCG_TARGET_HAS_bswap64_i64). */ | ||
26 | - case INDEX_op_not_i64: /* Optional (TCG_TARGET_HAS_not_i64). */ | ||
27 | - case INDEX_op_neg_i64: /* Optional (TCG_TARGET_HAS_neg_i64). */ | ||
28 | - case INDEX_op_ext8s_i64: /* Optional (TCG_TARGET_HAS_ext8s_i64). */ | ||
29 | - case INDEX_op_ext8u_i64: /* Optional (TCG_TARGET_HAS_ext8u_i64). */ | ||
30 | - case INDEX_op_ext16s_i64: /* Optional (TCG_TARGET_HAS_ext16s_i64). */ | ||
31 | - case INDEX_op_ext16u_i64: /* Optional (TCG_TARGET_HAS_ext16u_i64). */ | ||
32 | - case INDEX_op_ext32s_i64: /* Optional (TCG_TARGET_HAS_ext32s_i64). */ | ||
33 | - case INDEX_op_ext32u_i64: /* Optional (TCG_TARGET_HAS_ext32u_i64). */ | ||
34 | - case INDEX_op_ext_i32_i64: | ||
35 | - case INDEX_op_extu_i32_i64: | ||
36 | #endif /* TCG_TARGET_REG_BITS == 64 */ | ||
37 | - case INDEX_op_neg_i32: /* Optional (TCG_TARGET_HAS_neg_i32). */ | ||
38 | - case INDEX_op_not_i32: /* Optional (TCG_TARGET_HAS_not_i32). */ | ||
39 | - case INDEX_op_ext8s_i32: /* Optional (TCG_TARGET_HAS_ext8s_i32). */ | ||
40 | - case INDEX_op_ext16s_i32: /* Optional (TCG_TARGET_HAS_ext16s_i32). */ | ||
41 | - case INDEX_op_ext8u_i32: /* Optional (TCG_TARGET_HAS_ext8u_i32). */ | ||
42 | - case INDEX_op_ext16u_i32: /* Optional (TCG_TARGET_HAS_ext16u_i32). */ | ||
43 | - case INDEX_op_bswap16_i32: /* Optional (TCG_TARGET_HAS_bswap16_i32). */ | ||
44 | - case INDEX_op_bswap32_i32: /* Optional (TCG_TARGET_HAS_bswap32_i32). */ | ||
45 | + | ||
46 | + CASE_32_64(neg) /* Optional (TCG_TARGET_HAS_neg_*). */ | ||
47 | + CASE_32_64(not) /* Optional (TCG_TARGET_HAS_not_*). */ | ||
48 | + CASE_32_64(ext8s) /* Optional (TCG_TARGET_HAS_ext8s_*). */ | ||
49 | + CASE_32_64(ext8u) /* Optional (TCG_TARGET_HAS_ext8u_*). */ | ||
50 | + CASE_32_64(ext16s) /* Optional (TCG_TARGET_HAS_ext16s_*). */ | ||
51 | + CASE_32_64(ext16u) /* Optional (TCG_TARGET_HAS_ext16u_*). */ | ||
52 | + CASE_64(ext32s) /* Optional (TCG_TARGET_HAS_ext32s_i64). */ | ||
53 | + CASE_64(ext32u) /* Optional (TCG_TARGET_HAS_ext32u_i64). */ | ||
54 | + CASE_64(ext_i32) | ||
55 | + CASE_64(extu_i32) | ||
56 | + CASE_32_64(bswap16) /* Optional (TCG_TARGET_HAS_bswap16_*). */ | ||
57 | + CASE_32_64(bswap32) /* Optional (TCG_TARGET_HAS_bswap32_*). */ | ||
58 | + CASE_64(bswap64) /* Optional (TCG_TARGET_HAS_bswap64_i64). */ | ||
59 | tcg_out_r(s, args[0]); | ||
60 | tcg_out_r(s, args[1]); | ||
61 | break; | ||
62 | -- | ||
63 | 2.25.1 | ||
64 | |||
65 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use CASE_32_64 and CASE_64 to reduce ifdefs and merge | ||
2 | cases that are identical between 32-bit and 64-bit hosts. | ||
1 | 3 | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
6 | Message-Id: <20210217202036.1724901-5-richard.henderson@linaro.org> | ||
7 | [PMD: Split patch as 3/5] | ||
8 | Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
9 | Message-Id: <20210218232840.1760806-4-f4bug@amsat.org> | ||
10 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
11 | --- | ||
12 | tcg/tci/tcg-target.c.inc | 12 ++---------- | ||
13 | 1 file changed, 2 insertions(+), 10 deletions(-) | ||
14 | |||
15 | diff --git a/tcg/tci/tcg-target.c.inc b/tcg/tci/tcg-target.c.inc | ||
16 | index XXXXXXX..XXXXXXX 100644 | ||
17 | --- a/tcg/tci/tcg-target.c.inc | ||
18 | +++ b/tcg/tci/tcg-target.c.inc | ||
19 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
20 | tcg_out_r(s, args[1]); | ||
21 | tcg_out_r(s, args[2]); | ||
22 | break; | ||
23 | - case INDEX_op_deposit_i32: /* Optional (TCG_TARGET_HAS_deposit_i32). */ | ||
24 | + | ||
25 | + CASE_32_64(deposit) /* Optional (TCG_TARGET_HAS_deposit_*). */ | ||
26 | tcg_out_r(s, args[0]); | ||
27 | tcg_out_r(s, args[1]); | ||
28 | tcg_out_r(s, args[2]); | ||
29 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
30 | break; | ||
31 | |||
32 | #if TCG_TARGET_REG_BITS == 64 | ||
33 | - case INDEX_op_deposit_i64: /* Optional (TCG_TARGET_HAS_deposit_i64). */ | ||
34 | - tcg_out_r(s, args[0]); | ||
35 | - tcg_out_r(s, args[1]); | ||
36 | - tcg_out_r(s, args[2]); | ||
37 | - tcg_debug_assert(args[3] <= UINT8_MAX); | ||
38 | - tcg_out8(s, args[3]); | ||
39 | - tcg_debug_assert(args[4] <= UINT8_MAX); | ||
40 | - tcg_out8(s, args[4]); | ||
41 | - break; | ||
42 | case INDEX_op_brcond_i64: | ||
43 | tcg_out_r(s, args[0]); | ||
44 | tcg_out_r(s, args[1]); | ||
45 | -- | ||
46 | 2.25.1 | ||
47 | |||
48 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use CASE_32_64 and CASE_64 to reduce ifdefs and merge | ||
2 | cases that are identical between 32-bit and 64-bit hosts. | ||
1 | 3 | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
6 | Message-Id: <20210217202036.1724901-5-richard.henderson@linaro.org> | ||
7 | [PMD: Split patch as 4/5] | ||
8 | Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
9 | Message-Id: <20210218232840.1760806-5-f4bug@amsat.org> | ||
10 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
11 | --- | ||
12 | tcg/tci/tcg-target.c.inc | 23 ++++++----------------- | ||
13 | 1 file changed, 6 insertions(+), 17 deletions(-) | ||
14 | |||
15 | diff --git a/tcg/tci/tcg-target.c.inc b/tcg/tci/tcg-target.c.inc | ||
16 | index XXXXXXX..XXXXXXX 100644 | ||
17 | --- a/tcg/tci/tcg-target.c.inc | ||
18 | +++ b/tcg/tci/tcg-target.c.inc | ||
19 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
20 | } | ||
21 | set_jmp_reset_offset(s, args[0]); | ||
22 | break; | ||
23 | + | ||
24 | case INDEX_op_br: | ||
25 | tci_out_label(s, arg_label(args[0])); | ||
26 | break; | ||
27 | - case INDEX_op_setcond_i32: | ||
28 | + | ||
29 | + CASE_32_64(setcond) | ||
30 | tcg_out_r(s, args[0]); | ||
31 | tcg_out_r(s, args[1]); | ||
32 | tcg_out_r(s, args[2]); | ||
33 | tcg_out8(s, args[3]); /* condition */ | ||
34 | break; | ||
35 | + | ||
36 | #if TCG_TARGET_REG_BITS == 32 | ||
37 | case INDEX_op_setcond2_i32: | ||
38 | /* setcond2_i32 cond, t0, t1_low, t1_high, t2_low, t2_high */ | ||
39 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
40 | tcg_out_r(s, args[4]); | ||
41 | tcg_out8(s, args[5]); /* condition */ | ||
42 | break; | ||
43 | -#elif TCG_TARGET_REG_BITS == 64 | ||
44 | - case INDEX_op_setcond_i64: | ||
45 | - tcg_out_r(s, args[0]); | ||
46 | - tcg_out_r(s, args[1]); | ||
47 | - tcg_out_r(s, args[2]); | ||
48 | - tcg_out8(s, args[3]); /* condition */ | ||
49 | - break; | ||
50 | #endif | ||
51 | case INDEX_op_ld8u_i32: | ||
52 | case INDEX_op_ld8s_i32: | ||
53 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
54 | tcg_out8(s, args[4]); | ||
55 | break; | ||
56 | |||
57 | -#if TCG_TARGET_REG_BITS == 64 | ||
58 | - case INDEX_op_brcond_i64: | ||
59 | + CASE_32_64(brcond) | ||
60 | tcg_out_r(s, args[0]); | ||
61 | tcg_out_r(s, args[1]); | ||
62 | tcg_out8(s, args[2]); /* condition */ | ||
63 | tci_out_label(s, arg_label(args[3])); | ||
64 | break; | ||
65 | -#endif /* TCG_TARGET_REG_BITS == 64 */ | ||
66 | |||
67 | CASE_32_64(neg) /* Optional (TCG_TARGET_HAS_neg_*). */ | ||
68 | CASE_32_64(not) /* Optional (TCG_TARGET_HAS_not_*). */ | ||
69 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
70 | tcg_out_r(s, args[3]); | ||
71 | break; | ||
72 | #endif | ||
73 | - case INDEX_op_brcond_i32: | ||
74 | - tcg_out_r(s, args[0]); | ||
75 | - tcg_out_r(s, args[1]); | ||
76 | - tcg_out8(s, args[2]); /* condition */ | ||
77 | - tci_out_label(s, arg_label(args[3])); | ||
78 | - break; | ||
79 | + | ||
80 | case INDEX_op_qemu_ld_i32: | ||
81 | tcg_out_r(s, *args++); | ||
82 | tcg_out_r(s, *args++); | ||
83 | -- | ||
84 | 2.25.1 | ||
85 | |||
86 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use CASE_32_64 and CASE_64 to reduce ifdefs and merge | ||
2 | cases that are identical between 32-bit and 64-bit hosts. | ||
1 | 3 | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
6 | Message-Id: <20210217202036.1724901-5-richard.henderson@linaro.org> | ||
7 | [PMD: Split patch as 5/5] | ||
8 | Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
9 | Message-Id: <20210218232840.1760806-6-f4bug@amsat.org> | ||
10 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
11 | --- | ||
12 | tcg/tci/tcg-target.c.inc | 49 ++++++++++++---------------------------- | ||
13 | 1 file changed, 14 insertions(+), 35 deletions(-) | ||
14 | |||
15 | diff --git a/tcg/tci/tcg-target.c.inc b/tcg/tci/tcg-target.c.inc | ||
16 | index XXXXXXX..XXXXXXX 100644 | ||
17 | --- a/tcg/tci/tcg-target.c.inc | ||
18 | +++ b/tcg/tci/tcg-target.c.inc | ||
19 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
20 | tcg_out8(s, args[5]); /* condition */ | ||
21 | break; | ||
22 | #endif | ||
23 | - case INDEX_op_ld8u_i32: | ||
24 | - case INDEX_op_ld8s_i32: | ||
25 | - case INDEX_op_ld16u_i32: | ||
26 | - case INDEX_op_ld16s_i32: | ||
27 | + | ||
28 | + CASE_32_64(ld8u) | ||
29 | + CASE_32_64(ld8s) | ||
30 | + CASE_32_64(ld16u) | ||
31 | + CASE_32_64(ld16s) | ||
32 | case INDEX_op_ld_i32: | ||
33 | - case INDEX_op_st8_i32: | ||
34 | - case INDEX_op_st16_i32: | ||
35 | + CASE_64(ld32u) | ||
36 | + CASE_64(ld32s) | ||
37 | + CASE_64(ld) | ||
38 | + CASE_32_64(st8) | ||
39 | + CASE_32_64(st16) | ||
40 | case INDEX_op_st_i32: | ||
41 | - case INDEX_op_ld8u_i64: | ||
42 | - case INDEX_op_ld8s_i64: | ||
43 | - case INDEX_op_ld16u_i64: | ||
44 | - case INDEX_op_ld16s_i64: | ||
45 | - case INDEX_op_ld32u_i64: | ||
46 | - case INDEX_op_ld32s_i64: | ||
47 | - case INDEX_op_ld_i64: | ||
48 | - case INDEX_op_st8_i64: | ||
49 | - case INDEX_op_st16_i64: | ||
50 | - case INDEX_op_st32_i64: | ||
51 | - case INDEX_op_st_i64: | ||
52 | + CASE_64(st32) | ||
53 | + CASE_64(st) | ||
54 | stack_bounds_check(args[1], args[2]); | ||
55 | tcg_out_r(s, args[0]); | ||
56 | tcg_out_r(s, args[1]); | ||
57 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
58 | #endif | ||
59 | |||
60 | case INDEX_op_qemu_ld_i32: | ||
61 | - tcg_out_r(s, *args++); | ||
62 | - tcg_out_r(s, *args++); | ||
63 | - if (TARGET_LONG_BITS > TCG_TARGET_REG_BITS) { | ||
64 | - tcg_out_r(s, *args++); | ||
65 | - } | ||
66 | - tcg_out_i(s, *args++); | ||
67 | - break; | ||
68 | - case INDEX_op_qemu_ld_i64: | ||
69 | - tcg_out_r(s, *args++); | ||
70 | - if (TCG_TARGET_REG_BITS == 32) { | ||
71 | - tcg_out_r(s, *args++); | ||
72 | - } | ||
73 | - tcg_out_r(s, *args++); | ||
74 | - if (TARGET_LONG_BITS > TCG_TARGET_REG_BITS) { | ||
75 | - tcg_out_r(s, *args++); | ||
76 | - } | ||
77 | - tcg_out_i(s, *args++); | ||
78 | - break; | ||
79 | case INDEX_op_qemu_st_i32: | ||
80 | tcg_out_r(s, *args++); | ||
81 | tcg_out_r(s, *args++); | ||
82 | @@ -XXX,XX +XXX,XX @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc, const TCGArg *args, | ||
83 | } | ||
84 | tcg_out_i(s, *args++); | ||
85 | break; | ||
86 | + | ||
87 | + case INDEX_op_qemu_ld_i64: | ||
88 | case INDEX_op_qemu_st_i64: | ||
89 | tcg_out_r(s, *args++); | ||
90 | if (TCG_TARGET_REG_BITS == 32) { | ||
91 | -- | ||
92 | 2.25.1 | ||
93 | |||
94 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use explicit casts for ext8u opcodes, and allow truncation | ||
2 | to happen with the store for st8 opcodes. | ||
1 | 3 | ||
4 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
5 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
6 | --- | ||
7 | tcg/tci.c | 23 +++++------------------ | ||
8 | 1 file changed, 5 insertions(+), 18 deletions(-) | ||
9 | |||
10 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
11 | index XXXXXXX..XXXXXXX 100644 | ||
12 | --- a/tcg/tci.c | ||
13 | +++ b/tcg/tci.c | ||
14 | @@ -XXX,XX +XXX,XX @@ static int32_t tci_read_reg32s(const tcg_target_ulong *regs, TCGReg index) | ||
15 | } | ||
16 | #endif | ||
17 | |||
18 | -static uint8_t tci_read_reg8(const tcg_target_ulong *regs, TCGReg index) | ||
19 | -{ | ||
20 | - return (uint8_t)tci_read_reg(regs, index); | ||
21 | -} | ||
22 | - | ||
23 | static uint16_t tci_read_reg16(const tcg_target_ulong *regs, TCGReg index) | ||
24 | { | ||
25 | return (uint16_t)tci_read_reg(regs, index); | ||
26 | @@ -XXX,XX +XXX,XX @@ tci_read_r(const tcg_target_ulong *regs, const uint8_t **tb_ptr) | ||
27 | return value; | ||
28 | } | ||
29 | |||
30 | -/* Read indexed register (8 bit) from bytecode. */ | ||
31 | -static uint8_t tci_read_r8(const tcg_target_ulong *regs, const uint8_t **tb_ptr) | ||
32 | -{ | ||
33 | - uint8_t value = tci_read_reg8(regs, **tb_ptr); | ||
34 | - *tb_ptr += 1; | ||
35 | - return value; | ||
36 | -} | ||
37 | - | ||
38 | #if TCG_TARGET_HAS_ext8s_i32 || TCG_TARGET_HAS_ext8s_i64 | ||
39 | /* Read indexed register (8 bit signed) from bytecode. */ | ||
40 | static int8_t tci_read_r8s(const tcg_target_ulong *regs, const uint8_t **tb_ptr) | ||
41 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
42 | tci_write_reg(regs, t0, *(uint32_t *)(t1 + t2)); | ||
43 | break; | ||
44 | CASE_32_64(st8) | ||
45 | - t0 = tci_read_r8(regs, &tb_ptr); | ||
46 | + t0 = tci_read_r(regs, &tb_ptr); | ||
47 | t1 = tci_read_r(regs, &tb_ptr); | ||
48 | t2 = tci_read_s32(&tb_ptr); | ||
49 | *(uint8_t *)(t1 + t2) = t0; | ||
50 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
51 | #if TCG_TARGET_HAS_ext8u_i32 | ||
52 | case INDEX_op_ext8u_i32: | ||
53 | t0 = *tb_ptr++; | ||
54 | - t1 = tci_read_r8(regs, &tb_ptr); | ||
55 | - tci_write_reg(regs, t0, t1); | ||
56 | + t1 = tci_read_r(regs, &tb_ptr); | ||
57 | + tci_write_reg(regs, t0, (uint8_t)t1); | ||
58 | break; | ||
59 | #endif | ||
60 | #if TCG_TARGET_HAS_ext16u_i32 | ||
61 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
62 | #if TCG_TARGET_HAS_ext8u_i64 | ||
63 | case INDEX_op_ext8u_i64: | ||
64 | t0 = *tb_ptr++; | ||
65 | - t1 = tci_read_r8(regs, &tb_ptr); | ||
66 | - tci_write_reg(regs, t0, t1); | ||
67 | + t1 = tci_read_r(regs, &tb_ptr); | ||
68 | + tci_write_reg(regs, t0, (uint8_t)t1); | ||
69 | break; | ||
70 | #endif | ||
71 | #if TCG_TARGET_HAS_ext8s_i64 | ||
72 | -- | ||
73 | 2.25.1 | ||
74 | |||
75 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use explicit casts for ext8s opcodes. | ||
1 | 2 | ||
3 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | --- | ||
6 | tcg/tci.c | 25 ++++--------------------- | ||
7 | 1 file changed, 4 insertions(+), 21 deletions(-) | ||
8 | |||
9 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
10 | index XXXXXXX..XXXXXXX 100644 | ||
11 | --- a/tcg/tci.c | ||
12 | +++ b/tcg/tci.c | ||
13 | @@ -XXX,XX +XXX,XX @@ static tcg_target_ulong tci_read_reg(const tcg_target_ulong *regs, TCGReg index) | ||
14 | return regs[index]; | ||
15 | } | ||
16 | |||
17 | -#if TCG_TARGET_HAS_ext8s_i32 || TCG_TARGET_HAS_ext8s_i64 | ||
18 | -static int8_t tci_read_reg8s(const tcg_target_ulong *regs, TCGReg index) | ||
19 | -{ | ||
20 | - return (int8_t)tci_read_reg(regs, index); | ||
21 | -} | ||
22 | -#endif | ||
23 | - | ||
24 | #if TCG_TARGET_HAS_ext16s_i32 || TCG_TARGET_HAS_ext16s_i64 | ||
25 | static int16_t tci_read_reg16s(const tcg_target_ulong *regs, TCGReg index) | ||
26 | { | ||
27 | @@ -XXX,XX +XXX,XX @@ tci_read_r(const tcg_target_ulong *regs, const uint8_t **tb_ptr) | ||
28 | return value; | ||
29 | } | ||
30 | |||
31 | -#if TCG_TARGET_HAS_ext8s_i32 || TCG_TARGET_HAS_ext8s_i64 | ||
32 | -/* Read indexed register (8 bit signed) from bytecode. */ | ||
33 | -static int8_t tci_read_r8s(const tcg_target_ulong *regs, const uint8_t **tb_ptr) | ||
34 | -{ | ||
35 | - int8_t value = tci_read_reg8s(regs, **tb_ptr); | ||
36 | - *tb_ptr += 1; | ||
37 | - return value; | ||
38 | -} | ||
39 | -#endif | ||
40 | - | ||
41 | /* Read indexed register (16 bit) from bytecode. */ | ||
42 | static uint16_t tci_read_r16(const tcg_target_ulong *regs, | ||
43 | const uint8_t **tb_ptr) | ||
44 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
45 | #if TCG_TARGET_HAS_ext8s_i32 | ||
46 | case INDEX_op_ext8s_i32: | ||
47 | t0 = *tb_ptr++; | ||
48 | - t1 = tci_read_r8s(regs, &tb_ptr); | ||
49 | - tci_write_reg(regs, t0, t1); | ||
50 | + t1 = tci_read_r(regs, &tb_ptr); | ||
51 | + tci_write_reg(regs, t0, (int8_t)t1); | ||
52 | break; | ||
53 | #endif | ||
54 | #if TCG_TARGET_HAS_ext16s_i32 | ||
55 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
56 | #if TCG_TARGET_HAS_ext8s_i64 | ||
57 | case INDEX_op_ext8s_i64: | ||
58 | t0 = *tb_ptr++; | ||
59 | - t1 = tci_read_r8s(regs, &tb_ptr); | ||
60 | - tci_write_reg(regs, t0, t1); | ||
61 | + t1 = tci_read_r(regs, &tb_ptr); | ||
62 | + tci_write_reg(regs, t0, (int8_t)t1); | ||
63 | break; | ||
64 | #endif | ||
65 | #if TCG_TARGET_HAS_ext16s_i64 | ||
66 | -- | ||
67 | 2.25.1 | ||
68 | |||
69 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use explicit casts for ext16u opcodes, and allow truncation | ||
2 | to happen with the store for st16 opcodes, and with the call | ||
3 | for bswap16 opcodes. | ||
1 | 4 | ||
5 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
6 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
7 | --- | ||
8 | tcg/tci.c | 28 +++++++--------------------- | ||
9 | 1 file changed, 7 insertions(+), 21 deletions(-) | ||
10 | |||
11 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
12 | index XXXXXXX..XXXXXXX 100644 | ||
13 | --- a/tcg/tci.c | ||
14 | +++ b/tcg/tci.c | ||
15 | @@ -XXX,XX +XXX,XX @@ static int32_t tci_read_reg32s(const tcg_target_ulong *regs, TCGReg index) | ||
16 | } | ||
17 | #endif | ||
18 | |||
19 | -static uint16_t tci_read_reg16(const tcg_target_ulong *regs, TCGReg index) | ||
20 | -{ | ||
21 | - return (uint16_t)tci_read_reg(regs, index); | ||
22 | -} | ||
23 | - | ||
24 | static uint32_t tci_read_reg32(const tcg_target_ulong *regs, TCGReg index) | ||
25 | { | ||
26 | return (uint32_t)tci_read_reg(regs, index); | ||
27 | @@ -XXX,XX +XXX,XX @@ tci_read_r(const tcg_target_ulong *regs, const uint8_t **tb_ptr) | ||
28 | return value; | ||
29 | } | ||
30 | |||
31 | -/* Read indexed register (16 bit) from bytecode. */ | ||
32 | -static uint16_t tci_read_r16(const tcg_target_ulong *regs, | ||
33 | - const uint8_t **tb_ptr) | ||
34 | -{ | ||
35 | - uint16_t value = tci_read_reg16(regs, **tb_ptr); | ||
36 | - *tb_ptr += 1; | ||
37 | - return value; | ||
38 | -} | ||
39 | - | ||
40 | #if TCG_TARGET_HAS_ext16s_i32 || TCG_TARGET_HAS_ext16s_i64 | ||
41 | /* Read indexed register (16 bit signed) from bytecode. */ | ||
42 | static int16_t tci_read_r16s(const tcg_target_ulong *regs, | ||
43 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
44 | *(uint8_t *)(t1 + t2) = t0; | ||
45 | break; | ||
46 | CASE_32_64(st16) | ||
47 | - t0 = tci_read_r16(regs, &tb_ptr); | ||
48 | + t0 = tci_read_r(regs, &tb_ptr); | ||
49 | t1 = tci_read_r(regs, &tb_ptr); | ||
50 | t2 = tci_read_s32(&tb_ptr); | ||
51 | *(uint16_t *)(t1 + t2) = t0; | ||
52 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
53 | #if TCG_TARGET_HAS_ext16u_i32 | ||
54 | case INDEX_op_ext16u_i32: | ||
55 | t0 = *tb_ptr++; | ||
56 | - t1 = tci_read_r16(regs, &tb_ptr); | ||
57 | - tci_write_reg(regs, t0, t1); | ||
58 | + t1 = tci_read_r(regs, &tb_ptr); | ||
59 | + tci_write_reg(regs, t0, (uint16_t)t1); | ||
60 | break; | ||
61 | #endif | ||
62 | #if TCG_TARGET_HAS_bswap16_i32 | ||
63 | case INDEX_op_bswap16_i32: | ||
64 | t0 = *tb_ptr++; | ||
65 | - t1 = tci_read_r16(regs, &tb_ptr); | ||
66 | + t1 = tci_read_r(regs, &tb_ptr); | ||
67 | tci_write_reg(regs, t0, bswap16(t1)); | ||
68 | break; | ||
69 | #endif | ||
70 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
71 | #if TCG_TARGET_HAS_ext16u_i64 | ||
72 | case INDEX_op_ext16u_i64: | ||
73 | t0 = *tb_ptr++; | ||
74 | - t1 = tci_read_r16(regs, &tb_ptr); | ||
75 | - tci_write_reg(regs, t0, t1); | ||
76 | + t1 = tci_read_r(regs, &tb_ptr); | ||
77 | + tci_write_reg(regs, t0, (uint16_t)t1); | ||
78 | break; | ||
79 | #endif | ||
80 | #if TCG_TARGET_HAS_ext32s_i64 | ||
81 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
82 | #if TCG_TARGET_HAS_bswap16_i64 | ||
83 | case INDEX_op_bswap16_i64: | ||
84 | t0 = *tb_ptr++; | ||
85 | - t1 = tci_read_r16(regs, &tb_ptr); | ||
86 | + t1 = tci_read_r(regs, &tb_ptr); | ||
87 | tci_write_reg(regs, t0, bswap16(t1)); | ||
88 | break; | ||
89 | #endif | ||
90 | -- | ||
91 | 2.25.1 | ||
92 | |||
93 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use explicit casts for ext16s opcodes. | ||
1 | 2 | ||
3 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | --- | ||
6 | tcg/tci.c | 26 ++++---------------------- | ||
7 | 1 file changed, 4 insertions(+), 22 deletions(-) | ||
8 | |||
9 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
10 | index XXXXXXX..XXXXXXX 100644 | ||
11 | --- a/tcg/tci.c | ||
12 | +++ b/tcg/tci.c | ||
13 | @@ -XXX,XX +XXX,XX @@ static tcg_target_ulong tci_read_reg(const tcg_target_ulong *regs, TCGReg index) | ||
14 | return regs[index]; | ||
15 | } | ||
16 | |||
17 | -#if TCG_TARGET_HAS_ext16s_i32 || TCG_TARGET_HAS_ext16s_i64 | ||
18 | -static int16_t tci_read_reg16s(const tcg_target_ulong *regs, TCGReg index) | ||
19 | -{ | ||
20 | - return (int16_t)tci_read_reg(regs, index); | ||
21 | -} | ||
22 | -#endif | ||
23 | - | ||
24 | #if TCG_TARGET_REG_BITS == 64 | ||
25 | static int32_t tci_read_reg32s(const tcg_target_ulong *regs, TCGReg index) | ||
26 | { | ||
27 | @@ -XXX,XX +XXX,XX @@ tci_read_r(const tcg_target_ulong *regs, const uint8_t **tb_ptr) | ||
28 | return value; | ||
29 | } | ||
30 | |||
31 | -#if TCG_TARGET_HAS_ext16s_i32 || TCG_TARGET_HAS_ext16s_i64 | ||
32 | -/* Read indexed register (16 bit signed) from bytecode. */ | ||
33 | -static int16_t tci_read_r16s(const tcg_target_ulong *regs, | ||
34 | - const uint8_t **tb_ptr) | ||
35 | -{ | ||
36 | - int16_t value = tci_read_reg16s(regs, **tb_ptr); | ||
37 | - *tb_ptr += 1; | ||
38 | - return value; | ||
39 | -} | ||
40 | -#endif | ||
41 | - | ||
42 | /* Read indexed register (32 bit) from bytecode. */ | ||
43 | static uint32_t tci_read_r32(const tcg_target_ulong *regs, | ||
44 | const uint8_t **tb_ptr) | ||
45 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
46 | #if TCG_TARGET_HAS_ext16s_i32 | ||
47 | case INDEX_op_ext16s_i32: | ||
48 | t0 = *tb_ptr++; | ||
49 | - t1 = tci_read_r16s(regs, &tb_ptr); | ||
50 | - tci_write_reg(regs, t0, t1); | ||
51 | + t1 = tci_read_r(regs, &tb_ptr); | ||
52 | + tci_write_reg(regs, t0, (int16_t)t1); | ||
53 | break; | ||
54 | #endif | ||
55 | #if TCG_TARGET_HAS_ext8u_i32 | ||
56 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
57 | #if TCG_TARGET_HAS_ext16s_i64 | ||
58 | case INDEX_op_ext16s_i64: | ||
59 | t0 = *tb_ptr++; | ||
60 | - t1 = tci_read_r16s(regs, &tb_ptr); | ||
61 | - tci_write_reg(regs, t0, t1); | ||
62 | + t1 = tci_read_r(regs, &tb_ptr); | ||
63 | + tci_write_reg(regs, t0, (int16_t)t1); | ||
64 | break; | ||
65 | #endif | ||
66 | #if TCG_TARGET_HAS_ext16u_i64 | ||
67 | -- | ||
68 | 2.25.1 | ||
69 | |||
70 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | 1 | Use explicit casts for ext32u opcodes, and allow truncation | |
2 | to happen for other users. | ||
3 | |||
4 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
5 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
6 | --- | ||
7 | tcg/tci.c | 122 ++++++++++++++++++++++++------------------------------ | ||
8 | 1 file changed, 54 insertions(+), 68 deletions(-) | ||
9 | |||
10 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
11 | index XXXXXXX..XXXXXXX 100644 | ||
12 | --- a/tcg/tci.c | ||
13 | +++ b/tcg/tci.c | ||
14 | @@ -XXX,XX +XXX,XX @@ static int32_t tci_read_reg32s(const tcg_target_ulong *regs, TCGReg index) | ||
15 | } | ||
16 | #endif | ||
17 | |||
18 | -static uint32_t tci_read_reg32(const tcg_target_ulong *regs, TCGReg index) | ||
19 | -{ | ||
20 | - return (uint32_t)tci_read_reg(regs, index); | ||
21 | -} | ||
22 | - | ||
23 | #if TCG_TARGET_REG_BITS == 64 | ||
24 | static uint64_t tci_read_reg64(const tcg_target_ulong *regs, TCGReg index) | ||
25 | { | ||
26 | @@ -XXX,XX +XXX,XX @@ tci_read_r(const tcg_target_ulong *regs, const uint8_t **tb_ptr) | ||
27 | return value; | ||
28 | } | ||
29 | |||
30 | -/* Read indexed register (32 bit) from bytecode. */ | ||
31 | -static uint32_t tci_read_r32(const tcg_target_ulong *regs, | ||
32 | - const uint8_t **tb_ptr) | ||
33 | -{ | ||
34 | - uint32_t value = tci_read_reg32(regs, **tb_ptr); | ||
35 | - *tb_ptr += 1; | ||
36 | - return value; | ||
37 | -} | ||
38 | - | ||
39 | #if TCG_TARGET_REG_BITS == 32 | ||
40 | /* Read two indexed registers (2 * 32 bit) from bytecode. */ | ||
41 | static uint64_t tci_read_r64(const tcg_target_ulong *regs, | ||
42 | const uint8_t **tb_ptr) | ||
43 | { | ||
44 | - uint32_t low = tci_read_r32(regs, tb_ptr); | ||
45 | - return tci_uint64(tci_read_r32(regs, tb_ptr), low); | ||
46 | + uint32_t low = tci_read_r(regs, tb_ptr); | ||
47 | + return tci_uint64(tci_read_r(regs, tb_ptr), low); | ||
48 | } | ||
49 | #elif TCG_TARGET_REG_BITS == 64 | ||
50 | /* Read indexed register (32 bit signed) from bytecode. */ | ||
51 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
52 | continue; | ||
53 | case INDEX_op_setcond_i32: | ||
54 | t0 = *tb_ptr++; | ||
55 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
56 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
57 | + t1 = tci_read_r(regs, &tb_ptr); | ||
58 | + t2 = tci_read_r(regs, &tb_ptr); | ||
59 | condition = *tb_ptr++; | ||
60 | tci_write_reg(regs, t0, tci_compare32(t1, t2, condition)); | ||
61 | break; | ||
62 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
63 | #endif | ||
64 | case INDEX_op_mov_i32: | ||
65 | t0 = *tb_ptr++; | ||
66 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
67 | + t1 = tci_read_r(regs, &tb_ptr); | ||
68 | tci_write_reg(regs, t0, t1); | ||
69 | break; | ||
70 | case INDEX_op_tci_movi_i32: | ||
71 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
72 | break; | ||
73 | case INDEX_op_st_i32: | ||
74 | CASE_64(st32) | ||
75 | - t0 = tci_read_r32(regs, &tb_ptr); | ||
76 | + t0 = tci_read_r(regs, &tb_ptr); | ||
77 | t1 = tci_read_r(regs, &tb_ptr); | ||
78 | t2 = tci_read_s32(&tb_ptr); | ||
79 | *(uint32_t *)(t1 + t2) = t0; | ||
80 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
81 | |||
82 | case INDEX_op_add_i32: | ||
83 | t0 = *tb_ptr++; | ||
84 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
85 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
86 | + t1 = tci_read_r(regs, &tb_ptr); | ||
87 | + t2 = tci_read_r(regs, &tb_ptr); | ||
88 | tci_write_reg(regs, t0, t1 + t2); | ||
89 | break; | ||
90 | case INDEX_op_sub_i32: | ||
91 | t0 = *tb_ptr++; | ||
92 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
93 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
94 | + t1 = tci_read_r(regs, &tb_ptr); | ||
95 | + t2 = tci_read_r(regs, &tb_ptr); | ||
96 | tci_write_reg(regs, t0, t1 - t2); | ||
97 | break; | ||
98 | case INDEX_op_mul_i32: | ||
99 | t0 = *tb_ptr++; | ||
100 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
101 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
102 | + t1 = tci_read_r(regs, &tb_ptr); | ||
103 | + t2 = tci_read_r(regs, &tb_ptr); | ||
104 | tci_write_reg(regs, t0, t1 * t2); | ||
105 | break; | ||
106 | case INDEX_op_div_i32: | ||
107 | t0 = *tb_ptr++; | ||
108 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
109 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
110 | + t1 = tci_read_r(regs, &tb_ptr); | ||
111 | + t2 = tci_read_r(regs, &tb_ptr); | ||
112 | tci_write_reg(regs, t0, (int32_t)t1 / (int32_t)t2); | ||
113 | break; | ||
114 | case INDEX_op_divu_i32: | ||
115 | t0 = *tb_ptr++; | ||
116 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
117 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
118 | - tci_write_reg(regs, t0, t1 / t2); | ||
119 | + t1 = tci_read_r(regs, &tb_ptr); | ||
120 | + t2 = tci_read_r(regs, &tb_ptr); | ||
121 | + tci_write_reg(regs, t0, (uint32_t)t1 / (uint32_t)t2); | ||
122 | break; | ||
123 | case INDEX_op_rem_i32: | ||
124 | t0 = *tb_ptr++; | ||
125 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
126 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
127 | + t1 = tci_read_r(regs, &tb_ptr); | ||
128 | + t2 = tci_read_r(regs, &tb_ptr); | ||
129 | tci_write_reg(regs, t0, (int32_t)t1 % (int32_t)t2); | ||
130 | break; | ||
131 | case INDEX_op_remu_i32: | ||
132 | t0 = *tb_ptr++; | ||
133 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
134 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
135 | - tci_write_reg(regs, t0, t1 % t2); | ||
136 | + t1 = tci_read_r(regs, &tb_ptr); | ||
137 | + t2 = tci_read_r(regs, &tb_ptr); | ||
138 | + tci_write_reg(regs, t0, (uint32_t)t1 % (uint32_t)t2); | ||
139 | break; | ||
140 | case INDEX_op_and_i32: | ||
141 | t0 = *tb_ptr++; | ||
142 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
143 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
144 | + t1 = tci_read_r(regs, &tb_ptr); | ||
145 | + t2 = tci_read_r(regs, &tb_ptr); | ||
146 | tci_write_reg(regs, t0, t1 & t2); | ||
147 | break; | ||
148 | case INDEX_op_or_i32: | ||
149 | t0 = *tb_ptr++; | ||
150 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
151 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
152 | + t1 = tci_read_r(regs, &tb_ptr); | ||
153 | + t2 = tci_read_r(regs, &tb_ptr); | ||
154 | tci_write_reg(regs, t0, t1 | t2); | ||
155 | break; | ||
156 | case INDEX_op_xor_i32: | ||
157 | t0 = *tb_ptr++; | ||
158 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
159 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
160 | + t1 = tci_read_r(regs, &tb_ptr); | ||
161 | + t2 = tci_read_r(regs, &tb_ptr); | ||
162 | tci_write_reg(regs, t0, t1 ^ t2); | ||
163 | break; | ||
164 | |||
165 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
166 | |||
167 | case INDEX_op_shl_i32: | ||
168 | t0 = *tb_ptr++; | ||
169 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
170 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
171 | - tci_write_reg(regs, t0, t1 << (t2 & 31)); | ||
172 | + t1 = tci_read_r(regs, &tb_ptr); | ||
173 | + t2 = tci_read_r(regs, &tb_ptr); | ||
174 | + tci_write_reg(regs, t0, (uint32_t)t1 << (t2 & 31)); | ||
175 | break; | ||
176 | case INDEX_op_shr_i32: | ||
177 | t0 = *tb_ptr++; | ||
178 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
179 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
180 | - tci_write_reg(regs, t0, t1 >> (t2 & 31)); | ||
181 | + t1 = tci_read_r(regs, &tb_ptr); | ||
182 | + t2 = tci_read_r(regs, &tb_ptr); | ||
183 | + tci_write_reg(regs, t0, (uint32_t)t1 >> (t2 & 31)); | ||
184 | break; | ||
185 | case INDEX_op_sar_i32: | ||
186 | t0 = *tb_ptr++; | ||
187 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
188 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
189 | - tci_write_reg(regs, t0, ((int32_t)t1 >> (t2 & 31))); | ||
190 | + t1 = tci_read_r(regs, &tb_ptr); | ||
191 | + t2 = tci_read_r(regs, &tb_ptr); | ||
192 | + tci_write_reg(regs, t0, (int32_t)t1 >> (t2 & 31)); | ||
193 | break; | ||
194 | #if TCG_TARGET_HAS_rot_i32 | ||
195 | case INDEX_op_rotl_i32: | ||
196 | t0 = *tb_ptr++; | ||
197 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
198 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
199 | + t1 = tci_read_r(regs, &tb_ptr); | ||
200 | + t2 = tci_read_r(regs, &tb_ptr); | ||
201 | tci_write_reg(regs, t0, rol32(t1, t2 & 31)); | ||
202 | break; | ||
203 | case INDEX_op_rotr_i32: | ||
204 | t0 = *tb_ptr++; | ||
205 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
206 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
207 | + t1 = tci_read_r(regs, &tb_ptr); | ||
208 | + t2 = tci_read_r(regs, &tb_ptr); | ||
209 | tci_write_reg(regs, t0, ror32(t1, t2 & 31)); | ||
210 | break; | ||
211 | #endif | ||
212 | #if TCG_TARGET_HAS_deposit_i32 | ||
213 | case INDEX_op_deposit_i32: | ||
214 | t0 = *tb_ptr++; | ||
215 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
216 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
217 | + t1 = tci_read_r(regs, &tb_ptr); | ||
218 | + t2 = tci_read_r(regs, &tb_ptr); | ||
219 | tmp16 = *tb_ptr++; | ||
220 | tmp8 = *tb_ptr++; | ||
221 | tmp32 = (((1 << tmp8) - 1) << tmp16); | ||
222 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
223 | break; | ||
224 | #endif | ||
225 | case INDEX_op_brcond_i32: | ||
226 | - t0 = tci_read_r32(regs, &tb_ptr); | ||
227 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
228 | + t0 = tci_read_r(regs, &tb_ptr); | ||
229 | + t1 = tci_read_r(regs, &tb_ptr); | ||
230 | condition = *tb_ptr++; | ||
231 | label = tci_read_label(&tb_ptr); | ||
232 | if (tci_compare32(t0, t1, condition)) { | ||
233 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
234 | case INDEX_op_mulu2_i32: | ||
235 | t0 = *tb_ptr++; | ||
236 | t1 = *tb_ptr++; | ||
237 | - t2 = tci_read_r32(regs, &tb_ptr); | ||
238 | - tmp64 = tci_read_r32(regs, &tb_ptr); | ||
239 | - tci_write_reg64(regs, t1, t0, t2 * tmp64); | ||
240 | + t2 = tci_read_r(regs, &tb_ptr); | ||
241 | + tmp64 = (uint32_t)tci_read_r(regs, &tb_ptr); | ||
242 | + tci_write_reg64(regs, t1, t0, (uint32_t)t2 * tmp64); | ||
243 | break; | ||
244 | #endif /* TCG_TARGET_REG_BITS == 32 */ | ||
245 | #if TCG_TARGET_HAS_ext8s_i32 | ||
246 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
247 | #if TCG_TARGET_HAS_bswap32_i32 | ||
248 | case INDEX_op_bswap32_i32: | ||
249 | t0 = *tb_ptr++; | ||
250 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
251 | + t1 = tci_read_r(regs, &tb_ptr); | ||
252 | tci_write_reg(regs, t0, bswap32(t1)); | ||
253 | break; | ||
254 | #endif | ||
255 | #if TCG_TARGET_HAS_not_i32 | ||
256 | case INDEX_op_not_i32: | ||
257 | t0 = *tb_ptr++; | ||
258 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
259 | + t1 = tci_read_r(regs, &tb_ptr); | ||
260 | tci_write_reg(regs, t0, ~t1); | ||
261 | break; | ||
262 | #endif | ||
263 | #if TCG_TARGET_HAS_neg_i32 | ||
264 | case INDEX_op_neg_i32: | ||
265 | t0 = *tb_ptr++; | ||
266 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
267 | + t1 = tci_read_r(regs, &tb_ptr); | ||
268 | tci_write_reg(regs, t0, -t1); | ||
269 | break; | ||
270 | #endif | ||
271 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
272 | #endif | ||
273 | case INDEX_op_extu_i32_i64: | ||
274 | t0 = *tb_ptr++; | ||
275 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
276 | - tci_write_reg(regs, t0, t1); | ||
277 | + t1 = tci_read_r(regs, &tb_ptr); | ||
278 | + tci_write_reg(regs, t0, (uint32_t)t1); | ||
279 | break; | ||
280 | #if TCG_TARGET_HAS_bswap16_i64 | ||
281 | case INDEX_op_bswap16_i64: | ||
282 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
283 | #if TCG_TARGET_HAS_bswap32_i64 | ||
284 | case INDEX_op_bswap32_i64: | ||
285 | t0 = *tb_ptr++; | ||
286 | - t1 = tci_read_r32(regs, &tb_ptr); | ||
287 | + t1 = tci_read_r(regs, &tb_ptr); | ||
288 | tci_write_reg(regs, t0, bswap32(t1)); | ||
289 | break; | ||
290 | #endif | ||
291 | -- | ||
292 | 2.25.1 | ||
293 | |||
294 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Use explicit casts for ext32s opcodes. | ||
1 | 2 | ||
3 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | --- | ||
6 | tcg/tci.c | 20 ++------------------ | ||
7 | 1 file changed, 2 insertions(+), 18 deletions(-) | ||
8 | |||
9 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
10 | index XXXXXXX..XXXXXXX 100644 | ||
11 | --- a/tcg/tci.c | ||
12 | +++ b/tcg/tci.c | ||
13 | @@ -XXX,XX +XXX,XX @@ static tcg_target_ulong tci_read_reg(const tcg_target_ulong *regs, TCGReg index) | ||
14 | return regs[index]; | ||
15 | } | ||
16 | |||
17 | -#if TCG_TARGET_REG_BITS == 64 | ||
18 | -static int32_t tci_read_reg32s(const tcg_target_ulong *regs, TCGReg index) | ||
19 | -{ | ||
20 | - return (int32_t)tci_read_reg(regs, index); | ||
21 | -} | ||
22 | -#endif | ||
23 | - | ||
24 | #if TCG_TARGET_REG_BITS == 64 | ||
25 | static uint64_t tci_read_reg64(const tcg_target_ulong *regs, TCGReg index) | ||
26 | { | ||
27 | @@ -XXX,XX +XXX,XX @@ static uint64_t tci_read_r64(const tcg_target_ulong *regs, | ||
28 | return tci_uint64(tci_read_r(regs, tb_ptr), low); | ||
29 | } | ||
30 | #elif TCG_TARGET_REG_BITS == 64 | ||
31 | -/* Read indexed register (32 bit signed) from bytecode. */ | ||
32 | -static int32_t tci_read_r32s(const tcg_target_ulong *regs, | ||
33 | - const uint8_t **tb_ptr) | ||
34 | -{ | ||
35 | - int32_t value = tci_read_reg32s(regs, **tb_ptr); | ||
36 | - *tb_ptr += 1; | ||
37 | - return value; | ||
38 | -} | ||
39 | - | ||
40 | /* Read indexed register (64 bit) from bytecode. */ | ||
41 | static uint64_t tci_read_r64(const tcg_target_ulong *regs, | ||
42 | const uint8_t **tb_ptr) | ||
43 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
44 | #endif | ||
45 | case INDEX_op_ext_i32_i64: | ||
46 | t0 = *tb_ptr++; | ||
47 | - t1 = tci_read_r32s(regs, &tb_ptr); | ||
48 | - tci_write_reg(regs, t0, t1); | ||
49 | + t1 = tci_read_r(regs, &tb_ptr); | ||
50 | + tci_write_reg(regs, t0, (int32_t)t1); | ||
51 | break; | ||
52 | #if TCG_TARGET_HAS_ext32u_i64 | ||
53 | case INDEX_op_ext32u_i64: | ||
54 | -- | ||
55 | 2.25.1 | ||
56 | |||
57 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | 1 | In all cases restricted to 64-bit hosts, tcg_read_r is | |
2 | identical. We retain the 64-bit symbol for the single | ||
3 | case of INDEX_op_qemu_st_i64. | ||
4 | |||
5 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
6 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
7 | --- | ||
8 | tcg/tci.c | 93 +++++++++++++++++++++++++------------------------------ | ||
9 | 1 file changed, 42 insertions(+), 51 deletions(-) | ||
10 | |||
11 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
12 | index XXXXXXX..XXXXXXX 100644 | ||
13 | --- a/tcg/tci.c | ||
14 | +++ b/tcg/tci.c | ||
15 | @@ -XXX,XX +XXX,XX @@ static tcg_target_ulong tci_read_reg(const tcg_target_ulong *regs, TCGReg index) | ||
16 | return regs[index]; | ||
17 | } | ||
18 | |||
19 | -#if TCG_TARGET_REG_BITS == 64 | ||
20 | -static uint64_t tci_read_reg64(const tcg_target_ulong *regs, TCGReg index) | ||
21 | -{ | ||
22 | - return tci_read_reg(regs, index); | ||
23 | -} | ||
24 | -#endif | ||
25 | - | ||
26 | static void | ||
27 | tci_write_reg(tcg_target_ulong *regs, TCGReg index, tcg_target_ulong value) | ||
28 | { | ||
29 | @@ -XXX,XX +XXX,XX @@ static uint64_t tci_read_r64(const tcg_target_ulong *regs, | ||
30 | static uint64_t tci_read_r64(const tcg_target_ulong *regs, | ||
31 | const uint8_t **tb_ptr) | ||
32 | { | ||
33 | - uint64_t value = tci_read_reg64(regs, **tb_ptr); | ||
34 | - *tb_ptr += 1; | ||
35 | - return value; | ||
36 | + return tci_read_r(regs, tb_ptr); | ||
37 | } | ||
38 | #endif | ||
39 | |||
40 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
41 | #elif TCG_TARGET_REG_BITS == 64 | ||
42 | case INDEX_op_setcond_i64: | ||
43 | t0 = *tb_ptr++; | ||
44 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
45 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
46 | + t1 = tci_read_r(regs, &tb_ptr); | ||
47 | + t2 = tci_read_r(regs, &tb_ptr); | ||
48 | condition = *tb_ptr++; | ||
49 | tci_write_reg(regs, t0, tci_compare64(t1, t2, condition)); | ||
50 | break; | ||
51 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
52 | #if TCG_TARGET_REG_BITS == 64 | ||
53 | case INDEX_op_mov_i64: | ||
54 | t0 = *tb_ptr++; | ||
55 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
56 | + t1 = tci_read_r(regs, &tb_ptr); | ||
57 | tci_write_reg(regs, t0, t1); | ||
58 | break; | ||
59 | case INDEX_op_tci_movi_i64: | ||
60 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
61 | tci_write_reg(regs, t0, *(uint64_t *)(t1 + t2)); | ||
62 | break; | ||
63 | case INDEX_op_st_i64: | ||
64 | - t0 = tci_read_r64(regs, &tb_ptr); | ||
65 | + t0 = tci_read_r(regs, &tb_ptr); | ||
66 | t1 = tci_read_r(regs, &tb_ptr); | ||
67 | t2 = tci_read_s32(&tb_ptr); | ||
68 | *(uint64_t *)(t1 + t2) = t0; | ||
69 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
70 | |||
71 | case INDEX_op_add_i64: | ||
72 | t0 = *tb_ptr++; | ||
73 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
74 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
75 | + t1 = tci_read_r(regs, &tb_ptr); | ||
76 | + t2 = tci_read_r(regs, &tb_ptr); | ||
77 | tci_write_reg(regs, t0, t1 + t2); | ||
78 | break; | ||
79 | case INDEX_op_sub_i64: | ||
80 | t0 = *tb_ptr++; | ||
81 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
82 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
83 | + t1 = tci_read_r(regs, &tb_ptr); | ||
84 | + t2 = tci_read_r(regs, &tb_ptr); | ||
85 | tci_write_reg(regs, t0, t1 - t2); | ||
86 | break; | ||
87 | case INDEX_op_mul_i64: | ||
88 | t0 = *tb_ptr++; | ||
89 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
90 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
91 | + t1 = tci_read_r(regs, &tb_ptr); | ||
92 | + t2 = tci_read_r(regs, &tb_ptr); | ||
93 | tci_write_reg(regs, t0, t1 * t2); | ||
94 | break; | ||
95 | case INDEX_op_div_i64: | ||
96 | t0 = *tb_ptr++; | ||
97 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
98 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
99 | + t1 = tci_read_r(regs, &tb_ptr); | ||
100 | + t2 = tci_read_r(regs, &tb_ptr); | ||
101 | tci_write_reg(regs, t0, (int64_t)t1 / (int64_t)t2); | ||
102 | break; | ||
103 | case INDEX_op_divu_i64: | ||
104 | t0 = *tb_ptr++; | ||
105 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
106 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
107 | + t1 = tci_read_r(regs, &tb_ptr); | ||
108 | + t2 = tci_read_r(regs, &tb_ptr); | ||
109 | tci_write_reg(regs, t0, (uint64_t)t1 / (uint64_t)t2); | ||
110 | break; | ||
111 | case INDEX_op_rem_i64: | ||
112 | t0 = *tb_ptr++; | ||
113 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
114 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
115 | + t1 = tci_read_r(regs, &tb_ptr); | ||
116 | + t2 = tci_read_r(regs, &tb_ptr); | ||
117 | tci_write_reg(regs, t0, (int64_t)t1 % (int64_t)t2); | ||
118 | break; | ||
119 | case INDEX_op_remu_i64: | ||
120 | t0 = *tb_ptr++; | ||
121 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
122 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
123 | + t1 = tci_read_r(regs, &tb_ptr); | ||
124 | + t2 = tci_read_r(regs, &tb_ptr); | ||
125 | tci_write_reg(regs, t0, (uint64_t)t1 % (uint64_t)t2); | ||
126 | break; | ||
127 | case INDEX_op_and_i64: | ||
128 | t0 = *tb_ptr++; | ||
129 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
130 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
131 | + t1 = tci_read_r(regs, &tb_ptr); | ||
132 | + t2 = tci_read_r(regs, &tb_ptr); | ||
133 | tci_write_reg(regs, t0, t1 & t2); | ||
134 | break; | ||
135 | case INDEX_op_or_i64: | ||
136 | t0 = *tb_ptr++; | ||
137 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
138 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
139 | + t1 = tci_read_r(regs, &tb_ptr); | ||
140 | + t2 = tci_read_r(regs, &tb_ptr); | ||
141 | tci_write_reg(regs, t0, t1 | t2); | ||
142 | break; | ||
143 | case INDEX_op_xor_i64: | ||
144 | t0 = *tb_ptr++; | ||
145 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
146 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
147 | + t1 = tci_read_r(regs, &tb_ptr); | ||
148 | + t2 = tci_read_r(regs, &tb_ptr); | ||
149 | tci_write_reg(regs, t0, t1 ^ t2); | ||
150 | break; | ||
151 | |||
152 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
153 | |||
154 | case INDEX_op_shl_i64: | ||
155 | t0 = *tb_ptr++; | ||
156 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
157 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
158 | + t1 = tci_read_r(regs, &tb_ptr); | ||
159 | + t2 = tci_read_r(regs, &tb_ptr); | ||
160 | tci_write_reg(regs, t0, t1 << (t2 & 63)); | ||
161 | break; | ||
162 | case INDEX_op_shr_i64: | ||
163 | t0 = *tb_ptr++; | ||
164 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
165 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
166 | + t1 = tci_read_r(regs, &tb_ptr); | ||
167 | + t2 = tci_read_r(regs, &tb_ptr); | ||
168 | tci_write_reg(regs, t0, t1 >> (t2 & 63)); | ||
169 | break; | ||
170 | case INDEX_op_sar_i64: | ||
171 | t0 = *tb_ptr++; | ||
172 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
173 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
174 | + t1 = tci_read_r(regs, &tb_ptr); | ||
175 | + t2 = tci_read_r(regs, &tb_ptr); | ||
176 | tci_write_reg(regs, t0, ((int64_t)t1 >> (t2 & 63))); | ||
177 | break; | ||
178 | #if TCG_TARGET_HAS_rot_i64 | ||
179 | case INDEX_op_rotl_i64: | ||
180 | t0 = *tb_ptr++; | ||
181 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
182 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
183 | + t1 = tci_read_r(regs, &tb_ptr); | ||
184 | + t2 = tci_read_r(regs, &tb_ptr); | ||
185 | tci_write_reg(regs, t0, rol64(t1, t2 & 63)); | ||
186 | break; | ||
187 | case INDEX_op_rotr_i64: | ||
188 | t0 = *tb_ptr++; | ||
189 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
190 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
191 | + t1 = tci_read_r(regs, &tb_ptr); | ||
192 | + t2 = tci_read_r(regs, &tb_ptr); | ||
193 | tci_write_reg(regs, t0, ror64(t1, t2 & 63)); | ||
194 | break; | ||
195 | #endif | ||
196 | #if TCG_TARGET_HAS_deposit_i64 | ||
197 | case INDEX_op_deposit_i64: | ||
198 | t0 = *tb_ptr++; | ||
199 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
200 | - t2 = tci_read_r64(regs, &tb_ptr); | ||
201 | + t1 = tci_read_r(regs, &tb_ptr); | ||
202 | + t2 = tci_read_r(regs, &tb_ptr); | ||
203 | tmp16 = *tb_ptr++; | ||
204 | tmp8 = *tb_ptr++; | ||
205 | tmp64 = (((1ULL << tmp8) - 1) << tmp16); | ||
206 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
207 | break; | ||
208 | #endif | ||
209 | case INDEX_op_brcond_i64: | ||
210 | - t0 = tci_read_r64(regs, &tb_ptr); | ||
211 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
212 | + t0 = tci_read_r(regs, &tb_ptr); | ||
213 | + t1 = tci_read_r(regs, &tb_ptr); | ||
214 | condition = *tb_ptr++; | ||
215 | label = tci_read_label(&tb_ptr); | ||
216 | if (tci_compare64(t0, t1, condition)) { | ||
217 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
218 | #if TCG_TARGET_HAS_bswap64_i64 | ||
219 | case INDEX_op_bswap64_i64: | ||
220 | t0 = *tb_ptr++; | ||
221 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
222 | + t1 = tci_read_r(regs, &tb_ptr); | ||
223 | tci_write_reg(regs, t0, bswap64(t1)); | ||
224 | break; | ||
225 | #endif | ||
226 | #if TCG_TARGET_HAS_not_i64 | ||
227 | case INDEX_op_not_i64: | ||
228 | t0 = *tb_ptr++; | ||
229 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
230 | + t1 = tci_read_r(regs, &tb_ptr); | ||
231 | tci_write_reg(regs, t0, ~t1); | ||
232 | break; | ||
233 | #endif | ||
234 | #if TCG_TARGET_HAS_neg_i64 | ||
235 | case INDEX_op_neg_i64: | ||
236 | t0 = *tb_ptr++; | ||
237 | - t1 = tci_read_r64(regs, &tb_ptr); | ||
238 | + t1 = tci_read_r(regs, &tb_ptr); | ||
239 | tci_write_reg(regs, t0, -t1); | ||
240 | break; | ||
241 | #endif | ||
242 | -- | ||
243 | 2.25.1 | ||
244 | |||
245 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | This includes add, sub, mul, and, or, xor. | ||
1 | 2 | ||
3 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | --- | ||
6 | tcg/tci.c | 83 +++++++++++++++++-------------------------------------- | ||
7 | 1 file changed, 25 insertions(+), 58 deletions(-) | ||
8 | |||
9 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
10 | index XXXXXXX..XXXXXXX 100644 | ||
11 | --- a/tcg/tci.c | ||
12 | +++ b/tcg/tci.c | ||
13 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
14 | *(uint32_t *)(t1 + t2) = t0; | ||
15 | break; | ||
16 | |||
17 | - /* Arithmetic operations (32 bit). */ | ||
18 | + /* Arithmetic operations (mixed 32/64 bit). */ | ||
19 | |||
20 | - case INDEX_op_add_i32: | ||
21 | + CASE_32_64(add) | ||
22 | t0 = *tb_ptr++; | ||
23 | t1 = tci_read_r(regs, &tb_ptr); | ||
24 | t2 = tci_read_r(regs, &tb_ptr); | ||
25 | tci_write_reg(regs, t0, t1 + t2); | ||
26 | break; | ||
27 | - case INDEX_op_sub_i32: | ||
28 | + CASE_32_64(sub) | ||
29 | t0 = *tb_ptr++; | ||
30 | t1 = tci_read_r(regs, &tb_ptr); | ||
31 | t2 = tci_read_r(regs, &tb_ptr); | ||
32 | tci_write_reg(regs, t0, t1 - t2); | ||
33 | break; | ||
34 | - case INDEX_op_mul_i32: | ||
35 | + CASE_32_64(mul) | ||
36 | t0 = *tb_ptr++; | ||
37 | t1 = tci_read_r(regs, &tb_ptr); | ||
38 | t2 = tci_read_r(regs, &tb_ptr); | ||
39 | tci_write_reg(regs, t0, t1 * t2); | ||
40 | break; | ||
41 | + CASE_32_64(and) | ||
42 | + t0 = *tb_ptr++; | ||
43 | + t1 = tci_read_r(regs, &tb_ptr); | ||
44 | + t2 = tci_read_r(regs, &tb_ptr); | ||
45 | + tci_write_reg(regs, t0, t1 & t2); | ||
46 | + break; | ||
47 | + CASE_32_64(or) | ||
48 | + t0 = *tb_ptr++; | ||
49 | + t1 = tci_read_r(regs, &tb_ptr); | ||
50 | + t2 = tci_read_r(regs, &tb_ptr); | ||
51 | + tci_write_reg(regs, t0, t1 | t2); | ||
52 | + break; | ||
53 | + CASE_32_64(xor) | ||
54 | + t0 = *tb_ptr++; | ||
55 | + t1 = tci_read_r(regs, &tb_ptr); | ||
56 | + t2 = tci_read_r(regs, &tb_ptr); | ||
57 | + tci_write_reg(regs, t0, t1 ^ t2); | ||
58 | + break; | ||
59 | + | ||
60 | + /* Arithmetic operations (32 bit). */ | ||
61 | + | ||
62 | case INDEX_op_div_i32: | ||
63 | t0 = *tb_ptr++; | ||
64 | t1 = tci_read_r(regs, &tb_ptr); | ||
65 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
66 | t2 = tci_read_r(regs, &tb_ptr); | ||
67 | tci_write_reg(regs, t0, (uint32_t)t1 % (uint32_t)t2); | ||
68 | break; | ||
69 | - case INDEX_op_and_i32: | ||
70 | - t0 = *tb_ptr++; | ||
71 | - t1 = tci_read_r(regs, &tb_ptr); | ||
72 | - t2 = tci_read_r(regs, &tb_ptr); | ||
73 | - tci_write_reg(regs, t0, t1 & t2); | ||
74 | - break; | ||
75 | - case INDEX_op_or_i32: | ||
76 | - t0 = *tb_ptr++; | ||
77 | - t1 = tci_read_r(regs, &tb_ptr); | ||
78 | - t2 = tci_read_r(regs, &tb_ptr); | ||
79 | - tci_write_reg(regs, t0, t1 | t2); | ||
80 | - break; | ||
81 | - case INDEX_op_xor_i32: | ||
82 | - t0 = *tb_ptr++; | ||
83 | - t1 = tci_read_r(regs, &tb_ptr); | ||
84 | - t2 = tci_read_r(regs, &tb_ptr); | ||
85 | - tci_write_reg(regs, t0, t1 ^ t2); | ||
86 | - break; | ||
87 | |||
88 | /* Shift/rotate operations (32 bit). */ | ||
89 | |||
90 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
91 | |||
92 | /* Arithmetic operations (64 bit). */ | ||
93 | |||
94 | - case INDEX_op_add_i64: | ||
95 | - t0 = *tb_ptr++; | ||
96 | - t1 = tci_read_r(regs, &tb_ptr); | ||
97 | - t2 = tci_read_r(regs, &tb_ptr); | ||
98 | - tci_write_reg(regs, t0, t1 + t2); | ||
99 | - break; | ||
100 | - case INDEX_op_sub_i64: | ||
101 | - t0 = *tb_ptr++; | ||
102 | - t1 = tci_read_r(regs, &tb_ptr); | ||
103 | - t2 = tci_read_r(regs, &tb_ptr); | ||
104 | - tci_write_reg(regs, t0, t1 - t2); | ||
105 | - break; | ||
106 | - case INDEX_op_mul_i64: | ||
107 | - t0 = *tb_ptr++; | ||
108 | - t1 = tci_read_r(regs, &tb_ptr); | ||
109 | - t2 = tci_read_r(regs, &tb_ptr); | ||
110 | - tci_write_reg(regs, t0, t1 * t2); | ||
111 | - break; | ||
112 | case INDEX_op_div_i64: | ||
113 | t0 = *tb_ptr++; | ||
114 | t1 = tci_read_r(regs, &tb_ptr); | ||
115 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
116 | t2 = tci_read_r(regs, &tb_ptr); | ||
117 | tci_write_reg(regs, t0, (uint64_t)t1 % (uint64_t)t2); | ||
118 | break; | ||
119 | - case INDEX_op_and_i64: | ||
120 | - t0 = *tb_ptr++; | ||
121 | - t1 = tci_read_r(regs, &tb_ptr); | ||
122 | - t2 = tci_read_r(regs, &tb_ptr); | ||
123 | - tci_write_reg(regs, t0, t1 & t2); | ||
124 | - break; | ||
125 | - case INDEX_op_or_i64: | ||
126 | - t0 = *tb_ptr++; | ||
127 | - t1 = tci_read_r(regs, &tb_ptr); | ||
128 | - t2 = tci_read_r(regs, &tb_ptr); | ||
129 | - tci_write_reg(regs, t0, t1 | t2); | ||
130 | - break; | ||
131 | - case INDEX_op_xor_i64: | ||
132 | - t0 = *tb_ptr++; | ||
133 | - t1 = tci_read_r(regs, &tb_ptr); | ||
134 | - t2 = tci_read_r(regs, &tb_ptr); | ||
135 | - tci_write_reg(regs, t0, t1 ^ t2); | ||
136 | - break; | ||
137 | |||
138 | /* Shift/rotate operations (64 bit). */ | ||
139 | |||
140 | -- | ||
141 | 2.25.1 | ||
142 | |||
143 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | This includes ext8s, ext8u, ext16s, ext16u. | ||
1 | 2 | ||
3 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | --- | ||
6 | tcg/tci.c | 44 ++++++++------------------------------------ | ||
7 | 1 file changed, 8 insertions(+), 36 deletions(-) | ||
8 | |||
9 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
10 | index XXXXXXX..XXXXXXX 100644 | ||
11 | --- a/tcg/tci.c | ||
12 | +++ b/tcg/tci.c | ||
13 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
14 | tci_write_reg64(regs, t1, t0, (uint32_t)t2 * tmp64); | ||
15 | break; | ||
16 | #endif /* TCG_TARGET_REG_BITS == 32 */ | ||
17 | -#if TCG_TARGET_HAS_ext8s_i32 | ||
18 | - case INDEX_op_ext8s_i32: | ||
19 | +#if TCG_TARGET_HAS_ext8s_i32 || TCG_TARGET_HAS_ext8s_i64 | ||
20 | + CASE_32_64(ext8s) | ||
21 | t0 = *tb_ptr++; | ||
22 | t1 = tci_read_r(regs, &tb_ptr); | ||
23 | tci_write_reg(regs, t0, (int8_t)t1); | ||
24 | break; | ||
25 | #endif | ||
26 | -#if TCG_TARGET_HAS_ext16s_i32 | ||
27 | - case INDEX_op_ext16s_i32: | ||
28 | +#if TCG_TARGET_HAS_ext16s_i32 || TCG_TARGET_HAS_ext16s_i64 | ||
29 | + CASE_32_64(ext16s) | ||
30 | t0 = *tb_ptr++; | ||
31 | t1 = tci_read_r(regs, &tb_ptr); | ||
32 | tci_write_reg(regs, t0, (int16_t)t1); | ||
33 | break; | ||
34 | #endif | ||
35 | -#if TCG_TARGET_HAS_ext8u_i32 | ||
36 | - case INDEX_op_ext8u_i32: | ||
37 | +#if TCG_TARGET_HAS_ext8u_i32 || TCG_TARGET_HAS_ext8u_i64 | ||
38 | + CASE_32_64(ext8u) | ||
39 | t0 = *tb_ptr++; | ||
40 | t1 = tci_read_r(regs, &tb_ptr); | ||
41 | tci_write_reg(regs, t0, (uint8_t)t1); | ||
42 | break; | ||
43 | #endif | ||
44 | -#if TCG_TARGET_HAS_ext16u_i32 | ||
45 | - case INDEX_op_ext16u_i32: | ||
46 | +#if TCG_TARGET_HAS_ext16u_i32 || TCG_TARGET_HAS_ext16u_i64 | ||
47 | + CASE_32_64(ext16u) | ||
48 | t0 = *tb_ptr++; | ||
49 | t1 = tci_read_r(regs, &tb_ptr); | ||
50 | tci_write_reg(regs, t0, (uint16_t)t1); | ||
51 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
52 | continue; | ||
53 | } | ||
54 | break; | ||
55 | -#if TCG_TARGET_HAS_ext8u_i64 | ||
56 | - case INDEX_op_ext8u_i64: | ||
57 | - t0 = *tb_ptr++; | ||
58 | - t1 = tci_read_r(regs, &tb_ptr); | ||
59 | - tci_write_reg(regs, t0, (uint8_t)t1); | ||
60 | - break; | ||
61 | -#endif | ||
62 | -#if TCG_TARGET_HAS_ext8s_i64 | ||
63 | - case INDEX_op_ext8s_i64: | ||
64 | - t0 = *tb_ptr++; | ||
65 | - t1 = tci_read_r(regs, &tb_ptr); | ||
66 | - tci_write_reg(regs, t0, (int8_t)t1); | ||
67 | - break; | ||
68 | -#endif | ||
69 | -#if TCG_TARGET_HAS_ext16s_i64 | ||
70 | - case INDEX_op_ext16s_i64: | ||
71 | - t0 = *tb_ptr++; | ||
72 | - t1 = tci_read_r(regs, &tb_ptr); | ||
73 | - tci_write_reg(regs, t0, (int16_t)t1); | ||
74 | - break; | ||
75 | -#endif | ||
76 | -#if TCG_TARGET_HAS_ext16u_i64 | ||
77 | - case INDEX_op_ext16u_i64: | ||
78 | - t0 = *tb_ptr++; | ||
79 | - t1 = tci_read_r(regs, &tb_ptr); | ||
80 | - tci_write_reg(regs, t0, (uint16_t)t1); | ||
81 | - break; | ||
82 | -#endif | ||
83 | #if TCG_TARGET_HAS_ext32s_i64 | ||
84 | case INDEX_op_ext32s_i64: | ||
85 | #endif | ||
86 | -- | ||
87 | 2.25.1 | ||
88 | |||
89 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | This includes bswap16 and bswap32. | ||
1 | 2 | ||
3 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
4 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
5 | --- | ||
6 | tcg/tci.c | 22 ++++------------------ | ||
7 | 1 file changed, 4 insertions(+), 18 deletions(-) | ||
8 | |||
9 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
10 | index XXXXXXX..XXXXXXX 100644 | ||
11 | --- a/tcg/tci.c | ||
12 | +++ b/tcg/tci.c | ||
13 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
14 | tci_write_reg(regs, t0, (uint16_t)t1); | ||
15 | break; | ||
16 | #endif | ||
17 | -#if TCG_TARGET_HAS_bswap16_i32 | ||
18 | - case INDEX_op_bswap16_i32: | ||
19 | +#if TCG_TARGET_HAS_bswap16_i32 || TCG_TARGET_HAS_bswap16_i64 | ||
20 | + CASE_32_64(bswap16) | ||
21 | t0 = *tb_ptr++; | ||
22 | t1 = tci_read_r(regs, &tb_ptr); | ||
23 | tci_write_reg(regs, t0, bswap16(t1)); | ||
24 | break; | ||
25 | #endif | ||
26 | -#if TCG_TARGET_HAS_bswap32_i32 | ||
27 | - case INDEX_op_bswap32_i32: | ||
28 | +#if TCG_TARGET_HAS_bswap32_i32 || TCG_TARGET_HAS_bswap32_i64 | ||
29 | + CASE_32_64(bswap32) | ||
30 | t0 = *tb_ptr++; | ||
31 | t1 = tci_read_r(regs, &tb_ptr); | ||
32 | tci_write_reg(regs, t0, bswap32(t1)); | ||
33 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
34 | t1 = tci_read_r(regs, &tb_ptr); | ||
35 | tci_write_reg(regs, t0, (uint32_t)t1); | ||
36 | break; | ||
37 | -#if TCG_TARGET_HAS_bswap16_i64 | ||
38 | - case INDEX_op_bswap16_i64: | ||
39 | - t0 = *tb_ptr++; | ||
40 | - t1 = tci_read_r(regs, &tb_ptr); | ||
41 | - tci_write_reg(regs, t0, bswap16(t1)); | ||
42 | - break; | ||
43 | -#endif | ||
44 | -#if TCG_TARGET_HAS_bswap32_i64 | ||
45 | - case INDEX_op_bswap32_i64: | ||
46 | - t0 = *tb_ptr++; | ||
47 | - t1 = tci_read_r(regs, &tb_ptr); | ||
48 | - tci_write_reg(regs, t0, bswap32(t1)); | ||
49 | - break; | ||
50 | -#endif | ||
51 | #if TCG_TARGET_HAS_bswap64_i64 | ||
52 | case INDEX_op_bswap64_i64: | ||
53 | t0 = *tb_ptr++; | ||
54 | -- | ||
55 | 2.25.1 | ||
56 | |||
57 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | ||
2 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
3 | --- | ||
4 | tcg/tci.c | 29 +++++------------------------ | ||
5 | 1 file changed, 5 insertions(+), 24 deletions(-) | ||
1 | 6 | ||
7 | diff --git a/tcg/tci.c b/tcg/tci.c | ||
8 | index XXXXXXX..XXXXXXX 100644 | ||
9 | --- a/tcg/tci.c | ||
10 | +++ b/tcg/tci.c | ||
11 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
12 | tci_write_reg(regs, t0, tci_compare64(t1, t2, condition)); | ||
13 | break; | ||
14 | #endif | ||
15 | - case INDEX_op_mov_i32: | ||
16 | + CASE_32_64(mov) | ||
17 | t0 = *tb_ptr++; | ||
18 | t1 = tci_read_r(regs, &tb_ptr); | ||
19 | tci_write_reg(regs, t0, t1); | ||
20 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
21 | tci_write_reg(regs, t0, bswap32(t1)); | ||
22 | break; | ||
23 | #endif | ||
24 | -#if TCG_TARGET_HAS_not_i32 | ||
25 | - case INDEX_op_not_i32: | ||
26 | +#if TCG_TARGET_HAS_not_i32 || TCG_TARGET_HAS_not_i64 | ||
27 | + CASE_32_64(not) | ||
28 | t0 = *tb_ptr++; | ||
29 | t1 = tci_read_r(regs, &tb_ptr); | ||
30 | tci_write_reg(regs, t0, ~t1); | ||
31 | break; | ||
32 | #endif | ||
33 | -#if TCG_TARGET_HAS_neg_i32 | ||
34 | - case INDEX_op_neg_i32: | ||
35 | +#if TCG_TARGET_HAS_neg_i32 || TCG_TARGET_HAS_neg_i64 | ||
36 | + CASE_32_64(neg) | ||
37 | t0 = *tb_ptr++; | ||
38 | t1 = tci_read_r(regs, &tb_ptr); | ||
39 | tci_write_reg(regs, t0, -t1); | ||
40 | break; | ||
41 | #endif | ||
42 | #if TCG_TARGET_REG_BITS == 64 | ||
43 | - case INDEX_op_mov_i64: | ||
44 | - t0 = *tb_ptr++; | ||
45 | - t1 = tci_read_r(regs, &tb_ptr); | ||
46 | - tci_write_reg(regs, t0, t1); | ||
47 | - break; | ||
48 | case INDEX_op_tci_movi_i64: | ||
49 | t0 = *tb_ptr++; | ||
50 | t1 = tci_read_i64(&tb_ptr); | ||
51 | @@ -XXX,XX +XXX,XX @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env, | ||
52 | tci_write_reg(regs, t0, bswap64(t1)); | ||
53 | break; | ||
54 | #endif | ||
55 | -#if TCG_TARGET_HAS_not_i64 | ||
56 | - case INDEX_op_not_i64: | ||
57 | - t0 = *tb_ptr++; | ||
58 | - t1 = tci_read_r(regs, &tb_ptr); | ||
59 | - tci_write_reg(regs, t0, ~t1); | ||
60 | - break; | ||
61 | -#endif | ||
62 | -#if TCG_TARGET_HAS_neg_i64 | ||
63 | - case INDEX_op_neg_i64: | ||
64 | - t0 = *tb_ptr++; | ||
65 | - t1 = tci_read_r(regs, &tb_ptr); | ||
66 | - tci_write_reg(regs, t0, -t1); | ||
67 | - break; | ||
68 | -#endif | ||
69 | #endif /* TCG_TARGET_REG_BITS == 64 */ | ||
70 | |||
71 | /* QEMU specific operations. */ | ||
72 | -- | ||
73 | 2.25.1 | ||
74 | |||
75 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | From: Alex Bennée <alex.bennee@linaro.org> | ||
1 | 2 | ||
3 | Having a function return either and valid TB and some system state | ||
4 | seems excessive. It will make the subsequent re-factoring easier if we | ||
5 | lookup the current state where we are. | ||
6 | |||
7 | Signed-off-by: Alex Bennée <alex.bennee@linaro.org> | ||
8 | Message-Id: <20210224165811.11567-2-alex.bennee@linaro.org> | ||
9 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
10 | --- | ||
11 | include/exec/tb-lookup.h | 18 ++++++++---------- | ||
12 | accel/tcg/cpu-exec.c | 10 ++++++++-- | ||
13 | accel/tcg/tcg-runtime.c | 4 +++- | ||
14 | 3 files changed, 19 insertions(+), 13 deletions(-) | ||
15 | |||
16 | diff --git a/include/exec/tb-lookup.h b/include/exec/tb-lookup.h | ||
17 | index XXXXXXX..XXXXXXX 100644 | ||
18 | --- a/include/exec/tb-lookup.h | ||
19 | +++ b/include/exec/tb-lookup.h | ||
20 | @@ -XXX,XX +XXX,XX @@ | ||
21 | #include "exec/tb-hash.h" | ||
22 | |||
23 | /* Might cause an exception, so have a longjmp destination ready */ | ||
24 | -static inline TranslationBlock * | ||
25 | -tb_lookup__cpu_state(CPUState *cpu, target_ulong *pc, target_ulong *cs_base, | ||
26 | - uint32_t *flags, uint32_t cf_mask) | ||
27 | +static inline TranslationBlock * tb_lookup(CPUState *cpu, | ||
28 | + target_ulong pc, target_ulong cs_base, | ||
29 | + uint32_t flags, uint32_t cf_mask) | ||
30 | { | ||
31 | - CPUArchState *env = (CPUArchState *)cpu->env_ptr; | ||
32 | TranslationBlock *tb; | ||
33 | uint32_t hash; | ||
34 | |||
35 | - cpu_get_tb_cpu_state(env, pc, cs_base, flags); | ||
36 | - hash = tb_jmp_cache_hash_func(*pc); | ||
37 | + hash = tb_jmp_cache_hash_func(pc); | ||
38 | tb = qatomic_rcu_read(&cpu->tb_jmp_cache[hash]); | ||
39 | |||
40 | cf_mask &= ~CF_CLUSTER_MASK; | ||
41 | cf_mask |= cpu->cluster_index << CF_CLUSTER_SHIFT; | ||
42 | |||
43 | if (likely(tb && | ||
44 | - tb->pc == *pc && | ||
45 | - tb->cs_base == *cs_base && | ||
46 | - tb->flags == *flags && | ||
47 | + tb->pc == pc && | ||
48 | + tb->cs_base == cs_base && | ||
49 | + tb->flags == flags && | ||
50 | tb->trace_vcpu_dstate == *cpu->trace_dstate && | ||
51 | (tb_cflags(tb) & (CF_HASH_MASK | CF_INVALID)) == cf_mask)) { | ||
52 | return tb; | ||
53 | } | ||
54 | - tb = tb_htable_lookup(cpu, *pc, *cs_base, *flags, cf_mask); | ||
55 | + tb = tb_htable_lookup(cpu, pc, cs_base, flags, cf_mask); | ||
56 | if (tb == NULL) { | ||
57 | return NULL; | ||
58 | } | ||
59 | diff --git a/accel/tcg/cpu-exec.c b/accel/tcg/cpu-exec.c | ||
60 | index XXXXXXX..XXXXXXX 100644 | ||
61 | --- a/accel/tcg/cpu-exec.c | ||
62 | +++ b/accel/tcg/cpu-exec.c | ||
63 | @@ -XXX,XX +XXX,XX @@ static void cpu_exec_exit(CPUState *cpu) | ||
64 | |||
65 | void cpu_exec_step_atomic(CPUState *cpu) | ||
66 | { | ||
67 | + CPUArchState *env = (CPUArchState *)cpu->env_ptr; | ||
68 | TranslationBlock *tb; | ||
69 | target_ulong cs_base, pc; | ||
70 | uint32_t flags; | ||
71 | @@ -XXX,XX +XXX,XX @@ void cpu_exec_step_atomic(CPUState *cpu) | ||
72 | g_assert(!cpu->running); | ||
73 | cpu->running = true; | ||
74 | |||
75 | - tb = tb_lookup__cpu_state(cpu, &pc, &cs_base, &flags, cf_mask); | ||
76 | + cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags); | ||
77 | + tb = tb_lookup(cpu, pc, cs_base, flags, cf_mask); | ||
78 | + | ||
79 | if (tb == NULL) { | ||
80 | mmap_lock(); | ||
81 | tb = tb_gen_code(cpu, pc, cs_base, flags, cflags); | ||
82 | @@ -XXX,XX +XXX,XX @@ static inline TranslationBlock *tb_find(CPUState *cpu, | ||
83 | TranslationBlock *last_tb, | ||
84 | int tb_exit, uint32_t cf_mask) | ||
85 | { | ||
86 | + CPUArchState *env = (CPUArchState *)cpu->env_ptr; | ||
87 | TranslationBlock *tb; | ||
88 | target_ulong cs_base, pc; | ||
89 | uint32_t flags; | ||
90 | |||
91 | - tb = tb_lookup__cpu_state(cpu, &pc, &cs_base, &flags, cf_mask); | ||
92 | + cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags); | ||
93 | + | ||
94 | + tb = tb_lookup(cpu, pc, cs_base, flags, cf_mask); | ||
95 | if (tb == NULL) { | ||
96 | mmap_lock(); | ||
97 | tb = tb_gen_code(cpu, pc, cs_base, flags, cf_mask); | ||
98 | diff --git a/accel/tcg/tcg-runtime.c b/accel/tcg/tcg-runtime.c | ||
99 | index XXXXXXX..XXXXXXX 100644 | ||
100 | --- a/accel/tcg/tcg-runtime.c | ||
101 | +++ b/accel/tcg/tcg-runtime.c | ||
102 | @@ -XXX,XX +XXX,XX @@ const void *HELPER(lookup_tb_ptr)(CPUArchState *env) | ||
103 | target_ulong cs_base, pc; | ||
104 | uint32_t flags; | ||
105 | |||
106 | - tb = tb_lookup__cpu_state(cpu, &pc, &cs_base, &flags, curr_cflags()); | ||
107 | + cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags); | ||
108 | + | ||
109 | + tb = tb_lookup(cpu, pc, cs_base, flags, curr_cflags()); | ||
110 | if (tb == NULL) { | ||
111 | return tcg_code_gen_epilogue; | ||
112 | } | ||
113 | -- | ||
114 | 2.25.1 | ||
115 | |||
116 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | From: Alex Bennée <alex.bennee@linaro.org> | ||
1 | 2 | ||
3 | There is nothing special about this compile flag that doesn't mean we | ||
4 | can't just compute it with curr_cflags() which we should be using when | ||
5 | building a new set. | ||
6 | |||
7 | Signed-off-by: Alex Bennée <alex.bennee@linaro.org> | ||
8 | Message-Id: <20210224165811.11567-3-alex.bennee@linaro.org> | ||
9 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
10 | --- | ||
11 | include/exec/exec-all.h | 8 +++++--- | ||
12 | include/exec/tb-lookup.h | 3 --- | ||
13 | accel/tcg/cpu-exec.c | 9 ++++----- | ||
14 | accel/tcg/tcg-runtime.c | 2 +- | ||
15 | accel/tcg/translate-all.c | 6 +++--- | ||
16 | softmmu/physmem.c | 2 +- | ||
17 | 6 files changed, 14 insertions(+), 16 deletions(-) | ||
18 | |||
19 | diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h | ||
20 | index XXXXXXX..XXXXXXX 100644 | ||
21 | --- a/include/exec/exec-all.h | ||
22 | +++ b/include/exec/exec-all.h | ||
23 | @@ -XXX,XX +XXX,XX @@ static inline uint32_t tb_cflags(const TranslationBlock *tb) | ||
24 | } | ||
25 | |||
26 | /* current cflags for hashing/comparison */ | ||
27 | -static inline uint32_t curr_cflags(void) | ||
28 | +static inline uint32_t curr_cflags(CPUState *cpu) | ||
29 | { | ||
30 | - return (parallel_cpus ? CF_PARALLEL : 0) | ||
31 | - | (icount_enabled() ? CF_USE_ICOUNT : 0); | ||
32 | + uint32_t cflags = deposit32(0, CF_CLUSTER_SHIFT, 8, cpu->cluster_index); | ||
33 | + cflags |= parallel_cpus ? CF_PARALLEL : 0; | ||
34 | + cflags |= icount_enabled() ? CF_USE_ICOUNT : 0; | ||
35 | + return cflags; | ||
36 | } | ||
37 | |||
38 | /* TranslationBlock invalidate API */ | ||
39 | diff --git a/include/exec/tb-lookup.h b/include/exec/tb-lookup.h | ||
40 | index XXXXXXX..XXXXXXX 100644 | ||
41 | --- a/include/exec/tb-lookup.h | ||
42 | +++ b/include/exec/tb-lookup.h | ||
43 | @@ -XXX,XX +XXX,XX @@ static inline TranslationBlock * tb_lookup(CPUState *cpu, | ||
44 | hash = tb_jmp_cache_hash_func(pc); | ||
45 | tb = qatomic_rcu_read(&cpu->tb_jmp_cache[hash]); | ||
46 | |||
47 | - cf_mask &= ~CF_CLUSTER_MASK; | ||
48 | - cf_mask |= cpu->cluster_index << CF_CLUSTER_SHIFT; | ||
49 | - | ||
50 | if (likely(tb && | ||
51 | tb->pc == pc && | ||
52 | tb->cs_base == cs_base && | ||
53 | diff --git a/accel/tcg/cpu-exec.c b/accel/tcg/cpu-exec.c | ||
54 | index XXXXXXX..XXXXXXX 100644 | ||
55 | --- a/accel/tcg/cpu-exec.c | ||
56 | +++ b/accel/tcg/cpu-exec.c | ||
57 | @@ -XXX,XX +XXX,XX @@ void cpu_exec_step_atomic(CPUState *cpu) | ||
58 | TranslationBlock *tb; | ||
59 | target_ulong cs_base, pc; | ||
60 | uint32_t flags; | ||
61 | - uint32_t cflags = 1; | ||
62 | - uint32_t cf_mask = cflags & CF_HASH_MASK; | ||
63 | + uint32_t cflags = (curr_cflags(cpu) & ~CF_PARALLEL) | 1; | ||
64 | int tb_exit; | ||
65 | |||
66 | if (sigsetjmp(cpu->jmp_env, 0) == 0) { | ||
67 | @@ -XXX,XX +XXX,XX @@ void cpu_exec_step_atomic(CPUState *cpu) | ||
68 | cpu->running = true; | ||
69 | |||
70 | cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags); | ||
71 | - tb = tb_lookup(cpu, pc, cs_base, flags, cf_mask); | ||
72 | + tb = tb_lookup(cpu, pc, cs_base, flags, cflags); | ||
73 | |||
74 | if (tb == NULL) { | ||
75 | mmap_lock(); | ||
76 | @@ -XXX,XX +XXX,XX @@ static inline bool cpu_handle_exception(CPUState *cpu, int *ret) | ||
77 | if (replay_has_exception() | ||
78 | && cpu_neg(cpu)->icount_decr.u16.low + cpu->icount_extra == 0) { | ||
79 | /* Execute just one insn to trigger exception pending in the log */ | ||
80 | - cpu->cflags_next_tb = (curr_cflags() & ~CF_USE_ICOUNT) | 1; | ||
81 | + cpu->cflags_next_tb = (curr_cflags(cpu) & ~CF_USE_ICOUNT) | 1; | ||
82 | } | ||
83 | #endif | ||
84 | return false; | ||
85 | @@ -XXX,XX +XXX,XX @@ int cpu_exec(CPUState *cpu) | ||
86 | have CF_INVALID set, -1 is a convenient invalid value that | ||
87 | does not require tcg headers for cpu_common_reset. */ | ||
88 | if (cflags == -1) { | ||
89 | - cflags = curr_cflags(); | ||
90 | + cflags = curr_cflags(cpu); | ||
91 | } else { | ||
92 | cpu->cflags_next_tb = -1; | ||
93 | } | ||
94 | diff --git a/accel/tcg/tcg-runtime.c b/accel/tcg/tcg-runtime.c | ||
95 | index XXXXXXX..XXXXXXX 100644 | ||
96 | --- a/accel/tcg/tcg-runtime.c | ||
97 | +++ b/accel/tcg/tcg-runtime.c | ||
98 | @@ -XXX,XX +XXX,XX @@ const void *HELPER(lookup_tb_ptr)(CPUArchState *env) | ||
99 | |||
100 | cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags); | ||
101 | |||
102 | - tb = tb_lookup(cpu, pc, cs_base, flags, curr_cflags()); | ||
103 | + tb = tb_lookup(cpu, pc, cs_base, flags, curr_cflags(cpu)); | ||
104 | if (tb == NULL) { | ||
105 | return tcg_code_gen_epilogue; | ||
106 | } | ||
107 | diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c | ||
108 | index XXXXXXX..XXXXXXX 100644 | ||
109 | --- a/accel/tcg/translate-all.c | ||
110 | +++ b/accel/tcg/translate-all.c | ||
111 | @@ -XXX,XX +XXX,XX @@ tb_invalidate_phys_page_range__locked(struct page_collection *pages, | ||
112 | if (current_tb_modified) { | ||
113 | page_collection_unlock(pages); | ||
114 | /* Force execution of one insn next time. */ | ||
115 | - cpu->cflags_next_tb = 1 | curr_cflags(); | ||
116 | + cpu->cflags_next_tb = 1 | curr_cflags(cpu); | ||
117 | mmap_unlock(); | ||
118 | cpu_loop_exit_noexc(cpu); | ||
119 | } | ||
120 | @@ -XXX,XX +XXX,XX @@ static bool tb_invalidate_phys_page(tb_page_addr_t addr, uintptr_t pc) | ||
121 | #ifdef TARGET_HAS_PRECISE_SMC | ||
122 | if (current_tb_modified) { | ||
123 | /* Force execution of one insn next time. */ | ||
124 | - cpu->cflags_next_tb = 1 | curr_cflags(); | ||
125 | + cpu->cflags_next_tb = 1 | curr_cflags(cpu); | ||
126 | return true; | ||
127 | } | ||
128 | #endif | ||
129 | @@ -XXX,XX +XXX,XX @@ void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr) | ||
130 | * operations only (which execute after completion) so we don't | ||
131 | * double instrument the instruction. | ||
132 | */ | ||
133 | - cpu->cflags_next_tb = curr_cflags() | CF_MEMI_ONLY | CF_LAST_IO | n; | ||
134 | + cpu->cflags_next_tb = curr_cflags(cpu) | CF_MEMI_ONLY | CF_LAST_IO | n; | ||
135 | |||
136 | qemu_log_mask_and_addr(CPU_LOG_EXEC, tb->pc, | ||
137 | "cpu_io_recompile: rewound execution of TB to " | ||
138 | diff --git a/softmmu/physmem.c b/softmmu/physmem.c | ||
139 | index XXXXXXX..XXXXXXX 100644 | ||
140 | --- a/softmmu/physmem.c | ||
141 | +++ b/softmmu/physmem.c | ||
142 | @@ -XXX,XX +XXX,XX @@ void cpu_check_watchpoint(CPUState *cpu, vaddr addr, vaddr len, | ||
143 | cpu_loop_exit_restore(cpu, ra); | ||
144 | } else { | ||
145 | /* Force execution of one insn next time. */ | ||
146 | - cpu->cflags_next_tb = 1 | curr_cflags(); | ||
147 | + cpu->cflags_next_tb = 1 | curr_cflags(cpu); | ||
148 | mmap_unlock(); | ||
149 | if (ra) { | ||
150 | cpu_restore_state(cpu, ra, true); | ||
151 | -- | ||
152 | 2.25.1 | ||
153 | |||
154 | diff view generated by jsdifflib |
1 | From: Claudio Fontana <cfontana@suse.de> | 1 | From: Alex Bennée <alex.bennee@linaro.org> |
---|---|---|---|
2 | 2 | ||
3 | Signed-off-by: Claudio Fontana <cfontana@suse.de> | 3 | We don't really deal in cf_mask most of the time. The one time it's |
4 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> | 4 | relevant is when we want to remove an invalidated TB from the QHT |
5 | Message-Id: <20201015143217.29337-4-cfontana@suse.de> | 5 | lookup. Everywhere else we should be looking up things without |
6 | CF_INVALID set. | ||
7 | |||
8 | Signed-off-by: Alex Bennée <alex.bennee@linaro.org> | ||
9 | Message-Id: <20210224165811.11567-4-alex.bennee@linaro.org> | ||
6 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | 10 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> |
7 | --- | 11 | --- |
8 | accel/tcg/tcg-cpus-icount.h | 6 +-- | 12 | include/exec/exec-all.h | 4 +--- |
9 | accel/tcg/tcg-cpus-rr.h | 2 +- | 13 | include/exec/tb-lookup.h | 9 ++++++--- |
10 | accel/tcg/tcg-cpus.h | 6 +-- | 14 | accel/tcg/cpu-exec.c | 16 ++++++++-------- |
11 | accel/tcg/tcg-cpus-icount.c | 24 ++++++------ | 15 | accel/tcg/tcg-runtime.c | 2 +- |
12 | accel/tcg/tcg-cpus-mttcg.c | 10 ++--- | 16 | accel/tcg/translate-all.c | 8 +++++--- |
13 | accel/tcg/tcg-cpus-rr.c | 74 ++++++++++++++++++------------------- | 17 | 5 files changed, 21 insertions(+), 18 deletions(-) |
14 | accel/tcg/tcg-cpus.c | 6 +-- | 18 | |
15 | 7 files changed, 64 insertions(+), 64 deletions(-) | 19 | diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h |
16 | 20 | index XXXXXXX..XXXXXXX 100644 | |
17 | diff --git a/accel/tcg/tcg-cpus-icount.h b/accel/tcg/tcg-cpus-icount.h | 21 | --- a/include/exec/exec-all.h |
18 | index XXXXXXX..XXXXXXX 100644 | 22 | +++ b/include/exec/exec-all.h |
19 | --- a/accel/tcg/tcg-cpus-icount.h | 23 | @@ -XXX,XX +XXX,XX @@ struct TranslationBlock { |
20 | +++ b/accel/tcg/tcg-cpus-icount.h | 24 | #define CF_PARALLEL 0x00080000 /* Generate code for a parallel context */ |
25 | #define CF_CLUSTER_MASK 0xff000000 /* Top 8 bits are cluster ID */ | ||
26 | #define CF_CLUSTER_SHIFT 24 | ||
27 | -/* cflags' mask for hashing/comparison, basically ignore CF_INVALID */ | ||
28 | -#define CF_HASH_MASK (~CF_INVALID) | ||
29 | |||
30 | /* Per-vCPU dynamic tracing state used to generate this TB */ | ||
31 | uint32_t trace_vcpu_dstate; | ||
32 | @@ -XXX,XX +XXX,XX @@ void tb_flush(CPUState *cpu); | ||
33 | void tb_phys_invalidate(TranslationBlock *tb, tb_page_addr_t page_addr); | ||
34 | TranslationBlock *tb_htable_lookup(CPUState *cpu, target_ulong pc, | ||
35 | target_ulong cs_base, uint32_t flags, | ||
36 | - uint32_t cf_mask); | ||
37 | + uint32_t cflags); | ||
38 | void tb_set_jmp_target(TranslationBlock *tb, int n, uintptr_t addr); | ||
39 | |||
40 | /* GETPC is the true target of the return instruction that we'll execute. */ | ||
41 | diff --git a/include/exec/tb-lookup.h b/include/exec/tb-lookup.h | ||
42 | index XXXXXXX..XXXXXXX 100644 | ||
43 | --- a/include/exec/tb-lookup.h | ||
44 | +++ b/include/exec/tb-lookup.h | ||
21 | @@ -XXX,XX +XXX,XX @@ | 45 | @@ -XXX,XX +XXX,XX @@ |
22 | #ifndef TCG_CPUS_ICOUNT_H | 46 | /* Might cause an exception, so have a longjmp destination ready */ |
23 | #define TCG_CPUS_ICOUNT_H | 47 | static inline TranslationBlock * tb_lookup(CPUState *cpu, |
24 | 48 | target_ulong pc, target_ulong cs_base, | |
25 | -void handle_icount_deadline(void); | 49 | - uint32_t flags, uint32_t cf_mask) |
26 | -void prepare_icount_for_run(CPUState *cpu); | 50 | + uint32_t flags, uint32_t cflags) |
27 | -void process_icount_data(CPUState *cpu); | 51 | { |
28 | +void icount_handle_deadline(void); | 52 | TranslationBlock *tb; |
29 | +void icount_prepare_for_run(CPUState *cpu); | 53 | uint32_t hash; |
30 | +void icount_process_data(CPUState *cpu); | 54 | |
31 | 55 | + /* we should never be trying to look up an INVALID tb */ | |
32 | #endif /* TCG_CPUS_ICOUNT_H */ | 56 | + tcg_debug_assert(!(cflags & CF_INVALID)); |
33 | diff --git a/accel/tcg/tcg-cpus-rr.h b/accel/tcg/tcg-cpus-rr.h | 57 | + |
34 | index XXXXXXX..XXXXXXX 100644 | 58 | hash = tb_jmp_cache_hash_func(pc); |
35 | --- a/accel/tcg/tcg-cpus-rr.h | 59 | tb = qatomic_rcu_read(&cpu->tb_jmp_cache[hash]); |
36 | +++ b/accel/tcg/tcg-cpus-rr.h | 60 | |
61 | @@ -XXX,XX +XXX,XX @@ static inline TranslationBlock * tb_lookup(CPUState *cpu, | ||
62 | tb->cs_base == cs_base && | ||
63 | tb->flags == flags && | ||
64 | tb->trace_vcpu_dstate == *cpu->trace_dstate && | ||
65 | - (tb_cflags(tb) & (CF_HASH_MASK | CF_INVALID)) == cf_mask)) { | ||
66 | + tb_cflags(tb) == cflags)) { | ||
67 | return tb; | ||
68 | } | ||
69 | - tb = tb_htable_lookup(cpu, pc, cs_base, flags, cf_mask); | ||
70 | + tb = tb_htable_lookup(cpu, pc, cs_base, flags, cflags); | ||
71 | if (tb == NULL) { | ||
72 | return NULL; | ||
73 | } | ||
74 | diff --git a/accel/tcg/cpu-exec.c b/accel/tcg/cpu-exec.c | ||
75 | index XXXXXXX..XXXXXXX 100644 | ||
76 | --- a/accel/tcg/cpu-exec.c | ||
77 | +++ b/accel/tcg/cpu-exec.c | ||
78 | @@ -XXX,XX +XXX,XX @@ struct tb_desc { | ||
79 | CPUArchState *env; | ||
80 | tb_page_addr_t phys_page1; | ||
81 | uint32_t flags; | ||
82 | - uint32_t cf_mask; | ||
83 | + uint32_t cflags; | ||
84 | uint32_t trace_vcpu_dstate; | ||
85 | }; | ||
86 | |||
87 | @@ -XXX,XX +XXX,XX @@ static bool tb_lookup_cmp(const void *p, const void *d) | ||
88 | tb->cs_base == desc->cs_base && | ||
89 | tb->flags == desc->flags && | ||
90 | tb->trace_vcpu_dstate == desc->trace_vcpu_dstate && | ||
91 | - (tb_cflags(tb) & (CF_HASH_MASK | CF_INVALID)) == desc->cf_mask) { | ||
92 | + tb_cflags(tb) == desc->cflags) { | ||
93 | /* check next page if needed */ | ||
94 | if (tb->page_addr[1] == -1) { | ||
95 | return true; | ||
96 | @@ -XXX,XX +XXX,XX @@ static bool tb_lookup_cmp(const void *p, const void *d) | ||
97 | |||
98 | TranslationBlock *tb_htable_lookup(CPUState *cpu, target_ulong pc, | ||
99 | target_ulong cs_base, uint32_t flags, | ||
100 | - uint32_t cf_mask) | ||
101 | + uint32_t cflags) | ||
102 | { | ||
103 | tb_page_addr_t phys_pc; | ||
104 | struct tb_desc desc; | ||
105 | @@ -XXX,XX +XXX,XX @@ TranslationBlock *tb_htable_lookup(CPUState *cpu, target_ulong pc, | ||
106 | desc.env = (CPUArchState *)cpu->env_ptr; | ||
107 | desc.cs_base = cs_base; | ||
108 | desc.flags = flags; | ||
109 | - desc.cf_mask = cf_mask; | ||
110 | + desc.cflags = cflags; | ||
111 | desc.trace_vcpu_dstate = *cpu->trace_dstate; | ||
112 | desc.pc = pc; | ||
113 | phys_pc = get_page_addr_code(desc.env, pc); | ||
114 | @@ -XXX,XX +XXX,XX @@ TranslationBlock *tb_htable_lookup(CPUState *cpu, target_ulong pc, | ||
115 | return NULL; | ||
116 | } | ||
117 | desc.phys_page1 = phys_pc & TARGET_PAGE_MASK; | ||
118 | - h = tb_hash_func(phys_pc, pc, flags, cf_mask, *cpu->trace_dstate); | ||
119 | + h = tb_hash_func(phys_pc, pc, flags, cflags, *cpu->trace_dstate); | ||
120 | return qht_lookup_custom(&tb_ctx.htable, &desc, h, tb_lookup_cmp); | ||
121 | } | ||
122 | |||
123 | @@ -XXX,XX +XXX,XX @@ static inline void tb_add_jump(TranslationBlock *tb, int n, | ||
124 | |||
125 | static inline TranslationBlock *tb_find(CPUState *cpu, | ||
126 | TranslationBlock *last_tb, | ||
127 | - int tb_exit, uint32_t cf_mask) | ||
128 | + int tb_exit, uint32_t cflags) | ||
129 | { | ||
130 | CPUArchState *env = (CPUArchState *)cpu->env_ptr; | ||
131 | TranslationBlock *tb; | ||
132 | @@ -XXX,XX +XXX,XX @@ static inline TranslationBlock *tb_find(CPUState *cpu, | ||
133 | |||
134 | cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags); | ||
135 | |||
136 | - tb = tb_lookup(cpu, pc, cs_base, flags, cf_mask); | ||
137 | + tb = tb_lookup(cpu, pc, cs_base, flags, cflags); | ||
138 | if (tb == NULL) { | ||
139 | mmap_lock(); | ||
140 | - tb = tb_gen_code(cpu, pc, cs_base, flags, cf_mask); | ||
141 | + tb = tb_gen_code(cpu, pc, cs_base, flags, cflags); | ||
142 | mmap_unlock(); | ||
143 | /* We add the TB in the virtual pc hash table for the fast lookup */ | ||
144 | qatomic_set(&cpu->tb_jmp_cache[tb_jmp_cache_hash_func(pc)], tb); | ||
145 | diff --git a/accel/tcg/tcg-runtime.c b/accel/tcg/tcg-runtime.c | ||
146 | index XXXXXXX..XXXXXXX 100644 | ||
147 | --- a/accel/tcg/tcg-runtime.c | ||
148 | +++ b/accel/tcg/tcg-runtime.c | ||
37 | @@ -XXX,XX +XXX,XX @@ | 149 | @@ -XXX,XX +XXX,XX @@ |
38 | #define TCG_KICK_PERIOD (NANOSECONDS_PER_SECOND / 10) | 150 | #include "exec/helper-proto.h" |
39 | 151 | #include "exec/cpu_ldst.h" | |
40 | /* Kick all RR vCPUs. */ | 152 | #include "exec/exec-all.h" |
41 | -void qemu_cpu_kick_rr_cpus(CPUState *unused); | 153 | -#include "exec/tb-lookup.h" |
42 | +void rr_kick_vcpu_thread(CPUState *unused); | 154 | #include "disas/disas.h" |
43 | 155 | #include "exec/log.h" | |
44 | /* start the round robin vcpu thread */ | 156 | #include "tcg/tcg.h" |
45 | void rr_start_vcpu_thread(CPUState *cpu); | 157 | +#include "exec/tb-lookup.h" |
46 | diff --git a/accel/tcg/tcg-cpus.h b/accel/tcg/tcg-cpus.h | 158 | |
47 | index XXXXXXX..XXXXXXX 100644 | 159 | /* 32-bit helpers */ |
48 | --- a/accel/tcg/tcg-cpus.h | 160 | |
49 | +++ b/accel/tcg/tcg-cpus.h | 161 | diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c |
50 | @@ -XXX,XX +XXX,XX @@ extern const CpusAccel tcg_cpus_mttcg; | 162 | index XXXXXXX..XXXXXXX 100644 |
51 | extern const CpusAccel tcg_cpus_icount; | 163 | --- a/accel/tcg/translate-all.c |
52 | extern const CpusAccel tcg_cpus_rr; | 164 | +++ b/accel/tcg/translate-all.c |
53 | 165 | @@ -XXX,XX +XXX,XX @@ static bool tb_cmp(const void *ap, const void *bp) | |
54 | -void qemu_tcg_destroy_vcpu(CPUState *cpu); | 166 | return a->pc == b->pc && |
55 | -int tcg_cpu_exec(CPUState *cpu); | 167 | a->cs_base == b->cs_base && |
56 | -void tcg_handle_interrupt(CPUState *cpu, int mask); | 168 | a->flags == b->flags && |
57 | +void tcg_cpus_destroy(CPUState *cpu); | 169 | - (tb_cflags(a) & CF_HASH_MASK) == (tb_cflags(b) & CF_HASH_MASK) && |
58 | +int tcg_cpus_exec(CPUState *cpu); | 170 | + (tb_cflags(a) & ~CF_INVALID) == (tb_cflags(b) & ~CF_INVALID) && |
59 | +void tcg_cpus_handle_interrupt(CPUState *cpu, int mask); | 171 | a->trace_vcpu_dstate == b->trace_vcpu_dstate && |
60 | 172 | a->page_addr[0] == b->page_addr[0] && | |
61 | #endif /* TCG_CPUS_H */ | 173 | a->page_addr[1] == b->page_addr[1]; |
62 | diff --git a/accel/tcg/tcg-cpus-icount.c b/accel/tcg/tcg-cpus-icount.c | 174 | @@ -XXX,XX +XXX,XX @@ static void do_tb_phys_invalidate(TranslationBlock *tb, bool rm_from_page_list) |
63 | index XXXXXXX..XXXXXXX 100644 | 175 | PageDesc *p; |
64 | --- a/accel/tcg/tcg-cpus-icount.c | 176 | uint32_t h; |
65 | +++ b/accel/tcg/tcg-cpus-icount.c | 177 | tb_page_addr_t phys_pc; |
66 | @@ -XXX,XX +XXX,XX @@ | 178 | + uint32_t orig_cflags = tb_cflags(tb); |
67 | #include "tcg-cpus-icount.h" | 179 | |
68 | #include "tcg-cpus-rr.h" | 180 | assert_memory_lock(); |
69 | 181 | ||
70 | -static int64_t tcg_get_icount_limit(void) | 182 | @@ -XXX,XX +XXX,XX @@ static void do_tb_phys_invalidate(TranslationBlock *tb, bool rm_from_page_list) |
71 | +static int64_t icount_get_limit(void) | 183 | |
72 | { | 184 | /* remove the TB from the hash list */ |
73 | int64_t deadline; | 185 | phys_pc = tb->page_addr[0] + (tb->pc & ~TARGET_PAGE_MASK); |
74 | 186 | - h = tb_hash_func(phys_pc, tb->pc, tb->flags, tb_cflags(tb) & CF_HASH_MASK, | |
75 | @@ -XXX,XX +XXX,XX @@ static int64_t tcg_get_icount_limit(void) | 187 | + h = tb_hash_func(phys_pc, tb->pc, tb->flags, orig_cflags, |
76 | } | 188 | tb->trace_vcpu_dstate); |
77 | } | 189 | if (!qht_remove(&tb_ctx.htable, tb, h)) { |
78 | 190 | return; | |
79 | -static void notify_aio_contexts(void) | 191 | @@ -XXX,XX +XXX,XX @@ tb_link_page(TranslationBlock *tb, tb_page_addr_t phys_pc, |
80 | +static void icount_notify_aio_contexts(void) | 192 | uint32_t h; |
81 | { | 193 | |
82 | /* Wake up other AioContexts. */ | 194 | assert_memory_lock(); |
83 | qemu_clock_notify(QEMU_CLOCK_VIRTUAL); | 195 | + tcg_debug_assert(!(tb->cflags & CF_INVALID)); |
84 | qemu_clock_run_timers(QEMU_CLOCK_VIRTUAL); | ||
85 | } | ||
86 | |||
87 | -void handle_icount_deadline(void) | ||
88 | +void icount_handle_deadline(void) | ||
89 | { | ||
90 | assert(qemu_in_vcpu_thread()); | ||
91 | int64_t deadline = qemu_clock_deadline_ns_all(QEMU_CLOCK_VIRTUAL, | ||
92 | QEMU_TIMER_ATTR_ALL); | ||
93 | |||
94 | if (deadline == 0) { | ||
95 | - notify_aio_contexts(); | ||
96 | + icount_notify_aio_contexts(); | ||
97 | } | ||
98 | } | ||
99 | |||
100 | -void prepare_icount_for_run(CPUState *cpu) | ||
101 | +void icount_prepare_for_run(CPUState *cpu) | ||
102 | { | ||
103 | int insns_left; | ||
104 | 196 | ||
105 | /* | 197 | /* |
106 | - * These should always be cleared by process_icount_data after | 198 | * Add the TB to the page list, acquiring first the pages's locks. |
107 | + * These should always be cleared by icount_process_data after | 199 | @@ -XXX,XX +XXX,XX @@ tb_link_page(TranslationBlock *tb, tb_page_addr_t phys_pc, |
108 | * each vCPU execution. However u16.high can be raised | 200 | } |
109 | - * asynchronously by cpu_exit/cpu_interrupt/tcg_handle_interrupt | 201 | |
110 | + * asynchronously by cpu_exit/cpu_interrupt/tcg_cpus_handle_interrupt | 202 | /* add in the hash table */ |
111 | */ | 203 | - h = tb_hash_func(phys_pc, tb->pc, tb->flags, tb->cflags & CF_HASH_MASK, |
112 | g_assert(cpu_neg(cpu)->icount_decr.u16.low == 0); | 204 | + h = tb_hash_func(phys_pc, tb->pc, tb->flags, tb->cflags, |
113 | g_assert(cpu->icount_extra == 0); | 205 | tb->trace_vcpu_dstate); |
114 | 206 | qht_insert(&tb_ctx.htable, tb, h, &existing_tb); | |
115 | - cpu->icount_budget = tcg_get_icount_limit(); | ||
116 | + cpu->icount_budget = icount_get_limit(); | ||
117 | insns_left = MIN(0xffff, cpu->icount_budget); | ||
118 | cpu_neg(cpu)->icount_decr.u16.low = insns_left; | ||
119 | cpu->icount_extra = cpu->icount_budget - insns_left; | ||
120 | @@ -XXX,XX +XXX,XX @@ void prepare_icount_for_run(CPUState *cpu) | ||
121 | replay_mutex_lock(); | ||
122 | |||
123 | if (cpu->icount_budget == 0 && replay_has_checkpoint()) { | ||
124 | - notify_aio_contexts(); | ||
125 | + icount_notify_aio_contexts(); | ||
126 | } | ||
127 | } | ||
128 | |||
129 | -void process_icount_data(CPUState *cpu) | ||
130 | +void icount_process_data(CPUState *cpu) | ||
131 | { | ||
132 | /* Account for executed instructions */ | ||
133 | icount_update(cpu); | ||
134 | @@ -XXX,XX +XXX,XX @@ static void icount_handle_interrupt(CPUState *cpu, int mask) | ||
135 | { | ||
136 | int old_mask = cpu->interrupt_request; | ||
137 | |||
138 | - tcg_handle_interrupt(cpu, mask); | ||
139 | + tcg_cpus_handle_interrupt(cpu, mask); | ||
140 | if (qemu_cpu_is_self(cpu) && | ||
141 | !cpu->can_do_io | ||
142 | && (mask & ~old_mask) != 0) { | ||
143 | @@ -XXX,XX +XXX,XX @@ static void icount_handle_interrupt(CPUState *cpu, int mask) | ||
144 | |||
145 | const CpusAccel tcg_cpus_icount = { | ||
146 | .create_vcpu_thread = rr_start_vcpu_thread, | ||
147 | - .kick_vcpu_thread = qemu_cpu_kick_rr_cpus, | ||
148 | + .kick_vcpu_thread = rr_kick_vcpu_thread, | ||
149 | |||
150 | .handle_interrupt = icount_handle_interrupt, | ||
151 | .get_virtual_clock = icount_get, | ||
152 | diff --git a/accel/tcg/tcg-cpus-mttcg.c b/accel/tcg/tcg-cpus-mttcg.c | ||
153 | index XXXXXXX..XXXXXXX 100644 | ||
154 | --- a/accel/tcg/tcg-cpus-mttcg.c | ||
155 | +++ b/accel/tcg/tcg-cpus-mttcg.c | ||
156 | @@ -XXX,XX +XXX,XX @@ | ||
157 | * current CPUState for a given thread. | ||
158 | */ | ||
159 | |||
160 | -static void *tcg_cpu_thread_fn(void *arg) | ||
161 | +static void *mttcg_cpu_thread_fn(void *arg) | ||
162 | { | ||
163 | CPUState *cpu = arg; | ||
164 | |||
165 | @@ -XXX,XX +XXX,XX @@ static void *tcg_cpu_thread_fn(void *arg) | ||
166 | if (cpu_can_run(cpu)) { | ||
167 | int r; | ||
168 | qemu_mutex_unlock_iothread(); | ||
169 | - r = tcg_cpu_exec(cpu); | ||
170 | + r = tcg_cpus_exec(cpu); | ||
171 | qemu_mutex_lock_iothread(); | ||
172 | switch (r) { | ||
173 | case EXCP_DEBUG: | ||
174 | @@ -XXX,XX +XXX,XX @@ static void *tcg_cpu_thread_fn(void *arg) | ||
175 | qemu_wait_io_event(cpu); | ||
176 | } while (!cpu->unplug || cpu_can_run(cpu)); | ||
177 | |||
178 | - qemu_tcg_destroy_vcpu(cpu); | ||
179 | + tcg_cpus_destroy(cpu); | ||
180 | qemu_mutex_unlock_iothread(); | ||
181 | rcu_unregister_thread(); | ||
182 | return NULL; | ||
183 | @@ -XXX,XX +XXX,XX @@ static void mttcg_start_vcpu_thread(CPUState *cpu) | ||
184 | snprintf(thread_name, VCPU_THREAD_NAME_SIZE, "CPU %d/TCG", | ||
185 | cpu->cpu_index); | ||
186 | |||
187 | - qemu_thread_create(cpu->thread, thread_name, tcg_cpu_thread_fn, | ||
188 | + qemu_thread_create(cpu->thread, thread_name, mttcg_cpu_thread_fn, | ||
189 | cpu, QEMU_THREAD_JOINABLE); | ||
190 | |||
191 | #ifdef _WIN32 | ||
192 | @@ -XXX,XX +XXX,XX @@ const CpusAccel tcg_cpus_mttcg = { | ||
193 | .create_vcpu_thread = mttcg_start_vcpu_thread, | ||
194 | .kick_vcpu_thread = mttcg_kick_vcpu_thread, | ||
195 | |||
196 | - .handle_interrupt = tcg_handle_interrupt, | ||
197 | + .handle_interrupt = tcg_cpus_handle_interrupt, | ||
198 | }; | ||
199 | diff --git a/accel/tcg/tcg-cpus-rr.c b/accel/tcg/tcg-cpus-rr.c | ||
200 | index XXXXXXX..XXXXXXX 100644 | ||
201 | --- a/accel/tcg/tcg-cpus-rr.c | ||
202 | +++ b/accel/tcg/tcg-cpus-rr.c | ||
203 | @@ -XXX,XX +XXX,XX @@ | ||
204 | #include "tcg-cpus-icount.h" | ||
205 | |||
206 | /* Kick all RR vCPUs */ | ||
207 | -void qemu_cpu_kick_rr_cpus(CPUState *unused) | ||
208 | +void rr_kick_vcpu_thread(CPUState *unused) | ||
209 | { | ||
210 | CPUState *cpu; | ||
211 | |||
212 | @@ -XXX,XX +XXX,XX @@ void qemu_cpu_kick_rr_cpus(CPUState *unused) | ||
213 | * idleness is complete. | ||
214 | */ | ||
215 | |||
216 | -static QEMUTimer *tcg_kick_vcpu_timer; | ||
217 | -static CPUState *tcg_current_rr_cpu; | ||
218 | +static QEMUTimer *rr_kick_vcpu_timer; | ||
219 | +static CPUState *rr_current_cpu; | ||
220 | |||
221 | #define TCG_KICK_PERIOD (NANOSECONDS_PER_SECOND / 10) | ||
222 | |||
223 | -static inline int64_t qemu_tcg_next_kick(void) | ||
224 | +static inline int64_t rr_next_kick_time(void) | ||
225 | { | ||
226 | return qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + TCG_KICK_PERIOD; | ||
227 | } | ||
228 | |||
229 | /* Kick the currently round-robin scheduled vCPU to next */ | ||
230 | -static void qemu_cpu_kick_rr_next_cpu(void) | ||
231 | +static void rr_kick_next_cpu(void) | ||
232 | { | ||
233 | CPUState *cpu; | ||
234 | do { | ||
235 | - cpu = qatomic_mb_read(&tcg_current_rr_cpu); | ||
236 | + cpu = qatomic_mb_read(&rr_current_cpu); | ||
237 | if (cpu) { | ||
238 | cpu_exit(cpu); | ||
239 | } | ||
240 | - } while (cpu != qatomic_mb_read(&tcg_current_rr_cpu)); | ||
241 | + } while (cpu != qatomic_mb_read(&rr_current_cpu)); | ||
242 | } | ||
243 | |||
244 | -static void kick_tcg_thread(void *opaque) | ||
245 | +static void rr_kick_thread(void *opaque) | ||
246 | { | ||
247 | - timer_mod(tcg_kick_vcpu_timer, qemu_tcg_next_kick()); | ||
248 | - qemu_cpu_kick_rr_next_cpu(); | ||
249 | + timer_mod(rr_kick_vcpu_timer, rr_next_kick_time()); | ||
250 | + rr_kick_next_cpu(); | ||
251 | } | ||
252 | |||
253 | -static void start_tcg_kick_timer(void) | ||
254 | +static void rr_start_kick_timer(void) | ||
255 | { | ||
256 | - if (!tcg_kick_vcpu_timer && CPU_NEXT(first_cpu)) { | ||
257 | - tcg_kick_vcpu_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL, | ||
258 | - kick_tcg_thread, NULL); | ||
259 | + if (!rr_kick_vcpu_timer && CPU_NEXT(first_cpu)) { | ||
260 | + rr_kick_vcpu_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL, | ||
261 | + rr_kick_thread, NULL); | ||
262 | } | ||
263 | - if (tcg_kick_vcpu_timer && !timer_pending(tcg_kick_vcpu_timer)) { | ||
264 | - timer_mod(tcg_kick_vcpu_timer, qemu_tcg_next_kick()); | ||
265 | + if (rr_kick_vcpu_timer && !timer_pending(rr_kick_vcpu_timer)) { | ||
266 | + timer_mod(rr_kick_vcpu_timer, rr_next_kick_time()); | ||
267 | } | ||
268 | } | ||
269 | |||
270 | -static void stop_tcg_kick_timer(void) | ||
271 | +static void rr_stop_kick_timer(void) | ||
272 | { | ||
273 | - if (tcg_kick_vcpu_timer && timer_pending(tcg_kick_vcpu_timer)) { | ||
274 | - timer_del(tcg_kick_vcpu_timer); | ||
275 | + if (rr_kick_vcpu_timer && timer_pending(rr_kick_vcpu_timer)) { | ||
276 | + timer_del(rr_kick_vcpu_timer); | ||
277 | } | ||
278 | } | ||
279 | |||
280 | -static void qemu_tcg_rr_wait_io_event(void) | ||
281 | +static void rr_wait_io_event(void) | ||
282 | { | ||
283 | CPUState *cpu; | ||
284 | |||
285 | while (all_cpu_threads_idle()) { | ||
286 | - stop_tcg_kick_timer(); | ||
287 | + rr_stop_kick_timer(); | ||
288 | qemu_cond_wait_iothread(first_cpu->halt_cond); | ||
289 | } | ||
290 | |||
291 | - start_tcg_kick_timer(); | ||
292 | + rr_start_kick_timer(); | ||
293 | |||
294 | CPU_FOREACH(cpu) { | ||
295 | qemu_wait_io_event_common(cpu); | ||
296 | @@ -XXX,XX +XXX,XX @@ static void qemu_tcg_rr_wait_io_event(void) | ||
297 | * Destroy any remaining vCPUs which have been unplugged and have | ||
298 | * finished running | ||
299 | */ | ||
300 | -static void deal_with_unplugged_cpus(void) | ||
301 | +static void rr_deal_with_unplugged_cpus(void) | ||
302 | { | ||
303 | CPUState *cpu; | ||
304 | |||
305 | CPU_FOREACH(cpu) { | ||
306 | if (cpu->unplug && !cpu_can_run(cpu)) { | ||
307 | - qemu_tcg_destroy_vcpu(cpu); | ||
308 | + tcg_cpus_destroy(cpu); | ||
309 | break; | ||
310 | } | ||
311 | } | ||
312 | @@ -XXX,XX +XXX,XX @@ static void deal_with_unplugged_cpus(void) | ||
313 | * elsewhere. | ||
314 | */ | ||
315 | |||
316 | -static void *tcg_rr_cpu_thread_fn(void *arg) | ||
317 | +static void *rr_cpu_thread_fn(void *arg) | ||
318 | { | ||
319 | CPUState *cpu = arg; | ||
320 | |||
321 | @@ -XXX,XX +XXX,XX @@ static void *tcg_rr_cpu_thread_fn(void *arg) | ||
322 | } | ||
323 | } | ||
324 | |||
325 | - start_tcg_kick_timer(); | ||
326 | + rr_start_kick_timer(); | ||
327 | |||
328 | cpu = first_cpu; | ||
329 | |||
330 | @@ -XXX,XX +XXX,XX @@ static void *tcg_rr_cpu_thread_fn(void *arg) | ||
331 | * Run the timers here. This is much more efficient than | ||
332 | * waking up the I/O thread and waiting for completion. | ||
333 | */ | ||
334 | - handle_icount_deadline(); | ||
335 | + icount_handle_deadline(); | ||
336 | } | ||
337 | |||
338 | replay_mutex_unlock(); | ||
339 | @@ -XXX,XX +XXX,XX @@ static void *tcg_rr_cpu_thread_fn(void *arg) | ||
340 | |||
341 | while (cpu && cpu_work_list_empty(cpu) && !cpu->exit_request) { | ||
342 | |||
343 | - qatomic_mb_set(&tcg_current_rr_cpu, cpu); | ||
344 | + qatomic_mb_set(&rr_current_cpu, cpu); | ||
345 | current_cpu = cpu; | ||
346 | |||
347 | qemu_clock_enable(QEMU_CLOCK_VIRTUAL, | ||
348 | @@ -XXX,XX +XXX,XX @@ static void *tcg_rr_cpu_thread_fn(void *arg) | ||
349 | |||
350 | qemu_mutex_unlock_iothread(); | ||
351 | if (icount_enabled()) { | ||
352 | - prepare_icount_for_run(cpu); | ||
353 | + icount_prepare_for_run(cpu); | ||
354 | } | ||
355 | - r = tcg_cpu_exec(cpu); | ||
356 | + r = tcg_cpus_exec(cpu); | ||
357 | if (icount_enabled()) { | ||
358 | - process_icount_data(cpu); | ||
359 | + icount_process_data(cpu); | ||
360 | } | ||
361 | qemu_mutex_lock_iothread(); | ||
362 | |||
363 | @@ -XXX,XX +XXX,XX @@ static void *tcg_rr_cpu_thread_fn(void *arg) | ||
364 | } /* while (cpu && !cpu->exit_request).. */ | ||
365 | |||
366 | /* Does not need qatomic_mb_set because a spurious wakeup is okay. */ | ||
367 | - qatomic_set(&tcg_current_rr_cpu, NULL); | ||
368 | + qatomic_set(&rr_current_cpu, NULL); | ||
369 | |||
370 | if (cpu && cpu->exit_request) { | ||
371 | qatomic_mb_set(&cpu->exit_request, 0); | ||
372 | @@ -XXX,XX +XXX,XX @@ static void *tcg_rr_cpu_thread_fn(void *arg) | ||
373 | qemu_notify_event(); | ||
374 | } | ||
375 | |||
376 | - qemu_tcg_rr_wait_io_event(); | ||
377 | - deal_with_unplugged_cpus(); | ||
378 | + rr_wait_io_event(); | ||
379 | + rr_deal_with_unplugged_cpus(); | ||
380 | } | ||
381 | |||
382 | rcu_unregister_thread(); | ||
383 | @@ -XXX,XX +XXX,XX @@ void rr_start_vcpu_thread(CPUState *cpu) | ||
384 | /* share a single thread for all cpus with TCG */ | ||
385 | snprintf(thread_name, VCPU_THREAD_NAME_SIZE, "ALL CPUs/TCG"); | ||
386 | qemu_thread_create(cpu->thread, thread_name, | ||
387 | - tcg_rr_cpu_thread_fn, | ||
388 | + rr_cpu_thread_fn, | ||
389 | cpu, QEMU_THREAD_JOINABLE); | ||
390 | |||
391 | single_tcg_halt_cond = cpu->halt_cond; | ||
392 | @@ -XXX,XX +XXX,XX @@ void rr_start_vcpu_thread(CPUState *cpu) | ||
393 | |||
394 | const CpusAccel tcg_cpus_rr = { | ||
395 | .create_vcpu_thread = rr_start_vcpu_thread, | ||
396 | - .kick_vcpu_thread = qemu_cpu_kick_rr_cpus, | ||
397 | + .kick_vcpu_thread = rr_kick_vcpu_thread, | ||
398 | |||
399 | - .handle_interrupt = tcg_handle_interrupt, | ||
400 | + .handle_interrupt = tcg_cpus_handle_interrupt, | ||
401 | }; | ||
402 | diff --git a/accel/tcg/tcg-cpus.c b/accel/tcg/tcg-cpus.c | ||
403 | index XXXXXXX..XXXXXXX 100644 | ||
404 | --- a/accel/tcg/tcg-cpus.c | ||
405 | +++ b/accel/tcg/tcg-cpus.c | ||
406 | @@ -XXX,XX +XXX,XX @@ | ||
407 | |||
408 | /* common functionality among all TCG variants */ | ||
409 | |||
410 | -void qemu_tcg_destroy_vcpu(CPUState *cpu) | ||
411 | +void tcg_cpus_destroy(CPUState *cpu) | ||
412 | { | ||
413 | cpu_thread_signal_destroyed(cpu); | ||
414 | } | ||
415 | |||
416 | -int tcg_cpu_exec(CPUState *cpu) | ||
417 | +int tcg_cpus_exec(CPUState *cpu) | ||
418 | { | ||
419 | int ret; | ||
420 | #ifdef CONFIG_PROFILER | ||
421 | @@ -XXX,XX +XXX,XX @@ int tcg_cpu_exec(CPUState *cpu) | ||
422 | } | ||
423 | |||
424 | /* mask must never be zero, except for A20 change call */ | ||
425 | -void tcg_handle_interrupt(CPUState *cpu, int mask) | ||
426 | +void tcg_cpus_handle_interrupt(CPUState *cpu, int mask) | ||
427 | { | ||
428 | g_assert(qemu_mutex_iothread_locked()); | ||
429 | 207 | ||
430 | -- | 208 | -- |
431 | 2.25.1 | 209 | 2.25.1 |
432 | 210 | ||
433 | 211 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | From: Alex Bennée <alex.bennee@linaro.org> | ||
1 | 2 | ||
3 | Lets make sure all the flags we compare when looking up blocks are | ||
4 | together in the same place. | ||
5 | |||
6 | Signed-off-by: Alex Bennée <alex.bennee@linaro.org> | ||
7 | Message-Id: <20210224165811.11567-5-alex.bennee@linaro.org> | ||
8 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
9 | --- | ||
10 | include/exec/exec-all.h | 8 +++++--- | ||
11 | 1 file changed, 5 insertions(+), 3 deletions(-) | ||
12 | |||
13 | diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h | ||
14 | index XXXXXXX..XXXXXXX 100644 | ||
15 | --- a/include/exec/exec-all.h | ||
16 | +++ b/include/exec/exec-all.h | ||
17 | @@ -XXX,XX +XXX,XX @@ struct TranslationBlock { | ||
18 | target_ulong pc; /* simulated PC corresponding to this block (EIP + CS base) */ | ||
19 | target_ulong cs_base; /* CS base for this block */ | ||
20 | uint32_t flags; /* flags defining in which context the code was generated */ | ||
21 | - uint16_t size; /* size of target code for this block (1 <= | ||
22 | - size <= TARGET_PAGE_SIZE) */ | ||
23 | - uint16_t icount; | ||
24 | uint32_t cflags; /* compile flags */ | ||
25 | #define CF_COUNT_MASK 0x00007fff | ||
26 | #define CF_LAST_IO 0x00008000 /* Last insn may be an IO access. */ | ||
27 | @@ -XXX,XX +XXX,XX @@ struct TranslationBlock { | ||
28 | /* Per-vCPU dynamic tracing state used to generate this TB */ | ||
29 | uint32_t trace_vcpu_dstate; | ||
30 | |||
31 | + /* Above fields used for comparing */ | ||
32 | + uint16_t size; /* size of target code for this block (1 <= | ||
33 | + size <= TARGET_PAGE_SIZE) */ | ||
34 | + uint16_t icount; | ||
35 | + | ||
36 | struct tb_tc tc; | ||
37 | |||
38 | /* first and second physical page containing code. The lower bit | ||
39 | -- | ||
40 | 2.25.1 | ||
41 | |||
42 | diff view generated by jsdifflib |
1 | From: Claudio Fontana <cfontana@suse.de> | 1 | The primary motivation is to remove a dozen insns along |
---|---|---|---|
2 | 2 | the fast-path in tb_lookup. As a byproduct, this allows | |
3 | after the initial split into 3 tcg variants, we proceed to also | 3 | us to completely remove parallel_cpus. |
4 | split tcg_start_vcpu_thread. | 4 | |
5 | |||
6 | We actually split it in 2 this time, since the icount variant | ||
7 | just uses the round robin function. | ||
8 | |||
9 | Suggested-by: Richard Henderson <richard.henderson@linaro.org> | ||
10 | Signed-off-by: Claudio Fontana <cfontana@suse.de> | ||
11 | Message-Id: <20201015143217.29337-3-cfontana@suse.de> | ||
12 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | 5 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> |
13 | --- | 6 | --- |
14 | accel/tcg/tcg-cpus-mttcg.h | 21 -------------- | 7 | accel/tcg/tcg-accel-ops.h | 1 + |
15 | accel/tcg/tcg-cpus-rr.h | 3 +- | 8 | include/exec/exec-all.h | 7 +------ |
16 | accel/tcg/tcg-cpus.h | 1 - | 9 | include/hw/core/cpu.h | 2 ++ |
17 | accel/tcg/tcg-all.c | 5 ++++ | 10 | accel/tcg/cpu-exec.c | 3 --- |
18 | accel/tcg/tcg-cpus-icount.c | 2 +- | 11 | accel/tcg/tcg-accel-ops-mttcg.c | 3 +-- |
19 | accel/tcg/tcg-cpus-mttcg.c | 29 +++++++++++++++++-- | 12 | accel/tcg/tcg-accel-ops-rr.c | 2 +- |
20 | accel/tcg/tcg-cpus-rr.c | 39 +++++++++++++++++++++++-- | 13 | accel/tcg/tcg-accel-ops.c | 8 ++++++++ |
21 | accel/tcg/tcg-cpus.c | 58 ------------------------------------- | 14 | accel/tcg/translate-all.c | 4 ---- |
22 | 8 files changed, 71 insertions(+), 87 deletions(-) | 15 | linux-user/main.c | 1 + |
23 | delete mode 100644 accel/tcg/tcg-cpus-mttcg.h | 16 | linux-user/sh4/signal.c | 8 +++++--- |
24 | 17 | linux-user/syscall.c | 18 ++++++++++-------- | |
25 | diff --git a/accel/tcg/tcg-cpus-mttcg.h b/accel/tcg/tcg-cpus-mttcg.h | 18 | 11 files changed, 30 insertions(+), 27 deletions(-) |
26 | deleted file mode 100644 | 19 | |
27 | index XXXXXXX..XXXXXXX | 20 | diff --git a/accel/tcg/tcg-accel-ops.h b/accel/tcg/tcg-accel-ops.h |
28 | --- a/accel/tcg/tcg-cpus-mttcg.h | 21 | index XXXXXXX..XXXXXXX 100644 |
29 | +++ /dev/null | 22 | --- a/accel/tcg/tcg-accel-ops.h |
23 | +++ b/accel/tcg/tcg-accel-ops.h | ||
30 | @@ -XXX,XX +XXX,XX @@ | 24 | @@ -XXX,XX +XXX,XX @@ |
31 | -/* | 25 | void tcg_cpus_destroy(CPUState *cpu); |
32 | - * QEMU TCG Multi Threaded vCPUs implementation | 26 | int tcg_cpus_exec(CPUState *cpu); |
33 | - * | 27 | void tcg_handle_interrupt(CPUState *cpu, int mask); |
34 | - * Copyright 2020 SUSE LLC | 28 | +void tcg_cpu_init_cflags(CPUState *cpu, bool parallel); |
35 | - * | 29 | |
36 | - * This work is licensed under the terms of the GNU GPL, version 2 or later. | 30 | #endif /* TCG_CPUS_H */ |
37 | - * See the COPYING file in the top-level directory. | 31 | diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h |
38 | - */ | 32 | index XXXXXXX..XXXXXXX 100644 |
39 | - | 33 | --- a/include/exec/exec-all.h |
40 | -#ifndef TCG_CPUS_MTTCG_H | 34 | +++ b/include/exec/exec-all.h |
41 | -#define TCG_CPUS_MTTCG_H | 35 | @@ -XXX,XX +XXX,XX @@ struct TranslationBlock { |
42 | - | 36 | uintptr_t jmp_dest[2]; |
43 | -/* | 37 | }; |
44 | - * In the multi-threaded case each vCPU has its own thread. The TLS | 38 | |
45 | - * variable current_cpu can be used deep in the code to find the | 39 | -extern bool parallel_cpus; |
46 | - * current CPUState for a given thread. | 40 | - |
47 | - */ | 41 | /* Hide the qatomic_read to make code a little easier on the eyes */ |
48 | - | 42 | static inline uint32_t tb_cflags(const TranslationBlock *tb) |
49 | -void *tcg_cpu_thread_fn(void *arg); | 43 | { |
50 | - | 44 | @@ -XXX,XX +XXX,XX @@ static inline uint32_t tb_cflags(const TranslationBlock *tb) |
51 | -#endif /* TCG_CPUS_MTTCG_H */ | 45 | /* current cflags for hashing/comparison */ |
52 | diff --git a/accel/tcg/tcg-cpus-rr.h b/accel/tcg/tcg-cpus-rr.h | 46 | static inline uint32_t curr_cflags(CPUState *cpu) |
53 | index XXXXXXX..XXXXXXX 100644 | 47 | { |
54 | --- a/accel/tcg/tcg-cpus-rr.h | 48 | - uint32_t cflags = deposit32(0, CF_CLUSTER_SHIFT, 8, cpu->cluster_index); |
55 | +++ b/accel/tcg/tcg-cpus-rr.h | 49 | - cflags |= parallel_cpus ? CF_PARALLEL : 0; |
50 | - cflags |= icount_enabled() ? CF_USE_ICOUNT : 0; | ||
51 | - return cflags; | ||
52 | + return cpu->tcg_cflags; | ||
53 | } | ||
54 | |||
55 | /* TranslationBlock invalidate API */ | ||
56 | diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h | ||
57 | index XXXXXXX..XXXXXXX 100644 | ||
58 | --- a/include/hw/core/cpu.h | ||
59 | +++ b/include/hw/core/cpu.h | ||
60 | @@ -XXX,XX +XXX,XX @@ struct qemu_work_item; | ||
61 | * to a cluster this will be UNASSIGNED_CLUSTER_INDEX; otherwise it will | ||
62 | * be the same as the cluster-id property of the CPU object's TYPE_CPU_CLUSTER | ||
63 | * QOM parent. | ||
64 | + * @tcg_cflags: Pre-computed cflags for this cpu. | ||
65 | * @nr_cores: Number of cores within this CPU package. | ||
66 | * @nr_threads: Number of threads within this CPU. | ||
67 | * @running: #true if CPU is currently running (lockless). | ||
68 | @@ -XXX,XX +XXX,XX @@ struct CPUState { | ||
69 | /* TODO Move common fields from CPUArchState here. */ | ||
70 | int cpu_index; | ||
71 | int cluster_index; | ||
72 | + uint32_t tcg_cflags; | ||
73 | uint32_t halted; | ||
74 | uint32_t can_do_io; | ||
75 | int32_t exception_index; | ||
76 | diff --git a/accel/tcg/cpu-exec.c b/accel/tcg/cpu-exec.c | ||
77 | index XXXXXXX..XXXXXXX 100644 | ||
78 | --- a/accel/tcg/cpu-exec.c | ||
79 | +++ b/accel/tcg/cpu-exec.c | ||
80 | @@ -XXX,XX +XXX,XX @@ void cpu_exec_step_atomic(CPUState *cpu) | ||
81 | mmap_unlock(); | ||
82 | } | ||
83 | |||
84 | - /* Since we got here, we know that parallel_cpus must be true. */ | ||
85 | - parallel_cpus = false; | ||
86 | cpu_exec_enter(cpu); | ||
87 | /* execute the generated code */ | ||
88 | trace_exec_tb(tb, pc); | ||
89 | @@ -XXX,XX +XXX,XX @@ void cpu_exec_step_atomic(CPUState *cpu) | ||
90 | * the execution. | ||
91 | */ | ||
92 | g_assert(cpu_in_exclusive_context(cpu)); | ||
93 | - parallel_cpus = true; | ||
94 | cpu->running = false; | ||
95 | end_exclusive(); | ||
96 | } | ||
97 | diff --git a/accel/tcg/tcg-accel-ops-mttcg.c b/accel/tcg/tcg-accel-ops-mttcg.c | ||
98 | index XXXXXXX..XXXXXXX 100644 | ||
99 | --- a/accel/tcg/tcg-accel-ops-mttcg.c | ||
100 | +++ b/accel/tcg/tcg-accel-ops-mttcg.c | ||
101 | @@ -XXX,XX +XXX,XX @@ void mttcg_start_vcpu_thread(CPUState *cpu) | ||
102 | char thread_name[VCPU_THREAD_NAME_SIZE]; | ||
103 | |||
104 | g_assert(tcg_enabled()); | ||
105 | - | ||
106 | - parallel_cpus = (current_machine->smp.max_cpus > 1); | ||
107 | + tcg_cpu_init_cflags(cpu, current_machine->smp.max_cpus > 1); | ||
108 | |||
109 | cpu->thread = g_malloc0(sizeof(QemuThread)); | ||
110 | cpu->halt_cond = g_malloc0(sizeof(QemuCond)); | ||
111 | diff --git a/accel/tcg/tcg-accel-ops-rr.c b/accel/tcg/tcg-accel-ops-rr.c | ||
112 | index XXXXXXX..XXXXXXX 100644 | ||
113 | --- a/accel/tcg/tcg-accel-ops-rr.c | ||
114 | +++ b/accel/tcg/tcg-accel-ops-rr.c | ||
115 | @@ -XXX,XX +XXX,XX @@ void rr_start_vcpu_thread(CPUState *cpu) | ||
116 | static QemuThread *single_tcg_cpu_thread; | ||
117 | |||
118 | g_assert(tcg_enabled()); | ||
119 | - parallel_cpus = false; | ||
120 | + tcg_cpu_init_cflags(cpu, false); | ||
121 | |||
122 | if (!single_tcg_cpu_thread) { | ||
123 | cpu->thread = g_malloc0(sizeof(QemuThread)); | ||
124 | diff --git a/accel/tcg/tcg-accel-ops.c b/accel/tcg/tcg-accel-ops.c | ||
125 | index XXXXXXX..XXXXXXX 100644 | ||
126 | --- a/accel/tcg/tcg-accel-ops.c | ||
127 | +++ b/accel/tcg/tcg-accel-ops.c | ||
56 | @@ -XXX,XX +XXX,XX @@ | 128 | @@ -XXX,XX +XXX,XX @@ |
57 | /* Kick all RR vCPUs. */ | 129 | |
58 | void qemu_cpu_kick_rr_cpus(CPUState *unused); | 130 | /* common functionality among all TCG variants */ |
59 | 131 | ||
60 | -void *tcg_rr_cpu_thread_fn(void *arg); | 132 | +void tcg_cpu_init_cflags(CPUState *cpu, bool parallel) |
61 | +/* start the round robin vcpu thread */ | ||
62 | +void rr_start_vcpu_thread(CPUState *cpu); | ||
63 | |||
64 | #endif /* TCG_CPUS_RR_H */ | ||
65 | diff --git a/accel/tcg/tcg-cpus.h b/accel/tcg/tcg-cpus.h | ||
66 | index XXXXXXX..XXXXXXX 100644 | ||
67 | --- a/accel/tcg/tcg-cpus.h | ||
68 | +++ b/accel/tcg/tcg-cpus.h | ||
69 | @@ -XXX,XX +XXX,XX @@ extern const CpusAccel tcg_cpus_mttcg; | ||
70 | extern const CpusAccel tcg_cpus_icount; | ||
71 | extern const CpusAccel tcg_cpus_rr; | ||
72 | |||
73 | -void tcg_start_vcpu_thread(CPUState *cpu); | ||
74 | void qemu_tcg_destroy_vcpu(CPUState *cpu); | ||
75 | int tcg_cpu_exec(CPUState *cpu); | ||
76 | void tcg_handle_interrupt(CPUState *cpu, int mask); | ||
77 | diff --git a/accel/tcg/tcg-all.c b/accel/tcg/tcg-all.c | ||
78 | index XXXXXXX..XXXXXXX 100644 | ||
79 | --- a/accel/tcg/tcg-all.c | ||
80 | +++ b/accel/tcg/tcg-all.c | ||
81 | @@ -XXX,XX +XXX,XX @@ static int tcg_init(MachineState *ms) | ||
82 | tcg_exec_init(s->tb_size * 1024 * 1024); | ||
83 | mttcg_enabled = s->mttcg_enabled; | ||
84 | |||
85 | + /* | ||
86 | + * Initialize TCG regions | ||
87 | + */ | ||
88 | + tcg_region_init(); | ||
89 | + | ||
90 | if (mttcg_enabled) { | ||
91 | cpus_register_accel(&tcg_cpus_mttcg); | ||
92 | } else if (icount_enabled()) { | ||
93 | diff --git a/accel/tcg/tcg-cpus-icount.c b/accel/tcg/tcg-cpus-icount.c | ||
94 | index XXXXXXX..XXXXXXX 100644 | ||
95 | --- a/accel/tcg/tcg-cpus-icount.c | ||
96 | +++ b/accel/tcg/tcg-cpus-icount.c | ||
97 | @@ -XXX,XX +XXX,XX @@ static void icount_handle_interrupt(CPUState *cpu, int mask) | ||
98 | } | ||
99 | |||
100 | const CpusAccel tcg_cpus_icount = { | ||
101 | - .create_vcpu_thread = tcg_start_vcpu_thread, | ||
102 | + .create_vcpu_thread = rr_start_vcpu_thread, | ||
103 | .kick_vcpu_thread = qemu_cpu_kick_rr_cpus, | ||
104 | |||
105 | .handle_interrupt = icount_handle_interrupt, | ||
106 | diff --git a/accel/tcg/tcg-cpus-mttcg.c b/accel/tcg/tcg-cpus-mttcg.c | ||
107 | index XXXXXXX..XXXXXXX 100644 | ||
108 | --- a/accel/tcg/tcg-cpus-mttcg.c | ||
109 | +++ b/accel/tcg/tcg-cpus-mttcg.c | ||
110 | @@ -XXX,XX +XXX,XX @@ | ||
111 | #include "hw/boards.h" | ||
112 | |||
113 | #include "tcg-cpus.h" | ||
114 | -#include "tcg-cpus-mttcg.h" | ||
115 | |||
116 | /* | ||
117 | * In the multi-threaded case each vCPU has its own thread. The TLS | ||
118 | @@ -XXX,XX +XXX,XX @@ | ||
119 | * current CPUState for a given thread. | ||
120 | */ | ||
121 | |||
122 | -void *tcg_cpu_thread_fn(void *arg) | ||
123 | +static void *tcg_cpu_thread_fn(void *arg) | ||
124 | { | ||
125 | CPUState *cpu = arg; | ||
126 | |||
127 | @@ -XXX,XX +XXX,XX @@ static void mttcg_kick_vcpu_thread(CPUState *cpu) | ||
128 | cpu_exit(cpu); | ||
129 | } | ||
130 | |||
131 | +static void mttcg_start_vcpu_thread(CPUState *cpu) | ||
132 | +{ | 133 | +{ |
133 | + char thread_name[VCPU_THREAD_NAME_SIZE]; | 134 | + uint32_t cflags = cpu->cluster_index << CF_CLUSTER_SHIFT; |
134 | + | 135 | + cflags |= parallel ? CF_PARALLEL : 0; |
135 | + g_assert(tcg_enabled()); | 136 | + cflags |= icount_enabled() ? CF_USE_ICOUNT : 0; |
136 | + | 137 | + cpu->tcg_cflags = cflags; |
137 | + parallel_cpus = (current_machine->smp.max_cpus > 1); | ||
138 | + | ||
139 | + cpu->thread = g_malloc0(sizeof(QemuThread)); | ||
140 | + cpu->halt_cond = g_malloc0(sizeof(QemuCond)); | ||
141 | + qemu_cond_init(cpu->halt_cond); | ||
142 | + | ||
143 | + /* create a thread per vCPU with TCG (MTTCG) */ | ||
144 | + snprintf(thread_name, VCPU_THREAD_NAME_SIZE, "CPU %d/TCG", | ||
145 | + cpu->cpu_index); | ||
146 | + | ||
147 | + qemu_thread_create(cpu->thread, thread_name, tcg_cpu_thread_fn, | ||
148 | + cpu, QEMU_THREAD_JOINABLE); | ||
149 | + | ||
150 | +#ifdef _WIN32 | ||
151 | + cpu->hThread = qemu_thread_get_handle(cpu->thread); | ||
152 | +#endif | ||
153 | +} | 138 | +} |
154 | + | 139 | + |
155 | const CpusAccel tcg_cpus_mttcg = { | 140 | void tcg_cpus_destroy(CPUState *cpu) |
156 | - .create_vcpu_thread = tcg_start_vcpu_thread, | 141 | { |
157 | + .create_vcpu_thread = mttcg_start_vcpu_thread, | 142 | cpu_thread_signal_destroyed(cpu); |
158 | .kick_vcpu_thread = mttcg_kick_vcpu_thread, | 143 | diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c |
159 | 144 | index XXXXXXX..XXXXXXX 100644 | |
160 | .handle_interrupt = tcg_handle_interrupt, | 145 | --- a/accel/tcg/translate-all.c |
161 | diff --git a/accel/tcg/tcg-cpus-rr.c b/accel/tcg/tcg-cpus-rr.c | 146 | +++ b/accel/tcg/translate-all.c |
162 | index XXXXXXX..XXXXXXX 100644 | 147 | @@ -XXX,XX +XXX,XX @@ static void *l1_map[V_L1_MAX_SIZE]; |
163 | --- a/accel/tcg/tcg-cpus-rr.c | 148 | TCGContext tcg_init_ctx; |
164 | +++ b/accel/tcg/tcg-cpus-rr.c | 149 | __thread TCGContext *tcg_ctx; |
165 | @@ -XXX,XX +XXX,XX @@ static void deal_with_unplugged_cpus(void) | 150 | TBContext tb_ctx; |
166 | * elsewhere. | 151 | -bool parallel_cpus; |
167 | */ | 152 | |
168 | 153 | static void page_table_config_init(void) | |
169 | -void *tcg_rr_cpu_thread_fn(void *arg) | 154 | { |
170 | +static void *tcg_rr_cpu_thread_fn(void *arg) | 155 | @@ -XXX,XX +XXX,XX @@ TranslationBlock *tb_gen_code(CPUState *cpu, |
171 | { | 156 | cflags = (cflags & ~CF_COUNT_MASK) | 1; |
172 | CPUState *cpu = arg; | 157 | } |
173 | 158 | ||
174 | @@ -XXX,XX +XXX,XX @@ void *tcg_rr_cpu_thread_fn(void *arg) | 159 | - cflags &= ~CF_CLUSTER_MASK; |
175 | return NULL; | 160 | - cflags |= cpu->cluster_index << CF_CLUSTER_SHIFT; |
161 | - | ||
162 | max_insns = cflags & CF_COUNT_MASK; | ||
163 | if (max_insns == 0) { | ||
164 | max_insns = CF_COUNT_MASK; | ||
165 | diff --git a/linux-user/main.c b/linux-user/main.c | ||
166 | index XXXXXXX..XXXXXXX 100644 | ||
167 | --- a/linux-user/main.c | ||
168 | +++ b/linux-user/main.c | ||
169 | @@ -XXX,XX +XXX,XX @@ CPUArchState *cpu_copy(CPUArchState *env) | ||
170 | /* Reset non arch specific state */ | ||
171 | cpu_reset(new_cpu); | ||
172 | |||
173 | + new_cpu->tcg_cflags = cpu->tcg_cflags; | ||
174 | memcpy(new_env, env, sizeof(CPUArchState)); | ||
175 | |||
176 | /* Clone all break/watchpoints. | ||
177 | diff --git a/linux-user/sh4/signal.c b/linux-user/sh4/signal.c | ||
178 | index XXXXXXX..XXXXXXX 100644 | ||
179 | --- a/linux-user/sh4/signal.c | ||
180 | +++ b/linux-user/sh4/signal.c | ||
181 | @@ -XXX,XX +XXX,XX @@ static abi_ulong get_sigframe(struct target_sigaction *ka, | ||
182 | return (sp - frame_size) & -8ul; | ||
176 | } | 183 | } |
177 | 184 | ||
178 | +void rr_start_vcpu_thread(CPUState *cpu) | 185 | -/* Notice when we're in the middle of a gUSA region and reset. |
179 | +{ | 186 | - Note that this will only occur for !parallel_cpus, as we will |
180 | + char thread_name[VCPU_THREAD_NAME_SIZE]; | 187 | - translate such sequences differently in a parallel context. */ |
181 | + static QemuCond *single_tcg_halt_cond; | 188 | +/* |
182 | + static QemuThread *single_tcg_cpu_thread; | 189 | + * Notice when we're in the middle of a gUSA region and reset. |
190 | + * Note that this will only occur when #CF_PARALLEL is unset, as we | ||
191 | + * will translate such sequences differently in a parallel context. | ||
192 | + */ | ||
193 | static void unwind_gusa(CPUSH4State *regs) | ||
194 | { | ||
195 | /* If the stack pointer is sufficiently negative, and we haven't | ||
196 | diff --git a/linux-user/syscall.c b/linux-user/syscall.c | ||
197 | index XXXXXXX..XXXXXXX 100644 | ||
198 | --- a/linux-user/syscall.c | ||
199 | +++ b/linux-user/syscall.c | ||
200 | @@ -XXX,XX +XXX,XX @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp, | ||
201 | /* Grab a mutex so that thread setup appears atomic. */ | ||
202 | pthread_mutex_lock(&clone_lock); | ||
203 | |||
204 | + /* | ||
205 | + * If this is our first additional thread, we need to ensure we | ||
206 | + * generate code for parallel execution and flush old translations. | ||
207 | + * Do this now so that the copy gets CF_PARALLEL too. | ||
208 | + */ | ||
209 | + if (!(cpu->tcg_cflags & CF_PARALLEL)) { | ||
210 | + cpu->tcg_cflags |= CF_PARALLEL; | ||
211 | + tb_flush(cpu); | ||
212 | + } | ||
183 | + | 213 | + |
184 | + g_assert(tcg_enabled()); | 214 | /* we create a new CPU instance. */ |
185 | + parallel_cpus = false; | 215 | new_env = cpu_copy(env); |
186 | + | 216 | /* Init regs that differ from the parent. */ |
187 | + if (!single_tcg_cpu_thread) { | 217 | @@ -XXX,XX +XXX,XX @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp, |
188 | + cpu->thread = g_malloc0(sizeof(QemuThread)); | 218 | sigprocmask(SIG_BLOCK, &sigmask, &info.sigmask); |
189 | + cpu->halt_cond = g_malloc0(sizeof(QemuCond)); | 219 | cpu->random_seed = qemu_guest_random_seed_thread_part1(); |
190 | + qemu_cond_init(cpu->halt_cond); | 220 | |
191 | + | 221 | - /* If this is our first additional thread, we need to ensure we |
192 | + /* share a single thread for all cpus with TCG */ | 222 | - * generate code for parallel execution and flush old translations. |
193 | + snprintf(thread_name, VCPU_THREAD_NAME_SIZE, "ALL CPUs/TCG"); | 223 | - */ |
194 | + qemu_thread_create(cpu->thread, thread_name, | 224 | - if (!parallel_cpus) { |
195 | + tcg_rr_cpu_thread_fn, | 225 | - parallel_cpus = true; |
196 | + cpu, QEMU_THREAD_JOINABLE); | 226 | - tb_flush(cpu); |
197 | + | ||
198 | + single_tcg_halt_cond = cpu->halt_cond; | ||
199 | + single_tcg_cpu_thread = cpu->thread; | ||
200 | +#ifdef _WIN32 | ||
201 | + cpu->hThread = qemu_thread_get_handle(cpu->thread); | ||
202 | +#endif | ||
203 | + } else { | ||
204 | + /* we share the thread */ | ||
205 | + cpu->thread = single_tcg_cpu_thread; | ||
206 | + cpu->halt_cond = single_tcg_halt_cond; | ||
207 | + cpu->thread_id = first_cpu->thread_id; | ||
208 | + cpu->can_do_io = 1; | ||
209 | + cpu->created = true; | ||
210 | + } | ||
211 | +} | ||
212 | + | ||
213 | const CpusAccel tcg_cpus_rr = { | ||
214 | - .create_vcpu_thread = tcg_start_vcpu_thread, | ||
215 | + .create_vcpu_thread = rr_start_vcpu_thread, | ||
216 | .kick_vcpu_thread = qemu_cpu_kick_rr_cpus, | ||
217 | |||
218 | .handle_interrupt = tcg_handle_interrupt, | ||
219 | diff --git a/accel/tcg/tcg-cpus.c b/accel/tcg/tcg-cpus.c | ||
220 | index XXXXXXX..XXXXXXX 100644 | ||
221 | --- a/accel/tcg/tcg-cpus.c | ||
222 | +++ b/accel/tcg/tcg-cpus.c | ||
223 | @@ -XXX,XX +XXX,XX @@ | ||
224 | #include "hw/boards.h" | ||
225 | |||
226 | #include "tcg-cpus.h" | ||
227 | -#include "tcg-cpus-mttcg.h" | ||
228 | -#include "tcg-cpus-rr.h" | ||
229 | |||
230 | /* common functionality among all TCG variants */ | ||
231 | |||
232 | -void tcg_start_vcpu_thread(CPUState *cpu) | ||
233 | -{ | ||
234 | - char thread_name[VCPU_THREAD_NAME_SIZE]; | ||
235 | - static QemuCond *single_tcg_halt_cond; | ||
236 | - static QemuThread *single_tcg_cpu_thread; | ||
237 | - static int tcg_region_inited; | ||
238 | - | ||
239 | - assert(tcg_enabled()); | ||
240 | - /* | ||
241 | - * Initialize TCG regions--once. Now is a good time, because: | ||
242 | - * (1) TCG's init context, prologue and target globals have been set up. | ||
243 | - * (2) qemu_tcg_mttcg_enabled() works now (TCG init code runs before the | ||
244 | - * -accel flag is processed, so the check doesn't work then). | ||
245 | - */ | ||
246 | - if (!tcg_region_inited) { | ||
247 | - tcg_region_inited = 1; | ||
248 | - tcg_region_init(); | ||
249 | - parallel_cpus = qemu_tcg_mttcg_enabled() && current_machine->smp.max_cpus > 1; | ||
250 | - } | ||
251 | - | ||
252 | - if (qemu_tcg_mttcg_enabled() || !single_tcg_cpu_thread) { | ||
253 | - cpu->thread = g_malloc0(sizeof(QemuThread)); | ||
254 | - cpu->halt_cond = g_malloc0(sizeof(QemuCond)); | ||
255 | - qemu_cond_init(cpu->halt_cond); | ||
256 | - | ||
257 | - if (qemu_tcg_mttcg_enabled()) { | ||
258 | - /* create a thread per vCPU with TCG (MTTCG) */ | ||
259 | - snprintf(thread_name, VCPU_THREAD_NAME_SIZE, "CPU %d/TCG", | ||
260 | - cpu->cpu_index); | ||
261 | - | ||
262 | - qemu_thread_create(cpu->thread, thread_name, tcg_cpu_thread_fn, | ||
263 | - cpu, QEMU_THREAD_JOINABLE); | ||
264 | - | ||
265 | - } else { | ||
266 | - /* share a single thread for all cpus with TCG */ | ||
267 | - snprintf(thread_name, VCPU_THREAD_NAME_SIZE, "ALL CPUs/TCG"); | ||
268 | - qemu_thread_create(cpu->thread, thread_name, | ||
269 | - tcg_rr_cpu_thread_fn, | ||
270 | - cpu, QEMU_THREAD_JOINABLE); | ||
271 | - | ||
272 | - single_tcg_halt_cond = cpu->halt_cond; | ||
273 | - single_tcg_cpu_thread = cpu->thread; | ||
274 | - } | 227 | - } |
275 | -#ifdef _WIN32 | 228 | - |
276 | - cpu->hThread = qemu_thread_get_handle(cpu->thread); | 229 | ret = pthread_create(&info.thread, &attr, clone_func, &info); |
277 | -#endif | 230 | /* TODO: Free new CPU state if thread creation failed. */ |
278 | - } else { | 231 | |
279 | - /* For non-MTTCG cases we share the thread */ | ||
280 | - cpu->thread = single_tcg_cpu_thread; | ||
281 | - cpu->halt_cond = single_tcg_halt_cond; | ||
282 | - cpu->thread_id = first_cpu->thread_id; | ||
283 | - cpu->can_do_io = 1; | ||
284 | - cpu->created = true; | ||
285 | - } | ||
286 | -} | ||
287 | - | ||
288 | void qemu_tcg_destroy_vcpu(CPUState *cpu) | ||
289 | { | ||
290 | cpu_thread_signal_destroyed(cpu); | ||
291 | -- | 232 | -- |
292 | 2.25.1 | 233 | 2.25.1 |
293 | 234 | ||
294 | 235 | diff view generated by jsdifflib |