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

[Julian Braha]

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.

Though I guess you have bigger issues to address from other reviewers.

- Julian Braha