From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH 00/10] Tweaks to follow guidelines in style guide
Date: Thu, 5 Oct 2017 13:48:31 -0700 [thread overview]
Message-ID: <20171005204831.GX3521@linux.vnet.ibm.com> (raw)
In-Reply-To: <d36d262c-2fe4-284c-7818-516a4ad59ec7@gmail.com>
On Fri, Oct 06, 2017 at 12:47:21AM +0900, Akira Yokosawa wrote:
> >From 2890e0069882321553c16aac213d4bb8d0a06fb7 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Wed, 5 Oct 2017 22:57:46 +0900
> Subject: [PATCH 00/10] Tweaks to follow guidelines in style guide
>
> Hi Paul,
>
> This patch set consists of minor tweaks in regard to the suggestions
> having been presented in style guide for a while.
>
> Patches #1 -- #5 are trivial changes.
>
> Patch #6 attempts to improve consistency in denoting POWER series CPU
> by defining a macro "\Power{}".
>
> Patch #7 substitutes "GCC" for "gcc". There are a few exceptions as
> mentioned in commit log.
>
> Patch #8 substitutes "IRQ" for "irq" in the same way. You might like
> to skip this one, as I see "irq" more often than "IRQ" in Linux
> documentations.
>
> Patch #9 is somewhat invasive. It switches Times font to that of
> "newtxtext" and "newtxmath" packages. The reason of the change
> is to have access to both upright and slated glyphs of Greek letters.
> Recent versions of these font packages give better looking result,
> especially in math mode. As noted in the commit log, newtxmath in
> TeX Live 2013/Debian has a few issues which have been fixed in later
> versions. It also switches font choice for the experimental target "1csf".
>
> Patch #10 updates style guide to reflect the changes made in this
> patch set.
They look fine, so I applied them, thank you! We might want to go with
"irq", but let's see how people react. Easy to change, aside from
beginnings of sentences!
Thanx, Paul
> Thanks, Akira
> --
> Akira Yokosawa (10):
> debugging: Insert narrow space in front of percent symbol
> debugging: Use upright font for Euler's number
> future/QC: Insert narrow space in front of percent symbol
> future/QC: Use non-breakable hyphen for axis names
> treewide: Insert narrow space in front of percent symbol
> treewide: Use \Power{} macro for POWER CPU family
> treewide: Call GNU C compiler as "GCC"
> treewide: Use "IRQ" instead of "irq" used as abbreviation
> future/QC: Use upright glyph for math constant and descriptive suffix
> styleguide: Reflect recent style improvements
>
> FAQ-BUILD.txt | 4 +-
> Makefile | 2 +-
> SMPdesign/SMPdesign.tex | 2 +-
> SMPdesign/beyond.tex | 14 ++---
> advsync/advsync.tex | 2 +-
> appendix/styleguide/styleguide.tex | 64 ++++++++---------------
> appendix/toyrcu/toyrcu.tex | 26 +++++-----
> count/count.tex | 28 +++++-----
> cpu/hwfreelunch.tex | 4 +-
> datastruct/datastruct.tex | 2 +-
> debugging/debugging.tex | 104 ++++++++++++++++++-------------------
> defer/rcuapi.tex | 6 +--
> defer/rcuusage.tex | 4 +-
> formal/dyntickrcu.tex | 52 +++++++++----------
> formal/formal.tex | 2 +-
> formal/spinhint.tex | 2 +-
> future/QC.tex | 92 ++++++++++++++++----------------
> future/htm.tex | 2 +-
> future/tm.tex | 4 +-
> intro/intro.tex | 8 +--
> memorder/memorder.tex | 32 ++++++------
> perfbook.tex | 20 +++++--
> rt/rt.tex | 22 ++++----
> toolsoftrade/toolsoftrade.tex | 24 ++++-----
> 24 files changed, 257 insertions(+), 265 deletions(-)
>
> --
> 2.7.4
>
>
prev parent reply other threads:[~2017-10-05 20:48 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-05 15:47 [PATCH 00/10] Tweaks to follow guidelines in style guide Akira Yokosawa
2017-10-05 15:48 ` [PATCH 01/10] debugging: Insert narrow space in front of percent symbol Akira Yokosawa
2017-10-05 15:49 ` [PATCH 02/10] debugging: Use upright font for Euler's number Akira Yokosawa
2017-10-05 15:51 ` [PATCH 03/10] future/QC: Insert narrow space in front of percent symbol Akira Yokosawa
2017-10-05 15:52 ` [PATCH 04/10] future/QC: Use non-breakable hyphen for axis names Akira Yokosawa
2017-10-05 15:53 ` [PATCH 05/10] treewide: Insert narrow space in front of percent symbol Akira Yokosawa
2017-10-05 15:54 ` [PATCH 06/10] treewide: Use \Power{} macro for POWER CPU family Akira Yokosawa
2017-10-05 15:55 ` [PATCH 07/10] treewide: Call GNU C compiler as "GCC" Akira Yokosawa
2017-10-05 15:56 ` [PATCH 08/10] treewide: Use "IRQ" instead of "irq" used as abbreviation Akira Yokosawa
2017-10-05 15:59 ` [PATCH 09/10] future/QC: Use upright glyph for math constant and descriptive suffix Akira Yokosawa
2017-10-05 16:00 ` [PATCH 10/10] styleguide: Reflect recent style improvements Akira Yokosawa
2017-10-05 20:48 ` Paul E. McKenney [this message]
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=20171005204831.GX3521@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox