qemu-e2k/target/hexagon
Peter Maydell 1d6f147f04 The Hexagon target was silently failing the SIGSEGV test because
the signal handler was not called.
 
 Patch 1/2 fixes the Hexagon target
 Patch 2/2 drops include qemu.h from target/hexagon/op_helper.c
 
 **** Changes in v2 ****
 Drop changes to linux-test.c due to intermittent failures on riscv
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1
 
 iQEcBAABAgAGBQJg/doaAAoJEHsCRPsS3kQi/gIH+gJ6GBmIb7NNDt+tRYjsnOpZ
 QgmkM/cvOBhqo+dUkxWIDXA7i7ZzytBHHG5GoplVkZjm/S+e5aEsuEyqwL6KbcK7
 kB6NvnHA3n9npf5MGcUduHlvPPzDsO7Z4SLrfwkIliiWL/AJ4FzKqEGoviWv2YnN
 k+29YDSv11B1jgXriADBJVnWtCf2CGPsF7BiKMcguZ6Bj+q+fH1cPpe2EWN8R8n2
 D+La/M5qWEC2FcWPCkrCs61Pi/cV+L4M0IA6JAEm8K+MtoDWmsCNWaVsakiNWWg+
 FRiHg45z3cCBvQ+SLQQ4SvsaQriI3M/yIKD6ABNgAfurIiTj4YbHAbeTmfEFYOs=
 =YdBn
 -----END PGP SIGNATURE-----

Merge remote-tracking branch 'remotes/quic/tags/pull-hex-20210725' into staging

The Hexagon target was silently failing the SIGSEGV test because
the signal handler was not called.

Patch 1/2 fixes the Hexagon target
Patch 2/2 drops include qemu.h from target/hexagon/op_helper.c

**** Changes in v2 ****
Drop changes to linux-test.c due to intermittent failures on riscv

# gpg: Signature made Sun 25 Jul 2021 22:39:38 BST
# gpg:                using RSA key 7B0244FB12DE4422
# gpg: Good signature from "Taylor Simpson (Rock on) <tsimpson@quicinc.com>" [undefined]
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: 3635 C788 CE62 B91F D4C5  9AB4 7B02 44FB 12DE 4422

