[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH V13 6/9] target/mips: Add loongson-ext lsdc2 group of instruc
From: |
Huacai Chen |
Subject: |
Re: [PATCH V13 6/9] target/mips: Add loongson-ext lsdc2 group of instructions |
Date: |
Mon, 12 Oct 2020 18:33:21 +0800 |
Hi, Philippe,
On Sun, Oct 11, 2020 at 7:13 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> On 10/11/20 5:02 AM, Huacai Chen wrote:
> > Hi, Philippe,
> >
> > On Sat, Oct 10, 2020 at 9:07 PM Philippe Mathieu-Daudé <f4bug@amsat.org>
> > wrote:
> >>
> >> On 10/7/20 10:39 AM, Huacai Chen wrote:
> >>> From: Jiaxun Yang <jiaxun.yang@flygoat.com>
> >>>
> >>> LDC2/SDC2 opcodes have been rewritten as "load & store with offset"
> >>> group of instructions by loongson-ext ASE.
> >>>
> >>> This patch add implementation of these instructions:
> >>> gslbx: load 1 bytes to GPR
> >>> gslhx: load 2 bytes to GPR
> >>> gslwx: load 4 bytes to GPR
> >>> gsldx: load 8 bytes to GPR
> >>> gslwxc1: load 4 bytes to FPR
> >>> gsldxc1: load 8 bytes to FPR
> >>> gssbx: store 1 bytes from GPR
> >>> gsshx: store 2 bytes from GPR
> >>> gsswx: store 4 bytes from GPR
> >>> gssdx: store 8 bytes from GPR
> >>> gsswxc1: store 4 bytes from FPR
> >>> gssdxc1: store 8 bytes from FPR
> >>>
> >>> Details of Loongson-EXT is here:
> >>> https://github.com/FlyGoat/loongson-insn/blob/master/loongson-ext.md
> >>>
> >>> Signed-off-by: Huacai Chen <chenhc@lemote.com>
> >>> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> >>
> >> If this patch is from Jiaxun, Huacai's S-o-b should come *after*.
> > OK, I will do that.
> >
> >>
> >>> ---
> >>> target/mips/translate.c | 179
> >>> ++++++++++++++++++++++++++++++++++++++++++++++++
> >>> 1 file changed, 179 insertions(+)
> >>>
> >>> diff --git a/target/mips/translate.c b/target/mips/translate.c
> >>> index 916b57f..4d42cfc 100644
> >>> --- a/target/mips/translate.c
> >>> +++ b/target/mips/translate.c
> >>> @@ -484,6 +484,24 @@ enum {
> >>> OPC_GSSDRC1 = 0x7 | OPC_GSSHFS,
> >>> };
> >>>
> >>> +/* Loongson EXT LDC2/SDC2 opcodes */
> >>> +#define MASK_LOONGSON_LSDC2(op) (MASK_OP_MAJOR(op) | (op &
> >>> 0x7))
> >>> +
> >>> +enum {
> >>> + OPC_GSLBX = 0x0 | OPC_LDC2,
> >>> + OPC_GSLHX = 0x1 | OPC_LDC2,
> >>> + OPC_GSLWX = 0x2 | OPC_LDC2,
> >>> + OPC_GSLDX = 0x3 | OPC_LDC2,
> >>> + OPC_GSLWXC1 = 0x6 | OPC_LDC2,
> >>> + OPC_GSLDXC1 = 0x7 | OPC_LDC2,
> >>> + OPC_GSSBX = 0x0 | OPC_SDC2,
> >>> + OPC_GSSHX = 0x1 | OPC_SDC2,
> >>> + OPC_GSSWX = 0x2 | OPC_SDC2,
> >>> + OPC_GSSDX = 0x3 | OPC_SDC2,
> >>> + OPC_GSSWXC1 = 0x6 | OPC_SDC2,
> >>> + OPC_GSSDXC1 = 0x7 | OPC_SDC2,
> >>> +};
> >>> +
> >>> /* BSHFL opcodes */
> >>> #define MASK_BSHFL(op) (MASK_SPECIAL3(op) | (op & (0x1F
> >>> << 6)))
> >>>
> >>> @@ -6172,6 +6190,165 @@ static void gen_loongson_lswc2(DisasContext *ctx,
> >>> int rt,
> >>> tcg_temp_free(t0);
> >>> }
> >>>
> >>> +/* Loongson EXT LDC2/SDC2 */
> >>> +static void gen_loongson_lsdc2(DisasContext *ctx, int rt,
> >>> + int rs, int rd)
> >>
> >> Alignment off (various occurences in this series).
> > OK, thanks.
> >
> >>
> >>> +{
> >>> + int offset = (int8_t)(ctx->opcode >> 3);
> >>
> >> Please use sextract32() which is easier to read:
> >>
> >> int32_t offset = sextract32(ctx->opcode, 3, 8);
> > OK, thanks.
> >
> >>
> >>> + uint32_t opc = MASK_LOONGSON_LSDC2(ctx->opcode);
> >>> + TCGv t0, t1;
> >>> + TCGv_i32 fp0;
> >>> +
> >>> + /* Pre-conditions */
> >>> + switch (opc) {
> >>> + case OPC_GSLBX:
> >>> + case OPC_GSLHX:
> >>> + case OPC_GSLWX:
> >>> + case OPC_GSLDX:
> >>> + /* prefetch, implement as NOP */
> >>> + if (rt == 0) {
> >>> + return;
> >>> + }
> >>> + break;
> >>> + case OPC_GSSBX:
> >>> + case OPC_GSSHX:
> >>> + case OPC_GSSWX:
> >>> + case OPC_GSSDX:
> >>> + break;
> >>> + case OPC_GSLWXC1:
> >>> +#if defined(TARGET_MIPS64)
> >>> + case OPC_GSLDXC1:
> >>> +#endif
> >>> + check_cp1_enabled(ctx);
> >>> + /* prefetch, implement as NOP */
> >>> + if (rt == 0) {
> >>> + return;
> >>> + }
> >>> + break;
> >>> + case OPC_GSSWXC1:
> >>> +#if defined(TARGET_MIPS64)
> >>> + case OPC_GSSDXC1:
> >>> +#endif
> >>> + check_cp1_enabled(ctx);
> >>> + break;
> >>> + default:
> >>> + MIPS_INVAL("loongson_lsdc2");
> >>> + generate_exception_end(ctx, EXCP_RI);
> >>> + return;
> >>> + break;
> >>> + }
> >>> +
> >>> + t0 = tcg_temp_new();
> >>> +
> >>> + gen_base_offset_addr(ctx, t0, rs, offset);
> >>> + gen_op_addr_add(ctx, t0, cpu_gpr[rd], t0);
> >>> +
> >>> + switch (opc) {
> >>> + case OPC_GSLBX:
> >>> + tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_SB);
> >>> + gen_store_gpr(t0, rt);
> >>> + break;
> >>> + case OPC_GSLHX:
> >>> + tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_TESW |
> >>> + ctx->default_tcg_memop_mask);
> >>
> >> Do Loongson EXT plan to support unaligned accesses?
> > Not support in hardware, and Linux kernel emulate the unaligned cases.
>
> OK, that was my understanding. So we don't need to use
> default_tcg_memop_mask, we can directly use MO_ALIGN in
> place instead.
I read the code again, and found that MIPSR6 uses MO_UNALN while
others use MO_ALIGN. And I also realized that Loongson-3A4000 supports
unaligned access in hardware (the same as R6). So, I think I should
keep default_tcg_memop_mask here. And if possible, I want to set
MO_UNALN for all Loongson-3 processors.
Huacai
>
> Regards,
>
> Phil.
- Re: [PATCH V13 1/9] linux-headers: Update MIPS KVM type defintition, (continued)
- [PATCH V13 2/9] meson.build: Re-enable KVM support for MIPS, Huacai Chen, 2020/10/07
- [PATCH V13 3/9] target/mips: Fix PageMask with variable page size, Huacai Chen, 2020/10/07
- [PATCH V13 4/9] target/mips: Add loongson-ext lswc2 group of instructions (Part 1), Huacai Chen, 2020/10/07
- [PATCH V13 5/9] target/mips: Add loongson-ext lswc2 group of instructions (Part 2), Huacai Chen, 2020/10/07
- [PATCH V13 6/9] target/mips: Add loongson-ext lsdc2 group of instructions, Huacai Chen, 2020/10/07
[PATCH V13 7/9] hw/mips: Implement fw_cfg_arch_key_name(), Huacai Chen, 2020/10/07
[PATCH V13 8/9] hw/mips: Add Loongson-3 machine support, Huacai Chen, 2020/10/07
- Re: [PATCH V13 8/9] hw/mips: Add Loongson-3 machine support, Philippe Mathieu-Daudé, 2020/10/10
- Re: [PATCH V13 8/9] hw/mips: Add Loongson-3 machine support, Huacai Chen, 2020/10/10
- Re: [PATCH V13 8/9] hw/mips: Add Loongson-3 machine support, Philippe Mathieu-Daudé, 2020/10/12
- Re: [PATCH V13 8/9] hw/mips: Add Loongson-3 machine support, Huacai Chen, 2020/10/13
- Re: [PATCH V13 8/9] hw/mips: Add Loongson-3 machine support, Philippe Mathieu-Daudé, 2020/10/13
- Re: [PATCH V13 8/9] hw/mips: Add Loongson-3 machine support, Huacai Chen, 2020/10/13
[PATCH V13 9/9] docs/system: Update MIPS machine documentation, Huacai Chen, 2020/10/07