Received-SPF: pass (google.com: domain of linux-kernel+bounces-65824-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240208220604.140859-1-seanjc@google.com> <CAKwvOdk_obRUkD6WQHhS9uoFVe3HrgqH5h+FpqsNNgmj4cmvCQ@mail.gmail.com>
 <DM6PR02MB40587AD6ABBF1814E9CCFA7CB84B2@DM6PR02MB4058.namprd02.prod.outlook.com>
 <CAHk-=wi3p5C1n03UYoQhgVDJbh_0ogCpwbgVGnOdGn6RJ6hnKA@mail.gmail.com>
 <ZcZyWrawr1NUCiQZ@google.com> <CAKwvOdmKaYYxf7vjvPf2vbn-Ly+4=JZ_zf+OcjYOkWCkgyU_kA@mail.gmail.com>
 <CAHk-=wgEABCwu7HkJufpWC=K7u_say8k6Tp9eHvAXFa4DNXgzQ@mail.gmail.com>
 <CAHk-=wgBt9SsYjyHWn1ZH5V0Q7P6thqv_urVCTYqyWNUWSJ6_g@mail.gmail.com> <CAFULd4ZUa56KDLXSoYjoQkX0BcJwaipy3ZrEW+0tbi_Lz3FYAw@mail.gmail.com>
In-Reply-To: <CAFULd4ZUa56KDLXSoYjoQkX0BcJwaipy3ZrEW+0tbi_Lz3FYAw@mail.gmail.com>
From: Linus Torvalds <torvalds@linux-foundation.org>
Date: Wed, 14 Feb 2024 10:43:50 -0800
Message-ID: <CAHk-=wiRQKkgUSRsLHNkgi3M4M-mwPq+9-RST=neGibMR=ubUw@mail.gmail.com>
Subject: Re: [PATCH] Kconfig: Explicitly disable asm goto w/ outputs on gcc-11
 (and earlier)
To: Uros Bizjak <ubizjak@gmail.com>
Cc: Nick Desaulniers <ndesaulniers@google.com>, Jakub Jelinek <jakub@redhat.com>, 
	Sean Christopherson <seanjc@google.com>, "Andrew Pinski (QUIC)" <quic_apinski@quicinc.com>, 
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Masahiro Yamada <masahiroy@kernel.org>, 
	Peter Zijlstra <peterz@infradead.org>, "kvm@vger.kernel.org" <kvm@vger.kernel.org>
Content-Type: multipart/mixed; boundary="000000000000540e2206115be5e4"

--000000000000540e2206115be5e4
Content-Type: text/plain; charset="UTF-8"

On Sun, 11 Feb 2024 at 03:12, Uros Bizjak <ubizjak@gmail.com> wrote:
>
> So, I'd suggest at least limit the workaround to known-bad compilers.

Based on the current state of

    https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113921

I would suggest this attached kernel patch, which makes the manual
"volatile" the default case (since it should make no difference except
for the known gcc issue), and limits the extra empty asm serialization
to gcc versions older than 12.1.0.

But Jakub is clearly currently trying to figure out exactly what was
going wrong, so things may change. Maybe the commit he bisected to
happened to just accidentally hide the real issue.

                Linus

--000000000000540e2206115be5e4
Content-Type: text/x-patch; charset="US-ASCII"; name="patch.diff"
Content-Disposition: attachment; filename="patch.diff"
Content-Transfer-Encoding: base64
Content-ID: <f_lsm537ms0>
X-Attachment-Id: f_lsm537ms0

IGluY2x1ZGUvbGludXgvY29tcGlsZXItZ2NjLmggICB8IDEyICsrKy0tLS0tLS0tLQogaW5jbHVk
ZS9saW51eC9jb21waWxlcl90eXBlcy5oIHwgMTEgKysrKysrKysrKy0KIDIgZmlsZXMgY2hhbmdl
ZCwgMTMgaW5zZXJ0aW9ucygrKSwgMTAgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvaW5jbHVk
ZS9saW51eC9jb21waWxlci1nY2MuaCBiL2luY2x1ZGUvbGludXgvY29tcGlsZXItZ2NjLmgKaW5k
ZXggYzFhOTYzYmU3ZDI4Li5kMTgxZDI3MDNiYmEgMTAwNjQ0Ci0tLSBhL2luY2x1ZGUvbGludXgv
Y29tcGlsZXItZ2NjLmgKKysrIGIvaW5jbHVkZS9saW51eC9jb21waWxlci1nY2MuaApAQCAtNjcs
MjEgKzY3LDE1IEBACiAvKgogICogR0NDICdhc20gZ290bycgd2l0aCBvdXRwdXRzIG1pc2NvbXBp
bGVzIGNlcnRhaW4gY29kZSBzZXF1ZW5jZXM6CiAgKgotICogICBodHRwczovL2djYy5nbnUub3Jn
L2J1Z3ppbGxhL3Nob3dfYnVnLmNnaT9pZD0xMTA0MjAKLSAqICAgaHR0cHM6Ly9nY2MuZ251Lm9y
Zy9idWd6aWxsYS9zaG93X2J1Zy5jZ2k/aWQ9MTEwNDIyCisgKiAgIGh0dHBzOi8vZ2NjLmdudS5v
cmcvYnVnemlsbGEvc2hvd19idWcuY2dpP2lkPTExMzkyMQogICoKICAqIFdvcmsgaXQgYXJvdW5k
IHZpYSB0aGUgc2FtZSBjb21waWxlciBiYXJyaWVyIHF1aXJrIHRoYXQgd2UgdXNlZAogICogdG8g
dXNlIGZvciB0aGUgb2xkICdhc20gZ290bycgd29ya2Fyb3VuZC4KLSAqCi0gKiBBbHNvLCBhbHdh
eXMgbWFyayBzdWNoICdhc20gZ290bycgc3RhdGVtZW50cyBhcyB2b2xhdGlsZTogYWxsCi0gKiBh
c20gZ290byBzdGF0ZW1lbnRzIGFyZSBzdXBwb3NlZCB0byBiZSB2b2xhdGlsZSBhcyBwZXIgdGhl
Ci0gKiBkb2N1bWVudGF0aW9uLCBidXQgc29tZSB2ZXJzaW9ucyBvZiBnY2MgZGlkbid0IGFjdHVh
bGx5IGRvCi0gKiB0aGF0IGZvciBhc21zIHdpdGggb3V0cHV0czoKLSAqCi0gKiAgICBodHRwczov
L2djYy5nbnUub3JnL2J1Z3ppbGxhL3Nob3dfYnVnLmNnaT9pZD05ODYxOQogICovCisjaWYgR0ND
X1ZFUlNJT04gPCAxMjAxMDAKICNkZWZpbmUgYXNtX2dvdG9fb3V0cHV0KHguLi4pIFwKIAlkbyB7
IGFzbSB2b2xhdGlsZSBnb3RvKHgpOyBhc20gKCIiKTsgfSB3aGlsZSAoMCkKKyNlbmRpZgogCiAj
aWYgZGVmaW5lZChDT05GSUdfQVJDSF9VU0VfQlVJTFRJTl9CU1dBUCkKICNkZWZpbmUgX19IQVZF
X0JVSUxUSU5fQlNXQVAzMl9fCmRpZmYgLS1naXQgYS9pbmNsdWRlL2xpbnV4L2NvbXBpbGVyX3R5
cGVzLmggYi9pbmNsdWRlL2xpbnV4L2NvbXBpbGVyX3R5cGVzLmgKaW5kZXggNjYzZDg3OTFjODcx
Li4zYmI1YTlkMTZlYWEgMTAwNjQ0Ci0tLSBhL2luY2x1ZGUvbGludXgvY29tcGlsZXJfdHlwZXMu
aAorKysgYi9pbmNsdWRlL2xpbnV4L2NvbXBpbGVyX3R5cGVzLmgKQEAgLTM2Miw4ICszNjIsMTcg
QEAgc3RydWN0IGZ0cmFjZV9saWtlbHlfZGF0YSB7CiAjZGVmaW5lIF9fbWVtYmVyX3NpemUocCkJ
X19idWlsdGluX29iamVjdF9zaXplKHAsIDEpCiAjZW5kaWYKIAorLyoKKyAqICJhc20gZ290byIg
aXMgZG9jdW1lbnRlZCB0byBhbHdheXMgYmUgdm9sYXRpbGUsIGJ1dCBzb21lIHZlcnNpb25zCisg
KiBvZiBnY2MgZG9uJ3QgYWN0dWFsbHkgZG8gdGhhdDoKKyAqCisgKiAgIGh0dHBzOi8vZ2NjLmdu
dS5vcmcvYnVnemlsbGEvc2hvd19idWcuY2dpP2lkPTEwMzk3OQorICoKKyAqIFNvIHdlJ2xsIGp1
c3QgZG8gaXQgbWFudWFsbHkgdW5sZXNzIHdlIGhhdmUgb3RoZXIgbW9yZSBleHRlbnNpdmUKKyAq
IHdvcmthcm91bmRzLgorICovCiAjaWZuZGVmIGFzbV9nb3RvX291dHB1dAotI2RlZmluZSBhc21f
Z290b19vdXRwdXQoeC4uLikgYXNtIGdvdG8oeCkKKyNkZWZpbmUgYXNtX2dvdG9fb3V0cHV0KHgu
Li4pIGFzbSB2b2xhdGlsZSBnb3RvKHgpCiAjZW5kaWYKIAogI2lmZGVmIENPTkZJR19DQ19IQVNf
QVNNX0lOTElORQo=
--000000000000540e2206115be5e4--