|  |  | 
  
    |  |  | 
 
  |   |  | 
  
    |  |  | 
  
    |  |  | 
  
    |   xen-devel
RE: [Xen-devel] [Patch] Enable SMEP CPU feature support for  XEN	itself 
| To: | Keir Fraser <keir.xen@xxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>,	"Yang, Wei Y" <wei.y.yang@xxxxxxxxx> |  
| Subject: | RE: [Xen-devel] [Patch] Enable SMEP CPU feature support for  XEN	itself |  
| From: | "Li, Xin" <xin.li@xxxxxxxxx> |  
| Date: | Fri, 3 Jun 2011 06:49:55 +0800 |  
| Accept-language: | zh-CN, en-US |  
| Acceptlanguage: | zh-CN, en-US |  
| Cc: | "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx> |  
| Delivery-date: | Thu, 02 Jun 2011 15:50:37 -0700 |  
| Envelope-to: | www-data@xxxxxxxxxxxxxxxxxxx |  
| In-reply-to: | <CA0DA1F8.1B89A%keir.xen@xxxxxxxxx> |  
| List-help: | <mailto:xen-devel-request@lists.xensource.com?subject=help> |  
| List-id: | Xen developer discussion <xen-devel.lists.xensource.com> |  
| List-post: | <mailto:xen-devel@lists.xensource.com> |  
| List-subscribe: | <http://lists.xensource.com/mailman/listinfo/xen-devel>,	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |  
| List-unsubscribe: | <http://lists.xensource.com/mailman/listinfo/xen-devel>,	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |  
| References: | <FC2FB65B4D919844ADE4BE3C2BB739AD5AB9E58C@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>	<CA0DA1F8.1B89A%keir.xen@xxxxxxxxx> |  
| Sender: | xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |  
| Thread-index: | AcwhKSFqC1+9aLNySkqezZkJGvVaUgACElDwAApQ4SEABxMYEA== |  
| Thread-topic: | [Xen-devel] [Patch] Enable SMEP CPU feature support for  XEN	itself |  
| > >> } else
> >>> setup_clear_cpu_cap(X86_FEATURE_SMEP);
> >
> > This needs to be done on APs too.  Thus I think we still need define
> > setup_smep as __cpuinit.
> 
> The whole point of setup_clear_cpu_cap() is that it only needs to run on the
> BP and the change gets picked up by every AP automatically as it boots.
Yeah, and we can't do leaf 7.0 detect in generic_identify. Or will again set 
SMEP in capabilities, but it should be ok.
Thanks!
-Xin
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
 | 
 |  | 
  
    |  |  |