Re: [PATCH RFC tip/core/rcu 02/14] rcu/nocb: Add bypass callback queueing

From: Paul E. McKenney
Date: Tue Aug 06 2019 - 23:48:46 EST


On Tue, Aug 06, 2019 at 09:24:15PM -0400, Steven Rostedt wrote:
> On Tue, 6 Aug 2019 18:17:07 -0700
> "Paul E. McKenney" <paulmck@xxxxxxxxxxxxx> wrote:
>
> > On Tue, Aug 06, 2019 at 08:40:55PM -0400, Steven Rostedt wrote:
> > > On Tue, 6 Aug 2019 17:35:01 -0700
> > > "Paul E. McKenney" <paulmck@xxxxxxxxxxxxx> wrote:
> > >
> > > > > > + // Don't use ->nocb_bypass during early boot.
> > > > >
> > > > > Very minor nit: comment style should be /* */
> > > >
> > > > I thought that Linus said that "//" was now OK. Am I confused?
> > >
> > > Have a link?
> >
> > https://lkml.org/lkml/2016/7/8/625
>
> The (c) form is particularly good for things like enum or structure
> member comments at the end of code, where you might want to align
> things up, but the ending comment marker ends up being visually pretty
> distracting (and lining _that_ up is too much make-believe work).
>
> I think it's still for special occasions, and the above example doesn't
> look like one of them ;-)

It does say "particularly good for", not "only good for. ;-)

> I basically avoid the '//' comment, as it just adds inconstancy.

It saves me two whacks on the shift key and three whacks on other
keys. ;-)

Thanx, Paul