On Wed, Aug 07, 2013 at 08:31:04AM +0000, Bhushan Bharat-R65777 wrote: > > > > -----Original Message----- > > From: Paul Mackerras [mailto:[email protected]] > > Sent: Wednesday, August 07, 2013 1:58 PM > > To: Bhushan Bharat-R65777 > > Cc: Alexander Graf; Benjamin Herrenschmidt; [email protected]; > > [email protected] > > Subject: Re: [PATCH 21/23] KVM: PPC: Book3S PR: Use mmu_notifier_retry() in > > kvmppc_mmu_map_page() > > > > On Wed, Aug 07, 2013 at 05:17:29AM +0000, Bhushan Bharat-R65777 wrote: > > > > > > Pauls, I am trying to understand the flow; does retry mean that we do not > > create the mapping and return to guest, which will fault again and then we > > will > > retry? > > > > Yes, and you do put_page or kvm_release_pfn_clean for any page that you got. > > Ok, but what is the value to return back to guest when we know it is again > going to generate fault. > Cannot we retry within KVM?
You can, though you should make sure you include a preemption point. Going back to the guest gets you a preemption point because of the cond_resched() call in kvmppc_prepare_to_enter(). Paul. -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
