On 21/02/26 6:49 AM, SeongJae Park wrote:
> On Thu, 12 Feb 2026 17:33:30 +0100 (CET) Mikulas Patocka 
> <[email protected]> wrote:
> 
>> The commit 07003531e03c8 ("mm/vmalloc: warn on invalid vmalloc gfp
>> flags") breaks the device mapper VDO target. The VDO target calls vmalloc
>> with __GFP_RETRY_MAYFAIL and this flag is not in the mask of allowed
>> flags.
>>
>> There is no reason why vmalloc couldn't support __GFP_RETRY_MAYFAIL, so 
>> let's add this flag to GFP_VMALLOC_SUPPORTED.
>>
>> Signed-off-by: Mikulas Patocka <[email protected]>
>> Reported-by: Zdenek Kabelac <[email protected]>
>> Fixes: 07003531e03c ("mm/vmalloc: warn on invalid vmalloc gfp flags")
>> Cc: [email protected]   # v6.19
> 
> Acked-by: SeongJae Park <[email protected]>
> 
>>
>> ---
>>  mm/vmalloc.c |    1 +
>>  1 file changed, 1 insertion(+)
>>
>> Index: linux-2.6/mm/vmalloc.c
>> ===================================================================
>> --- linux-2.6.orig/mm/vmalloc.c
>> +++ linux-2.6/mm/vmalloc.c
>> @@ -3928,6 +3928,7 @@ fail:
>>   */
>>  #define GFP_VMALLOC_SUPPORTED (GFP_KERNEL | GFP_ATOMIC | GFP_NOWAIT |\
>>                              __GFP_NOFAIL |  __GFP_ZERO | __GFP_NORETRY |\
>> +                            __GFP_RETRY_MAYFAIL |\
>>                              GFP_NOFS | GFP_NOIO | GFP_KERNEL_ACCOUNT |\
>>                              GFP_USER | __GFP_NOLOCKDEP)
> 
> By the way, I think this list might better to be alphabetically sorted in

Agreed.

> future.  Obviously just a trivial and random thought that never required for
> this patch.

Reviewed-by: Anshuman Khandual <[email protected]>

Reply via email to