All of lore.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 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.