Re: [Xen-devel] [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-25 Thread Konrad Rzeszutek Wilk
> > > > Yes, I think that this version looks better > > > > > > But doesn't boot: > > > > > > (XEN) vmx_hybrid.c:710:d0 Dom:0 EPT violation 0x181 (r--/---), gpa > > > 0x00bf421e1c, mfn 0x, type 4. (XEN) > > > p2m-ept.c:642:d0 Walking EPT tables for domain 0 gfn bf421 (XEN) >

Re: [Xen-devel] [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-25 Thread Konrad Rzeszutek Wilk
Yes, I think that this version looks better But doesn't boot: (XEN) vmx_hybrid.c:710:d0 Dom:0 EPT violation 0x181 (r--/---), gpa 0x00bf421e1c, mfn 0x, type 4. (XEN) p2m-ept.c:642:d0 Walking EPT tables for domain 0 gfn bf421 (XEN) p2m-ept.c:648:d0 gfn

Re: [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-23 Thread Mukesh Rathor
On Tue, 23 Oct 2012 14:07:06 +0100 Stefano Stabellini wrote: > On Mon, 22 Oct 2012, Konrad Rzeszutek Wilk wrote: > > On Mon, Oct 22, 2012 at 02:34:44PM +0100, Stefano Stabellini wrote: > > > On Sat, 20 Oct 2012, Konrad Rzeszutek Wilk wrote: > > > > From: Mukesh Rathor > > > > > > > >

Re: [Xen-devel] [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-23 Thread Mukesh Rathor
On Tue, 23 Oct 2012 17:03:10 -0700 Mukesh Rathor wrote: > On Tue, 23 Oct 2012 16:47:29 -0700 > Mukesh Rathor wrote: > > > On Tue, 23 Oct 2012 14:07:06 +0100 > > Stefano Stabellini wrote: > > > > > On Mon, 22 Oct 2012, Konrad Rzeszutek Wilk wrote: > > > > On Mon, Oct 22, 2012 at 02:34:44PM

Re: [Xen-devel] [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-23 Thread Mukesh Rathor
On Tue, 23 Oct 2012 16:47:29 -0700 Mukesh Rathor wrote: > On Tue, 23 Oct 2012 14:07:06 +0100 > Stefano Stabellini wrote: > > > On Mon, 22 Oct 2012, Konrad Rzeszutek Wilk wrote: > > > On Mon, Oct 22, 2012 at 02:34:44PM +0100, Stefano Stabellini > > > wrote: > > > > On Sat, 20 Oct 2012, Konrad

Re: [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-23 Thread Stefano Stabellini
On Mon, 22 Oct 2012, Konrad Rzeszutek Wilk wrote: > On Mon, Oct 22, 2012 at 02:34:44PM +0100, Stefano Stabellini wrote: > > On Sat, 20 Oct 2012, Konrad Rzeszutek Wilk wrote: > > > From: Mukesh Rathor > > > > > > In enlighten.c for PVH we can trap cpuid via vmexit, so don't > > > need to use

Re: [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-23 Thread Stefano Stabellini
On Mon, 22 Oct 2012, Konrad Rzeszutek Wilk wrote: On Mon, Oct 22, 2012 at 02:34:44PM +0100, Stefano Stabellini wrote: On Sat, 20 Oct 2012, Konrad Rzeszutek Wilk wrote: From: Mukesh Rathor mukesh.rat...@oracle.com In enlighten.c for PVH we can trap cpuid via vmexit, so don't need to

Re: [Xen-devel] [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-23 Thread Mukesh Rathor
On Tue, 23 Oct 2012 16:47:29 -0700 Mukesh Rathor mukesh.rat...@oracle.com wrote: On Tue, 23 Oct 2012 14:07:06 +0100 Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: On Mon, 22 Oct 2012, Konrad Rzeszutek Wilk wrote: On Mon, Oct 22, 2012 at 02:34:44PM +0100, Stefano Stabellini

Re: [Xen-devel] [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-23 Thread Mukesh Rathor
On Tue, 23 Oct 2012 17:03:10 -0700 Mukesh Rathor mukesh.rat...@oracle.com wrote: On Tue, 23 Oct 2012 16:47:29 -0700 Mukesh Rathor mukesh.rat...@oracle.com wrote: On Tue, 23 Oct 2012 14:07:06 +0100 Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: On Mon, 22 Oct 2012, Konrad

Re: [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-23 Thread Mukesh Rathor
On Tue, 23 Oct 2012 14:07:06 +0100 Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: On Mon, 22 Oct 2012, Konrad Rzeszutek Wilk wrote: On Mon, Oct 22, 2012 at 02:34:44PM +0100, Stefano Stabellini wrote: On Sat, 20 Oct 2012, Konrad Rzeszutek Wilk wrote: From: Mukesh Rathor

Re: [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-22 Thread Konrad Rzeszutek Wilk
On Mon, Oct 22, 2012 at 02:34:44PM +0100, Stefano Stabellini wrote: > On Sat, 20 Oct 2012, Konrad Rzeszutek Wilk wrote: > > From: Mukesh Rathor > > > > In enlighten.c for PVH we can trap cpuid via vmexit, so don't > > need to use emulated prefix call. We also check for vector callback > > early

Re: [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-22 Thread Stefano Stabellini
On Sat, 20 Oct 2012, Konrad Rzeszutek Wilk wrote: > From: Mukesh Rathor > > In enlighten.c for PVH we can trap cpuid via vmexit, so don't > need to use emulated prefix call. We also check for vector callback > early on, as it is a required feature. PVH also runs at default kernel > IOPL. > > In

Re: [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-22 Thread Stefano Stabellini
On Sat, 20 Oct 2012, Konrad Rzeszutek Wilk wrote: From: Mukesh Rathor mukesh.rat...@oracle.com In enlighten.c for PVH we can trap cpuid via vmexit, so don't need to use emulated prefix call. We also check for vector callback early on, as it is a required feature. PVH also runs at default

Re: [PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-22 Thread Konrad Rzeszutek Wilk
On Mon, Oct 22, 2012 at 02:34:44PM +0100, Stefano Stabellini wrote: On Sat, 20 Oct 2012, Konrad Rzeszutek Wilk wrote: From: Mukesh Rathor mukesh.rat...@oracle.com In enlighten.c for PVH we can trap cpuid via vmexit, so don't need to use emulated prefix call. We also check for vector

[PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-19 Thread Konrad Rzeszutek Wilk
From: Mukesh Rathor In enlighten.c for PVH we can trap cpuid via vmexit, so don't need to use emulated prefix call. We also check for vector callback early on, as it is a required feature. PVH also runs at default kernel IOPL. In setup.c, in xen_add_extra_mem() we can skip updating P2M as it's

[PATCH 4/6] xen/pvh: bootup and setup related changes.

2012-10-19 Thread Konrad Rzeszutek Wilk
From: Mukesh Rathor mukesh.rat...@oracle.com In enlighten.c for PVH we can trap cpuid via vmexit, so don't need to use emulated prefix call. We also check for vector callback early on, as it is a required feature. PVH also runs at default kernel IOPL. In setup.c, in xen_add_extra_mem() we can