Re: [PATCH v1] kernel: add a simple timer based software watchpoint

[Feng Tang]

On Tue, Jun 23, 2026 at 06:26:54PM +0100, Julian Braha wrote:
> Hi Feng,
> 
> On 6/22/26 09:14, Feng Tang wrote:
> > +config SOFTWARE_WATCHPOINT
> > +	bool "Softwre watchpoint to watch memory corruption"
> > +	depends on  DEBUG_KERNEL
> > +	help
> > +	  Using periodc timer checking specific physical DRAM/MMIO address
> > +	  to capture memory corruption triggered by kernel, hardware devices,
> > +	  BIOS runtime code or silicon.
> > +
> 
> You've got some typo issues here in the kconfig:
> 1. "Softwre" in the prompt,
> 2. extra space in the 'depends on',
> 3. 'periodc' in the help text.

Thanks for the catch!

> Though I guess you have bigger issues to address from other reviewers.
 
I agree :)

> - Julian Braha