[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V9 2/7] nios2: Add architecture emulation suppor
From: |
Marek Vasut |
Subject: |
Re: [Qemu-devel] [PATCH V9 2/7] nios2: Add architecture emulation support |
Date: |
Wed, 18 Jan 2017 22:53:53 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.4.0 |
On 01/18/2017 08:33 PM, Richard Henderson wrote:
> On 01/16/2017 04:44 PM, Marek Vasut wrote:
>> + TCGv_i32 tmp = tcg_const_i32(instr.imm5 + 32);
>> + gen_helper_mmu_write(dc->cpu_env, tmp, load_gpr(dc, instr.a));
>> + tcg_temp_free_i32(tmp);
>> +#endif
>> + break;
>> + }
>> +
>> + default:
>> + tcg_gen_mov_tl(dc->cpu_R[instr.imm5 + 32], load_gpr(dc,
>> instr.a));
>> + break;
>> + }
>> +
>> + /* If interrupts were enabled using WRCTL, trigger them. */
>> +#if !defined(CONFIG_USER_ONLY)
>> + if ((instr.imm5 + 32) == CR_STATUS) {
>
> Did you want CR_BASE here instead of hard-coding 32?
Fixed
>> +static const char *regnames[] = {
>
> const char * const
>
>> + "reserved", "reserved", "reserved", "reserved",
>> + "reserved", "reserved", "reserved", "reserved",
>> + "reserved", "reserved", "reserved", "reserved",
>> + "reserved", "reserved", "reserved", "reserved",
>
> You might as well number them, e.g. resXX, to make it obvious in the
> dumps what's happening.
Hm, OK
>> + if (num_insns + 1 == max_insns && (tb->cflags & CF_LAST_IO)) {
>> + gen_io_start();
>> + }
>
> num_insns has already be incremented; don't + 1 here.
OK
>> + && qemu_log_in_addr_range(tb->pc)) {
>> + qemu_log("IN: %s\n", lookup_symbol(tb->pc));
>> + log_target_disas(cs, tb->pc, dc->pc - tb->pc, 0);
>> + qemu_log("\n");
>> + }
>
> qemu_log_lock / qemu_log_unlock around the dump.
OK
--
Best regards,
Marek Vasut
- [Qemu-devel] [PATCH 0/7] Nios2 architecture support, Marek Vasut, 2017/01/16
- [Qemu-devel] [PATCH V5 7/7] nios2: Add support for Nios-II R1, Marek Vasut, 2017/01/16
- [Qemu-devel] [PATCH V5 4/7] nios2: Add IIC interrupt controller emulation, Marek Vasut, 2017/01/16
- [Qemu-devel] [PATCH V5 5/7] nios2: Add periodic timer emulation, Marek Vasut, 2017/01/16
- [Qemu-devel] [PATCH V6 6/7] nios2: Add Altera 10M50 GHRD emulation, Marek Vasut, 2017/01/16
- [Qemu-devel] [PATCH V5 3/7] nios2: Add usermode binaries emulation, Marek Vasut, 2017/01/16
- [Qemu-devel] [PATCH V9 2/7] nios2: Add architecture emulation support, Marek Vasut, 2017/01/16
- [Qemu-devel] [PATCH V4 1/7] nios2: Add disas entries, Marek Vasut, 2017/01/16
- Re: [Qemu-devel] [PATCH 0/7] Nios2 architecture support, Alexander Graf, 2017/01/17
- [Qemu-devel] [PATCH 0/7] Nios2 architecture support, Marek Vasut, 2017/01/18
- [Qemu-devel] [PATCH V5 4/7] nios2: Add IIC interrupt controller emulation, Marek Vasut, 2017/01/18
- [Qemu-devel] [PATCH V6 7/7] nios2: Add support for Nios-II R1, Marek Vasut, 2017/01/18
- [Qemu-devel] [PATCH V5 5/7] nios2: Add periodic timer emulation, Marek Vasut, 2017/01/18
- [Qemu-devel] [PATCH V5 3/7] nios2: Add usermode binaries emulation, Marek Vasut, 2017/01/18
- [Qemu-devel] [PATCH V6 6/7] nios2: Add Altera 10M50 GHRD emulation, Marek Vasut, 2017/01/18
- [Qemu-devel] [PATCH V10 2/7] nios2: Add architecture emulation support, Marek Vasut, 2017/01/18
- [Qemu-devel] [PATCH V4 1/7] nios2: Add disas entries, Marek Vasut, 2017/01/18