pvh: disable pse feature for now

All of lore.kernel.org
 help / color / mirror / Atom feed

* pvh: disable pse feature for now
@ 2014-01-28  2:18 Mukesh Rathor
  2014-01-28  2:18 ` [V0 PATCH] pvh: Disable PSE " Mukesh Rathor
  0 siblings, 1 reply; 8+ messages in thread
From: Mukesh Rathor @ 2014-01-28  2:18 UTC (permalink / raw)
  To: konrad.wilk; +Cc: roger.pau, Xen-devel, linux-kernel

Konrad,

Following will turn off PSE in linux until we can get to it. It's better
to turn it off here than in xen, so if BSD gets there sooner, they are not 
dependent on us.

thanks
Mukesh

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [V0 PATCH] pvh: Disable PSE feature for now
  2014-01-28  2:18 pvh: disable pse feature for now Mukesh Rathor
@ 2014-01-28  2:18 ` Mukesh Rathor
  2014-01-28  3:46   ` Konrad Rzeszutek Wilk
                     ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Mukesh Rathor @ 2014-01-28  2:18 UTC (permalink / raw)
  To: konrad.wilk; +Cc: roger.pau, Xen-devel, linux-kernel

Until now, xen did not expose PSE to pvh guest, but a patch was submitted
to xen list to enable bunch of features for a pvh guest. PSE has not been
looked into for PVH, so until we can do that and test it to make sure it
works, disable the feature to avoid flood of bugs.

Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
---
 arch/x86/xen/enlighten.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c
index a4d7b64..4e952046 100644
--- a/arch/x86/xen/enlighten.c
+++ b/arch/x86/xen/enlighten.c
@@ -1497,6 +1497,11 @@ static void __init xen_pvh_early_guest_init(void)
 	xen_have_vector_callback = 1;
 	xen_pvh_set_cr_flags(0);
 
+        /* pvh guests are not quite ready for large pages yet */
+        setup_clear_cpu_cap(X86_FEATURE_PSE);
+        setup_clear_cpu_cap(X86_FEATURE_PSE36);
+
+
 #ifdef CONFIG_X86_32
 	BUG(); /* PVH: Implement proper support. */
 #endif
-- 
1.7.2.3


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [V0 PATCH] pvh: Disable PSE feature for now
  2014-01-28  2:18 ` [V0 PATCH] pvh: Disable PSE " Mukesh Rathor
@ 2014-01-28  3:46   ` Konrad Rzeszutek Wilk
  2014-01-29  2:28     ` Mukesh Rathor
  2014-01-28 10:39   ` [Xen-devel] " Jan Beulich
  2014-01-28 10:39   ` Jan Beulich
  2 siblings, 1 reply; 8+ messages in thread
From: Konrad Rzeszutek Wilk @ 2014-01-28  3:46 UTC (permalink / raw)
  To: Mukesh Rathor; +Cc: roger.pau, Xen-devel, linux-kernel

On Mon, Jan 27, 2014 at 06:18:39PM -0800, Mukesh Rathor wrote:
> Until now, xen did not expose PSE to pvh guest, but a patch was submitted
> to xen list to enable bunch of features for a pvh guest. PSE has not been

Which 'patch'?

> looked into for PVH, so until we can do that and test it to make sure it
> works, disable the feature to avoid flood of bugs.

I think we want a flood of bugs, no?
> 
> Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
> ---
>  arch/x86/xen/enlighten.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c
> index a4d7b64..4e952046 100644
> --- a/arch/x86/xen/enlighten.c
> +++ b/arch/x86/xen/enlighten.c
> @@ -1497,6 +1497,11 @@ static void __init xen_pvh_early_guest_init(void)
>  	xen_have_vector_callback = 1;
>  	xen_pvh_set_cr_flags(0);
>  
> +        /* pvh guests are not quite ready for large pages yet */
> +        setup_clear_cpu_cap(X86_FEATURE_PSE);
> +        setup_clear_cpu_cap(X86_FEATURE_PSE36);
> +
> +
>  #ifdef CONFIG_X86_32
>  	BUG(); /* PVH: Implement proper support. */
>  #endif
> -- 
> 1.7.2.3
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [V0 PATCH] pvh: Disable PSE feature for now
  2014-01-28  3:46   ` Konrad Rzeszutek Wilk
@ 2014-01-29  2:28     ` Mukesh Rathor
  0 siblings, 0 replies; 8+ messages in thread
From: Mukesh Rathor @ 2014-01-29  2:28 UTC (permalink / raw)
  To: Konrad Rzeszutek Wilk; +Cc: roger.pau, Xen-devel, linux-kernel

