All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [GIT PULL] Cosmetic fixes part 2 and typo fixes
Date: Sun, 3 Jul 2016 17:10:47 -0700	[thread overview]
Message-ID: <20160704001047.GI4650@linux.vnet.ibm.com> (raw)
In-Reply-To: <014c725d-2252-5892-ab4c-1621d89e2768@gmail.com>

On Sat, Jul 02, 2016 at 08:26:31PM +0900, Akira Yokosawa wrote:
> Hi Paul,
> 
> Here is part 2 of cosmetic fixes.
> Most commits do the same changes I did in the previous patch series.
> I also did some typo fixes in the last 3 commits.
> Most of them are trivial fixes, but there may be some wrong ones.
> Please review them with skeptical eyes.

They look good, thank you!

I applied the patch shown below to the quick-quiz answer -- it needed
a complete rewrite rather than wordsmithing.  Thank you for calling my
attention to it.

							Thanx, Paul

>                                             Thanks, Akira
> 
> ---
> The following changes since commit 91ac20e1ab95767f1e33b84aee474d0cd38cc748:
> 
>   hlist_for_each_entry_rcu() now has only three arguments (2016-07-01 13:31:02 -0700)
> 
> are available in the git repository at:
> 
>   https://github.com/akiyks/perfbook.git cosmetic-fixes-and-typo-fixes
> 
> for you to fetch changes up to 644ca3567d857cf2059f5820d30d4fd472f142c8:
> 
>   howto: Miscellaneous fixes (2016-07-02 20:08:43 +0900)
> 
> ----------------------------------------------------------------
> Akira Yokosawa (24):
>       locking: Centering figures of verbatim code examples
>       locking: Use \path command for path names
>       defer: Centering figures of verbatim code examples
>       defer: Tweak font size of Figure 9.5
>       defer: Align left margins of Figurs 9.28, 9.29 and 9.30
>       defer: Use \path command for path names
>       Use \co{} properly to give room for line-breaks
>       datastruct: Centering figures of verbatim code examples
>       datastruct: Fix indent of Figure 'Alignment for 64-Byte Cache Lines'
>       datastruct: Use \path command for path names
>       debugging: Centering figures of verbatim code examples
>       debugging: Use \path command for path names
>       debugging: Enclose shell command example in \co{} macro
>       formal: Centering figures of verbatim code examples
>       formal: Use \path command for path names
>       together: Centering figures of verbatim code examples
>       advsync: Centering figures of verbatim code examples
>       advsync: Add \centering in some tables
>       advsync: Use \path command for path names
>       rt: Centering figures of verbatim code examples
>       rt: Use \path{} for path names
>       rt: Consistently use 'real-time' as adjective
>       rt & advsync: Typo fixes
>       howto: Miscellaneous fixes
> 
>  SMPdesign/SMPdesign.tex              |   4 +-
>  advsync/advsync.tex                  |  10 +-
>  advsync/memorybarriers.tex           |  34 ++++---
>  advsync/rcu.tex                      |  38 +++----
>  appendix/primitives/primitives.tex   |   8 +-
>  appendix/rcuimpl/srcu.tex            |  38 +++----
>  appendix/whymb/whymemorybarriers.tex | 102 +++++++++----------
>  datastruct/datastruct.tex            | 132 +++++++++++++++----------
>  debugging/debugging.tex              |  24 +++--
>  defer/hazptr.tex                     |   8 +-
>  defer/rcuapi.tex                     |  20 ++--
>  defer/rcuexercises.tex               |   2 +-
>  defer/rcufundamental.tex             |  60 ++++++-----
>  defer/rcuusage.tex                   |  56 ++++++-----
>  defer/refcnt.tex                     |  34 ++++---
>  defer/seqlock.tex                    |  26 +++--
>  defer/toyrcu.tex                     | 186 +++++++++++++++++++++--------------
>  formal/axiomatic.tex                 |  18 ++--
>  formal/dyntickrcu.tex                |  88 ++++++++++-------
>  formal/ppcmem.tex                    |  32 +++---
>  formal/sat.tex                       |   2 +-
>  formal/spinhint.tex                  | 142 +++++++++++++++-----------
>  howto/howto.tex                      |  16 +--
>  locking/locking-existence.tex        |  16 +--
>  locking/locking.tex                  |  66 ++++++++-----
>  rt/rt.tex                            |  76 +++++++-------
>  together/applyrcu.tex                |  48 +++++----
>  27 files changed, 750 insertions(+), 536 deletions(-)
> 


      reply	other threads:[~2016-07-04  0:10 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-02 11:26 [GIT PULL] Cosmetic fixes part 2 and typo fixes Akira Yokosawa
2016-07-04  0:10 ` 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=20160704001047.GI4650@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.