From: Olaf Hering <olaf@aepfle.de>
To: Tim Deegan <Tim.Deegan@citrix.com>
Cc: xen-devel@lists.xensource.com
Subject: Re: [PATCH 0 of 5] update xenctx to dump pagetables
Date: Mon, 20 Jun 2011 19:51:58 +0200 [thread overview]
Message-ID: <20110620175158.GA17413@aepfle.de> (raw)
In-Reply-To: <20110620173345.GO17634@whitby.uk.xensource.com>
On Mon, Jun 20, Tim Deegan wrote:
> Hi,
>
> At 17:26 +0200 on 20 Jun (1308590816), Olaf Hering wrote:
> > The code to walk the pagetables is/was based on
> > xc_translate_foreign_address(), but I still think I have some major bugs in
> > there (the last patch). I cant figure out why some l3/l2/l1 entries can not be mapped.
> > Any help with getting that output fixed for at least a a 64bit PV guest is much
> > appreciated.
>
> I didn't spot anything broken in the walker (though it will need a bunch
> of cleaning up) and the output looks very plausible. I take it this is
> a PV guest? Is it a well-behaved one or a broken post-migrate one?
Yes, its a PV guest. I cant reproduce the migrate crashes, it happens
only on very few systems or on a certain configuration. The logs I
posted are from my test system.
Any ideas why some mfns are not accessible?
Are there any other paging states maintained outside of the guests
memory?
> One thing that might be causing trouble is if the domain's not paused
> while you do the walk, then you might see inconsistent tables, though
> I'd expect them to be garbage rather than looking like this. Your patch
> 3/5 does seem to make the pausing conditional where before it always
> happened.
It tries it preserve the state, if the guest was paused it probably
should not be unpaused. Is the dominfo.paused flag somehow unreliable, I
thought it comes right from struct domain?
next prev parent reply other threads:[~2011-06-20 17:51 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-20 15:26 [PATCH 0 of 5] update xenctx to dump pagetables Olaf Hering
2011-06-20 15:26 ` [PATCH 1 of 5] xenctx: recognize also -S option for --stack-trace Olaf Hering
2011-06-20 15:26 ` [PATCH 2 of 5] xenctx: move all globals into struct xenctx Olaf Hering
2011-06-21 12:36 ` Ian Jackson
2011-06-21 15:27 ` Olaf Hering
2011-06-21 15:31 ` Ian Jackson
2011-06-20 15:26 ` [PATCH 3 of 5] xenctx: move xc_* access out of dump_ctx Olaf Hering
2011-06-20 15:27 ` [PATCH 4 of 5] xenctx: add option -C to dump context for all vcpus Olaf Hering
2011-06-20 15:27 ` [PATCH 5 of 5] xenctx: dump pagetable Olaf Hering
2011-06-21 12:33 ` Ian Jackson
2011-06-21 15:25 ` Olaf Hering
2011-06-20 17:33 ` [PATCH 0 of 5] update xenctx to dump pagetables Tim Deegan
2011-06-20 17:51 ` Olaf Hering [this message]
2011-06-20 21:56 ` Konrad Rzeszutek Wilk
2011-06-21 9:59 ` Tim Deegan
2011-06-21 10:31 ` Jan Beulich
2011-06-22 13:50 ` Olaf Hering
2011-06-22 14:34 ` 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=20110620175158.GA17413@aepfle.de \
--to=olaf@aepfle.de \
--cc=Tim.Deegan@citrix.com \
--cc=xen-devel@lists.xensource.com \
/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.