Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-29 Thread Tamas K Lengyel
On Thu, Jul 28, 2016 at 10:16 PM, Razvan Cojocaru wrote: > On 07/29/16 00:25, Julien Grall wrote: >> >> >> On 28/07/2016 22:05, Tamas K Lengyel wrote: >>> On Thu, Jul 28, 2016 at 3:01 PM, Julien Grall >>> wrote: >>> That's not how we do it with vm_event. Even on x86 we only selectively >>> set re

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-29 Thread Tamas K Lengyel
On Jul 29, 2016 08:21, "Razvan Cojocaru" wrote: > > On 07/29/2016 05:11 PM, Julien Grall wrote: > > Which lead to my next question. For instance, we have an app A which is > > built for Xen 4.N and they want to also support Xen 4.(N + 1) which will > > set more registers and take advantage of it.

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-29 Thread Razvan Cojocaru
On 07/29/2016 05:11 PM, Julien Grall wrote: > Which lead to my next question. For instance, we have an app A which is > built for Xen 4.N and they want to also support Xen 4.(N + 1) which will > set more registers and take advantage of it. How the app will > differentiate the 2 versions? We curren

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-29 Thread Julien Grall
Hi Tamas, On 28/07/16 23:45, Tamas K Lengyel wrote: On Thu, Jul 28, 2016 at 4:03 PM, Julien Grall wrote: On 28/07/2016 22:33, Tamas K Lengyel wrote: On Jul 28, 2016 15:25, "Julien Grall" mailto:julien.gr...@arm.com>> wrote: On 28/07/2016 22:05, Tamas K Lengyel wrote: On Thu, Jul 28

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Razvan Cojocaru
On 07/29/16 00:25, Julien Grall wrote: > > > On 28/07/2016 22:05, Tamas K Lengyel wrote: >> On Thu, Jul 28, 2016 at 3:01 PM, Julien Grall >> wrote: >> That's not how we do it with vm_event. Even on x86 we only selectively >> set registers using the VM_EVENT_FLAG_SET_REGISTERS flag (albeit it >>

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Tamas K Lengyel
On Thu, Jul 28, 2016 at 4:03 PM, Julien Grall wrote: > > > On 28/07/2016 22:33, Tamas K Lengyel wrote: >> >> On Jul 28, 2016 15:25, "Julien Grall" > > wrote: >>> >>> >>> >>> >>> On 28/07/2016 22:05, Tamas K Lengyel wrote: On Thu, Jul 28, 2016 at 3:01 PM,

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Julien Grall
On 28/07/2016 22:33, Tamas K Lengyel wrote: On Jul 28, 2016 15:25, "Julien Grall" mailto:julien.gr...@arm.com>> wrote: On 28/07/2016 22:05, Tamas K Lengyel wrote: On Thu, Jul 28, 2016 at 3:01 PM, Julien Grall > wrote: That's not how we do it with vm_event. E

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Tamas K Lengyel
On Jul 28, 2016 15:25, "Julien Grall" wrote: > > > > On 28/07/2016 22:05, Tamas K Lengyel wrote: >> >> On Thu, Jul 28, 2016 at 3:01 PM, Julien Grall wrote: >> That's not how we do it with vm_event. Even on x86 we only selectively >> set registers using the VM_EVENT_FLAG_SET_REGISTERS flag (albeit

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Julien Grall
On 28/07/2016 22:05, Tamas K Lengyel wrote: On Thu, Jul 28, 2016 at 3:01 PM, Julien Grall wrote: That's not how we do it with vm_event. Even on x86 we only selectively set registers using the VM_EVENT_FLAG_SET_REGISTERS flag (albeit it not being documented in the header). As for "not exposing

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Tamas K Lengyel
On Thu, Jul 28, 2016 at 3:01 PM, Julien Grall wrote: > > > On 28/07/2016 21:48, Tamas K Lengyel wrote: >> >> On Thu, Jul 28, 2016 at 2:41 PM, Andrew Cooper >> wrote: >>> >>> On 28/07/2016 21:36, Tamas K Lengyel wrote: On Thu, Jul 28, 2016 at 2:26 PM, Andrew Cooper wrote: > >>>

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Julien Grall
On 28/07/2016 21:48, Tamas K Lengyel wrote: On Thu, Jul 28, 2016 at 2:41 PM, Andrew Cooper wrote: On 28/07/2016 21:36, Tamas K Lengyel wrote: On Thu, Jul 28, 2016 at 2:26 PM, Andrew Cooper wrote: On 28/07/2016 21:05, Tamas K Lengyel wrote: Add support for getting/setting registers through

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Tamas K Lengyel
On Thu, Jul 28, 2016 at 2:41 PM, Andrew Cooper wrote: > On 28/07/2016 21:36, Tamas K Lengyel wrote: >> On Thu, Jul 28, 2016 at 2:26 PM, Andrew Cooper >> wrote: >>> On 28/07/2016 21:05, Tamas K Lengyel wrote: Add support for getting/setting registers through vm_event on ARM. The set of r

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Andrew Cooper
On 28/07/2016 21:36, Tamas K Lengyel wrote: > On Thu, Jul 28, 2016 at 2:26 PM, Andrew Cooper > wrote: >> On 28/07/2016 21:05, Tamas K Lengyel wrote: >>> Add support for getting/setting registers through vm_event on ARM. >>> The set of registers can be expanded in the future to include other >>> r

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Tamas K Lengyel
On Thu, Jul 28, 2016 at 2:38 PM, Julien Grall wrote: > > > On 28/07/2016 21:36, Tamas K Lengyel wrote: >> >> On Thu, Jul 28, 2016 at 2:26 PM, Andrew Cooper >> wrote: >>> >>> On 28/07/2016 21:05, Tamas K Lengyel wrote: Add support for getting/setting registers through vm_event on ARM. >>

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Julien Grall
On 28/07/2016 21:36, Tamas K Lengyel wrote: On Thu, Jul 28, 2016 at 2:26 PM, Andrew Cooper wrote: On 28/07/2016 21:05, Tamas K Lengyel wrote: Add support for getting/setting registers through vm_event on ARM. The set of registers can be expanded in the future to include other registers as we

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Tamas K Lengyel
On Thu, Jul 28, 2016 at 2:26 PM, Andrew Cooper wrote: > On 28/07/2016 21:05, Tamas K Lengyel wrote: >> Add support for getting/setting registers through vm_event on ARM. >> The set of registers can be expanded in the future to include other registers >> as well if necessary but for now it is limit

Re: [Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Andrew Cooper
On 28/07/2016 21:05, Tamas K Lengyel wrote: > Add support for getting/setting registers through vm_event on ARM. > The set of registers can be expanded in the future to include other registers > as well if necessary but for now it is limited to TTB/CR/R0/R1, PC and CPSR. > > Signed-off-by: Tamas K

[Xen-devel] [PATCH] arm/vm_event: get/set registers

2016-07-28 Thread Tamas K Lengyel
Add support for getting/setting registers through vm_event on ARM. The set of registers can be expanded in the future to include other registers as well if necessary but for now it is limited to TTB/CR/R0/R1, PC and CPSR. Signed-off-by: Tamas K Lengyel --- Cc: Stefano Stabellini Cc: Julien Grall