All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: Xen-devel List <xen-devel@lists.xen.org>
Subject: Re: Xen 4.5 Development Update (RC4)
Date: Fri, 19 Dec 2014 15:19:44 +0000	[thread overview]
Message-ID: <54944210.6050402@citrix.com> (raw)
In-Reply-To: <20141216204900.GB11551@konrad-lan.dumpdata.com>

On 16/12/14 20:49, Konrad Rzeszutek Wilk wrote:
> On Tue, Dec 16, 2014 at 05:43:08PM +0000, Andrew Cooper wrote:
>> On 16/12/14 16:13, konrad.wilk@oracle.com wrote:
>>> Xen 4.5-rc4 was out on Monday (Dec 15th). This is the last RC and then
>>> we have the General Release on Jan 7th!
>>>
>>> Details for the test-day are at
>>>
>>> http://wiki.xen.org/wiki/Xen_4.5_RC4_test_instructions
>>>
>>> In terms of bugs, we have:
>> From the XenServer testing.
> Thank you for doing this testing!
>> * Fail to reliably boot on IBM Flex x222 blades, apparent regression
>> from 4.4
>>
>> I have declared this a latent BIOS bug, and not a regression from 4.4. 
>> Across regular reboots, the exact positions of the ACPI tables, and the
>> e820 layout is unstable.  The first consistent difference between 4.4
>> and 4.5 is that 4.4 reports 1 MBR signature while 4.5 reports 0.  This
>> is because the int $0x13, ah=2 call is returning differently.  I can get
>> the call to return differently (and correctly for 4.5) by simply making
>> the boot trampoline larger (with my debugging routines but not being
>> called).
> This sounds very familiar, but I can't place where I saw mention of
> a similar issue.
>> * VM fail to resume on upgrade from Xen < 4.5
>>
>> This is the issue I am currently looking into.  Currently, all the
>> "upgrade from older XenServer" tests are failing due to VMs crashing on
>> resume.  I have not yet identified whether this is a XenServer issue or
> Ugh.

I have got to the bottom of this, and it it turns out to be a legacy ->
migration v2 conversion bug which only surfaced now because Xen-4.5 is
more strict than Xen-4.4.

HVM_PARAM_PAE_ENABLED is sent out-of-band in legacy, but passed to
xc_domain_restore(), which does a set_param(), unconnected with any
contents of the stream.  Migration v2 saves and restores it properly,
but the legacy -> v2 conversion neglected to combine the out-of-band
information.  No VMs blew up because all versions of Xen at that point
were not correctly auditing updates to cr4 against the domain cpuid
policy.  Xen-4.5 now does, causing #GP faults on cr4 writes for guests
which had PAE enabled before migrate.

I shall be fixing this in the migration v2 series, and also looking for
any other obvious out-of-band information which needs injecting into a
converted stream.


With this fixed(^W hacked around for now), I have identified and solved
all discrepancies XenServer testing has noticed between Xen-4.4 and
Xen-4.5 so far.

There will be another full nightly test happening tonight (based on c/s
7e88c23 "libxl: Tell qemu to use raw format when using a tapdisk"), and
some stress and scale tests if time allows.

~Andrew

  reply	other threads:[~2014-12-19 15:19 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-16 16:13 Xen 4.5 Development Update (RC4) konrad.wilk
2014-12-16 16:34 ` Olaf Hering
2014-12-16 20:46   ` Konrad Rzeszutek Wilk
2014-12-17  7:55     ` Olaf Hering
2014-12-17 19:41       ` Konrad Rzeszutek Wilk
2014-12-17 21:43         ` Konrad Rzeszutek Wilk
2014-12-18  8:49           ` Olaf Hering
2014-12-16 17:43 ` Andrew Cooper
2014-12-16 20:49   ` Konrad Rzeszutek Wilk
2014-12-19 15:19     ` Andrew Cooper [this message]
2014-12-19 18:14       ` Konrad Rzeszutek Wilk
2014-12-23 15:10         ` Andrew Cooper
2014-12-26 12:20           ` Sander Eikelenboom
2014-12-26 13:34             ` Andrew Cooper
2014-12-26 19:07           ` Konrad Rzeszutek Wilk
2014-12-19 14:52 ` Xen 4.5 Development Update (RC4) - Documentation Updates Lars Kurth
2014-12-19 15:30   ` Jan Beulich
2014-12-19 19:55     ` Pasi Kärkkäinen
2014-12-19 20:05       ` Russ Pavlicek
2014-12-19 15:55   ` Meng Xu
2014-12-19 16:01     ` Lars Kurth
2014-12-19 16:15     ` Dario Faggioli
2014-12-19 17:10       ` Meng Xu
2014-12-19 18:02         ` Lars Kurth

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=54944210.6050402@citrix.com \
    --to=andrew.cooper3@citrix.com \
    --cc=konrad.wilk@oracle.com \
    --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.