mirror of https://lore.kernel.org/perfbook/git/0.git
 help / color / mirror / Atom feed
From: "Paul E. McKenney" <paulmck@kernel.org>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH -perfbook 2/4] memorder: Add index markers for 'address/control/data dependency', take 2
Date: Tue, 26 Sep 2023 11:04:16 -0700	[thread overview]
Message-ID: <e1f008ec-1358-4fd7-ab50-abc0fb72439e@paulmck-laptop> (raw)
In-Reply-To: <5615921f-7c33-bcbe-f076-9ed1cbe900ee@gmail.com>

On Tue, Sep 26, 2023 at 07:04:12PM +0900, Akira Yokosawa wrote:
> Hi,
> 
> On 2023/09/23 17:23, Paul E. McKenney wrote:
> > On Fri, Sep 22, 2023 at 10:55:44PM +0900, Akira Yokosawa wrote:
> [...]
> >>  
> >> -The low overheads of the address and data dependencies discussed in
> >> +The low overheads of the \IXalth{address}{address}{dependency}
> > 
> > This looks strange given the different parameters to the \IXalth above
> > and the \IXalth below, but it gets the right thing to the right place
> > in the index, so I queued and pushed all four.
> > 
> [...]
> >> +and \IXalth{data dependencies}{data}{dependency} discussed in
> 
> Let me explain what's happening here.
> 
> Original sentence looked like this:
> 
>     The low overheads of the address and data dependencies discussed in
> 
> Here, if "address dependencies" were there, I could have said:
> 
>     \IXalth{address dependencies}{address}{dependency}
> 
> Instead, I put an index marker of "Dependency, address" on the lonely
> "address", therefore it became:
> 
>     \IXalth{address}{address}{dependency}
> 
> \IXalth{}{}{} prints any string in #1 with an index marker of
> "#3, #2" put on it.
> 
> Macros of \IX*alt* are powerful and can do whatever you'd like.

Got it, thank you for the explanation!  And indeed they are powerful!

> > Thank you!!!  And yes, I am now generating a set of glossary entries.  ;-)
> 
> Thank you.  I like those cautionary remarks.

Glad you like them!

							Thanx, Paul

  reply	other threads:[~2023-09-26 18:04 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-22 13:52 [PATCH -perfbook 0/4] memorder: Add index markers Akira Yokosawa
2023-09-22 13:54 ` [PATCH -perfbook 1/4] memorder: Add index markers for 'address/control/data dependency', take 1 Akira Yokosawa
2023-09-22 13:55 ` [PATCH -perfbook 2/4] memorder: Add index markers for 'address/control/data dependency', take 2 Akira Yokosawa
2023-09-23  8:23   ` Paul E. McKenney
2023-09-26 10:04     ` Akira Yokosawa
2023-09-26 18:04       ` Paul E. McKenney [this message]
2023-09-22 13:56 ` [PATCH -perfbook 3/4] memorder: Add index marker for smp_read_barrier_depends() as classic API Akira Yokosawa
2023-09-22 13:57 ` [PATCH -perfbook 4/4] Fix definition of \indexh for flat index pages Akira Yokosawa

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=e1f008ec-1358-4fd7-ab50-abc0fb72439e@paulmck-laptop \
    --to=paulmck@kernel.org \
    --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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).