From: Linus Torvalds <torvalds@linux-foundation.org>
To: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Cc: Will Deacon <will@kernel.org>,
"Paul E. McKenney" <paulmck@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
Segher Boessenkool <segher@kernel.crashing.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Alan Stern <stern@rowland.harvard.edu>,
Andrea Parri <parri.andrea@gmail.com>,
Boqun Feng <boqun.feng@gmail.com>,
Nick Piggin <npiggin@gmail.com>,
David Howells <dhowells@redhat.com>,
Jade Alglave <j.alglave@ucl.ac.uk>,
Luc Maranget <luc.maranget@inria.fr>,
Akira Yokosawa <akiyks@gmail.com>,
linux-toolchains@vger.kernel.org,
linux-arch <linux-arch@vger.kernel.org>
Subject: Re: [RFC PATCH] LKMM: Add ctrl_dep() macro for control dependency
Date: Wed, 29 Sep 2021 07:54:14 -0700 [thread overview]
Message-ID: <CAHk-=wjd_BJiJYZ99PAoc4mQ3QTiZrt-tRdznj3g9kU8-gYsAA@mail.gmail.com> (raw)
In-Reply-To: <CAHk-=wg23CqjGWjjxDQ7yxrb+eF5at2KFU03GZa18Znx=+Xvow@mail.gmail.com>
On Wed, Sep 29, 2021 at 7:47 AM Linus Torvalds
<torvalds@linux-foundation.org> wrote:
>
> And if there *is* need ("look, we have that same store in both the if-
> and the else-statement" or whatever), then say so, and state that
> thing.
Side note: I'd also like the commit that introduces this to talk very
explicitly about the particular case that it is used doe and that it
fixes. No "this can happen". A "this happened, here's the _actual_
wrong code generation, and look how this new ctrl_dep() macro fixes
it".
When it's this subtle, I don't want theoretical arguments. I want
actual outstanding and real bugs.
Because I get the feeling that there were very few actual realistic
examples of this, only made-up theoretical cases that wouldn't ever
really be found in real code.
Linus
next prev parent reply other threads:[~2021-09-29 14:54 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-28 21:15 [RFC PATCH] LKMM: Add ctrl_dep() macro for control dependency Mathieu Desnoyers
2021-09-29 12:06 ` Marco Elver
2021-10-01 15:45 ` Mathieu Desnoyers
2021-10-01 16:20 ` Linus Torvalds
2021-10-01 17:28 ` Mathieu Desnoyers
2021-10-01 18:18 ` Linus Torvalds
2021-09-29 12:28 ` Florian Weimer
2021-09-29 17:41 ` Segher Boessenkool
2021-09-29 19:46 ` Florian Weimer
2021-10-01 16:13 ` Mathieu Desnoyers
2021-10-01 16:26 ` Florian Weimer
2021-10-01 16:35 ` Linus Torvalds
2021-10-10 14:02 ` Florian Weimer
2021-10-14 0:01 ` Paul E. McKenney
2021-10-14 2:14 ` Alan Stern
2021-10-14 16:14 ` Paul E. McKenney
2021-10-14 15:58 ` Florian Weimer
2021-10-14 16:23 ` Paul E. McKenney
2021-10-14 18:19 ` Florian Weimer
2021-10-14 21:09 ` Paul E. McKenney
2021-10-14 22:36 ` Linus Torvalds
2021-09-30 13:28 ` Mathieu Desnoyers
2021-09-29 14:47 ` Linus Torvalds
2021-09-29 14:54 ` Linus Torvalds [this message]
2021-09-29 19:50 ` Mathieu Desnoyers
2021-09-29 20:13 ` Mathieu Desnoyers
2021-09-29 19:27 ` Mathieu Desnoyers
2021-09-29 22:14 ` Linus Torvalds
2021-09-29 21:47 ` Segher Boessenkool
2021-09-29 23:57 ` Paul E. McKenney
2021-10-01 15:28 ` Mathieu Desnoyers
2021-10-01 22:53 ` Paul E. McKenney
2021-10-01 19:10 ` Segher Boessenkool
2021-10-01 22:50 ` Paul E. McKenney
2021-10-02 14:29 ` Alan Stern
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='CAHk-=wjd_BJiJYZ99PAoc4mQ3QTiZrt-tRdznj3g9kU8-gYsAA@mail.gmail.com' \
--to=torvalds@linux-foundation.org \
--cc=akiyks@gmail.com \
--cc=boqun.feng@gmail.com \
--cc=dhowells@redhat.com \
--cc=j.alglave@ucl.ac.uk \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-toolchains@vger.kernel.org \
--cc=luc.maranget@inria.fr \
--cc=mathieu.desnoyers@efficios.com \
--cc=npiggin@gmail.com \
--cc=parri.andrea@gmail.com \
--cc=paulmck@kernel.org \
--cc=peterz@infradead.org \
--cc=segher@kernel.crashing.org \
--cc=stern@rowland.harvard.edu \
--cc=will@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 NNTP newsgroup(s).