Re: [PATCH v4] padata: Fix refcnt handling in padata_free_shell()

From: Daniel Jordan
Date: Mon Oct 23 2023 - 09:59:02 EST


On Fri, Oct 20, 2023 at 01:03:59PM +0800, Herbert Xu wrote:
> WangJinchao <wangjinchao@xxxxxxxxxxx> wrote:
> > diff --git a/kernel/padata.c b/kernel/padata.c
> > index 222d60195de6..73108ac75f03 100644
> > --- a/kernel/padata.c
> > +++ b/kernel/padata.c
> > @@ -1102,12 +1102,15 @@ EXPORT_SYMBOL(padata_alloc_shell);
> > */
> > void padata_free_shell(struct padata_shell *ps)
> > {
> > + struct parallel_data *pd;
> > if (!ps)
> > return;

The \n after the declaration disappeared in this version but would be
nice for consistency with the file. Maybe when applying, no need for
another post.

> Daniel, could you please reconfirm that you're still with v4?

Yes, regardless of above,
Acked-by: Daniel Jordan <daniel.m.jordan@xxxxxxxxxx>