Re: [PATCH v2 15/19] docs: update rcu_dereference.rst reference

From: "Paul E. McKenney" <paulmck@kernel.org>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>,
	Linux Doc Mailing List <linux-doc@vger.kernel.org>,
	Jonathan Corbet <corbet@lwn.net>
Subject: Re: [PATCH v2 15/19] docs: update rcu_dereference.rst reference
Date: Wed, 7 Apr 2021 10:42:44 -0700	[thread overview]
Message-ID: <20210407174244.GG4510@paulmck-ThinkPad-P17-Gen-1> (raw)
In-Reply-To: <058f26dd-cc49-c58e-afde-58a565c3bba1@gmail.com>

On Wed, Apr 07, 2021 at 11:44:00PM +0900, Akira Yokosawa wrote:
> On Wed, 7 Apr 2021 06:51:06 -0700, Paul E. McKenney wrote:
> > On Wed, Apr 07, 2021 at 07:10:00PM +0900, Akira Yokosawa wrote:
> >> Hello Mauro,
> >>
> >> (Dropped most CC)
> >>
> >> On Wed,  7 Apr 2021 10:20:54 +0200, Mauro Carvalho Chehab wrote:
> >>> Changeset b00aedf978aa ("doc: Convert to rcu_dereference.txt to rcu_dereference.rst")
> >>> renamed: Documentation/RCU/rcu_dereference.txt
> >>> to: Documentation/RCU/rcu_dereference.rst.
> >>>
> >>> Update its cross-reference accordingly.
> >>>
> >>> Fixes: b00aedf978aa ("doc: Convert to rcu_dereference.txt to rcu_dereference.rst")
> >>> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> >>> ---
> >>
> >> Paul has sent a pull request to mingo for v5.13 including v1 of
> >> this patch.
> >>
> >>> Mauro Carvalho Chehab (2):
> >>>       rcu: Fix kfree_rcu() docbook errors
> >>>       doc: Update rcu_dereference.rst reference   <---
> >>
> >> See: https://lore.kernel.org/lkml/20210331192934.GA8127@paulmck-ThinkPad-P72/
> >>
> >> Hmm, your email address is on the CC list...
> > 
> > My approach would be simply to take Mauro's patch into the -rcu dev
> > branch.  If it affected kernel function in a bad way, I would submit it
> > into v5.13, but as it is, I would just run it through my normal process.
> > This would likely get it into v5.14.
> 
> Paul, v1 and v2 has the identical diff.  Only difference is the added
> Fixes: tag.
> 
> I don't think you can do anything further with regard to the v2 patch.

Ah, good point, thank you.

							Thanx, Paul

>         Thanks, Akira
> 
> > 
> > Seem reasonable?
> > 
> > 							Thanx, Paul
> > 
> >>         Thanks, Akira
> >>
> >>>  tools/memory-model/Documentation/glossary.txt | 2 +-
> >>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> diff --git a/tools/memory-model/Documentation/glossary.txt b/tools/memory-model/Documentation/glossary.txt
> >>> index b2da6365be63..6f3d16dbf467 100644
> >>> --- a/tools/memory-model/Documentation/glossary.txt
> >>> +++ b/tools/memory-model/Documentation/glossary.txt
> >>> @@ -19,7 +19,7 @@ Address Dependency:  When the address of a later memory access is computed
> >>>  	 from the value returned by the rcu_dereference() on line 2, the
> >>>  	 address dependency extends from that rcu_dereference() to that
> >>>  	 "p->a".  In rare cases, optimizing compilers can destroy address
> >>> -	 dependencies.	Please see Documentation/RCU/rcu_dereference.txt
> >>> +	 dependencies.	Please see Documentation/RCU/rcu_dereference.rst
> >>>  	 for more information.
> >>>  
> >>>  	 See also "Control Dependency" and "Data Dependency".
> >>>