Re: [GIT PULL -perfbook] Minor fixes and update for Edition 2, take 5

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

 



On Sat, Mar 20, 2021 at 08:34:01AM +0900, Akira Yokosawa wrote:
> On Fri, 19 Mar 2021 12:21:36 -0700, Paul E. McKenney wrote:
> > On Sat, Mar 20, 2021 at 12:01:45AM +0900, Akira Yokosawa wrote:
> >> Hi Paul,
> >>
> >> Today's pull request consists of minor typo fixes and an update only.
> >>
> >> However, I'm not sure if Patches 6/7 and 7/7 are really the right fixes
> >> or not.
> >> So please have a closer look at them.
> > 
> > They look good, thank you!  I have pulled this series and pushed it out.
> > 
> >>         Thanks, Akira
> >>
> >> (Please find a PS. part at the bottom.)
> >>
> >> ---------------
> >> The following changes since commit 82599ee517da15ce3957bc94b82f8702a2357d14:
> >>
> >>   memorder: Fix typos and grammar, take two (2021-03-18 17:29:17 -0700)
> >>
> >> are available in the Git repository at:
> >>
> >>   https://github.com/akiyks/perfbook.git tags/for-paul-2021.03.19a
> >>
> >> for you to fetch changes up to 0b1692098fb87a4c1d5f2fec9c078ee4462aac85:
> >>
> >>   advsync: Use Oxford comma (2021-03-19 22:19:10 +0900)
> >>
> >> ----------------------------------------------------------------
> >> Akira Yokosawa (7):
> >>       memorder: Fix staccato of 'another'
> >>       count: Fix typo
> >>       count: Move ',' to proper position
> >>       toolsoftrade: Fix typo
> >>       toolstotrade: Use path of atomic_t.txt under Linux-kernel source tree
> >>       advsync: Fix typo
> >>       advsync: Use Oxford comma
> >>
> >>  advsync/advsync.tex           | 4 ++--
> >>  count/count.tex               | 6 +++---
> >>  memorder/memorder.tex         | 2 +-
> >>  toolsoftrade/toolsoftrade.tex | 8 +++++---
> >>  4 files changed, 11 insertions(+), 9 deletions(-)
> >>
> >> ---------------------
> >> PS.
> >>
> >> Looking at the Glossaries, I thought entries of "Concurrent"
> >> and "Parallel" were missing.
> >>
> >> And you use these words almost interchangeably.
> >>
> >> (In-)distinction of them is the topic of Appendix A.2.
> >>
> >> So maybe it might help readers to put a footnote near the first use
> >> of "parallel" telling that in this book, parallel and concurrent are
> >> used as synonyms, and cite Appendix A.2.
> >>
> >> To make this point even clearer, adding entries like
> >>
> >>   Concurrent: In this book, synonym of parallel.
> >>
> >>   Parallel: In this book, synonym of concurrent.
> >>
> >> in the Glossaries might help readers.
> >>
> >> Thoughts?
> > 
> > Like this?
> 
> Looks good to me.

Thank you, pushed!  I also pushed the last change from my review of all
of my year-2021 changes.  How are things at your end?

							Thanx, Paul

>         Thanks, Akira
> 
> > 
> > 							Thanx, Paul
> > 
> > ------------------------------------------------------------------------
> > 
> > diff --git a/glossary.tex b/glossary.tex
> > index da54482..c6c9c62 100644
> > --- a/glossary.tex
> > +++ b/glossary.tex
> > @@ -145,6 +145,12 @@
> >  \item[Communication Miss:]\index{Communication miss}
> >  	A cache miss incurred because some other CPU has written to
> >  	the cache line since the last time this CPU accessed it.
> > +\item[Concurrent:]\index{Concurrent}
> > +	In this book, a synonym of parallel.
> > +	Please see \cref{sec:app:questions:What is the Difference Between ``Concurrent'' and ``Parallel''?}
> > +	on \cpageref{sec:app:questions:What is the Difference Between ``Concurrent'' and ``Parallel''?}
> > +	for a discussion of the recent distinction between these two
> > +	terms.
> >  \item[Critical Section:]\index{Critical section}
> >  	A section of code guarded by some synchronization mechanism,
> >  	so that its execution constrained by that primitive.
> > @@ -348,6 +354,12 @@
> >  	directly to the work that must be accomplished.
> >  	For example, lock acquisition and release is normally considered
> >  	to be overhead, and specifically to be synchronization overhead.
> > +\item[Parallel:]\index{Parallel}
> > +	In this book, a synonym of concurrent.
> > +	Please see \cref{sec:app:questions:What is the Difference Between ``Concurrent'' and ``Parallel''?}
> > +	on \cpageref{sec:app:questions:What is the Difference Between ``Concurrent'' and ``Parallel''?}
> > +	for a discussion of the recent distinction between these two
> > +	terms.
> >  \item[Performance:]\index{Performance}
> >  	Rate at which work is done, expressed as work per unit time.
> >  	If this work is fully serialized, then the performance will
> > 



[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