On Mon, 27 Jan 2014 22:46:34 -0500
Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> wrote:

> On Mon, Jan 27, 2014 at 06:18:39PM -0800, Mukesh Rathor wrote:
> > Until now, xen did not expose PSE to pvh guest, but a patch was
> > submitted to xen list to enable bunch of features for a pvh guest.
> > PSE has not been
> 
> Which 'patch'?
> 
> > looked into for PVH, so until we can do that and test it to make
> > sure it works, disable the feature to avoid flood of bugs.
> 
> I think we want a flood of bugs, no?

Ok, but lets document (via this email :)), that they are not tested.

thanks
Mukesh

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Xen-devel] [V0 PATCH] pvh: Disable PSE feature for now
  2014-01-28  2:18 ` [V0 PATCH] pvh: Disable PSE " Mukesh Rathor
  2014-01-28  3:46   ` Konrad Rzeszutek Wilk
@ 2014-01-28 10:39   ` Jan Beulich
  2014-01-29  2:30     ` Mukesh Rathor
  2014-01-29  2:30     ` Mukesh Rathor
  2014-01-28 10:39   ` Jan Beulich
  2 siblings, 2 replies; 8+ messages in thread
From: Jan Beulich @ 2014-01-28 10:39 UTC (permalink / raw)
  To: Mukesh Rathor; +Cc: roger.pau, xen-devel, konrad.wilk, linux-kernel

>>> On 28.01.14 at 03:18, Mukesh Rathor <mukesh.rathor@oracle.com> wrote:
> Until now, xen did not expose PSE to pvh guest, but a patch was submitted
> to xen list to enable bunch of features for a pvh guest. PSE has not been
> looked into for PVH, so until we can do that and test it to make sure it
> works, disable the feature to avoid flood of bugs.
> 
> Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
> ---
>  arch/x86/xen/enlighten.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c
> index a4d7b64..4e952046 100644
> --- a/arch/x86/xen/enlighten.c
> +++ b/arch/x86/xen/enlighten.c
> @@ -1497,6 +1497,11 @@ static void __init xen_pvh_early_guest_init(void)
>  	xen_have_vector_callback = 1;
>  	xen_pvh_set_cr_flags(0);
>  
> +        /* pvh guests are not quite ready for large pages yet */
> +        setup_clear_cpu_cap(X86_FEATURE_PSE);
> +        setup_clear_cpu_cap(X86_FEATURE_PSE36);

And why would you not want to also turn of 1Gb pages then?

Jan


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Xen-devel] [V0 PATCH] pvh: Disable PSE feature for now
  2014-01-28 10:39   ` [Xen-devel] " Jan Beulich
@ 2014-01-29  2:30     ` Mukesh Rathor
  2014-01-29  2:30     ` Mukesh Rathor
  1 sibling, 0 replies; 8+ messages in thread
From: Mukesh Rathor @ 2014-01-29  2:30 UTC (permalink / raw)
  To: Jan Beulich; +Cc: roger.pau, xen-devel, konrad.wilk, linux-kernel

On Tue, 28 Jan 2014 10:39:23 +0000
"Jan Beulich" <JBeulich@suse.com> wrote:

> >>> On 28.01.14 at 03:18, Mukesh Rathor <mukesh.rathor@oracle.com>
> >>> wrote:
> > Until now, xen did not expose PSE to pvh guest, but a patch was
> > submitted to xen list to enable bunch of features for a pvh guest.
> > PSE has not been looked into for PVH, so until we can do that and
> > test it to make sure it works, disable the feature to avoid flood
> > of bugs.
> > 
> > Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
> > ---
> >  arch/x86/xen/enlighten.c |    5 +++++
> >  1 files changed, 5 insertions(+), 0 deletions(-)
> > 
> > diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c
> > index a4d7b64..4e952046 100644
> > --- a/arch/x86/xen/enlighten.c
> > +++ b/arch/x86/xen/enlighten.c
> > @@ -1497,6 +1497,11 @@ static void __init
> > xen_pvh_early_guest_init(void) xen_have_vector_callback = 1;
> >  	xen_pvh_set_cr_flags(0);
> >  
> > +        /* pvh guests are not quite ready for large pages yet */
> > +        setup_clear_cpu_cap(X86_FEATURE_PSE);
> > +        setup_clear_cpu_cap(X86_FEATURE_PSE36);
> 
> And why would you not want to also turn of 1Gb pages then?

Right, that should be turned off too, but Konrad thinks we should
leave them on in linux and deal with issues as they come. I've not
tested them, or looked/thought about them, so had thought would be 
better to turn them on after I/someone gets to test them.

thanks
Mukesh

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [V0 PATCH] pvh: Disable PSE feature for now
  2014-01-28 10:39   ` [Xen-devel] " Jan Beulich
  2014-01-29  2:30     ` Mukesh Rathor
@ 2014-01-29  2:30     ` Mukesh Rathor
  1 sibling, 0 replies; 8+ messages in thread
