* [PATCH 0/4] count: Get rid of ACCESS_ONCE
@ 2017-12-23 3:42 Akira Yokosawa
2017-12-23 3:43 ` [PATCH 1/4] count: Get rid of ACCESS_ONCE() in code snippet Akira Yokosawa
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: Akira Yokosawa @ 2017-12-23 3:42 UTC (permalink / raw)
To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa
Hi Paul,
This patch set removes ACCESS_ONCE() in Chapter 5.
(Yes, I saw the update in todo.txt.)
Patch #1 updates code snippets and code under CodeSamples/count/.
Patch #2 updates text in Chapter 5 as well as Section 4.2.5 to adjust
context. Citations in QQA to LWN article "ACCESS_ONCE()" is
replaced by footnotes referencing definition in Section 4.2.5.
Patch #3 adds a couple of git commits to bibliography to be cited
by Patch #4.
Patch #4 adds a Quick Quiz in Section 4.2.5 to put a brief history of
ACCESS_ONCE().
As ACCESS_ONCE() has been removed during Linux 4.15 merge window,
it would be a good chance to put some memorial words in perfbook.
In patch #4, I put them in a QQA. An alternative would be to
put them in a dedicated section on "compiler barriers" in Section 4.2 or 4.3.
In any case, the wording of the brief history needs some polishing
by the inventor of ACCESS_ONCE(). So I don't expect patch #4 to be
applied as is.
BTW, I thought twice about citing the git commit adding ACCESS_ONCE()
in RCU code because of its title. Are you OK with it being listed in
bibliography?
Thanks, Akira
--
Akira Yokosawa (4):
count: Get rid of ACCESS_ONCE() in code snippet
count: Get rid of ACCESS_ONCE() in text
Update bibliography
toolsoftrade: Add quick quiz on ACCESS_ONCE()
CodeSamples/count/count_lim_sig.c | 26 ++++++++++-----------
CodeSamples/count/counttorture.h | 8 +++----
CodeSamples/count/limtorture.h | 18 +++++++-------
bib/RCU.bib | 33 +++++++++++++++++++++++++-
count/count.tex | 45 ++++++++++++++++++++---------------
toolsoftrade/toolsoftrade.tex | 49 ++++++++++++++++++++++++++++-----------
6 files changed, 119 insertions(+), 60 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/4] count: Get rid of ACCESS_ONCE() in code snippet
2017-12-23 3:42 [PATCH 0/4] count: Get rid of ACCESS_ONCE Akira Yokosawa
@ 2017-12-23 3:43 ` Akira Yokosawa
2017-12-23 3:44 ` [PATCH 2/4] count: Get rid of ACCESS_ONCE() in text Akira Yokosawa
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Akira Yokosawa @ 2017-12-23 3:43 UTC (permalink / raw)
To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/4] count: Get rid of ACCESS_ONCE() in text
2017-12-23 3:42 [PATCH 0/4] count: Get rid of ACCESS_ONCE Akira Yokosawa
2017-12-23 3:43 ` [PATCH 1/4] count: Get rid of ACCESS_ONCE() in code snippet Akira Yokosawa
@ 2017-12-23 3:44 ` Akira Yokosawa
2017-12-23 3:45 ` [PATCH 3/4] Update bibliography Akira Yokosawa
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Akira Yokosawa @ 2017-12-23 3:44 UTC (permalink / raw)
To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 3/4] Update bibliography
2017-12-23 3:42 [PATCH 0/4] count: Get rid of ACCESS_ONCE Akira Yokosawa
2017-12-23 3:43 ` [PATCH 1/4] count: Get rid of ACCESS_ONCE() in code snippet Akira Yokosawa
2017-12-23 3:44 ` [PATCH 2/4] count: Get rid of ACCESS_ONCE() in text Akira Yokosawa
@ 2017-12-23 3:45 ` Akira Yokosawa
2017-12-23 3:46 ` [PATCH 4/4] toolsoftrade: Add quick quiz on ACCESS_ONCE() Akira Yokosawa
2018-01-04 0:18 ` [PATCH 0/4] count: Get rid of ACCESS_ONCE Paul E. McKenney
4 siblings, 0 replies; 6+ messages in thread
From: Akira Yokosawa @ 2017-12-23 3:45 UTC (permalink / raw)
To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 4/4] toolsoftrade: Add quick quiz on ACCESS_ONCE()
2017-12-23 3:42 [PATCH 0/4] count: Get rid of ACCESS_ONCE Akira Yokosawa
` (2 preceding siblings ...)
2017-12-23 3:45 ` [PATCH 3/4] Update bibliography Akira Yokosawa
@ 2017-12-23 3:46 ` Akira Yokosawa
2018-01-04 0:18 ` [PATCH 0/4] count: Get rid of ACCESS_ONCE Paul E. McKenney
4 siblings, 0 replies; 6+ messages in thread
From: Akira Yokosawa @ 2017-12-23 3:46 UTC (permalink / raw)
To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/4] count: Get rid of ACCESS_ONCE
2017-12-23 3:42 [PATCH 0/4] count: Get rid of ACCESS_ONCE Akira Yokosawa
` (3 preceding siblings ...)
2017-12-23 3:46 ` [PATCH 4/4] toolsoftrade: Add quick quiz on ACCESS_ONCE() Akira Yokosawa
@ 2018-01-04 0:18 ` Paul E. McKenney
4 siblings, 0 replies; 6+ messages in thread
From: Paul E. McKenney @ 2018-01-04 0:18 UTC (permalink / raw)
To: Akira Yokosawa; +Cc: perfbook
On Sat, Dec 23, 2017 at 12:42:04PM +0900, Akira Yokosawa wrote:
> Hi Paul,
>
> This patch set removes ACCESS_ONCE() in Chapter 5.
> (Yes, I saw the update in todo.txt.)
>
> Patch #1 updates code snippets and code under CodeSamples/count/.
> Patch #2 updates text in Chapter 5 as well as Section 4.2.5 to adjust
> context. Citations in QQA to LWN article "ACCESS_ONCE()" is
> replaced by footnotes referencing definition in Section 4.2.5.
> Patch #3 adds a couple of git commits to bibliography to be cited
> by Patch #4.
> Patch #4 adds a Quick Quiz in Section 4.2.5 to put a brief history of
> ACCESS_ONCE().
>
> As ACCESS_ONCE() has been removed during Linux 4.15 merge window,
> it would be a good chance to put some memorial words in perfbook.
> In patch #4, I put them in a QQA. An alternative would be to
> put them in a dedicated section on "compiler barriers" in Section 4.2 or 4.3.
>
> In any case, the wording of the brief history needs some polishing
> by the inventor of ACCESS_ONCE(). So I don't expect patch #4 to be
> applied as is.
>
> BTW, I thought twice about citing the git commit adding ACCESS_ONCE()
> in RCU code because of its title. Are you OK with it being listed in
> bibliography?
I queued these, thank you!
I reworked the answer to the new quick quiz. We might want to revisit
the bibliography entries, but we are very likely going to have to cite
git commits, so we might as well get started. ;-)
Thanx, Paul
> Thanks, Akira
> --
> Akira Yokosawa (4):
> count: Get rid of ACCESS_ONCE() in code snippet
> count: Get rid of ACCESS_ONCE() in text
> Update bibliography
> toolsoftrade: Add quick quiz on ACCESS_ONCE()
>
> CodeSamples/count/count_lim_sig.c | 26 ++++++++++-----------
> CodeSamples/count/counttorture.h | 8 +++----
> CodeSamples/count/limtorture.h | 18 +++++++-------
> bib/RCU.bib | 33 +++++++++++++++++++++++++-
> count/count.tex | 45 ++++++++++++++++++++---------------
> toolsoftrade/toolsoftrade.tex | 49 ++++++++++++++++++++++++++++-----------
> 6 files changed, 119 insertions(+), 60 deletions(-)
>
> --
> 2.7.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe perfbook" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-01-04 0:17 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-23 3:42 [PATCH 0/4] count: Get rid of ACCESS_ONCE Akira Yokosawa
2017-12-23 3:43 ` [PATCH 1/4] count: Get rid of ACCESS_ONCE() in code snippet Akira Yokosawa
2017-12-23 3:44 ` [PATCH 2/4] count: Get rid of ACCESS_ONCE() in text Akira Yokosawa
2017-12-23 3:45 ` [PATCH 3/4] Update bibliography Akira Yokosawa
2017-12-23 3:46 ` [PATCH 4/4] toolsoftrade: Add quick quiz on ACCESS_ONCE() Akira Yokosawa
2018-01-04 0:18 ` [PATCH 0/4] count: Get rid of ACCESS_ONCE Paul E. McKenney
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox