Re: [PATCH v2 06/19] lib/stackdepot: fix and clean-up atomic annotations

From: Andrey Konovalov
Date: Mon Oct 23 2023 - 12:16:11 EST


On Fri, Oct 6, 2023 at 7:22 PM Alexander Potapenko <glider@xxxxxxxxxx> wrote:
>
> On Fri, Oct 6, 2023 at 6:14 PM Alexander Potapenko <glider@xxxxxxxxxx> wrote:
> >
> > On Wed, Sep 13, 2023 at 7:15 PM <andrey.konovalov@xxxxxxxxx> wrote:
> > >
> > > From: Andrey Konovalov <andreyknvl@xxxxxxxxxx>
> > >
> > > Simplify comments accompanying the use of atomic accesses in the
> > > stack depot code.
> > >
> > > Also drop smp_load_acquire from next_pool_required in depot_init_pool,
> > > as both depot_init_pool and the all smp_store_release's to this variable
> > > are executed under the stack depot lock.
>
> Maybe add this to the comment before "if (!next_pool_required)" ?

Will do in v3.

Re removed parentheses: will restore them in v3.

Thanks!