Re: [Xen-devel] [PATCH V2 1/2] x86/vm_event: added hvm/vm_event.{h, c}

2017-05-03 Thread Razvan Cojocaru
On 05/03/2017 11:32 PM, Tamas K Lengyel wrote: > On Wed, May 3, 2017 at 4:16 PM, Razvan Cojocaru > wrote: >> On 05/03/2017 11:05 PM, Tamas K Lengyel wrote: >>> >>> >>> On Wed, May 3, 2017 at 6:48 AM, Jan Beulich >> > wrote: >>> >>> >>> On 03.05.17 at 12:37, >>

Re: [Xen-devel] [PATCH V2 1/2] x86/vm_event: added hvm/vm_event.{h, c}

2017-05-03 Thread Tamas K Lengyel
On Wed, May 3, 2017 at 4:16 PM, Razvan Cojocaru wrote: > On 05/03/2017 11:05 PM, Tamas K Lengyel wrote: >> >> >> On Wed, May 3, 2017 at 6:48 AM, Jan Beulich > > wrote: >> >> >>> On 03.05.17 at 12:37, > > wrote: >> > On 05/03/17 12

Re: [Xen-devel] [PATCH V2 1/2] x86/vm_event: added hvm/vm_event.{h, c}

2017-05-03 Thread Razvan Cojocaru
On 05/03/2017 11:05 PM, Tamas K Lengyel wrote: > > > On Wed, May 3, 2017 at 6:48 AM, Jan Beulich > wrote: > > >>> On 03.05.17 at 12:37, > wrote: > > On 05/03/17 12:51, Jan Beulich wrote: > > On 03.05.17 at 11:10,

Re: [Xen-devel] [PATCH V2 1/2] x86/vm_event: added hvm/vm_event.{h, c}

2017-05-03 Thread Tamas K Lengyel
On Wed, May 3, 2017 at 6:48 AM, Jan Beulich wrote: > >>> On 03.05.17 at 12:37, wrote: > > On 05/03/17 12:51, Jan Beulich wrote: > > On 03.05.17 at 11:10, wrote: > >>> --- /dev/null > >>> +++ b/xen/arch/x86/hvm/vm_event.c > >>> @@ -0,0 +1,101 @@ > >>> +/* > >>> + * arch/x86/hvm/vm_event.c >

Re: [Xen-devel] [PATCH V2 1/2] x86/vm_event: added hvm/vm_event.{h, c}

2017-05-03 Thread Jan Beulich
>>> On 03.05.17 at 12:37, wrote: > On 05/03/17 12:51, Jan Beulich wrote: > On 03.05.17 at 11:10, wrote: >>> --- /dev/null >>> +++ b/xen/arch/x86/hvm/vm_event.c >>> @@ -0,0 +1,101 @@ >>> +/* >>> + * arch/x86/hvm/vm_event.c >>> + * >>> + * HVM vm_event handling routines >>> + * >>> + * Copyrigh

Re: [Xen-devel] [PATCH V2 1/2] x86/vm_event: added hvm/vm_event.{h, c}

2017-05-03 Thread Razvan Cojocaru
On 05/03/17 12:51, Jan Beulich wrote: On 03.05.17 at 11:10, wrote: >> @@ -483,67 +483,7 @@ void hvm_do_resume(struct vcpu *v) >> if ( !handle_hvm_io_completion(v) ) >> return; >> >> -if ( unlikely(v->arch.vm_event) ) >> -{ >> -struct monitor_write_data *w = &v-

Re: [Xen-devel] [PATCH V2 1/2] x86/vm_event: added hvm/vm_event.{h, c}

2017-05-03 Thread Jan Beulich
>>> On 03.05.17 at 11:10, wrote: > @@ -483,67 +483,7 @@ void hvm_do_resume(struct vcpu *v) > if ( !handle_hvm_io_completion(v) ) > return; > > -if ( unlikely(v->arch.vm_event) ) > -{ > -struct monitor_write_data *w = &v->arch.vm_event->write_data; > - > -if

[Xen-devel] [PATCH V2 1/2] x86/vm_event: added hvm/vm_event.{h,c}

2017-05-03 Thread Razvan Cojocaru
Created arch/x86/hvm/vm_event.c and include/asm-x86/hvm/vm_event.h, where HVM-specific vm_event-related code will live. This cleans up hvm_do_resume() and ensures that the vm_event maintainers are responsible for changes to that code. Signed-off-by: Razvan Cojocaru --- MAINTAINERS