Re: [PATCH] Documentation: RCU: whatisRCU: Fix formatting for section 2

From: Amol Grover
Date: Tue Nov 05 2019 - 11:20:45 EST


On Tue, Nov 05, 2019 at 07:52:59AM -0700, Shuah Khan wrote:
> Hi Amol,
>
> On 11/4/19 12:45 PM, Paul E. McKenney wrote:
> > On Mon, Nov 04, 2019 at 10:46:41PM +0530, Amol Grover wrote:
> > > On Mon, Nov 04, 2019 at 07:03:28AM -0800, Paul E. McKenney wrote:
> > > > On Mon, Nov 04, 2019 at 07:03:15PM +0530, Amol Grover wrote:
> > > > > Convert RCU API method text to sub-headings and
> > > > > add hyperlink and superscript to 2 literary notes
> > > > > under rcu_dereference() section
> > > > >
> > > > > Signed-off-by: Amol Grover <frextrite@xxxxxxxxx>
> > > >
> > > > Good stuff, but Phong Tran beat you to it. If you are suggesting
> > > > changes to that patch, please send a reply to her email, which
> > > > may be found here:
> > > >
>
> Please do a review and send comments and suggestions in an email
> instead of a patch.

Hi Shuah,

Sure thing! I'll do a review and send in the suggestions.

>
> > > > https://lore.kernel.org/lkml/20191030233128.14997-1-tranmanphong@xxxxxxxxx/
> > > >
> > > > There are several options for replying to this email listed at the
> > > > bottom of that web page.
> > >
> > > Thank you Paul! And that is correct, I was suggesting changes to
> > > that patch. However, since that patch was already integrated into
> > > the `dev` branch, I mistakenly believed this patch could be sent
> > > independently. Sorry for the trouble, I'll re-send the patch the
> > > correct way.
> >
>
> Please drop your patch and do a review for the patch as suggested by
> Paul. This should have been a review and not a patch on top.

Noted. Will definitely keep this in mind the next time.

Thank you
Amol

>
> thanks,
> -- Shuah