Re: [PATCH 1/2] mm/compaction: remove low watermark cap for proactive compaction

[Pasha Tatashin]

On Fri, Jan 24, 2025 at 1:21 PM Michal Clapinski <mclapinski@xxxxxxxxxx> wrote:
>
> Previously a min cap of 5 has been set in the commit introducing
> proactive compaction. This was to make sure users don't hurt themselves
> by setting the proactiveness to 100 and making their system
> unresponsive. But the compaction mechanism has a backoff mechanism that
> will sleep for 30s if no progress is made, so I don't see a significant
> risk here. My system (20GB of memory) has been perfectly fine with
> proactiveness set to 100 and leeway set to 0.
>
> Signed-off-by: Michal Clapinski <mclapinski@xxxxxxxxxx>

Reviewed-by: Pasha Tatashin <pasha.tatashin@xxxxxxxxxx>

Thank you,
Pasha