All of lore.kernel.org
 help / color / mirror / Atom feed
From: "PaX Team" <pageexec@freemail.hu>
To: x86@kernel.org
Cc: llvmlinux@lists.linuxfoundation.org, linux-kernel@vger.kernel.org
Subject: Re: [llvmlinux] [PATCH 4/4] x86, boot: Work around clang PR18415.
Date: Wed, 08 Jan 2014 13:04:51 +0100	[thread overview]
Message-ID: <52CD3EE3.6295.92E23548@pageexec.freemail.hu> (raw)
In-Reply-To: <1389180083-23249-4-git-send-email-David.Woodhouse@intel.com>

On 8 Jan 2014 at 11:21, David Woodhouse wrote:

> Clang's intrinsics ignore -mregparm=3 when they fall back to calling the
> out-of-line implementations. Putting the args on the stack when memcpy()
> expects them in registers is not a recipe for a happy kernel.
> 
> This bites with -m32 too, so clang is presumably catastrophically
> broken for the i386 kernel until this is fixed, unless I'm missing
> something.

llvmlinux already carries my old patch (and some more) here:
 arch/i586/patches/0026-Add-own-versions-of-memcpy-and-memset-for-compilatio.patch

(see also my comment on the clang PR)

cheers,
 PaX Team


  reply	other threads:[~2014-01-08 12:05 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-08 11:21 [PATCH 1/4] x86: Remove duplication of 16-bit CFLAGS David Woodhouse
2014-01-08 11:21 ` [PATCH 2/4] x86, boot: Use __attribute__((used)) to ensure videocard structs are emitted David Woodhouse
2014-01-22  2:03   ` [tip:x86/build] " tip-bot for David Woodhouse
2014-01-22 12:25   ` tip-bot for David Woodhouse
2014-01-08 11:21 ` [PATCH 3/4] x86: Allow building 16-bit code with -m16 with toolchains that support it David Woodhouse
2014-01-14 19:14   ` [llvmlinux] " Jan-Simon Möller
2014-01-14 23:59     ` Woodhouse, David
2014-01-22  0:54   ` David Woodhouse
2014-01-29 12:11   ` [PATCH] x86: Build 16-bit code with -m16 where possible David Woodhouse
2014-01-29 12:16     ` [PATCH v2] " David Woodhouse
2014-01-29 17:51       ` [tip:x86/build] x86, build: " tip-bot for David Woodhouse
2014-01-29 17:57       ` tip-bot for David Woodhouse
2014-01-30 16:10       ` tip-bot for David Woodhouse
2014-01-31 10:23     ` [PATCH] x86: " Pavel Machek
2014-01-08 11:21 ` [PATCH 4/4] x86, boot: Work around clang PR18415 David Woodhouse
2014-01-08 12:04   ` PaX Team [this message]
2014-01-08 13:03 ` [llvmlinux] [PATCH 1/4] x86: Remove duplication of 16-bit CFLAGS Jan-Simon Möller
2014-01-22  2:03 ` [tip:x86/build] " tip-bot for David Woodhouse
2014-01-22 12:25 ` tip-bot for David Woodhouse

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=52CD3EE3.6295.92E23548@pageexec.freemail.hu \
    --to=pageexec@freemail.hu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=llvmlinux@lists.linuxfoundation.org \
    --cc=x86@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.