From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gleb Natapov Subject: Re: [PATCH v3 07/15] KVM: MMU: introduce nulls desc Date: Mon, 25 Nov 2013 12:19:43 +0200 Message-ID: <20131125101943.GS2108@redhat.com> References: <1382534973-13197-1-git-send-email-xiaoguangrong@linux.vnet.ibm.com> <1382534973-13197-8-git-send-email-xiaoguangrong@linux.vnet.ibm.com> <20131122191429.GA13308@amt.cnet> <65EE805B-B5DB-4BD0-A057-E5FF78D96D67@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=cp1255 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Marcelo Tosatti , avi.kivity@gmail.com, "pbonzini@redhat.com Bonzini" , linux-kernel@vger.kernel.org, kvm@vger.kernel.org, Eric Dumazet , Peter Zijlstra To: Xiao Guangrong Return-path: Content-Disposition: inline In-Reply-To: <65EE805B-B5DB-4BD0-A057-E5FF78D96D67@linux.vnet.ibm.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Mon, Nov 25, 2013 at 02:11:31PM +0800, Xiao Guangrong wrote: > >=20 > > For example, nothing prevents lockless walker to move into some > > parent_ptes chain, right? >=20 > No. >=20 > The nulls can help us to detect this case, for parent_ptes, the nulls= points > to "shadow page" but for rmaps, the nulls points to slot.arch.rmap. T= here > is no chance that the =93rmap" is used as shadow page when slot-lock = is held. >=20 But meanwhile we will write protect non-last level sptes, no? Better to create separate slab caches for rmap and parent_ptes lists. -- Gleb.