All of lore.kernel.org
 help / color / mirror / Atom feed
* [RFC PATCH 5/8]: PVH: smp changes
@ 2012-08-16  1:04 Mukesh Rathor
  2012-08-17  9:42 ` Ian Campbell
  0 siblings, 1 reply; 4+ messages in thread
From: Mukesh Rathor @ 2012-08-16  1:04 UTC (permalink / raw)
  To: Konrad Rzeszutek Wilk, Xen-devel


---
 arch/x86/xen/smp.c |   33 +++++++++++++++++++++++++--------
 1 files changed, 25 insertions(+), 8 deletions(-)

diff --git a/arch/x86/xen/smp.c b/arch/x86/xen/smp.c
index cdf269d..017d7fa 100644
--- a/arch/x86/xen/smp.c
+++ b/arch/x86/xen/smp.c
@@ -68,9 +68,11 @@ static void __cpuinit cpu_bringup(void)
 	touch_softlockup_watchdog();
 	preempt_disable();
 
-	xen_enable_sysenter();
-	xen_enable_syscall();
-
+	/* PVH runs in ring 0 and allows us to do native syscalls. Yay! */
+	if (!xen_pvh_domain()) {
+		xen_enable_sysenter();
+		xen_enable_syscall();
+	}
 	cpu = smp_processor_id();
 	smp_store_cpu_info(cpu);
 	cpu_data(cpu).x86_max_cores = 1;
@@ -230,10 +232,11 @@ static void __init xen_smp_prepare_boot_cpu(void)
 	BUG_ON(smp_processor_id() != 0);
 	native_smp_prepare_boot_cpu();
 
-	/* We've switched to the "real" per-cpu gdt, so make sure the
-	   old memory can be recycled */
-	make_lowmem_page_readwrite(xen_initial_gdt);
-
+	if (!xen_pvh_domain()) {
+		/* We've switched to the "real" per-cpu gdt, so make sure the
+		 * old memory can be recycled */
+		make_lowmem_page_readwrite(xen_initial_gdt);
+	}
 	xen_filter_cpu_maps();
 	xen_setup_vcpu_info_placement();
 }
@@ -312,6 +315,7 @@ cpu_initialize_context(unsigned int cpu, struct task_struct *idle)
 
 	memset(&ctxt->fpu_ctxt, 0, sizeof(ctxt->fpu_ctxt));
 
+	if (!xen_pvh_domain()) {
 		ctxt->user_regs.ds = __USER_DS;
 		ctxt->user_regs.es = __USER_DS;
 
@@ -339,7 +343,20 @@ cpu_initialize_context(unsigned int cpu, struct task_struct *idle)
 					(unsigned long)xen_hypervisor_callback;
 		ctxt->failsafe_callback_eip = 
 					(unsigned long)xen_failsafe_callback;
-
+	} else {
+		ctxt->user_regs.ds = __KERNEL_DS;
+		ctxt->user_regs.es = 0;
+		ctxt->user_regs.gs = 0;
+
+		ctxt->gdt_frames[0] = (unsigned long)gdt;
+		ctxt->gdt_ents = (unsigned long)(GDT_SIZE - 1);
+
+		/* Note: PVH is not supported on x86_32. */
+#ifdef __x86_64__
+		ctxt->gs_base_user = (unsigned long)
+		                         per_cpu(irq_stack_union.gs_base, cpu);
+#endif
+	}
 	ctxt->user_regs.cs = __KERNEL_CS;
 	ctxt->user_regs.esp = idle->thread.sp0 - sizeof(struct pt_regs);
 
-- 
1.7.2.3

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [RFC PATCH 5/8]: PVH: smp changes
  2012-08-16  1:04 [RFC PATCH 5/8]: PVH: smp changes Mukesh Rathor
@ 2012-08-17  9:42 ` Ian Campbell
  2012-08-22  0:23   ` Mukesh Rathor
  0 siblings, 1 reply; 4+ messages in thread
From: Ian Campbell @ 2012-08-17  9:42 UTC (permalink / raw)
  To: Mukesh Rathor; +Cc: Xen-devel, Konrad Rzeszutek Wilk

On Thu, 2012-08-16 at 02:04 +0100, Mukesh Rathor wrote:
> ---
>  arch/x86/xen/smp.c |   33 +++++++++++++++++++++++++--------
>  1 files changed, 25 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/x86/xen/smp.c b/arch/x86/xen/smp.c
> index cdf269d..017d7fa 100644
> --- a/arch/x86/xen/smp.c
> +++ b/arch/x86/xen/smp.c
> @@ -68,9 +68,11 @@ static void __cpuinit cpu_bringup(void)
>  	touch_softlockup_watchdog();
>  	preempt_disable();
>  
> -	xen_enable_sysenter();
> -	xen_enable_syscall();
> -
> +	/* PVH runs in ring 0 and allows us to do native syscalls. Yay! */
> +	if (!xen_pvh_domain()) {

Can this be gated on X86_FEATURE_BLAH, with appropriate masking for the
PV vs PVH cases.

Looks like enable_sysenter actually does that already, maybe
enable_syscall could too?

> +		xen_enable_sysenter();
> +		xen_enable_syscall();
> +	}
>  	cpu = smp_processor_id();
>  	smp_store_cpu_info(cpu);
>  	cpu_data(cpu).x86_max_cores = 1;
> @@ -230,10 +232,11 @@ static void __init xen_smp_prepare_boot_cpu(void)
>  	BUG_ON(smp_processor_id() != 0);
>  	native_smp_prepare_boot_cpu();
>  
> -	/* We've switched to the "real" per-cpu gdt, so make sure the
> -	   old memory can be recycled */
> -	make_lowmem_page_readwrite(xen_initial_gdt);
> -
> +	if (!xen_pvh_domain()) {

XENFEAT_writable_pagetalbes, I think? And possibly pushed down into
make_lowmem_page_readwrite?

> +		/* We've switched to the "real" per-cpu gdt, so make sure the
> +		 * old memory can be recycled */
> +		make_lowmem_page_readwrite(xen_initial_gdt);
> +	}
>  	xen_filter_cpu_maps();
>  	xen_setup_vcpu_info_placement();
>  }
> @@ -312,6 +315,7 @@ cpu_initialize_context(unsigned int cpu, struct task_struct *idle)
>  
>  	memset(&ctxt->fpu_ctxt, 0, sizeof(ctxt->fpu_ctxt));
>  
> +	if (!xen_pvh_domain()) {
>  		ctxt->user_regs.ds = __USER_DS;
>  		ctxt->user_regs.es = __USER_DS;
>  
> @@ -339,7 +343,20 @@ cpu_initialize_context(unsigned int cpu, struct task_struct *idle)
>  					(unsigned long)xen_hypervisor_callback;
>  		ctxt->failsafe_callback_eip = 
>  					(unsigned long)xen_failsafe_callback;
> -
> +	} else {
> +		ctxt->user_regs.ds = __KERNEL_DS;
> +		ctxt->user_regs.es = 0;
> +		ctxt->user_regs.gs = 0;

Not __KERNEL_DS for es too?

Not sure about gs -- shouldn't that point to some per-cpu segment or
something? Maybe that happens somewhere else? (in which case a comment?)

> +
> +		ctxt->gdt_frames[0] = (unsigned long)gdt;
> +		ctxt->gdt_ents = (unsigned long)(GDT_SIZE - 1);
> +
> +		/* Note: PVH is not supported on x86_32. */
> +#ifdef __x86_64__

ITYM CONFIG_X86_64?

> +		ctxt->gs_base_user = (unsigned long)
> +		                         per_cpu(irq_stack_union.gs_base, cpu);
> +#endif
> +	}
>  	ctxt->user_regs.cs = __KERNEL_CS;
>  	ctxt->user_regs.esp = idle->thread.sp0 - sizeof(struct pt_regs);
>  

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [RFC PATCH 5/8]: PVH: smp changes
  2012-08-17  9:42 ` Ian Campbell
