Re: [PATCH v4 3/4] target/riscv: support vector extension csr

2020-02-11 Thread LIU Zhiwei
On 2020/2/12 0:11, Richard Henderson wrote: On 2/10/20 8:12 AM, LIU Zhiwei wrote: +static int vs(CPURISCVState *env, int csrno) +{ +return 0; +} This should at least be testing RVV, a-la smode(). Testing RVV is ok.  I'm not quite understand "a -1a smode()" here. Could you give more de

Re: [PATCH v4 3/4] target/riscv: support vector extension csr

2020-02-11 Thread Richard Henderson
On 2/10/20 8:12 AM, LIU Zhiwei wrote: > +static int vs(CPURISCVState *env, int csrno) > +{ > +return 0; > +} This should at least be testing RVV, a-la smode(). You probably want to have all of the other tests vs RVV in this file use this function, since this will need to grow the system mode

[PATCH v4 3/4] target/riscv: support vector extension csr

2020-02-10 Thread LIU Zhiwei
The v0.7.1 specification does not define vector status within mstatus. A future revision will define the privileged portion of the vector status. Signed-off-by: LIU Zhiwei --- target/riscv/cpu_bits.h | 15 + target/riscv/csr.c | 72 +++-- 2 files