Re: [Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-21 Thread Stefano Stabellini
On Fri, 20 May 2016, Jan Beulich wrote: > >>> On 20.05.16 at 17:54, wrote: > > On 20/05/16 17:36, Jan Beulich wrote: > > On 20.05.16 at 17:04, wrote: > >>> On 20/05/16 16:49, Jan Beulich wrote: > >>> On 20.05.16 at 15:22, wrote: > >

Re: [Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-20 Thread Jan Beulich
>>> On 20.05.16 at 17:54, wrote: > On 20/05/16 17:36, Jan Beulich wrote: > On 20.05.16 at 17:04, wrote: >>> On 20/05/16 16:49, Jan Beulich wrote: >>> On 20.05.16 at 15:22, wrote: > if ( guest_handle_is_null(runstate_guest(v)) )

Re: [Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-20 Thread Juergen Gross
On 20/05/16 17:36, Jan Beulich wrote: On 20.05.16 at 17:04, wrote: >> On 20/05/16 16:49, Jan Beulich wrote: >> On 20.05.16 at 15:22, wrote: if ( guest_handle_is_null(runstate_guest(v)) ) return 1; +update_flag =

Re: [Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-20 Thread Jan Beulich
>>> On 20.05.16 at 17:04, wrote: > On 20/05/16 16:49, Jan Beulich wrote: > On 20.05.16 at 15:22, wrote: >>> if ( guest_handle_is_null(runstate_guest(v)) ) >>> return 1; >>> >>> +update_flag = VM_ASSIST(v->domain, runstate_update_flag);

Re: [Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-20 Thread Juergen Gross
On 20/05/16 16:49, Jan Beulich wrote: On 20.05.16 at 15:22, wrote: >> --- a/xen/arch/x86/domain.c >> +++ b/xen/arch/x86/domain.c >> @@ -1925,13 +1925,37 @@ static void paravirt_ctxt_switch_to(struct vcpu *v) >> bool_t update_runstate_area(struct vcpu *v) >> { >>

Re: [Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-20 Thread Jan Beulich
>>> On 20.05.16 at 15:22, wrote: > --- a/xen/arch/x86/domain.c > +++ b/xen/arch/x86/domain.c > @@ -1925,13 +1925,37 @@ static void paravirt_ctxt_switch_to(struct vcpu *v) > bool_t update_runstate_area(struct vcpu *v) > { > bool_t rc; > +bool_t update_flag; I think

Re: [Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-20 Thread Andrew Cooper
On 20/05/16 14:22, Juergen Gross wrote: > diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c > index 1365b4a..91f256b 100644 > --- a/xen/arch/arm/domain.c > +++ b/xen/arch/arm/domain.c > @@ -239,10 +239,32 @@ static void ctxt_switch_to(struct vcpu *n) > /* Update per-VCPU guest runstate

Re: [Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-20 Thread Juergen Gross
On 20/05/16 15:34, Andrew Cooper wrote: > On 20/05/16 14:22, Juergen Gross wrote: >> diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c >> index 1365b4a..91f256b 100644 >> --- a/xen/arch/arm/domain.c >> +++ b/xen/arch/arm/domain.c >> @@ -239,10 +239,32 @@ static void ctxt_switch_to(struct

[Xen-devel] [PATCH 2/2] xen: add update indicator to vcpu_runstate_info

2016-05-20 Thread Juergen Gross
In order to support reading another vcpu's mapped vcpu_runstate_info an indicator for an occurring update of the runstate information is needed. Add the possibility to activate setting this indicator in the highest bit of state_entry_time via a vm_assist hypercall. When activated the update