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

Re: [Xen-devel] [PATCH] x86/xsave: Remove xfeat_mask checking from validate_xstate()



>>> Andrew Cooper <andrew.cooper3@xxxxxxxxxx> 05/30/14 10:39 AM >>>
>validate_xsave() is call from the HVM and PV codepaths which load new vcpu
>xsave state, usually as part of migration.  In both cases, this is the
>xfeature_mask of the saving Xen rather than the restoring Xen.
>
>Given that the xsave state itself is checked for consistency and validity on
>the current cpus, checking whether it was valid for the cpu before migration
>is not interesting (or indeed relevant, as the error can't be distinguished
>from the other validity checking).

While I'm not entirely opposed to this, I'm also not fully convinced - the data
being available, it can as well be used for additional sanity checking.

Jan


_______________________________________________
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®.