Re: [PATCH 2/2] target/riscv: fixes a bug against `ssamoswap` behavior in M-mode

2025-03-05 Thread Deepak Gupta
On Thu, Mar 06, 2025 at 04:22:52PM +1000, Alistair Francis wrote: On Thu, Mar 6, 2025 at 4:13 PM Deepak Gupta wrote: On Thu, Mar 06, 2025 at 03:29:00PM +1000, Alistair Francis wrote: >On Tue, Feb 18, 2025 at 12:57 PM Deepak Gupta wrote: >> >> Commit f06bfe3dc38c ("target/riscv: implement zicf

Re: [PATCH 2/2] target/riscv: fixes a bug against `ssamoswap` behavior in M-mode

2025-03-05 Thread Alistair Francis
On Thu, Mar 6, 2025 at 4:13 PM Deepak Gupta wrote: > > On Thu, Mar 06, 2025 at 03:29:00PM +1000, Alistair Francis wrote: > >On Tue, Feb 18, 2025 at 12:57 PM Deepak Gupta wrote: > >> > >> Commit f06bfe3dc38c ("target/riscv: implement zicfiss instructions") adds > >> `ssamoswap` instruction. `ssamo

Re: [PATCH 2/2] target/riscv: fixes a bug against `ssamoswap` behavior in M-mode

2025-03-05 Thread Deepak Gupta
On Thu, Mar 06, 2025 at 03:29:00PM +1000, Alistair Francis wrote: On Tue, Feb 18, 2025 at 12:57 PM Deepak Gupta wrote: Commit f06bfe3dc38c ("target/riscv: implement zicfiss instructions") adds `ssamoswap` instruction. `ssamoswap` takes the code-point from existing reserved encoding (and not a

Re: [PATCH 2/2] target/riscv: fixes a bug against `ssamoswap` behavior in M-mode

2025-03-05 Thread Alistair Francis
On Tue, Feb 18, 2025 at 12:57 PM Deepak Gupta wrote: > > Commit f06bfe3dc38c ("target/riscv: implement zicfiss instructions") adds > `ssamoswap` instruction. `ssamoswap` takes the code-point from existing > reserved encoding (and not a zimop like other shadow stack instructions). > If shadow stack

[PATCH 2/2] target/riscv: fixes a bug against `ssamoswap` behavior in M-mode

2025-02-17 Thread Deepak Gupta
Commit f06bfe3dc38c ("target/riscv: implement zicfiss instructions") adds `ssamoswap` instruction. `ssamoswap` takes the code-point from existing reserved encoding (and not a zimop like other shadow stack instructions). If shadow stack is not enabled (via xenvcfg.SSE), then `ssamoswap` must result