Re: [PATCH 03/11] RISC-V: Adding T-Head SYNC instructions

2022-12-12 Thread LIU Zhiwei
On 2022/9/8 15:29, Richard Henderson wrote: On 9/6/22 13:22, Christoph Muellner wrote: +NOP_PRIVCHECK(th_sfence_vmas, REQUIRE_PRIV_MHS) +NOP_PRIVCHECK(th_sync, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_i, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_is, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_s

Re: [PATCH 03/11] RISC-V: Adding T-Head SYNC instructions

2022-12-12 Thread LIU Zhiwei
On 2022/9/8 15:29, Richard Henderson wrote: On 9/6/22 13:22, Christoph Muellner wrote: +NOP_PRIVCHECK(th_sfence_vmas, REQUIRE_PRIV_MHS) +NOP_PRIVCHECK(th_sync, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_i, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_is, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_s

Re: [PATCH 03/11] RISC-V: Adding T-Head SYNC instructions

2022-09-09 Thread Christoph Müllner
On Thu, Sep 8, 2022 at 9:30 AM Richard Henderson < richard.hender...@linaro.org> wrote: > On 9/6/22 13:22, Christoph Muellner wrote: > > +NOP_PRIVCHECK(th_sfence_vmas, REQUIRE_PRIV_MHS) > > +NOP_PRIVCHECK(th_sync, REQUIRE_PRIV_MHSU) > > +NOP_PRIVCHECK(th_sync_i, REQUIRE_PRIV_MHSU) > > +NOP_PRIVCHE

Re: [PATCH 03/11] RISC-V: Adding T-Head SYNC instructions

2022-09-08 Thread Richard Henderson
On 9/6/22 13:22, Christoph Muellner wrote: +NOP_PRIVCHECK(th_sfence_vmas, REQUIRE_PRIV_MHS) +NOP_PRIVCHECK(th_sync, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_i, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_is, REQUIRE_PRIV_MHSU) +NOP_PRIVCHECK(th_sync_s, REQUIRE_PRIV_MHSU) These should not be nops