public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Bugs aren't features: X86_FEATURE_FXSAVE_LEAK
Date: Fri, 05 May 2006 09:36:00 -0700	[thread overview]
Message-ID: <445B7EF0.6090708@zytor.com> (raw)

The recent fix for the AMD FXSAVE information leak had a problematic side effect.  It 
introduced an entry in the x86 features vector which is a bug, not a feature.

The problem with this is that the features vector is designed so that it can be ANDed 
between CPUs to find out the common feature set.  However, bugs aren't features, and bugs 
should be ORd, not ANDed.  In that sense, the *absence* of a bug is a feature.

There are two possible ways of dealing with this:

a) put bugs in the features vector, but sense-inversed, i.e. 1 = bug absent; 0 = bug present.

b) add a separate bugs vector.

When I originally wrote the code I always meant to do (b), but never got around to it. 
It's clear, though, that whatever way we go about this, it should also incorporate the 
fdiv, hlt, f00f, and coma bugs.

	-hpa

             reply	other threads:[~2006-05-05 16:36 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-05 16:36 H. Peter Anvin [this message]
2006-05-05 18:11 ` Bugs aren't features: X86_FEATURE_FXSAVE_LEAK Andi Kleen
2006-05-05 18:18   ` H. Peter Anvin
2006-05-05 18:27     ` Andi Kleen
2006-05-06  4:16   ` Christer Weinigel
2006-05-09 14:38   ` Bill Davidsen
     [not found] <69bvw-2zO-5@gated-at.bofh.it>
     [not found] ` <69d4M-4Yx-19@gated-at.bofh.it>
     [not found]   ` <69mqY-1Ci-9@gated-at.bofh.it>
2006-05-06 17:37     ` Robert Hancock
2006-05-08 17:47       ` H. Peter Anvin

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=445B7EF0.6090708@zytor.com \
    --to=hpa@zytor.com \
    --cc=linux-kernel@vger.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