xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: "Egger, Christoph" <chegger@amazon.de>
To: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
Cc: Jacob Shin <Jacob.Shin@amd.com>,
	Sherry Hurwitz <sherry.hurwitz@amd.com>,
	Keir Fraser <keir@xen.org>, Jan Beulich <JBeulich@suse.com>,
	xen-devel <xen-devel@lists.xen.org>
Subject: Re: x86/AMD: Nested hvm crashes in 4.3
Date: Thu, 27 Jun 2013 11:50:40 +0200	[thread overview]
Message-ID: <51CC0AF0.1060101@amazon.de> (raw)
In-Reply-To: <51CC03C9.60800@amd.com>

Running a PV guest as L2 guest makes no difference how the p2m
code is used in L0 Xen (L0 == OS that runs on bare metal hardware).

I assume you use the default settings which means you use NPT-on-NPT.
Try shadow-on-npt. You can do this with

  cpuid="host,svm_npt=0"

in the guest config file. Then in the L1 guest you should see
the NPT svm feature bit not available.
Then launch a l2 guest and check if it still crashes.

I agree with Jan: Please provide the crash logs he requested.

Christoph

On 27.06.13 11:20, Suravee Suthikulpanit wrote:
> On 6/27/2013 3:22 AM, Jan Beulich wrote:
>>>>> On 27.06.13 at 02:24, Suravee Suthikulanit
>>>>> <suravee.suthikulpanit@amd.com> wrote:
>>> I have found an issue in where the system crash right when I start
>>> another HVM guest inside an HVM guest.  I have traced back to the patch
>>> which the issue started.
>>>
>>> commit f1bde87fc08ce8c818a1640a8fe4765d48923091
>>> Author: Jan Beulich <jbeulich@suse.com>
>>> Date:   Fri Feb 8 11:06:04 2013 +0100
>>>
>>>       x86: debugging code for testing 16Tb support on smaller memory
>>> systems
>>>
>>>       Signed-off-by: Jan Beulich <jbeulich@suse.com>
>>>       Acked-by: Keir Fraser <keir@xen.org>
>> We had issues exposed by this patch before, but any such issue
>> would just have been masked before that patch (and would
>> surface on a system with more than 5Tb of memory anyway).
> 
> The system I am having the issue has 48GB of memory.
> 
>> So it is very unlikely for the patch itself to be at fault.
> 
> I have traced the issue and found that the system crashing starts from
> this commit id and onward.
> (i.e. The system does not crash with commit id
> ed759d20249197cf87b338ff0ed328052ca3b8e7)
> So, I am still believe that this patch has somehow triggered the issue.
> 
>> Furthermore, the crash them supposedly is because of the code
>> added conditional upon NDEBUG, and hence would (on a smaller
>> memory system) otherwise not surface at all for a production
>> (debug=n) build.
>>
>>> The issue doesn't reproduce when starting a PV (L2) guest inside an HVM
>>> (L1) guest.
>> "Does not" or "does"? In the former case - what is this supposed to
>> tell me?
> 
> What I am trying to say here is that the system_does not_ crash when
> starting the PV guest as level 2 guest.
> This is meant to be another data point to help analyzing the issue.
> 
>> In any case - without you sharing technical details (register/stack
>> dump of the crash at the very least) I don't think I have anything
>> at hand to look for possible problems.
> 
> At this point, I am just reporting the issue.  I have not been able to
> get the crash dump because the system immediately reboot.
> I'll try to boot Xen with "noreboot" and inspect the log for more clues.
> Any suggestions are welcome.
> 
> Thank you,
> 
> Suravee

  reply	other threads:[~2013-06-27  9:50 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-27  0:24 x86/AMD: Nested hvm crashes in 4.3 Suravee Suthikulanit
2013-06-27  8:22 ` Jan Beulich
2013-06-27  9:20   ` Suravee Suthikulpanit
2013-06-27  9:50     ` Egger, Christoph [this message]
2013-06-27 10:08     ` Jan Beulich
2013-06-27 10:24       ` Suravee Suthikulpanit
2013-06-27 10:28         ` Andrew Cooper
2013-06-27 10:33         ` Egger, Christoph
2013-06-27 11:14           ` Suravee Suthikulpanit
2013-06-28  0:44             ` Suravee Suthikulanit
2013-06-28  7:58               ` Jan Beulich
2013-06-28 14:20                 ` Suravee Suthikulanit
2013-06-28 14:24                   ` Andrew Cooper
2013-06-28 14:52                   ` Jan Beulich
2013-06-28 15:05                     ` Egger, Christoph
2013-06-27 11:20         ` George Dunlap
2013-06-27 11:37         ` Jan Beulich

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=51CC0AF0.1060101@amazon.de \
    --to=chegger@amazon.de \
    --cc=JBeulich@suse.com \
    --cc=Jacob.Shin@amd.com \
    --cc=keir@xen.org \
    --cc=sherry.hurwitz@amd.com \
    --cc=suravee.suthikulpanit@amd.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).