From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH 0/5] Update table layout
Date: Sun, 22 Oct 2017 19:24:30 -0700 [thread overview]
Message-ID: <20171023022430.GJ3659@linux.vnet.ibm.com> (raw)
In-Reply-To: <6f0085b1-ca8e-d6f7-e613-1b1c8d119af6@gmail.com>
On Mon, Oct 23, 2017 at 12:20:14AM +0900, Akira Yokosawa wrote:
> >From d224db4bfed5d4ce0639062a920bafc6f9442e16 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Sun, 22 Oct 2017 23:55:49 +0900
> Subject: [PATCH 0/5] Update table layout
>
> Hi Paul,
>
> My first idea was to add a make target for "table captions at the top".
> But I changed my mind and promoted the layout as default.
> Instead, I added a target "perfbook-tcb.pdf" (tcb: Table Caption Bottom),
> which keeps the caption at the bottom.
>
> Patch #1 does the change in perfbook.tex and Makefile.
>
> Patches #2 and #3 are minor fixes I noticed while updating tables.
>
> Patch #4 updates most tables to use the new table layout experimented in
> Style Guide for a while. It contains changes to use "S" column specifier
> provided by the "siunitx" package.
>
> Patch #5 converts a table into a figure drawn by tabular.
>
> There remains several tables not yet touched. They are complex and large
> ones. They need extra work to use the new scheme.
>
> NOTE: Table 13.1 does not contain legends and does not use subscripts
> as shown in Table D.5 at the moment. In the text, it looks OK to me.
>
> Thoughts?
At a first scan, this all looks good. There are a few remaining
tables in the old format, including a couple that we discussed earlier
as being quite complex. Which should be fine, for the time being,
at least. But what are your thoughts on the remaining old-style tables?
Queued and pushed, thank you!
Thanx, Paul
> Thanks, Akira
> --
> Akira Yokosawa (5):
> Move caption of table to top
> Define \twocolumnwidth for 1c layout
> Use \O{} macro for 'order-of'
> Update tables using booktabs and alternate-row coloring scheme
> datastruct: Convert 'NUMA Topology of System Under Test' to figure
>
> Makefile | 6 ++-
> SMPdesign/SMPdesign.tex | 10 +++--
> advsync/rt.tex | 8 ++--
> appendix/questions/after.tex | 25 +++++++++----
> appendix/styleguide/styleguide.tex | 20 ++++++----
> count/count.tex | 54 ++++++++++++++++-----------
> cpu/overheads.tex | 76 +++++++++++++++++---------------------
> datastruct/datastruct.tex | 23 ++++++++----
> defer/rcuapi.tex | 16 ++++----
> defer/rcufundamental.tex | 16 ++++----
> defer/rcuusage.tex | 10 +++--
> defer/whichtochoose.tex | 27 +++++---------
> formal/spinhint.tex | 68 +++++++++++++++++-----------------
> future/QC.tex | 75 +++++++++++++++++++------------------
> memorder/memorder.tex | 8 ++--
> perfbook.tex | 17 +++++++++
> together/refcnt.tex | 25 +++++++------
> 17 files changed, 266 insertions(+), 218 deletions(-)
>
> --
> 2.7.4
>
next prev parent reply other threads:[~2017-10-23 2:24 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-22 15:20 [PATCH 0/5] Update table layout Akira Yokosawa
2017-10-22 15:22 ` [PATCH 1/5] Move caption of table to top Akira Yokosawa
2017-10-22 15:24 ` [PATCH 2/5] Define \twocolumnwidth for 1c layout Akira Yokosawa
2017-10-22 15:25 ` [PATCH 3/5] Use \O{} macro for 'order-of' Akira Yokosawa
2017-10-22 15:26 ` [PATCH 4/5] Update tables using booktabs and alternate-row coloring scheme Akira Yokosawa
2017-10-22 15:28 ` [PATCH 5/5] datastruct: Convert 'NUMA Topology of System Under Test' to figure Akira Yokosawa
2017-10-23 2:24 ` Paul E. McKenney [this message]
2017-10-23 15:25 ` [PATCH 0/5] Update table layout Akira Yokosawa
2017-10-23 17:49 ` Paul E. McKenney
2017-10-24 13:50 ` Akira Yokosawa
2017-10-25 19:19 ` Paul E. McKenney
2017-10-23 15:27 ` [PATCH 1/3] whymb: " Akira Yokosawa
2017-10-23 15:29 ` [PATCH 2/3] defer/rcuapi: " Akira Yokosawa
2017-10-23 15:30 ` [PATCH 3/3] Convert table of code fragments to 'listing' Akira Yokosawa
2017-10-23 18:00 ` [PATCH 1/3] whymb: Update table layout Paul E. McKenney
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171023022430.GJ3659@linux.vnet.ibm.com \
--to=paulmck@linux.vnet.ibm.com \
--cc=akiyks@gmail.com \
--cc=perfbook@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.