From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dor Laor Subject: Re: [PATCH] KVM: add KVM_SYNC_SHADOW_WITH_USER ioctl Date: Wed, 09 Jan 2008 00:56:07 +0200 Message-ID: <4783FF87.3080209@qumranet.com> References: <20080108153915.GB21726@dmt> Reply-To: dor.laor-atKUWr5tajBWk0Htik3J/w@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel , Avi Kivity , virtualization-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Marcelo Tosatti Return-path: In-Reply-To: <20080108153915.GB21726@dmt> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: kvm.vger.kernel.org Marcelo Tosatti wrote: > The host needs to zap its shadow entries before performing an inflate > operation to avoid the guest from using stale ones. > > So add an ioctl to interface with kvm_mmu_zap_all(). > > Signed-off-by: Marcelo Tosatti > > Index: kvm.quilt/virt/kvm/kvm_main.c > =================================================================== > --- kvm.quilt.orig/virt/kvm/kvm_main.c > +++ kvm.quilt/virt/kvm/kvm_main.c > @@ -971,6 +971,11 @@ static long kvm_vm_ioctl(struct file *fi > goto out; > break; > } > + case KVM_SYNC_SHADOW_WITH_USER: { > + r = 0; > + kvm_mmu_zap_all(kvm); > + break; > + } > It should go into the arch ioctl below since the mmu is under x86 > default: > r = kvm_arch_vm_ioctl(filp, ioctl, arg); > } > > > ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace