[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v3 07/56] target/m68k: rename cpu_halted to cpu_ha
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [RFC v3 07/56] target/m68k: rename cpu_halted to cpu_halt |
Date: |
Sun, 21 Oct 2018 14:38:38 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 |
On 10/21/18 1:53 PM, Richard Henderson wrote:
> On 10/19/18 2:05 AM, Emilio G. Cota wrote:
>> To avoid a name clash with the soon-to-be-defined cpu_halted() helper.
>>
>> Cc: Laurent Vivier <address@hidden>
>> Signed-off-by: Emilio G. Cota <address@hidden>
>> ---
>> target/m68k/translate.c | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> Although for this usage it's probably better to avoid the
> tcg_global_mem_new_i32 and just use tcg_gen_st_i32.
And, as I read further, you need to convert this use to a helper call.
Otherwise you've still got an unlocked direct modification to cpu->halted
from within the TCG generated code.
There are several other targets that do the same thing: alpha, cris, hppa,
mips, microblaze, ppc. And typically they will do exactly the same thing: set
the flag and then raise the halt exception.
r~
- [Qemu-devel] [RFC v3 01/56] cpu: convert queued work to a QSIMPLEQ, (continued)
- [Qemu-devel] [RFC v3 01/56] cpu: convert queued work to a QSIMPLEQ, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 08/56] cpu: define cpu_halted helpers, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 09/56] arm: convert to cpu_halted, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 16/56] riscv: convert to cpu_halted, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 07/56] target/m68k: rename cpu_halted to cpu_halt, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 20/56] gdbstub: convert to cpu_halted, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 23/56] cpu: define cpu_interrupt_request helpers, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 19/56] xtensa: convert to cpu_halted, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 26/56] i386: use cpu_reset_interrupt, Emilio G. Cota, 2018/10/18
- [Qemu-devel] [RFC v3 22/56] cpu-exec: convert to cpu_halted, Emilio G. Cota, 2018/10/18