> From: Euan Harris [mailto:euan.har...@citrix.com] > Sent: Thursday, June 09, 2016 9:21 PM > > On Thu, Jun 09, 2016 at 06:47:55AM -0600, Jan Beulich wrote: > > > /* > > > - * Those MSRs are available only when bit 55 of > > > - * MSR_IA32_VMX_BASIC is set. > > > + * These MSRs are only available when flags in other MSRs are set. > > > + * These prerequisites are listed in the Intel 64 and IA-32 > > > + * Architectures Software Developer’s Manual, Vol 3, Appendix A. > > > */ > > > - switch ( msr ) > > > - { > > > + switch ( msr ) { case MSR_IA32_VMX_PROCBASED_CTLS2: > > > > I hope you didn't really mean to produce a garbled line like this? > > With proper formatting restored > > Reviewed-by: Jan Beulich <jbeul...@suse.com> > > Oops, definitely not. I think this happened when I was re-wrapping > the comment above - !}fmt went further than I intended. >
Acked-by: Kevin Tian <kevin.t...@intel.com>, with above fixed. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel