Re: [PATCH memory-model 3/3] tools/memory-model: Add KCSAN LF mentorship session citation

[Akira Yokosawa <akiyks@gmail.com>]

On 2024/06/09 0:48, Paul E. McKenney wrote:
> On Sat, Jun 08, 2024 at 08:38:12AM +0900, Akira Yokosawa wrote:
>> On 2024/06/05 13:02, Paul E. McKenney wrote:
>>> On Wed, Jun 05, 2024 at 10:57:27AM +0900, Akira Yokosawa wrote:
>>>> On Tue,  4 Jun 2024 15:14:19 -0700, Paul E. McKenney wrote:
>>>>> Add a citation to Marco's LF mentorship session presentation entitled
>>>>> "The Kernel Concurrency Sanitizer"
>>>>>
>>>>> [ paulmck: Apply Marco Elver feedback. ]
>>>>>
>>>>> Reported-by: Marco Elver <elver@google.com>
>>>>> Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
>>>>> Cc: Alan Stern <stern@rowland.harvard.edu>
>>>>> Cc: Andrea Parri <parri.andrea@gmail.com>
>>>>> Cc: Will Deacon <will@kernel.org>
>>>>> Cc: Peter Zijlstra <peterz@infradead.org>
>>>>> Cc: Boqun Feng <boqun.feng@gmail.com>
>>>>> Cc: Nicholas Piggin <npiggin@gmail.com>
>>>>> Cc: David Howells <dhowells@redhat.com>
>>>>> Cc: Jade Alglave <j.alglave@ucl.ac.uk>
>>>>> Cc: Luc Maranget <luc.maranget@inria.fr>
>>>>> Cc: Akira Yokosawa <akiyks@gmail.com>
>>>>
>>>> Paul,
>>>>
>>>> While reviewing this, I noticed that
>>>> tools/memory-model/Documentation/README has no mention of
>>>> access-marking.txt.
>>>>
>>>> It has no mention of glossary.txt or locking.txt, either.
>>>>
>>>> I'm not sure where are the right places in README for them.
>>>> Can you update it in a follow-up change?
>>>>
>>>> Anyway, for this change,
>>>>
>>>> Reviewed-by: Akira Yokosawa <akiyks@gmail.com>
>>>
>>> Thank you, and good catch!  Does the patch below look appropriate?
>>
>> Well, I must say this is not what I expected.
>> Please see below.
> 
> OK, I was clearly in way too much of a hurry when doing this, and please
> accept my apologies for my inattention.  I am therefore going to do
> what I should have done in the first place, which is to ask you if you
> would like to send a patch fixing this.  If so, I would be quite happy
> to replace mine with yours.

OK.
I think I can submit a draft patch after fixing perfbook's build error
caused by changes in core LaTeX packages released last week.

Can you wait for a while ?

        Thanks, Akira