On 30/08/18 02:39, Tian, Kevin wrote:
>> From: Andrew Cooper [mailto:[email protected]]
>> Sent: Wednesday, August 29, 2018 1:39 AM
>>
>> By making {vmx,svm} in hvm_vcpu into an anonymous union (consistent
>> with
>> domain side of things), the hvm_{vmx,svm} defines can be dropped, and all
>> code
>> refer to the correctly-named fields.  This means that the data hierachy is no
>> longer obscured from grep/cscope/tags/etc.
>>
>> Reformat one comment and switch one bool_t to bool while making
>> changes.
>>
>> No functional change.
>>
>> Signed-off-by: Andrew Cooper <[email protected]>
> Reviewed-by: Kevin Tian <[email protected]>
>
> one small note. In coverletter:
>
>       " This series started by trying to address the bug in patch 7, 
> and ballooned somewhat."
>
> there is no bug per se in this patch, right?

I should probably have said issue rather than bug.  I was referring to
the obscuring of data from grep/cscope/tags/etc.

~Andrew

_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to