On Sun, Feb 20, 2022 at 08:51:21PM +0900, Akira Yokosawa wrote:
> Hi Paul,
>
> In commit 9c0c058db09d ("defer/rcuapi: Describe SRCU initialization and
> cleanup"), you added a row in Table 9.2.
>
> > --- a/defer/rcuapi.tex
>
> > +++ b/defer/rcuapi.tex
> > @@ -148,6 +148,14 @@ API~\cite{PaulEMcKenney2019RCUAPI}.\footnote{
> > {\bf Tasks RCU Rude}: Free idle-task tracing trampolines &
> > {\bf Tasks RCU Trace}: Protect sleepable BPF programs \\
> > \midrule
> > +{\bf Initialization and Cleanup} &
> > + &
> > + \tco{DEFINE_SRCU()} \tco{DEFINE_STATIC_SRCU()}
> > + \tco{init_srcu_struct()} \tco{cleanup_srcu_struct()} &
> > + &
> > + &
> > + \\
> > +\midrule
>
> Did you add this \midrule intentionally?
> The added row looks similar to the rows below it.
Now that you mention it, that does look rather silly. Thank you for
checking, and I have pushed out a fix with your Reported-by.
Thanx, Paul
> Thanks, Akira
>
> > {\bf Read-side critical-section markers} &
> > \tco{rcu_read_lock()}~! \tco{rcu_read_unlock()}~!
> > \tco{rcu_read_lock_bh()} \tco{rcu_read_unlock_bh()}
>
