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.
>
next prev parent 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