From: Michael Ellerman <mpe@ellerman.id.au>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: arnd@arndb.de, Nick Desaulniers <ndesaulniers@google.com>,
Linux List Kernel Mailing <linux-kernel@vger.kernel.org>,
clang-built-linux@googlegroups.com,
Nathan Chancellor <natechancellor@gmail.com>,
linuxppc-dev@lists.ozlabs.org
Subject: Re: [GIT PULL] Please pull powerpc/linux.git powerpc-5.3-4 tag
Date: Sun, 11 Aug 2019 08:51:19 +1000 [thread overview]
Message-ID: <87ftm8skgo.fsf@concordia.ellerman.id.au> (raw)
In-Reply-To: <CAHk-=whnEp5+EM53MaT-3ep1xjhrUqCdcfBfTF9YxByGsmDMRw@mail.gmail.com>
[ expanded Cc ]
Linus Torvalds <torvalds@linux-foundation.org> writes:
> On Sat, Aug 10, 2019 at 3:11 AM Michael Ellerman <mpe@ellerman.id.au> wrote:
>>
>> Just one fix, a revert of a commit that was meant to be a minor improvement to
>> some inline asm, but ended up having no real benefit with GCC and broke booting
>> 32-bit machines when using Clang.
>
> Pulled, but whenever there are possible subtle compiler issues I get
> nervous, and wonder if the problem was reported to the clang guys?
Yes, sorry I should have included more context. It was actually the
Clang Linux folks who noticed it and reported it to us:
https://github.com/ClangBuiltLinux/linux/issues/593
There's an LLVM bug filed:
https://bugs.llvm.org/show_bug.cgi?id=42762
And I think there's now agreement that the Clang behaviour is not
correct, Nick actually sent a revert as well but I already had one
queued:
https://patchwork.ozlabs.org/patch/1144980/
Arnd identified some work arounds, which we may end up using, but for
this cycle we thought it was preferable to just revert this change as it
didn't actually change code generation with GCC anyway.
cheers
WARNING: multiple messages have this Message-ID (diff)
From: Michael Ellerman <mpe@ellerman.id.au>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Linux List Kernel Mailing <linux-kernel@vger.kernel.org>,
linuxppc-dev@lists.ozlabs.org
Cc: segher@kernel.crashing.org, arnd@arndb.de,
Nick Desaulniers <ndesaulniers@google.com>,
Nathan Chancellor <natechancellor@gmail.com>,
clang-built-linux@googlegroups.com,
Christophe Leroy <christophe.leroy@c-s.fr>
Subject: Re: [GIT PULL] Please pull powerpc/linux.git powerpc-5.3-4 tag
Date: Sun, 11 Aug 2019 08:51:19 +1000 [thread overview]
Message-ID: <87ftm8skgo.fsf@concordia.ellerman.id.au> (raw)
In-Reply-To: <CAHk-=whnEp5+EM53MaT-3ep1xjhrUqCdcfBfTF9YxByGsmDMRw@mail.gmail.com>
[ expanded Cc ]
Linus Torvalds <torvalds@linux-foundation.org> writes:
> On Sat, Aug 10, 2019 at 3:11 AM Michael Ellerman <mpe@ellerman.id.au> wrote:
>>
>> Just one fix, a revert of a commit that was meant to be a minor improvement to
>> some inline asm, but ended up having no real benefit with GCC and broke booting
>> 32-bit machines when using Clang.
>
> Pulled, but whenever there are possible subtle compiler issues I get
> nervous, and wonder if the problem was reported to the clang guys?
Yes, sorry I should have included more context. It was actually the
Clang Linux folks who noticed it and reported it to us:
https://github.com/ClangBuiltLinux/linux/issues/593
There's an LLVM bug filed:
https://bugs.llvm.org/show_bug.cgi?id=42762
And I think there's now agreement that the Clang behaviour is not
correct, Nick actually sent a revert as well but I already had one
queued:
https://patchwork.ozlabs.org/patch/1144980/
Arnd identified some work arounds, which we may end up using, but for
this cycle we thought it was preferable to just revert this change as it
didn't actually change code generation with GCC anyway.
cheers
next prev parent reply other threads:[~2019-08-10 22:53 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-10 10:11 [GIT PULL] Please pull powerpc/linux.git powerpc-5.3-4 tag Michael Ellerman
2019-08-10 10:11 ` Michael Ellerman
2019-08-10 17:21 ` Linus Torvalds
2019-08-10 17:21 ` Linus Torvalds
2019-08-10 22:42 ` Nathan Chancellor
2019-08-10 22:42 ` Nathan Chancellor
2019-08-10 22:51 ` Michael Ellerman [this message]
2019-08-10 22:51 ` Michael Ellerman
2019-08-10 19:30 ` pr-tracker-bot
2019-08-10 19:30 ` pr-tracker-bot
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=87ftm8skgo.fsf@concordia.ellerman.id.au \
--to=mpe@ellerman.id.au \
--cc=arnd@arndb.de \
--cc=clang-built-linux@googlegroups.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=natechancellor@gmail.com \
--cc=ndesaulniers@google.com \
--cc=torvalds@linux-foundation.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.