Re: [PATCH -perfbook] cpu: add conj 'than' to a defective sentence

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

 



On Fri, Jun 18, 2021 at 12:17:49PM +0800, Zhouyi Zhou wrote:
> Hi Paul,
> 
> I guess '... represents a reasonably large system with no fewer
> 448 hardware threads' is a defective sentence, it needs a conj: 'than'.
> 
> Sorry for bothering you if I am wrong, because my English is not quite
> good :-)
> 
> Thanks
> Zhouyi
> 
> Signed-off-by: Zhouyi Zhou <zhouzhouyi@xxxxxxxxx> 

You are quite right, and thank you for spotting this and for sending the
patch!  I wordsmithed the commit log, in part to avoid the age-old debate
as to whether a given use of "than" is a conjunction or a preposition.
You probably had it right, but why take the chance?  ;-)

							Thanx, Paul

------------------------------------------------------------------------

commit 8fc0705251d3409591edfaf62a4373ffcec5794e
Author: Zhouyi Zhou <zhouzhouyi@xxxxxxxxx>
Date:   Fri Jun 18 12:17:49 2021 +0800

    cpu: Add 'than' to a defective sentence
    
    The sentence '... represents a reasonably large system with no fewer
    448 hardware threads' is missing a "than".  This commit adds it.
    
    Signed-off-by: Zhouyi Zhou <zhouzhouyi@xxxxxxxxx>
    Signed-off-by: Paul E. McKenney <paulmck@xxxxxxxxxx>

diff --git a/cpu/overheads.tex b/cpu/overheads.tex
index 4aeae6cd..28462d49 100644
--- a/cpu/overheads.tex
+++ b/cpu/overheads.tex
@@ -354,7 +354,7 @@ thousand clock cycles.
 	\cref{tab:cpu:CPU 0 View of Synchronization Mechanisms on 8-Socket System With Intel Xeon Platinum 8176 CPUs at 2.10GHz}
 	on
 	\cpageref{tab:cpu:CPU 0 View of Synchronization Mechanisms on 8-Socket System With Intel Xeon Platinum 8176 CPUs at 2.10GHz}
-	represents a reasonably large system with no fewer 448~hardware
+	represents a reasonably large system with no fewer than 448~hardware
 	threads.
 	Smaller systems often achieve better latency, as may be seen in
 	\cref{tab:cpu:Performance of Synchronization Mechanisms on 16-CPU 2.8GHz Intel X5550 (Nehalem) System},



[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