public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* VMI and kmap_atomic_pte paravirt_op
@ 2010-02-25 19:52 Ian Campbell
  2010-02-25 20:15 ` Jeremy Fitzhardinge
  2010-02-25 20:32 ` Alok Kataria
  0 siblings, 2 replies; 4+ messages in thread
From: Ian Campbell @ 2010-02-25 19:52 UTC (permalink / raw)
  To: Alok N Kataria
  Cc: Jeremy Fitzhardinge, Thomas Gleixner, Ingo Molnar, H. Peter Anvin,
	linux-kernel

Hi Alok,

How critical is the vmi_kmap_atomic_pte pv_mmu_op hook to VMI? Is it
required for correct functionality or is it simply an optimisation?

We think we may be able to do away with the Xen version of the hook and
if VMI (which is deprecated) can live without it and you agree then I'll
cook up a patch to remove it altogether.

Thanks,
Ian.



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

* Re: VMI and kmap_atomic_pte paravirt_op
  2010-02-25 19:52 VMI and kmap_atomic_pte paravirt_op Ian Campbell
@ 2010-02-25 20:15 ` Jeremy Fitzhardinge
  2010-02-25 20:32 ` Alok Kataria
  1 sibling, 0 replies; 4+ messages in thread
From: Jeremy Fitzhardinge @ 2010-02-25 20:15 UTC (permalink / raw)
  To: Ian Campbell
  Cc: Alok N Kataria, Thomas Gleixner, Ingo Molnar, H. Peter Anvin,
	linux-kernel

On 02/25/2010 11:52 AM, Ian Campbell wrote:
> How critical is the vmi_kmap_atomic_pte pv_mmu_op hook to VMI? Is it
> required for correct functionality or is it simply an optimisation?
>    

Or disable high ptes for VMI too.



     J

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

* Re: VMI and kmap_atomic_pte paravirt_op
  2010-02-25 19:52 VMI and kmap_atomic_pte paravirt_op Ian Campbell
  2010-02-25 20:15 ` Jeremy Fitzhardinge
@ 2010-02-25 20:32 ` Alok Kataria
  2010-02-26  9:04   ` Ian Campbell
  1 sibling, 1 reply; 4+ messages in thread
From: Alok Kataria @ 2010-02-25 20:32 UTC (permalink / raw)
  To: Ian Campbell
  Cc: Jeremy Fitzhardinge, Thomas Gleixner, Ingo Molnar, H. Peter Anvin,
	linux-kernel@vger.kernel.org

Hi Ian, 

On Thu, 2010-02-25 at 11:52 -0800, Ian Campbell wrote:
> Hi Alok,
> 
> How critical is the vmi_kmap_atomic_pte pv_mmu_op hook to VMI? Is it
> required for correct functionality or is it simply an optimisation?
> 
> We think we may be able to do away with the Xen version of the hook and
> if VMI (which is deprecated) can live without it and you agree then I'll
> cook up a patch to remove it altogether.

As Jeremy suggests we will be fine with disabling high ptes for VMI. 
Thanks for doing this. 

Alok
> 
> Thanks,
> Ian.
> 
> 


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

* Re: VMI and kmap_atomic_pte paravirt_op
  2010-02-25 20:32 ` Alok Kataria
@ 2010-02-26  9:04   ` Ian Campbell
  0 siblings, 0 replies; 4+ messages in thread
From: Ian Campbell @ 2010-02-26  9:04 UTC (permalink / raw)
  To: akataria@vmware.com
  Cc: Jeremy Fitzhardinge, Thomas Gleixner, Ingo Molnar, H. Peter Anvin,
	linux-kernel@vger.kernel.org

On Thu, 2010-02-25 at 20:32 +0000, Alok Kataria wrote:
> Hi Ian, 
> 
> On Thu, 2010-02-25 at 11:52 -0800, Ian Campbell wrote:
> > Hi Alok,
> > 
> > How critical is the vmi_kmap_atomic_pte pv_mmu_op hook to VMI? Is it
> > required for correct functionality or is it simply an optimisation?
> > 
> > We think we may be able to do away with the Xen version of the hook and
> > if VMI (which is deprecated) can live without it and you agree then I'll
> > cook up a patch to remove it altogether.
> 
> As Jeremy suggests we will be fine with disabling high ptes for VMI. 
> Thanks for doing this. 

OK, I'll do that.

Thanks,
Ian.



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

end of thread, other threads:[~2010-02-26  9:04 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-02-25 19:52 VMI and kmap_atomic_pte paravirt_op Ian Campbell
2010-02-25 20:15 ` Jeremy Fitzhardinge
2010-02-25 20:32 ` Alok Kataria
2010-02-26  9:04   ` Ian Campbell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox