Re: [PATCH] RFC: possible fixing a preposition

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

 



On Wed, May 19, 2021 at 02:21:54AM +0800, Zhouyi Zhou wrote:
> Thanks Paul for reviewing my incorrect proposal.

No problem!  After all, if I were ever to have to write a document
in Chinese, I would need quite a bit of help.  Like the help you are
currently providing by doing this translation.  ;-)

> On Wed, May 19, 2021 at 1:35 AM Paul E. McKenney <paulmck@xxxxxxxxxx> wrote:
> >
> > In this case, "add" as in "addition" as in arithmetic, similar to the
> > C-language "+=" operator.  But...
> I see, my English is improved a lot by this case, thank you
> >
> > Would "add the value one to the global free-running rcu_gp_ctr variable"
> > work better for you?
> "add the value one to the global free-running rcu_gp_ctr variable" looks
> great to me.

Very good!  How about as shown below?

							Thanx, Paul

------------------------------------------------------------------------

commit 498ad5f93ea7a531bd79592b08c072ebd73bb188
Author: Paul E. McKenney <paulmck@xxxxxxxxxx>
Date:   Tue May 18 11:34:25 2021 -0700

    appendix/toyrcu: Wordsmith "add the value one to"
    
    Reported-by: Zhouyi Zhou <zhouzhouyi@xxxxxxxxx>
    Signed-off-by: Paul E. McKenney <paulmck@xxxxxxxxxx>

diff --git a/appendix/toyrcu/toyrcu.tex b/appendix/toyrcu/toyrcu.tex
index 9e5c01d5..39e43e30 100644
--- a/appendix/toyrcu/toyrcu.tex
+++ b/appendix/toyrcu/toyrcu.tex
@@ -976,7 +976,7 @@ The resulting \co{rcu_read_lock()} implementation is extremely
 straightforward.
 \begin{fcvref}[ln:defer:rcu:read_lock_unlock:lock]
 \Clnref{gp1,gp2} simply
-add one to the global free-running \co{rcu_gp_ctr}
+add the value one to the global free-running \co{rcu_gp_ctr}
 variable and stores the resulting odd-numbered value into the
 \co{rcu_reader_gp} per-thread variable.
 \Clnref{mb} executes a memory barrier to prevent the content of the



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux