Re: [PATCH 21/24] rcu/tiny: move kvfree_call_rcu() out of header

From: Uladzislau Rezki
Date: Thu May 07 2020 - 13:34:53 EST


> >
> > Please see full log here: ftp://vps418301.ovh.net/incoming/include_mm_h_output.txt
> >
> > I can fix it by adding the kvfree() declaration to the rcutiny.h also:
> > extern void kvfree(const void *addr);
> >
> > what seems wired to me? Also it can be fixed if i move it to the tiny.c
> > so it will be aligned with the way how it is done for tree-RCU.
>
> If the mm guys are OK with the kvfree() declaration, that is the way
> to go. With the addition of a comment saying something like "Avoid
> #include hell".
>
> The compiler will complain if the definition changes given that there
> has to be somewhere that sees both the above and the real declaration,
> so this should not cause too much trouble.
>
> > Any valuable proposals?
>
> Otherwise, yes, the function would need to move to tiny.c and thus add
> bloat. :-(
>

OK. I will declare it one more time. Indeed if it is changed, the
compiler will emit some errors. Also, i will add some comments why
it is done.

Thanks!

--
Vlad Rezki