Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-21 Thread Bhupinder Thakur
Hi Julien, On 20 June 2017 at 16:46, Julien Grall wrote: > On 06/19/2017 02:11 PM, Bhupinder Thakur wrote: >> >> Hi Wei, > > > Hi Bhupinder, > > >> On 19 June 2017 at 17:17, Wei Liu wrote: >>> >>> On Mon, Jun 19, 2017 at 12:01:32PM +0100, Julien Grall

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-20 Thread Wei Liu
On Tue, Jun 20, 2017 at 12:16:52PM +0100, Julien Grall wrote: > On 06/19/2017 02:11 PM, Bhupinder Thakur wrote: > > Hi Wei, > > Hi Bhupinder, > > > On 19 June 2017 at 17:17, Wei Liu wrote: > > > On Mon, Jun 19, 2017 at 12:01:32PM +0100, Julien Grall wrote: > > > > > > > >

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-20 Thread Julien Grall
On 06/19/2017 02:11 PM, Bhupinder Thakur wrote: Hi Wei, Hi Bhupinder, On 19 June 2017 at 17:17, Wei Liu wrote: On Mon, Jun 19, 2017 at 12:01:32PM +0100, Julien Grall wrote: On 19/06/17 11:59, Bhupinder Thakur wrote: Hi Julien, I was mistaken in my earlier mail

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-19 Thread Bhupinder Thakur
Hi Wei, On 19 June 2017 at 17:17, Wei Liu wrote: > On Mon, Jun 19, 2017 at 12:01:32PM +0100, Julien Grall wrote: >> >> >> On 19/06/17 11:59, Bhupinder Thakur wrote: >> > Hi Julien, >> > >> > I was mistaken in my earlier mail about vpl011 init working if it is >> > moved to

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-19 Thread Wei Liu
On Mon, Jun 19, 2017 at 12:01:32PM +0100, Julien Grall wrote: > > > On 19/06/17 11:59, Bhupinder Thakur wrote: > > Hi Julien, > > > > I was mistaken in my earlier mail about vpl011 init working if it is > > moved to libxl__arch_domain_create(). It is failing because as you > > have mentioned

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-19 Thread Julien Grall
On 19/06/17 11:59, Bhupinder Thakur wrote: Hi Julien, I was mistaken in my earlier mail about vpl011 init working if it is moved to libxl__arch_domain_create(). It is failing because as you have mentioned vuart_pfn is allocated later in xc_dom_build_image(). Can we delay mapping of this page

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-19 Thread Bhupinder Thakur
Hi Julien, I was mistaken in my earlier mail about vpl011 init working if it is moved to libxl__arch_domain_create(). It is failing because as you have mentioned vuart_pfn is allocated later in xc_dom_build_image(). Can we delay mapping of this page in Xen until the ring buffer is actually

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-15 Thread Bhupinder Thakur
Hi Julien, diff --git a/tools/libxl/libxl_arch.h b/tools/libxl/libxl_arch.h index 5e1fc60..d1ca9c6 100644 --- a/tools/libxl/libxl_arch.h +++ b/tools/libxl/libxl_arch.h @@ -32,6 +32,13 @@ _hidden int libxl__arch_domain_create(libxl__gc *gc, libxl_domain_config

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-14 Thread Julien Grall
On 06/14/2017 10:16 AM, Bhupinder Thakur wrote: Hi Julien, Hi Bhupinder, diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h index 1629f41..77425dd 100644 --- a/tools/libxc/include/xenctrl.h +++ b/tools/libxc/include/xenctrl.h @@ -884,6 +884,23 @@ int

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-14 Thread Bhupinder Thakur
Hi Julien, >> diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h >> index 1629f41..77425dd 100644 >> --- a/tools/libxc/include/xenctrl.h >> +++ b/tools/libxc/include/xenctrl.h >> @@ -884,6 +884,23 @@ int xc_vcpu_getcontext(xc_interface *xch, >>

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-14 Thread Bhupinder Thakur
On 14 June 2017 at 13:05, Bhupinder Thakur wrote: > Hi Stefano, > > On 7 June 2017 at 04:56, Stefano Stabellini wrote: >>> static int set_vnuma_info(libxl__gc *gc, uint32_t domid, >>> @@ -788,6 +791,7 @@ int libxl__build_pv(libxl__gc *gc,

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-14 Thread Bhupinder Thakur
Hi Stefano, On 7 June 2017 at 04:56, Stefano Stabellini wrote: >> static int set_vnuma_info(libxl__gc *gc, uint32_t domid, >> @@ -788,6 +791,7 @@ int libxl__build_pv(libxl__gc *gc, uint32_t domid, >> if (xc_dom_translated(dom)) { >> state->console_mfn =

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-09 Thread Stefano Stabellini
On Fri, 9 Jun 2017, Julien Grall wrote: > Hi Stefano, > > On 07/06/17 00:26, Stefano Stabellini wrote: > > > diff --git a/tools/libxc/xc_domain.c b/tools/libxc/xc_domain.c > > > index 00909ad4..a8efd5e 100644 > > > --- a/tools/libxc/xc_domain.c > > > +++ b/tools/libxc/xc_domain.c > > > @@ -343,6

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-09 Thread Julien Grall
Hi Bhupinder, On 06/06/17 18:25, Bhupinder Thakur wrote: diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h index 1629f41..77425dd 100644 --- a/tools/libxc/include/xenctrl.h +++ b/tools/libxc/include/xenctrl.h @@ -884,6 +884,23 @@ int xc_vcpu_getcontext(xc_interface

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-09 Thread Julien Grall
Hi Stefano, On 07/06/17 00:26, Stefano Stabellini wrote: diff --git a/tools/libxc/xc_domain.c b/tools/libxc/xc_domain.c index 00909ad4..a8efd5e 100644 --- a/tools/libxc/xc_domain.c +++ b/tools/libxc/xc_domain.c @@ -343,6 +343,29 @@ int xc_domain_get_guest_width(xc_interface *xch, uint32_t

Re: [Xen-devel] [PATCH 06/14 v4] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-06-06 Thread Stefano Stabellini
On Tue, 6 Jun 2017, Bhupinder Thakur wrote: > Add a new domctl API to initialize vpl011. It takes the GFN and console > backend domid as input and returns an event channel to be used for > sending and receiving events from Xen. > > Xen will communicate with xenconsole using GFN as the ring buffer