Re: [PATCH tip/core/rcu 3/3] rcu: Fix occurrence of "the the" in checklist.txt

Michael Opdenacker
2013-09-25T18:22:10Z

On 09/25/2013 05:33 PM, Josh Triplett wrote:
> On Wed, Sep 25, 2013 at 06:55:27AM -0700, Paul E. McKenney wrote:
>> On Tue, Sep 24, 2013 at 08:47:39PM -0700, Josh Triplett wrote:
>>> On Tue, Sep 24, 2013 at 06:17:23PM -0700, Paul E. McKenney wrote:
>>>> From: Michael Opdenacker <michael.opdenacker@free-electrons.com>
>>>>
>>>> Signed-off-by: Michael Opdenacker <michael.opdenacker@free-electrons.com>
>>>> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
>>>> ---
>>> See comment below.
>>>
>>>> Documentation/RCU/checklist.txt | 2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/Documentation/RCU/checklist.txt b/Documentation/RCU/checklist.txt
>>>> index 7703ec7..ad6cba4 100644
>>>> --- a/Documentation/RCU/checklist.txt
>>>> +++ b/Documentation/RCU/checklist.txt
>>>> @@ -203,7 +203,7 @@ over a rather long period of time, but improvements are always welcome!
>>>> the corresponding readers must disable preemption, possibly
>>>> by calling rcu_read_lock_sched() and rcu_read_unlock_sched().
>>>> If the updater uses synchronize_srcu() or call_srcu(),
>>>> - the the corresponding readers must use srcu_read_lock() and
>>>> + the corresponding readers must use srcu_read_lock() and
>>> I think this may have actually been a typo for "then", as in "If the
>>> updater uses ..., then the corresponding readers must use ...".
>> Good point -- that would make the sentence construction parallel with
>> the surrounding sentences. Updated patch below.
>>
>> Thanx, Paul
Good catch! Thanks!

Michael.

--
Michael Opdenacker, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
+33 484 258 098