lkml.org 
[lkml]   [2023]   [Jan]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH 06/18] lib/stackdepot: annotate init and early init functions
    On Tue, Jan 31, 2023 at 11:31 AM Alexander Potapenko <glider@google.com> wrote:
    >
    > On Mon, Jan 30, 2023 at 9:50 PM <andrey.konovalov@linux.dev> wrote:
    > >
    > > From: Andrey Konovalov <andreyknvl@google.com>
    > >
    > > Add comments to stack_depot_early_init and stack_depot_init to explain
    > > certain parts of their implementation.
    > >
    > > Also add a pr_info message to stack_depot_early_init similar to the one
    > > in stack_depot_init.
    > >
    > > Also move the scale variable in stack_depot_init to the scope where it
    > > is being used.
    > >
    > > Signed-off-by: Andrey Konovalov <andreyknvl@google.com>
    > Reviewed-by: Alexander Potapenko <glider@google.com>
    > ...
    > >
    > > +/* Allocates a hash table via kvmalloc. Can be used after boot. */
    > Nit: kvcalloc? (Doesn't really matter much)

    Ah, right, forgot to fix this. I initially wanted to point out that
    early init allocates in memblock and late init in slab or vmalloc but
    then decided it's an unnecessary level of details. Will fix in v2.
    Thanks!

    \
     
     \ /
      Last update: 2023-03-27 00:04    [W:3.100 / U:0.000 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site