public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Jones <davej@codemonkey.org.uk>
To: Thomas Horsten <thomas@horsten.com>
Cc: LKML <linux-kernel@vger.kernel.org>,
	"H . Peter Anvin" <hpa@zytor.com>,
	Michael Elizabeth Chastain <mec@shout.net>
Subject: Re: [PATCH] 2.5.69 Changes to Kconfig and i386 Makefile to include support for various K7 optimizations
Date: Wed, 14 May 2003 19:55:39 +0100	[thread overview]
Message-ID: <20030514185539.GA25542@suse.de> (raw)
In-Reply-To: <200305141940.10999.thomas@horsten.com>

On Wed, May 14, 2003 at 07:40:10PM +0100, Thomas Horsten wrote:

 > I think when GCC supports the different cores, it should be supported by the 
 > kernel scripts, the differences between the cores are real enough to have 
 > potential optimizations at least in theory (as far as I could see the only 
 > difference in GCC 3.2 is whether to use SSE, but that could change in the 
 > future).

To use SSE in kernel space, we need to wrap it in kernel_fpu_begin() /
kernel_fpu_end() pairs anyway. gcc doesn't (and can't) know this.

 > I think it's a fairly simple patch that doesn't break anything

it's non-obvious that it'll break things when people use broken compilers.
read what I wrote before, several gcc versions got this horribly wrong.
"athlon4" was even tuned for 64bit at one point.

Take away the SSE, take away the broken compiler versions, and you'd be
left with something that would hardly show a blip on a benchmark.

		Dave

  reply	other threads:[~2003-05-14 18:42 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200305071834.26789.thomas@horsten.com>
2003-05-14 14:04 ` [PATCH] 2.5.69 Changes to Kconfig and i386 Makefile to include support for various K7 optimizations Dave Jones
2003-05-14 18:40   ` Thomas Horsten
2003-05-14 18:55     ` Dave Jones [this message]
2003-05-14 16:49 Ray Lee
2003-05-14 17:06 ` Dave Jones
  -- strict thread matches above, loose matches on Subject: below --
2003-05-07 19:28 Thomas Horsten
2003-05-07 20:06 ` Ken Witherow
2003-05-07 20:44   ` Thomas Horsten
2003-05-07 21:02     ` Thomas Horsten

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=20030514185539.GA25542@suse.de \
    --to=davej@codemonkey.org.uk \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mec@shout.net \
    --cc=thomas@horsten.com \
    /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