From: Mukesh Rathor @ 2014-01-29  2:30 UTC (permalink / raw)
  To: Jan Beulich; +Cc: xen-devel, linux-kernel, roger.pau

On Tue, 28 Jan 2014 10:39:23 +0000
"Jan Beulich" <JBeulich@suse.com> wrote:

> >>> On 28.01.14 at 03:18, Mukesh Rathor <mukesh.rathor@oracle.com>
> >>> wrote:
> > Until now, xen did not expose PSE to pvh guest, but a patch was
> > submitted to xen list to enable bunch of features for a pvh guest.
> > PSE has not been looked into for PVH, so until we can do that and
> > test it to make sure it works, disable the feature to avoid flood
> > of bugs.
> > 
> > Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
> > ---
> >  arch/x86/xen/enlighten.c |    5 +++++
> >  1 files changed, 5 insertions(+), 0 deletions(-)
> > 
> > diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c
> > index a4d7b64..4e952046 100644
> > --- a/arch/x86/xen/enlighten.c
> > +++ b/arch/x86/xen/enlighten.c
> > @@ -1497,6 +1497,11 @@ static void __init
> > xen_pvh_early_guest_init(void) xen_have_vector_callback = 1;
> >  	xen_pvh_set_cr_flags(0);
> >  
> > +        /* pvh guests are not quite ready for large pages yet */
> > +        setup_clear_cpu_cap(X86_FEATURE_PSE);
> > +        setup_clear_cpu_cap(X86_FEATURE_PSE36);
> 
> And why would you not want to also turn of 1Gb pages then?

Right, that should be turned off too, but Konrad thinks we should
leave them on in linux and deal with issues as they come. I've not
tested them, or looked/thought about them, so had thought would be 
better to turn them on after I/someone gets to test them.

thanks
Mukesh

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [V0 PATCH] pvh: Disable PSE feature for now
  2014-01-28  2:18 ` [V0 PATCH] pvh: Disable PSE " Mukesh Rathor
  2014-01-28  3:46   ` Konrad Rzeszutek Wilk
  2014-01-28 10:39   ` [Xen-devel] " Jan Beulich
@ 2014-01-28 10:39   ` Jan Beulich
  2 siblings, 0 replies; 8+ messages in thread
From: Jan Beulich @ 2014-01-28 10:39 UTC (permalink / raw)
  To: Mukesh Rathor; +Cc: xen-devel, linux-kernel, roger.pau

>>> On 28.01.14 at 03:18, Mukesh Rathor <mukesh.rathor@oracle.com> wrote:
> Until now, xen did not expose PSE to pvh guest, but a patch was submitted
> to xen list to enable bunch of features for a pvh guest. PSE has not been
> looked into for PVH, so until we can do that and test it to make sure it
> works, disable the feature to avoid flood of bugs.
> 
> Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
> ---
>  arch/x86/xen/enlighten.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c
> index a4d7b64..4e952046 100644
> --- a/arch/x86/xen/enlighten.c
> +++ b/arch/x86/xen/enlighten.c
> @@ -1497,6 +1497,11 @@ static void __init xen_pvh_early_guest_init(void)
>  	xen_have_vector_callback = 1;
>  	xen_pvh_set_cr_flags(0);
>  
> +        /* pvh guests are not quite ready for large pages yet */
> +        setup_clear_cpu_cap(X86_FEATURE_PSE);
> +        setup_clear_cpu_cap(X86_FEATURE_PSE36);

And why would you not want to also turn of 1Gb pages then?

Jan

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2014-01-29  2:30 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-28  2:18 pvh: disable pse feature for now Mukesh Rathor
2014-01-28  2:18 ` [V0 PATCH] pvh: Disable PSE " Mukesh Rathor
2014-01-28  3:46   ` Konrad Rzeszutek Wilk
2014-01-29  2:28     ` Mukesh Rathor
2014-01-28 10:39   ` [Xen-devel] " Jan Beulich
2014-01-29  2:30     ` Mukesh Rathor
2014-01-29  2:30     ` Mukesh Rathor
2014-01-28 10:39   ` Jan Beulich

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.