* remotes/quic/tags/pull-hex-20210725:
  target/hexagon: Drop include of qemu.h
  Hexagon (target/hexagon) remove put_user_*/get_user_*

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
2021-07-26 13:36:51 +01:00
..
imported Hexagon (target/hexagon) fix l2fetch instructions 2021-06-29 11:32:50 -05:00
arch.c Hexagon (target/hexagon) CABAC decode bin 2021-05-01 16:06:11 -07:00
arch.h Hexagon (target/hexagon) CABAC decode bin 2021-05-01 16:06:11 -07:00
attribs_def.h.inc Hexagon (target/hexagon) instruction attributes 2021-02-18 07:48:22 -08:00
attribs.h Hexagon (target/hexagon) instruction attributes 2021-02-18 07:48:22 -08:00
cpu_bits.h Hexagon (target/hexagon) change variables from int to bool when appropriate 2021-05-01 08:31:43 -07:00
cpu-param.h
cpu.c hw/core: Constify TCGCPUOps 2021-05-26 15:33:59 -07:00
cpu.h Hexagon (target/hexagon) use env_archcpu and env_cpu 2021-05-01 08:31:43 -07:00
decode.c Trivial patches pull request 20210503 2021-05-05 13:52:00 +01:00
decode.h Hexagon (target/hexagon) instruction/packet decode 2021-02-18 07:48:22 -08:00
dectree.py Hexagon (target/hexagon) generater phase 4 - decode tree 2021-02-18 07:48:22 -08:00
fma_emu.c Hexagon (target/hexagon) use softfloat for float-to-int conversions 2021-05-01 08:31:43 -07:00
fma_emu.h Hexagon (target/hexagon/fma_emu.[ch]) utility functions 2021-02-18 07:48:22 -08:00
gdbstub.c
gen_dectree_import.c Hexagon (target/hexagon) generator phase 3 - C preprocessor for decode tree 2021-02-18 07:48:22 -08:00
gen_helper_funcs.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
gen_helper_protos.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
gen_op_attribs.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
gen_op_regs.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
gen_opcodes_def.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
gen_printinsn.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
gen_semantics.c Hexagon (target/hexagon) generator phase 1 - C preprocessor for semantics 2021-02-18 07:48:22 -08:00
gen_shortcode.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
gen_tcg_func_table.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
gen_tcg_funcs.py Hexagon (target/hexagon) decide if pred has been written at TCG gen time 2021-05-01 08:31:43 -07:00
gen_tcg.h Hexagon (target/hexagon) cleanup gen_store_conditional[48] functions 2021-06-29 11:32:50 -05:00
genptr.c Hexagon (target/hexagon) remove unused TCG variables 2021-06-29 11:32:50 -05:00
genptr.h Hexagon (target/hexagon) TCG generation 2021-02-18 07:48:22 -08:00
helper.h Hexagon (target/hexagon) bit reverse (brev) addressing 2021-05-01 16:03:10 -07:00
hex_arch_types.h
hex_common.py Hexagon (target/hexagon) generator phase 2 - generate header files 2021-02-18 07:48:22 -08:00
hex_regs.h
iclass.c target/hexagon: remove unnecessary checks in find_iclass_slots 2021-05-01 08:31:42 -07:00
iclass.h Hexagon (target/hexagon) instruction classes 2021-02-18 07:48:22 -08:00
insn.h Hexagon (target/hexagon) change variables from int to bool when appropriate 2021-05-01 08:31:43 -07:00
internal.h Hexagon (target/hexagon) compile all debug code 2021-05-01 08:31:43 -07:00
macros.h Hexagon (target/hexagon) cleanup gen_store_conditional[48] functions 2021-06-29 11:32:50 -05:00
meson.build Hexagon (target/hexagon) use softfloat for float-to-int conversions 2021-05-01 08:31:43 -07:00
op_helper.c target/hexagon: Drop include of qemu.h 2021-07-21 15:54:02 -05:00
opcodes.c target/hexagon/opcodes: Add missing varargs cleanup 2021-03-06 17:35:43 -08:00
opcodes.h Hexagon (target/hexagon) opcode data structures 2021-02-18 07:48:22 -08:00
printinsn.c Hexagon (target/hexagon) instruction printing 2021-02-18 07:48:22 -08:00
printinsn.h Hexagon (target/hexagon) instruction printing 2021-02-18 07:48:22 -08:00
README
reg_fields_def.h.inc
reg_fields.c Hexagon (target/hexagon) cleanup reg_field_info definition 2021-05-01 08:31:43 -07:00
reg_fields.h Hexagon (target/hexagon) cleanup reg_field_info definition 2021-05-01 08:31:43 -07:00
translate.c accel/tcg: Remove TranslatorOps.breakpoint_check 2021-07-21 07:47:05 -10:00
translate.h Hexagon (target/hexagon) compile all debug code 2021-05-01 08:31:43 -07:00

Hexagon is Qualcomm's very long instruction word (VLIW) digital signal
processor(DSP).

The following versions of the Hexagon core are supported
    Scalar core: v67
    https://developer.qualcomm.com/downloads/qualcomm-hexagon-v67-programmer-s-reference-manual

We presented an overview of the project at the 2019 KVM Forum.
    https://kvmforum2019.sched.com/event/Tmwc/qemu-hexagon-automatic-translation-of-the-isa-manual-pseudcode-to-tiny-code-instructions-of-a-vliw-architecture-niccolo-izzo-revng-taylor-simpson-qualcomm-innovation-center

*** Tour of the code ***

The qemu-hexagon implementation is a combination of qemu and the Hexagon
architecture library (aka archlib).  The three primary directories with
Hexagon-specific code are

    qemu/target/hexagon
        This has all the instruction and packet semantics
    qemu/target/hexagon/imported
        These files are imported with very little modification from archlib
        *.idef                  Instruction semantics definition
        macros.def              Mapping of macros to instruction attributes
        encode*.def             Encoding patterns for each instruction
        iclass.def              Instruction class definitions used to determine
                                legal VLIW slots for each instruction
    qemu/linux-user/hexagon
        Helpers for loading the ELF file and making Linux system calls,
        signals, etc

