All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH-perfbook] count: Fix typos
@ 2022-01-29 22:18 Johann Klähn
  2022-01-31  1:55 ` Paul E. McKenney
  0 siblings, 1 reply; 4+ messages in thread
From: Johann Klähn @ 2022-01-29 22:18 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook

Signed-off-by: Johann Klähn <johann@jklaehn.de>
---
Hi Paul,

I'm really enjoying the book and hope that I can contribute back a little bit in
the form of small fixes.  As this is my first patch to this list, please point
out anything I could improve in the future (e.g., granularity of patches).

Thanks
Johann
---
 count/count.tex | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/count/count.tex b/count/count.tex
index d2b39904..f4e74c13 100644
--- a/count/count.tex
+++ b/count/count.tex
@@ -3237,7 +3237,7 @@ Summarizing the summary:
 	\Cref{fig:count:Atomic Increment Scalability on x86}
 	illustrates this point:
 	Atomic increment might be completely acceptable for a two-CPU
-	system, but be completely inadequate for an eight-CPU system.
+	system, but can be completely inadequate for an eight-CPU system.
 \end{enumerate}
 
 \begin{figure}
@@ -3251,7 +3251,7 @@ Summarizing still further, we have the ``big three'' methods of
 increasing performance and scalability, namely
 (1)~\emph{partitioning} over CPUs or threads,
 (2)~\emph{batching} so that more work can be done by each expensive
-synchronization operations, and
+synchronization operation, and
 (3)~\emph{weakening} synchronization operations where feasible.
 As a rough rule of thumb, you should apply these methods in this order,
 as was noted earlier in the discussion of
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH-perfbook] count: Fix typos
  2022-01-29 22:18 [PATCH-perfbook] count: Fix typos Johann Klähn
@ 2022-01-31  1:55 ` Paul E. McKenney
  2022-01-31 19:12   ` Johann Klähn
  0 siblings, 1 reply; 4+ messages in thread
From: Paul E. McKenney @ 2022-01-31  1:55 UTC (permalink / raw)
  To: Johann Klähn; +Cc: perfbook

On Sat, Jan 29, 2022 at 11:18:15PM +0100, Johann Klähn wrote:
> Signed-off-by: Johann Klähn <johann@jklaehn.de>
> ---
> Hi Paul,
> 
> I'm really enjoying the book and hope that I can contribute back a little bit in
> the form of small fixes.  As this is my first patch to this list, please point
> out anything I could improve in the future (e.g., granularity of patches).

Queued and pushed, thank you!  I did take the liberty of changing your
added "can" to a "nevertheless".  Does that work for you?

							Thanx, Paul

> Thanks
> Johann
> ---
>  count/count.tex | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/count/count.tex b/count/count.tex
> index d2b39904..f4e74c13 100644
> --- a/count/count.tex
> +++ b/count/count.tex
> @@ -3237,7 +3237,7 @@ Summarizing the summary:
>  	\Cref{fig:count:Atomic Increment Scalability on x86}
>  	illustrates this point:
>  	Atomic increment might be completely acceptable for a two-CPU
> -	system, but be completely inadequate for an eight-CPU system.
> +	system, but can be completely inadequate for an eight-CPU system.
>  \end{enumerate}
>  
>  \begin{figure}
> @@ -3251,7 +3251,7 @@ Summarizing still further, we have the ``big three'' methods of
>  increasing performance and scalability, namely
>  (1)~\emph{partitioning} over CPUs or threads,
>  (2)~\emph{batching} so that more work can be done by each expensive
> -synchronization operations, and
> +synchronization operation, and
>  (3)~\emph{weakening} synchronization operations where feasible.
>  As a rough rule of thumb, you should apply these methods in this order,
>  as was noted earlier in the discussion of
> -- 
> 2.34.1
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH-perfbook] count: Fix typos
  2022-01-31  1:55 ` Paul E. McKenney
@ 2022-01-31 19:12   ` Johann Klähn
  2022-02-01 17:57     ` Paul E. McKenney
  0 siblings, 1 reply; 4+ messages in thread
From: Johann Klähn @ 2022-01-31 19:12 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook

On Sun, Jan 30, 2022 at 17:55 -0800, Paul E. McKenney wrote:
> Queued and pushed, thank you!  I did take the liberty of changing your
> added "can" to a "nevertheless".  Does that work for you?

Sure, in that case maybe also remove the second “be”?
| Atomic increment might be completely acceptable for a two-CPU
| system, but nevertheless completely inadequate for an eight-CPU system.

Thanks
Johann

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH-perfbook] count: Fix typos
  2022-01-31 19:12   ` Johann Klähn
@ 2022-02-01 17:57     ` Paul E. McKenney
  0 siblings, 0 replies; 4+ messages in thread
From: Paul E. McKenney @ 2022-02-01 17:57 UTC (permalink / raw)
  To: Johann Klähn; +Cc: perfbook

On Mon, Jan 31, 2022 at 08:12:46PM +0100, Johann Klähn wrote:
> On Sun, Jan 30, 2022 at 17:55 -0800, Paul E. McKenney wrote:
> > Queued and pushed, thank you!  I did take the liberty of changing your
> > added "can" to a "nevertheless".  Does that work for you?
> 
> Sure, in that case maybe also remove the second “be”?
> | Atomic increment might be completely acceptable for a two-CPU
> | system, but nevertheless completely inadequate for an eight-CPU system.

Good point in that the sentence is still a bit awkward.  So what I have
done is to add a LaTeX comment that will remind me to think harder
about it.  If I don't get a good replacement by (say) March, please
remind me about it.

							Thanx, Paul

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-02-01 17:57 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-29 22:18 [PATCH-perfbook] count: Fix typos Johann Klähn
2022-01-31  1:55 ` Paul E. McKenney
2022-01-31 19:12   ` Johann Klähn
2022-02-01 17:57     ` Paul E. McKenney

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.