From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Mukesh Rathor <mukesh.rathor@oracle.com>
Cc: "boris.ostrovsky@oracle.com" <boris.ostrovsky@oracle.com>,
"Xen-devel@lists.xensource.com" <Xen-devel@lists.xensource.com>,
David Vrabel <david.vrabel@citrix.com>
Subject: Re: linux PVH broken in 3.16.0-rc7 (arch_gnttab_map_shared)
Date: Wed, 6 Aug 2014 21:44:18 -0400 [thread overview]
Message-ID: <20140807014417.GB2434@konrad-lan.dumpdata.com> (raw)
In-Reply-To: <20140806142325.2d8ec3bf@mantra.us.oracle.com>
On Wed, Aug 06, 2014 at 02:23:25PM -0700, Mukesh Rathor wrote:
> On Tue, 5 Aug 2014 12:00:12 +0100
> David Vrabel <david.vrabel@citrix.com> wrote:
>
> > On 02/08/14 00:46, Mukesh Rathor wrote:
> > > Hi,
> > >
> > > Not sure if this already got reported, but it appears
> > >
> > > commit b7dd0e3
> > > Author: David Vrabel <david.vrabel@citrix.com>
> > > Date: Fri Jul 11 16:42:34 2014 +0100
> > >
> > > x86/xen: safely map and unmap grant frames when in atomic
> > > context
> > >
> > > has broken boot of domU PVH. The reason being arch_gnttab_map_shared
> > > is getting called before gnttab_shared_vm_area gets initialized by
> > > arch_gnttab_init(). As the comment says,
> > >
> > > --->
> > > /* Call it _before_ __gnttab_init as we need to initialize the
> > > * * xen_auto_xlat_grant_frames first. */
> > > core_initcall(xen_pvh_gnttab_setup);
> > > <---
> > >
> > > we can't wait till arch_gnttab_init is called by __gnttab_init().
> >
> > Does this fix it? I can't test it because the only box I have easily
> > available for testing is AMD.
> >
> > David
> >
>
> Well, I don't see the crash, but guest hangs. My hunch is the pte
> entries that are populated are not correct, but I don't have time to
> look into it right now, so I'll let you debug.
If you revert David's patch (x86/xen: safely map and unmap ..) does it
continue booting without the issue below?
Thanks!
>
> [ 1.905278] systemd[1]: Started dracut pre-udev hook.
> [ 1.905870] systemd[1]: Starting udev Kernel Device Manager...
> [ 1.945204] systemd-udevd[275]: starting version 204
> [ 2.625252] systemd-sysctl (526) used greatest stack depth: 13192 bytes left
> [ 49.832064] random: nonblocking pool is initialized
> .. hang..
>
> thanks
> mukesh
>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2014-08-07 1:44 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-01 23:46 linux PVH broken in 3.16.0-rc7 (arch_gnttab_map_shared) Mukesh Rathor
2014-08-04 10:33 ` David Vrabel
2014-08-04 14:46 ` Konrad Rzeszutek Wilk
2014-08-05 11:00 ` David Vrabel
2014-08-06 21:23 ` Mukesh Rathor
2014-08-07 1:44 ` Konrad Rzeszutek Wilk [this message]
2014-08-07 9:44 ` David Vrabel
2014-08-07 23:45 ` Mukesh Rathor
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=20140807014417.GB2434@konrad-lan.dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=Xen-devel@lists.xensource.com \
--cc=boris.ostrovsky@oracle.com \
--cc=david.vrabel@citrix.com \
--cc=mukesh.rathor@oracle.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.