@ 2012-08-22  0:23   ` Mukesh Rathor
  2012-08-22  6:10     ` Ian Campbell
  0 siblings, 1 reply; 4+ messages in thread
From: Mukesh Rathor @ 2012-08-22  0:23 UTC (permalink / raw)
  To: Ian Campbell; +Cc: Xen-devel, Konrad Rzeszutek Wilk

On Fri, 17 Aug 2012 10:42:11 +0100
Ian Campbell <Ian.Campbell@citrix.com> wrote:

> > @@ -339,7 +343,20 @@ cpu_initialize_context(unsigned int cpu,
> > struct task_struct *idle) (unsigned long)xen_hypervisor_callback;
> >  		ctxt->failsafe_callback_eip = 
> >  					(unsigned
> > long)xen_failsafe_callback; -
> > +	} else {
> > +		ctxt->user_regs.ds = __KERNEL_DS;
> > +		ctxt->user_regs.es = 0;
> > +		ctxt->user_regs.gs = 0;
> 
> Not __KERNEL_DS for es too?

64bit, es ignored, right?

> Not sure about gs -- shouldn't that point to some per-cpu segment or
> something? Maybe that happens somewhere else? (in which case a
> comment?)

Gets set later. comment good idea.

> > +
> > +		ctxt->gdt_frames[0] = (unsigned long)gdt;
> > +		ctxt->gdt_ents = (unsigned long)(GDT_SIZE - 1);
> > +
> > +		/* Note: PVH is not supported on x86_32. */
> > +#ifdef __x86_64__
> 
> ITYM CONFIG_X86_64?

Yup.

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [RFC PATCH 5/8]: PVH: smp changes
  2012-08-22  0:23   ` Mukesh Rathor
@ 2012-08-22  6:10     ` Ian Campbell
  0 siblings, 0 replies; 4+ messages in thread
From: Ian Campbell @ 2012-08-22  6:10 UTC (permalink / raw)
  To: Mukesh Rathor; +Cc: Xen-devel, Konrad Rzeszutek Wilk

On Wed, 2012-08-22 at 01:23 +0100, Mukesh Rathor wrote:
> On Fri, 17 Aug 2012 10:42:11 +0100
> Ian Campbell <Ian.Campbell@citrix.com> wrote:
> 
> > > @@ -339,7 +343,20 @@ cpu_initialize_context(unsigned int cpu,
> > > struct task_struct *idle) (unsigned long)xen_hypervisor_callback;
> > >  		ctxt->failsafe_callback_eip = 
> > >  					(unsigned
> > > long)xen_failsafe_callback; -
> > > +	} else {
> > > +		ctxt->user_regs.ds = __KERNEL_DS;
> > > +		ctxt->user_regs.es = 0;
> > > +		ctxt->user_regs.gs = 0;
> > 
> > Not __KERNEL_DS for es too?
> 
> 64bit, es ignored, right?

Not 100% sure, the limit certainly is but the base and the privilege
level might not be? 

The reason I mentioned it was that I could see in the !pvh case:
                ctxt->user_regs.es = __USER_DS;

Ian.

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-08-22  6:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-08-16  1:04 [RFC PATCH 5/8]: PVH: smp changes Mukesh Rathor
2012-08-17  9:42 ` Ian Campbell
2012-08-22  0:23   ` Mukesh Rathor
2012-08-22  6:10     ` Ian Campbell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.