Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-04 Thread Tamas K Lengyel
On Wed, May 4, 2016 at 8:03 AM, Julien Grall wrote: > > > On 04/05/2016 14:30, Razvan Cojocaru wrote: >> >> On 05/04/2016 04:26 PM, Julien Grall wrote: >>> >>> I may misunderstand some parts of the vm event subsystem. To get/set the >>> full set of registers, the user will have to use the >>> DOMC

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-04 Thread Julien Grall
On 04/05/2016 14:30, Razvan Cojocaru wrote: On 05/04/2016 04:26 PM, Julien Grall wrote: I may misunderstand some parts of the vm event subsystem. To get/set the full set of registers, the user will have to use the DOMCTL_{set,get}vcpucontext, correct? So why does Xen expose a part of the vCPU

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-04 Thread Razvan Cojocaru
On 05/04/2016 04:26 PM, Julien Grall wrote: > I may misunderstand some parts of the vm event subsystem. To get/set the > full set of registers, the user will have to use the > DOMCTL_{set,get}vcpucontext, correct? So why does Xen expose a part of > the vCPU context through the vm_event? Because DO

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-04 Thread Julien Grall
Hi Tamas, I have just noticed that you use an old email address for Stefano. Please check MAINTAINERS file for any update on email address, new maintainers. On 04/05/2016 13:42, Tamas K Lengyel wrote: > On 03/05/2016 19:48, Tamas K Lengyel wrote: >> >> >> >> On Tue, May 3, 2016 at 5:31

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-04 Thread Tamas K Lengyel
> On 03/05/2016 19:48, Tamas K Lengyel wrote: >> >> >> >> On Tue, May 3, 2016 at 5:31 AM, Julien Grall > > wrote: >> On 29/04/16 19:07, Tamas K Lengyel wrote: >> >> The ARM SMC instructions are already configured to trap to Xen >> by default. In >>

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-04 Thread Julien Grall
Hi Tamas, Can you configure your email client to quote properly? I.e using ">" rather than tabulation to show the quoting. On 03/05/2016 19:48, Tamas K Lengyel wrote: On Tue, May 3, 2016 at 5:31 AM, Julien Grall mailto:julien.gr...@arm.com>> wrote: On 29/04/16 19:07, Tamas K Lengyel wro

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-03 Thread Tamas K Lengyel
On Tue, May 3, 2016 at 5:31 AM, Julien Grall wrote: > Hi Tamas, > > > On 29/04/16 19:07, Tamas K Lengyel wrote: > >> The ARM SMC instructions are already configured to trap to Xen by >> default. In >> this patch we allow a user-space process in a privileged domain to receive >> notification of wh

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-03 Thread Julien Grall
Hi Tamas, On 29/04/16 19:07, Tamas K Lengyel wrote: The ARM SMC instructions are already configured to trap to Xen by default. In this patch we allow a user-space process in a privileged domain to receive notification of when such event happens through the vm_event subsystem by introducing the P

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-05-02 Thread Wei Liu
On Fri, Apr 29, 2016 at 12:07:30PM -0600, Tamas K Lengyel wrote: > tools/libxc/include/xenctrl.h | 2 + > tools/libxc/xc_monitor.c| 26 +++- Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-04-29 Thread Razvan Cojocaru
On 04/29/16 23:27, Tamas K Lengyel wrote: > > > > > > diff --git a/xen/common/vm_event.c b/xen/common/vm_event.c > > index 2906407..a29bda8 100644 > > --- a/xen/common/vm_event.c > > +++ b/xen/common/vm_event.c > > @@ -818,7 +818,6 @@ int vm_event_mo

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-04-29 Thread Tamas K Lengyel
> > > > >> >> > diff --git a/xen/common/vm_event.c b/xen/common/vm_event.c >> > index 2906407..a29bda8 100644 >> > --- a/xen/common/vm_event.c >> > +++ b/xen/common/vm_event.c >> > @@ -818,7 +818,6 @@ int vm_event_monitor_traps(struct vcpu *v, uint8_t >> sync, >> > req->altp2m_idx = altp2m

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-04-29 Thread Tamas K Lengyel
> @@ -2491,6 +2492,21 @@ bad_data_abort: > > inject_dabt_exception(regs, info.gva, hsr.len); > > } > > > > +static void do_trap_smc(struct cpu_user_regs *regs, const union hsr hsr) > > +{ > > +int rc = 0; > > +if ( current->domain->arch.monitor.privileged_call_enabled ) > > +{ > >

Re: [Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-04-29 Thread Razvan Cojocaru
On 04/29/16 21:07, Tamas K Lengyel wrote: > The ARM SMC instructions are already configured to trap to Xen by default. In > this patch we allow a user-space process in a privileged domain to receive > notification of when such event happens through the vm_event subsystem by > introducing the PRIVIL

[Xen-devel] [PATCH v2 2/5] vm_event: Implement ARM SMC events

2016-04-29 Thread Tamas K Lengyel
The ARM SMC instructions are already configured to trap to Xen by default. In this patch we allow a user-space process in a privileged domain to receive notification of when such event happens through the vm_event subsystem by introducing the PRIVILEGED_CALL type. Signed-off-by: Tamas K Lengyel -