We start with scripts that generate a bunch of include files.  This
is a two step process.  The first step is to use the C preprocessor to expand
macros inside the architecture definition files.  This is done in
target/hexagon/gen_semantics.c.  This step produces
    <BUILD_DIR>/target/hexagon/semantics_generated.pyinc.
That file is consumed by the following python scripts to produce the indicated
header files in <BUILD_DIR>/target/hexagon
        gen_opcodes_def.py              -> opcodes_def_generated.h.inc
        gen_op_regs.py                  -> op_regs_generated.h.inc
        gen_printinsn.py                -> printinsn_generated.h.inc
        gen_op_attribs.py               -> op_attribs_generated.h.inc
        gen_helper_protos.py            -> helper_protos_generated.h.inc
        gen_shortcode.py                -> shortcode_generated.h.inc
        gen_tcg_funcs.py                -> tcg_funcs_generated.c.inc
        gen_tcg_func_table.py           -> tcg_func_table_generated.c.inc
        gen_helper_funcs.py             -> helper_funcs_generated.c.inc

Qemu helper functions have 3 parts
    DEF_HELPER declaration indicates the signature of the helper
    gen_helper_<NAME> will generate a TCG call to the helper function
    The helper implementation

Here's an example of the A2_add instruction.
    Instruction tag        A2_add
    Assembly syntax        "Rd32=add(Rs32,Rt32)"
    Instruction semantics  "{ RdV=RsV+RtV;}"

By convention, the operands are identified by letter
    RdV is the destination register
    RsV, RtV are source registers

The generator uses the operand naming conventions (see large comment in
hex_common.py) to determine the signature of the helper function.  Here are the
results for A2_add

helper_protos_generated.h.inc
    DEF_HELPER_3(A2_add, s32, env, s32, s32)

tcg_funcs_generated.c.inc
    static void generate_A2_add(
                    CPUHexagonState *env,
                    DisasContext *ctx,
                    Insn *insn,
                    Packet *pkt)
    {
        TCGv RdV = tcg_temp_local_new();
        const int RdN = insn->regno[0];
        TCGv RsV = hex_gpr[insn->regno[1]];
        TCGv RtV = hex_gpr[insn->regno[2]];
        gen_helper_A2_add(RdV, cpu_env, RsV, RtV);
        gen_log_reg_write(RdN, RdV);
        ctx_log_reg_write(ctx, RdN);
        tcg_temp_free(RdV);
    }

helper_funcs_generated.c.inc
    int32_t HELPER(A2_add)(CPUHexagonState *env, int32_t RsV, int32_t RtV)
    {
        uint32_t slot __attribute__((unused)) = 4;
        int32_t RdV = 0;
        { RdV=RsV+RtV;}
        return RdV;
    }

Note that generate_A2_add updates the disassembly context to be processed
when the packet commits (see "Packet Semantics" below).

The generator checks for fGEN_TCG_<tag> macro.  This allows us to generate
TCG code instead of a call to the helper.  If defined, the macro takes 1
argument.
    C semantics (aka short code)

This allows the code generator to override the auto-generated code.  In some
cases this is necessary for correct execution.  We can also override for
faster emulation.  For example, calling a helper for add is more expensive
than generating a TCG add operation.

The gen_tcg.h file has any overrides. For example, we could write
    #define fGEN_TCG_A2_add(GENHLPR, SHORTCODE) \
        tcg_gen_add_tl(RdV, RsV, RtV)

The instruction semantics C code relies heavily on macros.  In cases where the
C semantics are specified only with macros, we can override the default with
the short semantics option and #define the macros to generate TCG code.  One
example is L2_loadw_locked:
    Instruction tag        L2_loadw_locked
    Assembly syntax        "Rd32=memw_locked(Rs32)"
    Instruction semantics  "{ fEA_REG(RsV); fLOAD_LOCKED(1,4,u,EA,RdV) }"

In gen_tcg.h, we use the shortcode
#define fGEN_TCG_L2_loadw_locked(SHORTCODE) \
    SHORTCODE

