All of lore.kernel.org
 help / color / mirror / Atom feed
From: George Dunlap <george.dunlap@eu.citrix.com>
To: Andrew Cooper <andrew.cooper3@citrix.com>,
	Xen-devel <xen-devel@lists.xen.org>
Cc: Keir Fraser <keir@xen.org>, Jan Beulich <JBeulich@suse.com>
Subject: Re: [PATCH] x86/boot: Fix BIOS memory corruption on certain IBM systems
Date: Wed, 4 Dec 2013 10:09:06 +0000	[thread overview]
Message-ID: <529EFF42.50805@eu.citrix.com> (raw)
In-Reply-To: <1386102859-14477-1-git-send-email-andrew.cooper3@citrix.com>

On 12/03/2013 08:34 PM, Andrew Cooper wrote:
> IBM System x3530 M4 BIOSes (including the latest available at the time of this
> patch) will corrupt a byte at physical address 0x105ff1 to the value of 0x86
> if %esp has the value 0x00080000 when issuing an `int $0x15 (ax=0xec00)` to
> inform the system about our intended operating mode.
>
> Xen gets unhappy when the bootloader has placed it's .text section in over
> this specific region of RAM.
>
> After dropping into 16bit mode, initialise as much state as we possibly can to
> sane values. This includes 0 for all the GPRs and %cs for %fs and %gs which
> would otherwise be unreal segment selectors.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> CC: Keir Fraser <keir@xen.org>
> CC: Jan Beulich <JBeulich@suse.com>
> CC: George Dunlap <george.dunlap@eu.citrix.com>
>
> ---
>
> George:
>   * This fixes a memory corruption issue, so counts towards both #1 and #2 as
>     far as a freeze exception goes.

The general rule during the code freeze is that bug fixes are exempt 
from needing a freeze exception, unless the maintainers think that it's 
particularly risky.  I suppose one could quibble about whether this is a 
"bug fix" or a "work-around for broken BIOS" though, so just in case:

Release-acked-by: George Dunlap <george.dunlap@eu.citrix.com>

      parent reply	other threads:[~2013-12-04 10:09 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-03 20:34 [PATCH] x86/boot: Fix BIOS memory corruption on certain IBM systems Andrew Cooper
2013-12-04  7:17 ` Keir Fraser
2013-12-04 10:03 ` Jan Beulich
2013-12-04 10:35   ` Andrew Cooper
2013-12-04 10:47     ` Jan Beulich
2013-12-04 18:59       ` Keir Fraser
2013-12-04 10:09 ` George Dunlap [this message]

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=529EFF42.50805@eu.citrix.com \
    --to=george.dunlap@eu.citrix.com \
    --cc=JBeulich@suse.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=keir@xen.org \
    --cc=xen-devel@lists.xen.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.