linux-kernel - Re: [PATCH 0/2] add a knob to control whether to use other nodes at the same tier of the target node in DAMON

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <d70d2cc058fc4a588720076200c81214@inspur.com>
Date: Thu, 29 May 2025 03:14:00 +0000
From: Simon Wang (王传国) <wangchuanguo@...pur.com>
To: SeongJae Park <sj@...nel.org>
CC: "akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"hannes@...xchg.org" <hannes@...xchg.org>, "david@...hat.com"
	<david@...hat.com>, "mhocko@...nel.org" <mhocko@...nel.org>,
	"zhengqi.arch@...edance.com" <zhengqi.arch@...edance.com>,
	"shakeel.butt@...ux.dev" <shakeel.butt@...ux.dev>,
	"lorenzo.stoakes@...cle.com" <lorenzo.stoakes@...cle.com>,
	"linux-mm@...ck.org" <linux-mm@...ck.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "damon@...ts.linux.dev"
	<damon@...ts.linux.dev>
Subject: Re: [PATCH 0/2] add a knob to control whether to use other nodes at
 the same tier of the target node in DAMON


> Thank you for sending this patch series!
> 
> On Wed, 28 May 2025 19:10:36 +0800 wangchuanguo
> <wangchuanguo@...pur.com> wrote:
> 
> > In DAMON's migrate_hot and migrate_cold features, the code was
> > intended to migrate pages only to the node specified by target_nid.
> > However, during testing, it was observed that memory allocation and
> > migration could occur on any nodes, which is a BUG.
> > The first patch in this PR fix this issue.
> >
> > A use_nodes_of_tier file has been added under the directory
> > /sys/kernel/mm/damon/admin/kdamonds/<N>/contexts/<N>/schemes/<N>/
> > to control whether to use other nodes in the same tier as the target
> > node for migration.
> 
> I left a few comments on the patches.  Looking forward to discussions on
> each sub-thread :)
> 
> >
> > wangchuanguo (2):
> 
> I believe your name is Wang Chuanguo?  Sorry if I read/wrote it wrongly.
> But we disallow[1] anonymous contributions, and prefer more formal
> Signed-off-by:
> if possible.  Could you please use such formal Signed-off-by: identity, say,
> "Wang Chuanguo <wangchuanguo@...pur.com>" from next time?

I'm sorry, this was my mistake. I will be more careful next time.

> [1]
> https://docs.kernel.org/process/submitting-patches.html#developer-s-certifica
> te-of-origin-1-1
> 
> 
> Thanks,
> SJ
> 
> [...]