[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v4 0/2] x86/HVM: Properly handle SMAP check in certain cases




> -----Original Message-----
> From: Sander Eikelenboom [mailto:linux@xxxxxxxxxxxxxx]
> Sent: Wednesday, July 30, 2014 9:14 PM
> To: Jan Beulich
> Cc: Wu, Feng; xen-devel@xxxxxxxxxxxxx; keir@xxxxxxx; tim@xxxxxxx
> Subject: Re: [PATCH v4 0/2] x86/HVM: Properly handle SMAP check in certain
> cases
> 
> 
> Wednesday, July 30, 2014, 10:24:39 AM, you wrote:
> 
> >>>> On 30.07.14 at 03:35, <feng.wu@xxxxxxxxx> wrote:
> >> V4:
> >> - Adjust the branch handling in update_runstate_area().
> >> - Remove the pointless initial value of smap_check_policy_t.
> >> - Using __packed__ attribute for smap_check_policy_t to make
> >> its size down to one byte.
> >> - Adjust the position of 'smap_check_policy' in struct arch_vcpu
> >> to use the current padding in this structure.
> >> - Coding style.
> >>
> >> Feng Wu (2):
> >>   x86/hvm: Always do SMAP check when updating runstate_guest(v)
> >>   x86/hvm: Always do SMAP check when updating secondary system time
> for
> >>     guest
> >>
> >>  xen/arch/x86/domain.c        | 27 +++++++++++++++++++++++----
> >>  xen/arch/x86/mm/guest_walk.c | 39
> ++++++++++++++++++++++++++-------------
> >>  xen/arch/x86/time.c          | 10 +++++++++-
> >>  xen/include/asm-x86/domain.h | 19 +++++++++++++++++--
> >>  4 files changed, 75 insertions(+), 20 deletions(-)
> 
> > Sander,
> 
> > can you confirm this addresses the issue you had?
> 
> > Jan
> 
> Hi Jan / Feng,
> 
> It wasn't so much of an issue in the sense that i only got this warning 
> (didn't
> notice any other adverse effects my self):
> 
> (XEN) [2014-07-30 11:41:02] Segment register inaccessible for d1v0
> (XEN) [2014-07-30 11:41:02] (If you see this outside of debugging activity, 
> please
> report to xen-devel@xxxxxxxxxxxxxxxxxxxx)

Printing out this message means there are some issues, since the function 
returns
from that point directly without executing the remaining stuff, which is 
incorrect.
> 
> But i just tested the v4 patches and the warning is gone !

Thanks a lot for the testing!

Thanks,
Feng

> 
> Thanks,
> 
> Sander


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.