Date: Sat, 3 Jul 2010 13:13:02 +1000
From: Nick Piggin <npiggin@suse.de>
To: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>, Ingo Molnar <mingo@elte.hu>,
       Avi Kivity <avi@redhat.com>, LKML <linux-kernel@vger.kernel.org>,
       KVM list <kvm@vger.kernel.org>
Subject: Re: [PATCH v4 1/6] KVM: MMU: introduce gfn_to_pfn_atomic() function
Message-ID: <20100703031302.GD11732@laptop>
References: <4C2C9DC0.8050607@cn.fujitsu.com>
 <20100702164756.GA25969@amt.cnet>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20100702164756.GA25969@amt.cnet>
User-Agent: Mutt/1.5.20 (2009-06-14)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1666
Lines: 46

On Fri, Jul 02, 2010 at 01:47:56PM -0300, Marcelo Tosatti wrote:
> On Thu, Jul 01, 2010 at 09:53:04PM +0800, Xiao Guangrong wrote:
> > Introduce gfn_to_pfn_atomic(), it's the fast path and can used in atomic
> > context, the later patch will use it
> > 
> > Signed-off-by: Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>
> > ---
> >  arch/x86/mm/gup.c        |    2 ++
> >  include/linux/kvm_host.h |    1 +
> >  virt/kvm/kvm_main.c      |   32 +++++++++++++++++++++++++-------
> >  3 files changed, 28 insertions(+), 7 deletions(-)
> > 
> > diff --git a/arch/x86/mm/gup.c b/arch/x86/mm/gup.c
> > index 738e659..0c9034b 100644
> > --- a/arch/x86/mm/gup.c
> > +++ b/arch/x86/mm/gup.c
> > @@ -6,6 +6,7 @@
> >   */
> >  #include <linux/sched.h>
> >  #include <linux/mm.h>
> > +#include <linux/module.h>
> >  #include <linux/vmstat.h>
> >  #include <linux/highmem.h>
> >  
> > @@ -274,6 +275,7 @@ int __get_user_pages_fast(unsigned long start, int nr_pages, int write,
> >  
> >  	return nr;
> >  }
> > +EXPORT_SYMBOL_GPL(__get_user_pages_fast);
> >  
> >  /**
> >   * get_user_pages_fast() - pin user pages in memory
> 
> This should be a separate patch (overlooked that before). Ingo, Nick,
> can this go in through the kvm tree?

I'm happy for it to be exported. Yes put it in a seperate patch and
add an acked-by: me on it please. Beyond that I don't care what tree
it goes through :) although Ingo might have an opinion


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/