On Wed, Jul 13, 2016 at 11:37:35PM +0900, Akira Yokosawa wrote: > On 2016/07/13 1:34, Paul E. McKenney wrote: > >On Wed, Jul 13, 2016 at 12:32:01AM +0900, Akira Yokosawa wrote: > >>Hi Paul, > >> > >>Thanks to your addition in Chapter 9, it is getting more and more > >>convincing. > > > >Glad you like them! A little more to go... > > > >>However, the X-axis label in Figures 9.5, 9.9, and 9.16 is > >>"Number of CPUs/Threads." > >>At first glance, I read it as "Number of CPUs per Threads." > >>It took a few seconds for me to realize that it means > >>"Number of CPUs or Threads." > >> > >>In Figure 4.10, the x-axis label is "Number of CPUs (Threads)." > >>I think this is better to avoid ambiguity. > >> > >>What do you think? > > > >Sold!!! ;-) > > > >Made this change with your Signed-off-by. Make that Reported-by... > Hi, > > So, I searched "CPUs/Threads" in the pdf and 10 other labels hit. > See the list below. (Sorry, but with page numbers of 2 column pdf.) > > * p.38 Figure 5.3 > * p.77 Figure 6.22 > * p.78 Figure 6.23 and in the text just below > * p.166 Figures 10.7, 10.8, and 10.9 > * p.168 Figures 10.13 and 10.14 > * p.169 Figure 10.15 > * p.177 Figure 10.29 > > These figures existed before your recent rewrite of Chapter 9, > and I never found them ambiguous. So I might have been on a > different planet last night or something... > If you think it is still better to modify them, please consider > doing so. Might as well... Done! > There is another "CPUs/Threads" in p.437 at the definition of > "Linearizable," but it is not at all ambiguous in the context. Fixed that one as well. And added glossary.tex to the Makefile dependency list. ;-) Thanx, Paul -- To unsubscribe from this list: send the line "unsubscribe perfbook" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
