On 30/08/18 02:39, Tian, Kevin wrote: >> From: Andrew Cooper [mailto:andrew.coop...@citrix.com] >> 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 <andrew.coop...@citrix.com> > Reviewed-by: Kevin Tian <kevin.t...@intel.com> > > 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 Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
