Re: Re: [PATCH 0/2] Export debug triggers as an extension

2024-02-04 Thread Alistair Francis
On Mon, Feb 5, 2024 at 2:50 PM Anup Patel wrote: > > On Mon, Feb 5, 2024 at 9:36 AM Alistair Francis wrote: > > > > On Mon, Jan 22, 2024 at 7:16 PM Andrew Jones > > wrote: > > > > > > On Mon, Jan 22, 2024 at 03:42:10PM +1000, Alistair Francis wrote: > > > > > > From memory the "debug" property

Re: Re: [PATCH 0/2] Export debug triggers as an extension

2024-02-04 Thread Anup Patel
On Mon, Feb 5, 2024 at 9:36 AM Alistair Francis wrote: > > On Mon, Jan 22, 2024 at 7:16 PM Andrew Jones wrote: > > > > On Mon, Jan 22, 2024 at 03:42:10PM +1000, Alistair Francis wrote: > > > > > From memory the "debug" property is for the original debug spec: > > > > > https://github.com/riscv/ri

Re: Re: [PATCH 0/2] Export debug triggers as an extension

2024-02-04 Thread Alistair Francis
On Mon, Jan 22, 2024 at 7:16 PM Andrew Jones wrote: > > On Mon, Jan 22, 2024 at 03:42:10PM +1000, Alistair Francis wrote: > > > > From memory the "debug" property is for the original debug spec: > > > > https://github.com/riscv/riscv-debug-spec/releases/tag/task_group_vote > > > > > > > > That was

Re: Re: [PATCH 0/2] Export debug triggers as an extension

2024-01-22 Thread Andrew Jones
On Mon, Jan 22, 2024 at 03:42:10PM +1000, Alistair Francis wrote: > > > From memory the "debug" property is for the original debug spec: > > > https://github.com/riscv/riscv-debug-spec/releases/tag/task_group_vote > > > > > > That was ratified and is an official extension. AFAIK this is what is > >

Re: [PATCH 0/2] Export debug triggers as an extension

2024-01-21 Thread Alistair Francis
On Fri, Jan 12, 2024 at 11:34 PM Rob Bradford wrote: > > On Fri, 2024-01-12 at 13:52 +1000, Alistair Francis wrote: > > On Thu, Jan 11, 2024 at 5:20 AM Daniel Henrique Barboza > > wrote: > > > > > > Himanshu, > > > > > > We spoke offline but let's make everyone aware: > > > > > > - 'sdtrig' shoul

Re: [PATCH 0/2] Export debug triggers as an extension

2024-01-17 Thread Himanshu Chauhan
> On 17-Jan-2024, at 10:29 PM, Daniel Henrique Barboza > wrote: > > > > On 1/12/24 10:34, Rob Bradford wrote: >> On Fri, 2024-01-12 at 13:52 +1000, Alistair Francis wrote: >>> On Thu, Jan 11, 2024 at 5:20 AM Daniel Henrique Barboza >>> wrote: Himanshu, We spoke offlin

Re: [PATCH 0/2] Export debug triggers as an extension

2024-01-17 Thread Daniel Henrique Barboza
On 1/12/24 10:34, Rob Bradford wrote: On Fri, 2024-01-12 at 13:52 +1000, Alistair Francis wrote: On Thu, Jan 11, 2024 at 5:20 AM Daniel Henrique Barboza wrote: Himanshu, We spoke offline but let's make everyone aware: - 'sdtrig' should be marked with 'x-' and be an experimental extension

Re: [PATCH 0/2] Export debug triggers as an extension

2024-01-12 Thread Rob Bradford
On Fri, 2024-01-12 at 13:52 +1000, Alistair Francis wrote: > On Thu, Jan 11, 2024 at 5:20 AM Daniel Henrique Barboza > wrote: > > > > Himanshu, > > > > We spoke offline but let's make everyone aware: > > > > - 'sdtrig' should be marked with 'x-' and be an experimental > > extension since > > th

Re: [PATCH 0/2] Export debug triggers as an extension

2024-01-11 Thread Alistair Francis
On Thu, Jan 11, 2024 at 5:20 AM Daniel Henrique Barboza wrote: > > Himanshu, > > We spoke offline but let's make everyone aware: > > - 'sdtrig' should be marked with 'x-' and be an experimental extension since > the spec isn't yet frozen; > > - Alvin sent a patch to the ML adding the 'mcontext' CS

Re: [PATCH 0/2] Export debug triggers as an extension

2024-01-10 Thread Daniel Henrique Barboza
Himanshu, We spoke offline but let's make everyone aware: - 'sdtrig' should be marked with 'x-' and be an experimental extension since the spec isn't yet frozen; - Alvin sent a patch to the ML adding the 'mcontext' CSR for 'sdtrig' some time ago: "[PATCH v2] target/riscv: Implement optional CS

[PATCH 0/2] Export debug triggers as an extension

2024-01-09 Thread Himanshu Chauhan
All the CPUs may or may not implement the debug trigger (sdtrig) extension. The presence of it should be dynamically detectable. This patch exports the debug triggers as an extension which can be turned on or off by sdtrig= option. It is turned on by default. "sdtrig" is concatenated to ISA string