Re: [PATCH 6/8] mm/vmalloc: Defer freeing partly initialized vm_struct
From: Uladzislau Rezki
Date: Fri Aug 08 2025 - 06:37:39 EST
On Thu, Aug 07, 2025 at 01:25:01PM +0200, Michal Hocko wrote:
> On Thu 07-08-25 09:58:08, Uladzislau Rezki wrote:
> > __vmalloc_area_node() may call free_vmap_area() or vfree() on
> > error paths, both of which can sleep. This becomes problematic
> > if the function is invoked from an atomic context, such as when
> > GFP_ATOMIC or GFP_NOWAIT is passed via gfp_mask.
> >
> > To fix this, unify error paths and defer the cleanup of partly
> > initialized vm_struct objects to a workqueue. This ensures that
> > freeing happens in a process context and avoids invalid sleeps
> > in atomic regions.
> >
> > Signed-off-by: Uladzislau Rezki (Sony) <urezki@xxxxxxxxx>
>
> LGTM
> Acked-by: Michal Hocko <mhocko@xxxxxxxx>
> Thanks!
>
Thanks, applied Acked-by.
--
Uladzislau Rezki