On Mon, 2016-01-25 at 08:46 -0700, Jan Beulich wrote:
> > > > On 19.01.16 at 08:30, <huaitong....@intel.com> wrote: > > > > + write_cr4(cr4 | X86_CR4_PKE); > > + asm volatile (".byte 0x0f,0x01,0xee" > > + : "=a" (pkru) : "c" (0) : "dx"); > > + write_cr4(cr4); > > I think you will want to abstract out the actual writing of CR4 from > write_cr4(), as updating this_cpu(cr4) back and forth is quite > pointless here. > Updating this_cpu(cr4) back and forth is pointless, but using "asm volatile ( "mov %0,%%cr4" : : "r" (val) )" directly here is also bad code style, as there are two places you can update cr4 directly. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel