[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+CK2bB+NVzFW5DpmrL3jJPZFyaKyHdqge3KNf1eLBe0YeHhfA@mail.gmail.com>
Date: Sun, 17 Mar 2024 11:15:30 -0400
From: Pasha Tatashin <pasha.tatashin@...een.com>
To: Christophe JAILLET <christophe.jaillet@...adoo.fr>
Cc: linux-kernel@...r.kernel.org, linux-mm@...ck.org,
akpm@...ux-foundation.org, x86@...nel.org, bp@...en8.de, brauner@...nel.org,
bristot@...hat.com, bsegall@...gle.com, dave.hansen@...ux.intel.com,
dianders@...omium.org, dietmar.eggemann@....com, eric.devolder@...cle.com,
hca@...ux.ibm.com, hch@...radead.org, hpa@...or.com,
jacob.jun.pan@...ux.intel.com, jgg@...pe.ca, jpoimboe@...nel.org,
jroedel@...e.de, juri.lelli@...hat.com, kent.overstreet@...ux.dev,
kinseyho@...gle.com, kirill.shutemov@...ux.intel.com, lstoakes@...il.com,
luto@...nel.org, mgorman@...e.de, mic@...ikod.net,
michael.christie@...cle.com, mingo@...hat.com, mjguzik@...il.com,
mst@...hat.com, npiggin@...il.com, peterz@...radead.org, pmladek@...e.com,
rick.p.edgecombe@...el.com, rostedt@...dmis.org, surenb@...gle.com,
tglx@...utronix.de, urezki@...il.com, vincent.guittot@...aro.org
Subject: Re: [RFC 08/14] fork: separate vmap stack alloction and free calls
On Sun, Mar 17, 2024 at 10:52 AM Christophe JAILLET
<christophe.jaillet@...adoo.fr> wrote:
>
> Le 11/03/2024 à 17:46, Pasha Tatashin a écrit :
> > In preparation for the dynamic stacks, separate out the
> > __vmalloc_node_range and vfree calls from the vmap based stack
> > allocations. The dynamic stacks will use their own variants of these
> > functions.
> >
> > Signed-off-by: Pasha Tatashin <pasha.tatashin@...een.com>
> > ---
> > kernel/fork.c | 53 ++++++++++++++++++++++++++++++---------------------
> > 1 file changed, 31 insertions(+), 22 deletions(-)
> >
> > diff --git a/kernel/fork.c b/kernel/fork.c
> > index 3004e6ce6c65..bbae5f705773 100644
> > --- a/kernel/fork.c
> > +++ b/kernel/fork.c
> > @@ -204,6 +204,29 @@ static bool try_release_thread_stack_to_cache(struct vm_struct *vm_area)
> > return false;
> > }
> >
> > +static inline struct vm_struct *alloc_vmap_stack(int node)
> > +{
> > + void *stack;
> > +
> > + /*
> > + * Allocated stacks are cached and later reused by new threads,
> > + * so memcg accounting is performed manually on assigning/releasing
> > + * stacks to tasks. Drop __GFP_ACCOUNT.
> > + */
> > + stack = __vmalloc_node_range(THREAD_SIZE, THREAD_ALIGN,
> > + VMALLOC_START, VMALLOC_END,
> > + THREADINFO_GFP & ~__GFP_ACCOUNT,
> > + PAGE_KERNEL,
> > + 0, node, __builtin_return_address(0));
> > +
> > + return (stack) ? find_vm_area(stack) : NULL;
>
> Nit: superfluous ()
Thank you.
>
> > +}
>
> ...
>
Powered by blists - more mailing lists