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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux