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

2016-06-02 Thread Razvan Cojocaru
On 06/02/2016 12:38 PM, Jan Beulich wrote: On 02.06.16 at 10:26, wrote: >> On 06/02/2016 10:35 AM, Jan Beulich wrote: >>> The criteria for inclusion or exclusion should >>> follow a predictable model. I.e. if someone comes along and says >>> "I need register Y", then there should be rules tha

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

2016-06-02 Thread Jan Beulich
>>> On 02.06.16 at 10:26, wrote: > On 06/02/2016 10:35 AM, Jan Beulich wrote: >> The criteria for inclusion or exclusion should >> follow a predictable model. I.e. if someone comes along and says >> "I need register Y", then there should be rules that (s)he can apply >> up front to determine what

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

2016-06-02 Thread Razvan Cojocaru
On 06/02/2016 10:35 AM, Jan Beulich wrote: > The criteria for inclusion or exclusion should > follow a predictable model. I.e. if someone comes along and says > "I need register Y", then there should be rules that (s)he can apply > up front to determine what (at least in the vast majority of cases)

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

2016-06-02 Thread Jan Beulich
>>> On 01.06.16 at 21:34, wrote: > On 06/01/16 21:21, Tamas K Lengyel wrote: >> On Wed, Jun 1, 2016 at 5:24 AM, Julien Grall wrote: >>> On 01/06/16 09:41, Jan Beulich wrote: Once an ABI is set in stone, and if that ABI allows for optimizations (by consumers) like the one mentioned, I do

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

2016-06-01 Thread Tamas K Lengyel
On Wed, Jun 1, 2016 at 1:38 PM, Julien Grall wrote: > Hi Tamas, > > > On 01/06/16 19:21, Tamas K Lengyel wrote: >> >> On Wed, Jun 1, 2016 at 5:24 AM, Julien Grall wrote: >>> >>> Hi, >>> >>> >>> On 01/06/16 09:41, Jan Beulich wrote: >>> >>> >>> On 31.05.16 at 18:28, wrote: > >

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

2016-06-01 Thread Julien Grall
On 01/06/16 20:38, Julien Grall wrote: Hi Tamas, On 01/06/16 19:21, Tamas K Lengyel wrote: On Wed, Jun 1, 2016 at 5:24 AM, Julien Grall wrote: Hi, On 01/06/16 09:41, Jan Beulich wrote: On 31.05.16 at 18:28, wrote: On May 31, 2016 01:48, "Jan Beulich" wrote: On 30.05.16 at 21:47,

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

2016-06-01 Thread Julien Grall
Hi Razvan, On 01/06/16 20:34, Razvan Cojocaru wrote: On 06/01/16 21:21, Tamas K Lengyel wrote: On Wed, Jun 1, 2016 at 5:24 AM, Julien Grall wrote: The only purpose of having that information in the request is to quickly get things that are immediately necessary - otherwise the full context ca

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

2016-06-01 Thread Julien Grall
Hi Tamas, On 01/06/16 19:21, Tamas K Lengyel wrote: On Wed, Jun 1, 2016 at 5:24 AM, Julien Grall wrote: Hi, On 01/06/16 09:41, Jan Beulich wrote: On 31.05.16 at 18:28, wrote: On May 31, 2016 01:48, "Jan Beulich" wrote: On 30.05.16 at 21:47, wrote: On Mon, May 30, 2016 at 5:50 AM

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

2016-06-01 Thread Razvan Cojocaru
On 06/01/16 21:21, Tamas K Lengyel wrote: > On Wed, Jun 1, 2016 at 5:24 AM, Julien Grall wrote: >> Hi, >> >> >> On 01/06/16 09:41, Jan Beulich wrote: >> >> On 31.05.16 at 18:28, wrote: On May 31, 2016 01:48, "Jan Beulich" wrote: > > On 30.05.16 at 21:47, wrote

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

2016-06-01 Thread Tamas K Lengyel
On Wed, Jun 1, 2016 at 5:24 AM, Julien Grall wrote: > Hi, > > > On 01/06/16 09:41, Jan Beulich wrote: > > On 31.05.16 at 18:28, wrote: >>> >>> On May 31, 2016 01:48, "Jan Beulich" wrote: >>> On 30.05.16 at 21:47, wrote: > > On Mon, May 30, 2016 at 5:50 AM, Jan Beul

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

2016-06-01 Thread Julien Grall
Hi, On 01/06/16 09:41, Jan Beulich wrote: On 31.05.16 at 18:28, wrote: On May 31, 2016 01:48, "Jan Beulich" wrote: On 30.05.16 at 21:47, wrote: On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: On 30.05.16 at 00:37, wrote: +struct vm_event_regs_arm32 { +uint32_t r0_usr; +ui

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

2016-06-01 Thread Jan Beulich
>>> On 31.05.16 at 18:28, wrote: > On May 31, 2016 01:48, "Jan Beulich" wrote: >> >> >>> On 30.05.16 at 21:47, wrote: >> > On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: >> > On 30.05.16 at 00:37, wrote: >> >>> +struct vm_event_regs_arm32 { >> >>> +uint32_t r0_usr; >> >>> +uin

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

2016-05-31 Thread Tamas K Lengyel
On May 31, 2016 01:48, "Jan Beulich" wrote: > > >>> On 30.05.16 at 21:47, wrote: > > On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: > > On 30.05.16 at 00:37, wrote: > >>> +struct vm_event_regs_arm32 { > >>> +uint32_t r0_usr; > >>> +uint32_t r1_usr; > >>> +uint32_t r2_usr; >

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

2016-05-31 Thread Tamas K Lengyel
On May 31, 2016 01:54, "Jan Beulich" wrote: > > >>> On 30.05.16 at 22:37, wrote: > > On Mon, May 30, 2016 at 2:20 PM, Julien Grall wrote: > >> On 30/05/2016 20:47, Tamas K Lengyel wrote: > >>> On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: > > +struct vm_event_regs_arm64 { > > +

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

2016-05-31 Thread Jan Beulich
>>> On 31.05.16 at 10:06, wrote: > On 05/31/2016 10:54 AM, Jan Beulich wrote: > On 30.05.16 at 22:37, wrote: >>> On Mon, May 30, 2016 at 2:20 PM, Julien Grall wrote: On 30/05/2016 20:47, Tamas K Lengyel wrote: > On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: >>> +struct vm

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

2016-05-31 Thread Razvan Cojocaru
On 05/31/2016 10:54 AM, Jan Beulich wrote: On 30.05.16 at 22:37, wrote: >> On Mon, May 30, 2016 at 2:20 PM, Julien Grall wrote: >>> On 30/05/2016 20:47, Tamas K Lengyel wrote: On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: >> +struct vm_event_regs_arm64 { >> +uint64_t

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

2016-05-31 Thread Jan Beulich
>>> On 30.05.16 at 22:37, wrote: > On Mon, May 30, 2016 at 2:20 PM, Julien Grall wrote: >> On 30/05/2016 20:47, Tamas K Lengyel wrote: >>> On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: > +struct vm_event_regs_arm64 { > +uint64_t x0; > +uint64_t x1; > +uint64_t x

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

2016-05-31 Thread Jan Beulich
>>> On 30.05.16 at 21:47, wrote: > On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: > On 30.05.16 at 00:37, wrote: >>> +struct vm_event_regs_arm32 { >>> +uint32_t r0_usr; >>> +uint32_t r1_usr; >>> +uint32_t r2_usr; >>> +uint32_t r3_usr; >>> +uint32_t r4_usr; >>> +u

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

2016-05-30 Thread Tamas K Lengyel
On Mon, May 30, 2016 at 3:35 PM, Julien Grall wrote: > > > On 30/05/2016 21:37, Tamas K Lengyel wrote: >> >> On Mon, May 30, 2016 at 2:20 PM, Julien Grall >> wrote: >>> >>> Hi Tamas, >>> >>> On 30/05/2016 20:47, Tamas K Lengyel wrote: On Mon, May 30, 2016 at 5:50 AM, Jan Beulich w

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

2016-05-30 Thread Julien Grall
On 30/05/2016 21:37, Tamas K Lengyel wrote: On Mon, May 30, 2016 at 2:20 PM, Julien Grall wrote: Hi Tamas, On 30/05/2016 20:47, Tamas K Lengyel wrote: On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: +struct vm_event_regs_arm64 { +uint64_t x0; +uint64_t x1; +uint64_t x2;

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

2016-05-30 Thread Tamas K Lengyel
On Mon, May 30, 2016 at 2:46 PM, Razvan Cojocaru wrote: > On 05/30/16 23:37, Tamas K Lengyel wrote: >> Well, as we discussed it in the previous revision, there is no >> hard-set rule of what can and cannot be transmitted here. The only >> thing to keep in mind is to not grow this struct to be too

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

2016-05-30 Thread Razvan Cojocaru
On 05/30/16 23:37, Tamas K Lengyel wrote: > Well, as we discussed it in the previous revision, there is no > hard-set rule of what can and cannot be transmitted here. The only > thing to keep in mind is to not grow this struct to be too large. The > registers sent right now represent a "best guess"

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

2016-05-30 Thread Tamas K Lengyel
On Mon, May 30, 2016 at 2:20 PM, Julien Grall wrote: > Hi Tamas, > > On 30/05/2016 20:47, Tamas K Lengyel wrote: >> >> On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: +struct vm_event_regs_arm64 { +uint64_t x0; +uint64_t x1; +uint64_t x2; +uint64_

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

2016-05-30 Thread Julien Grall
Hi Tamas, On 30/05/2016 20:47, Tamas K Lengyel wrote: On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: +struct vm_event_regs_arm64 { +uint64_t x0; +uint64_t x1; +uint64_t x2; +uint64_t x3; +uint64_t x4; +uint64_t x5; +uint64_t x6; +uint64_t x7; +uint64_t

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

2016-05-30 Thread Tamas K Lengyel
On Mon, May 30, 2016 at 5:50 AM, Jan Beulich wrote: On 30.05.16 at 00:37, wrote: >> +struct vm_event_regs_arm32 { >> +uint32_t r0_usr; >> +uint32_t r1_usr; >> +uint32_t r2_usr; >> +uint32_t r3_usr; >> +uint32_t r4_usr; >> +uint32_t r5_usr; >> +uint32_t r6_usr; >>

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

2016-05-30 Thread Jan Beulich
>>> On 30.05.16 at 00:37, wrote: > +struct vm_event_regs_arm32 { > +uint32_t r0_usr; > +uint32_t r1_usr; > +uint32_t r2_usr; > +uint32_t r3_usr; > +uint32_t r4_usr; > +uint32_t r5_usr; > +uint32_t r6_usr; > +uint32_t r7_usr; > +uint32_t r8_usr; > +uint32_t r

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

2016-05-30 Thread Razvan Cojocaru
On 05/30/2016 01:37 AM, Tamas K Lengyel wrote: > Add support for getting/setting registers through vm_event on ARM. > > Signed-off-by: Tamas K Lengyel > --- > Cc: Stefano Stabellini > Cc: Julien Grall > Cc: Razvan Cojocaru > > v4: Use psr mode to determine whether to full 32-bit or 64-bit str

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

2016-05-29 Thread Tamas K Lengyel
Add support for getting/setting registers through vm_event on ARM. Signed-off-by: Tamas K Lengyel --- Cc: Stefano Stabellini Cc: Julien Grall Cc: Razvan Cojocaru v4: Use psr mode to determine whether to full 32-bit or 64-bit structs --- xen/arch/arm/Makefile | 1 + xen/arch/arm/vm