Re: [PATCH v1 1/1] Docs/mm/damon: Document DAMON actions when TRANSPARENT_HUGEPAGE is off

[Gutierrez Asier]

Hi SJ,

On 3/16/2026 5:48 PM, SeongJae Park wrote:
> Hello Asier,
> 
> 
> Thank you for sending this patch. :)
> 
> On Mon, 16 Mar 2026 14:08:21 +0000 <gutierrez.asier@xxxxxxxxxxxxxxxxxxx> wrote:
> 
>> From: Asier Gutierrez <gutierrez.asier@xxxxxxxxxxxxxxxxxxx>
>>
>> MADV_HUGEPAGE and MADV_NOHUGEPAGE are guarded and they
>> are not available when compiling the kernel without TRANSPARENT_HUGEPAGE
>> option. The DAMON behaviour is to silently fail[1] in when DAMOS_HUGEPAGE or
> 
> checkpatch.pl shows a warning for the above line:
> 
>     WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
>     #12:
>     option. The DAMON behaviour is to silently fail[1] in when DAMOS_HUGEPAGE or

Yes, this is due to the link. I tried to keep this patch compact, without a
cover letter. I will submit a new version without the link.

>> DAMOS_NOHUGEPAGE are used, but TRANSPARENT_HUGEPAGE is disabled. Update the
>> DAMON documentation to reflect this behaviour.
>>
>> [1]: https://lore.kernel.org/damon/66131775-180b-4b9f-b7ce-61a3e077b6e6@xxxxxxxxxxxxxxxxxxx/T/#m278d3c9bbedcceaf2efa51e3dc7b40a40654e51c
>>
>> Signed-off-by: Asier Gutierrez <gutierrez.asier@xxxxxxxxxxxxxxxxxxx>
> 
> Reviewed-by: SeongJae Park <sj@xxxxxxxxxx>
> 
> Andrew might add this to mm.git while fixing the checkpatch warning on his own.
> Let's give time for Andrew today.
> 
> I also added this to damon/next tree after fixing the warning.  If Andrew
> doesn't pick this with the fix today, I will post mine as v2 of this patch
> tomorrow.  If you prefer to do the v2 posting on your own, just let me know :)
> 
> 
> Thanks,
> SJ
> 
> [...]
> 

-- 
Asier Gutierrez
Huawei