xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Atom2 <ariel.atom2@web2web.at>,
	Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: Doug Goldstein <cardoe@cardoe.com>, xen-devel@lists.xen.org
Subject: Re: HVM domains crash after upgrade from XEN 4.5.1 to 4.5.2
Date: Mon, 16 Nov 2015 23:01:56 +0000	[thread overview]
Message-ID: <564A6064.4080800@citrix.com> (raw)
In-Reply-To: <564A2B91.2090501@web2web.at>

On 16/11/2015 19:16, Atom2 wrote:
>
>
> Am 16.11.15 um 16:31 schrieb Konrad Rzeszutek Wilk:
>>>>> Your analysis was absolutely spot on. After re-thinking this for a
>>>>> moment, I thought going down that route first would make a lot of
>>>>> sense
>>>>> as PV guests still do work and one of the differences to HVM domUs is
>>>>> that the former do _not_ require SeaBIOS. Looking at my log files of
>>>>> installed packages confirmed an upgrade from SeaBIOS 1.7.5 to
>>>>> 1.8.2 in
>>>>> the relevant timeframe which obviously had not made it to the
>>>>> hvmloader
>>>>> of xen-4.5.1 as I did not re-compile xen after the upgrade of
>>>>> SeaBIOS.
>>>>>
>>>>> So I re-compiled xen-4.5.1 (obviously now using the installed SeaBIOS
>>>>> 1.8.2) and the same error as with xen-4.5.2 popped up - and that
>>>>> seemed
>>>>> to strongly indicate that there indeed might be an issue with
>>>>> SeaBIOS as
>>>>> this probably was the only variable that had changed from the
>>>>> original
>>>>> install of xen-4.5.1.
>> I recall seeing this way back in Fedora 20 days. I narrowed it down the
>> SeaBIOS version that was a standalone package to not have CONFIG_XEN.
>>
>> Having that fixed in the SeaBIOS package fixed it.
> Hi Konrad, Doug, Andrew (specifically added to this part of the thread)!
> Konrad, you might have found an interesting point. I did have a look
> at the ebuild for the failing version and in there I found the
> following comment:
> ====== comment from ebuild =======
>     # Upstream hasn't released a new binary.  We snipe ours from
> Fedora for now.
>     # http://code.coreboot.org/p/seabios/downloads/get/bios.bin-${PV}.gz
> ====== end comment from ebuild =======
> which might in fact underline that there might be an issue similar to
> what you described above.
>
> What is also pretty interesting is the fact that the old (working)
> SeaBIOS version 1.7.5 installed as "bios.bin" under /usr/share/seabios
> is actually 262.144 bytes in size whereas the new (invalid) SeaBIOS
> 1.8.2 installed in the same location is only half as big: 131.072 bytes.
>
> I checked at the download site and the 1.8.2 binary version is indeed
> not available from http://code.coreboot.org/p/seabios/downloads/. But
> both the binary versions for 1.7.5 and 1.8.0 are available and both
> are acutually 262.144 bytes in size, so I'd be very surprised if the
> 1.8.2 version is really only half that size. By the way, the old
> working version (according to the ebuild) was directly downloaded from
> the above url and also shows an identical SHA1 digest to that version
> available for download there.
>
> To me this looks as if something is really wrong here. If anybody of
> you has access to a 1.8.2 version, could you please confirm whether
> there's really that big a size difference between the 1.7.5 and the
> 1.8.2 versions? Or is that difference probably attributable to the
> missing CONFIG_XEN option?
>
> Andrew: I havent't gotten around to run the debug version of the
> hypervisor again, but if the current suspicion turns out to be true,
> there's probably not much value in that anyways. Would you agree?

Sadly not.

I accept that this issue is possibly fixed in newer SeaBIOS by working
around the issue.

However, I stand by my original point.  *There is no way the guest
should be able to get into this situation in the first place*, and its
implication of *there is a genuine hypervisor bug which we should track
down*, irrespective of whether the issue has been worked around elsehow.

~Andrew

  parent reply	other threads:[~2015-11-16 23:01 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-12  1:08 HVM domains crash after upgrade from XEN 4.5.1 to 4.5.2 Atom2
2015-11-12 12:52 ` Jan Beulich
2015-11-12 13:01   ` Andrew Cooper
2015-11-12 14:29     ` Atom2
2015-11-12 15:32       ` Jan Beulich
2015-11-12 16:43       ` Andrew Cooper
2015-11-12 23:00         ` Atom2
2015-11-13  7:25           ` Jan Beulich
2015-11-13 10:09             ` Andrew Cooper
2015-11-14  0:16               ` Atom2
2015-11-14 20:32                 ` Andrew Cooper
2015-11-15  0:14                   ` Atom2
2015-11-15 15:12                     ` Andrew Cooper
2015-11-16  0:39                       ` Atom2
2015-11-16 10:02                         ` Andrew Cooper
2015-11-15 20:12                     ` Doug Goldstein
2015-11-16  1:05                       ` Atom2
2015-11-16 15:31                         ` Konrad Rzeszutek Wilk
2015-11-16 19:16                           ` Atom2
2015-11-16 19:25                             ` Konrad Rzeszutek Wilk
2015-11-16 19:39                               ` Doug Goldstein
2015-11-16 19:47                                 ` Konrad Rzeszutek Wilk
2015-11-16 19:45                               ` Atom2
2015-11-16 23:01                             ` Andrew Cooper [this message]
2015-11-16 23:10                               ` Atom2
2015-11-18 22:51                                 ` Atom2
2015-11-18 23:17                                   ` Andrew Cooper
2015-11-19  0:31                                     ` Atom2
2015-11-19  1:06                                       ` Andrew Cooper
2015-11-19 20:02                                         ` Atom2
2015-11-19 23:53                                           ` Andrew Cooper
2015-11-24 11:53                                             ` Atom2
2015-11-19 10:24                                     ` Jan Beulich
2015-11-19 10:38                                       ` Andrew Cooper
2015-11-19 19:51                                         ` Atom2
2015-11-20  7:57                                           ` Jan Beulich
2015-11-24 10:32                                             ` Atom2
2015-11-24 10:43                                               ` Jan Beulich
2015-11-27 22:51                                                 ` Atom2
2015-11-30  9:04                                                   ` Jan Beulich
2015-11-16 19:47                         ` Doug Goldstein
2015-11-16 20:14                           ` Atom2
2015-11-12 14:12   ` Atom2

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=564A6064.4080800@citrix.com \
    --to=andrew.cooper3@citrix.com \
    --cc=ariel.atom2@web2web.at \
    --cc=cardoe@cardoe.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).