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

Re: [Xen-devel] [PATCH] x86/HVM: move vendor independent CPU save/restore logic to shared code

>>> On 05.10.18 at 14:18, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 05/10/18 12:31, Jan Beulich wrote:
>> A few pieces of the handling here are (no longer?) vendor specific, and
>> hence there's no point in replicating the code.
> EFER probably was vendor specific originally.  The control registers
> really shouldn't have been...
>> Make sure not otherwise
>> pre-filled fields of struct hvm_hw_cpu instances are zero filled before
>> calling the vendor "save" hook, eliminating the need for the hook
>> functions to zero individual fields.
> The start of this sentence is rather hard to parse.  How about "Zero the
> full structure before calling the save hook, eliminating the need to
> zero individual fields." ?

Fine with me, changed.

>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> Code wise, this looks fine.  Reviewed-by: Andrew Cooper
> <andrew.cooper3@xxxxxxxxxx>



Xen-devel mailing list



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