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

Re: [Xen-devel] [PATCH v3 4/9] Clear AC bit in RFLAGS to protect Xen itself by SMAP




> -----Original Message-----
> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
> Sent: Tuesday, April 29, 2014 5:48 PM
> To: Wu, Feng
> Cc: andrew.cooper3@xxxxxxxxxx; ian.campbell@xxxxxxxxxx; Dong, Eddie;
> Nakajima, Jun; Tian, Kevin; xen-devel@xxxxxxxxxxxxx
> Subject: RE: [PATCH v3 4/9] Clear AC bit in RFLAGS to protect Xen itself by 
> SMAP
> 
> >>> On 29.04.14 at 11:27, <feng.wu@xxxxxxxxx> wrote:
> > May I move the definition of SAVE_ALL (or more related code, such as,
> > SAVE_VOLATILE, etc.) from " xen/include/asm-x86/x86_64/asm_defns.h " to "
> > xen/include/asm-x86/asm_defns.h ", since I got an build error after adding a
> > parameter to marco SAVE_ALL like this, which makes me think it should be
> > defined after ASM_CLAC?
> 
> No problem (and in fact I wouldn't mind you merging all of
> x86_64/asm_defns.h into the other file - that due at some point
> anyway), but I'm afraid we'd want this to be yet another separate
> prerequisite patch - not sure if you're up to that, or if you wouldn't
> find it simpler to keep SAVE_ALL being a C macro then.

I am fine with the merging task. But Is there anything hard to handle while 
merging x86_64/asm_defns.h to asm_defns.h, I am wondering
why x86_64/asm_defns.h is there for a long time.

> 
> Jan

Thanks,
Feng

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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