Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754345Ab0DYKAZ (ORCPT ); Sun, 25 Apr 2010 06:00:25 -0400 Received: from mx1.redhat.com ([209.132.183.28]:30367 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752939Ab0DYKAY (ORCPT ); Sun, 25 Apr 2010 06:00:24 -0400 Message-ID: <4BD412A0.1000101@redhat.com> Date: Sun, 25 Apr 2010 13:00:00 +0300 From: Avi Kivity User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100330 Fedora/3.0.4-1.fc12 Thunderbird/3.0.4 MIME-Version: 1.0 To: Xiao Guangrong CC: Marcelo Tosatti , KVM list , LKML Subject: Re: [PATCH v2 6/10] KVM MMU: don't write-protect if have new mapping to unsync page References: <4BD3E306.4020202@cn.fujitsu.com> <4BD3E8AB.4020704@cn.fujitsu.com> In-Reply-To: <4BD3E8AB.4020704@cn.fujitsu.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1387 Lines: 29 On 04/25/2010 10:00 AM, Xiao Guangrong wrote: > Two cases maybe happen in kvm_mmu_get_page() function: > > - one case is, the goal sp is already in cache, if the sp is unsync, > we only need update it to assure this mapping is valid, but not > mark it sync and not write-protect sp->gfn since it not broke unsync > rule(one shadow page for a gfn) > > - another case is, the goal sp not existed, we need create a new sp > for gfn, i.e, gfn (may)has another shadow page, to keep unsync rule, > we should sync(mark sync and write-protect) gfn's unsync shadow page. > After enabling multiple unsync shadows, we sync those shadow pages > only when the new sp not allow to become unsync(also for the unsyc > rule, the new rule is: allow all pte page become unsync) > Another interesting case is to create new shadow pages in the unsync state. That can help when the guest starts a short lived process: we can avoid write protecting its pagetables completely. Even if we do sync them, we can sync them in a batch instead of one by one, saving IPIs. -- error compiling committee.c: too many arguments to function -- 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/