Re: [PATCH 3/4] mm/damon/core: copy nid for eligible_mem_bp damos quota goal commit

[SJ Park]

On Thu,  3 Sep 2026 16:45:45 +0800 Kunwu Chan <kunwu.chan@xxxxxxxxx> wrote:

> On Tue,  1 Sep 2026 17:27:22 -0700 SJ Park <sj@xxxxxxxxxx> wrote:
[...]
> > This issue was discovered [1] by Sashiko.
> > 
> > [1] https://lore.kkernel.org/20260827045035.94611-1-sj@xxxxxxxxxx
> 
> Hi SJ,
> 
> The fix is consistent with the other node-based quota metrics. I also 
> checked that damos_get_node_eligible_mem_bp() directly uses goal->nid 
> when calculating the current value.
> 
> One minor typo: the [1] has an extra k (kkernel should be kernel).

Nice catch, thank you Kunwu!  Andrew, could you please fix this when you pick
this series up?  Let me know if you prefer reposting the series with the fix.

> 
> Reviewed-by: Kunwu Chan <kunwu.chan@xxxxxxxxx>

Appreciate all the reviews, Kunwu!


Thanks,
SJ

[...]