public inbox for perfbook@vger.kernel.org
 help / color / mirror / Atom feed
From: "Kunwu Chan" <kunwu.chan@linux.dev>
To: paulmck@kernel.org
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH] memorder: Reconcile Figure 15.15 notation with W+RWC z-flow semantics
Date: Thu, 12 Mar 2026 02:00:33 +0000	[thread overview]
Message-ID: <f23e0457d77d509a58fd9ee6e4bef5aa7feed4cb@linux.dev> (raw)
In-Reply-To: <15acbc05-3b54-4d9e-8a74-f763a990ba3f@paulmck-laptop>

March 11, 2026 at 10:39 PM, "Paul E. McKenney" <paulmck@kernel.org mailto:paulmck@kernel.org?to=%22Paul%20E.%20McKenney%22%20%3Cpaulmck%40kernel.org%3E > wrote:


> 
> On Wed, Mar 11, 2026 at 04:48:13PM +0800, Kunwu Chan wrote:
> 
> > 
> > State explicitly that Figure 15.15 uses x as a schematic variable name, while the surrounding W+RWC analysis in Listing 15.18 follows the concrete z-flow (P2 -> z -> P1). This eliminates notation drift across cross-references and preserves propagation-argument fidelity.
> >  
> >  Signed-off-by: Kunwu Chan <kunwu.chan@linux.dev>
> > 
> Again, good eyes, and thank you! I queued and pushed all three, and
> added a fourth commit that converted this to a footnote and mentioned
> both variables and the listing, shown below. Please check it to make
> sure that I have not messed something up.
> 
>  Thanx, Paul
> 
Hi Paul,
Checked — LGTM.
The footnote conversion is the right call. It cleanly resolves 
the Figure 15.15 x vs. W+RWC z notation mismatch.
Thanks for landing this promptly.

Best, Kunwu

> > 
> > ---
> >  memorder/memorder.tex | 2 ++
> >  1 file changed, 2 insertions(+)
> >  
> >  diff --git a/memorder/memorder.tex b/memorder/memorder.tex
> >  index 2357e7ea..8eb14a29 100644
> >  --- a/memorder/memorder.tex
> >  +++ b/memorder/memorder.tex
> >  @@ -2760,6 +2760,8 @@ that \co{P1()}'s read from \co{z} happens much later in time, but
> >  nevertheless still sees the old value of zero.
> >  This situation is depicted in
> >  \cref{fig:memorder:Load-to-Store is Counter-Temporal}:
> >  +The figure uses \co{x} as a generic variable name for the same
> >  +counter-temporal pattern.
> >  Just because a load sees the old value does \emph{not} mean that
> >  this load executed at an earlier time than did the store of the
> >  new value.
> >  -- 
> >  2.25.1
> > 
> ------------------------------------------------------------------------
> 
> commit d2c790e9077625e2e28ee919619b4d18175c3665
> Author: Paul E. McKenney <paulmck@kernel.org>
> Date: Wed Mar 11 07:34:07 2026 -0700
> 
>  memorder: Convert Figure 15.15 variable-x commentary to footnote
>  
>  Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
> 
> diff --git a/memorder/memorder.tex b/memorder/memorder.tex
> index 388903b6..4503c0a6 100644
> --- a/memorder/memorder.tex
> +++ b/memorder/memorder.tex
> @@ -2759,9 +2759,10 @@ It therefore takes time for the effect of the \co{P2()}'s store to
>  that \co{P1()}'s read from \co{z} happens much later in time, but
>  nevertheless still sees the old value of zero.
>  This situation is depicted in
> -\cref{fig:memorder:Load-to-Store is Counter-Temporal}:
> -The figure uses \co{x} as a generic variable name for the same
> -counter-temporal pattern.
> +\cref{fig:memorder:Load-to-Store is Counter-Temporal}:\footnote{
> + The figure uses \co{x} as a generic variable name for the same
> + counter-temporal pattern that uses the variable \co{z} in
> + \cref{lst:memorder:W+RWC Litmus Test With Release (No Ordering)}.}
>  Just because a load sees the old value does \emph{not} mean that
>  this load executed at an earlier time than did the store of the
>  new value.
>

  reply	other threads:[~2026-03-12  2:00 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-11  8:48 [PATCH] memorder: Reconcile Figure 15.15 notation with W+RWC z-flow semantics Kunwu Chan
2026-03-11 14:39 ` Paul E. McKenney
2026-03-12  2:00   ` Kunwu Chan [this message]
2026-03-12 21:15     ` 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=f23e0457d77d509a58fd9ee6e4bef5aa7feed4cb@linux.dev \
    --to=kunwu.chan@linux.dev \
    --cc=paulmck@kernel.org \
    --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