Re: [PATCH 1/3] mm: fix the format of the kernel parameter ``thp_anon=``

From: David Hildenbrand
Date: Mon Oct 28 2024 - 08:23:19 EST


On 27.10.24 18:36, Maíra Canal wrote:
If we add ``thp_anon=32,64KB:always`` to the kernel command line, we
will see the following error:

^ did you mean "64K" instead of "64KB" ?


[ 0.000000] huge_memory: thp_anon=32,64K:always: error parsing string, ignoring setting

This happens because the correct format isn't ``thp_anon=<size>,<size>[KMG]:<state>```,
as [KMG] must follow each number to especify its unit. So, the correct
format is ``thp_anon=<size>[KMG],<size>[KMG]:<state>```.

Therefore, adjust the documentation to reflect the correct format of the
parameter ``thp_anon=``.

Fixes: dd4d30d1cdbe ("mm: override mTHP "enabled" defaults at kernel cmdline")

As Barry says, this is a doc fix and we should make that clearer in the subject. With that:

Acked-by: David Hildenbrand <david@xxxxxxxxxx>

--
Cheers,

David / dhildenb