There are also cases where we brute force the TCG code generation.
Instructions with multiple definitions are examples.  These require special
handling because qemu helpers can only return a single value.

In addition to instruction semantics, we use a generator to create the decode
tree.  This generation is also a two step process.  The first step is to run
target/hexagon/gen_dectree_import.c to produce
    <BUILD_DIR>/target/hexagon/iset.py
This file is imported by target/hexagon/dectree.py to produce
    <BUILD_DIR>/target/hexagon/dectree_generated.h.inc

*** Key Files ***

cpu.h

This file contains the definition of the CPUHexagonState struct.  It is the
runtime information for each thread and contains stuff like the GPR and
predicate registers.

macros.h

The Hexagon arch lib relies heavily on macros for the instruction semantics.
This is a great advantage for qemu because we can override them for different
purposes.  You will also notice there are sometimes two definitions of a macro.
The QEMU_GENERATE variable determines whether we want the macro to generate TCG
code.  If QEMU_GENERATE is not defined, we want the macro to generate vanilla
C code that will work in the helper implementation.

translate.c

The functions in this file generate TCG code for a translation block.  Some
important functions in this file are

    gen_start_packet - initialize the data structures for packet semantics
    gen_commit_packet - commit the register writes, stores, etc for a packet
    decode_and_translate_packet - disassemble a packet and generate code

genptr.c
gen_tcg.h

These files create a function for each instruction.  It is mostly composed of
fGEN_TCG_<tag> definitions followed by including tcg_funcs_generated.c.inc.

op_helper.c

This file contains the implementations of all the helpers.  There are a few
general purpose helpers, but most of them are generated by including
helper_funcs_generated.c.inc.  There are also several helpers used for debugging.


*** Packet Semantics ***

VLIW packet semantics differ from serial semantics in that all input operands
are read, then the operations are performed, then all the results are written.
For exmaple, this packet performs a swap of registers r0 and r1
    { r0 = r1; r1 = r0 }
Note that the result is different if the instructions are executed serially.

Packet semantics dictate that we defer any changes of state until the entire
packet is committed.  We record the results of each instruction in a side data
structure, and update the visible processor state when we commit the packet.

The data structures are divided between the runtime state and the translation
context.

During the TCG generation (see translate.[ch]), we use the DisasContext to
track what needs to be done during packet commit.  Here are the relevant
fields

    reg_log            list of registers written
    reg_log_idx        index into ctx_reg_log
    pred_log           list of predicates written
    pred_log_idx       index into ctx_pred_log
    store_width        width of stores (indexed by slot)

During runtime, the following fields in CPUHexagonState (see cpu.h) are used

    new_value             new value of a given register
    reg_written           boolean indicating if register was written
    new_pred_value        new value of a predicate register
    pred_written          boolean indicating if predicate was written
    mem_log_stores        record of the stores (indexed by slot)

*** Debugging ***

You can turn on a lot of debugging by changing the HEX_DEBUG macro to 1 in
internal.h.  This will stream a lot of information as it generates TCG and
executes the code.

To track down nasty issues with Hexagon->TCG generation, we compare the
execution results with actual hardware running on a Hexagon Linux target.
Run qemu with the "-d cpu" option.  Then, we can diff the results and figure
out where qemu and hardware behave differently.

The stacks are located at different locations.  We handle this by changing
env->stack_adjust in translate.c.  First, set this to zero and run qemu.
Then, change env->stack_adjust to the difference between the two stack
locations.  Then rebuild qemu and run again. That will produce a very
clean diff.

Here are some handy places to set breakpoints

    At the call to gen_start_packet for a given PC (note that the line number
        might change in the future)
        br translate.c:602 if ctx->base.pc_next == 0xdeadbeef
    The helper function for each instruction is named helper_<TAG>, so here's
        an example that will set a breakpoint at the start
        br helper_A2_add
    If you have the HEX_DEBUG macro set, the following will be useful
        At the start of execution of a packet for a given PC
            br helper_debug_start_packet if env->gpr[41] == 0xdeadbeef
        At the end of execution of a packet for a given PC
            br helper_debug_commit_end if env->this_PC == 0xdeadbeef