From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH v5 6/9] KVM: MMU: introduce pte_prefetch_topup_memory_cache() Date: Tue, 13 Jul 2010 07:21:17 +0300 Message-ID: <4C3BE9BD.40007@redhat.com> References: <4C330918.6040709@cn.fujitsu.com> <4C330A37.8080709@cn.fujitsu.com> <4C39C1AB.6000606@redhat.com> <4C3A8694.1000401@cn.fujitsu.com> <4C3B09FD.3060307@redhat.com> <4C3BBE84.30708@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Marcelo Tosatti , LKML , KVM list To: Xiao Guangrong Return-path: Received: from mx1.redhat.com ([209.132.183.28]:44903 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751255Ab0GMEVZ (ORCPT ); Tue, 13 Jul 2010 00:21:25 -0400 In-Reply-To: <4C3BBE84.30708@cn.fujitsu.com> Sender: kvm-owner@vger.kernel.org List-ID: On 07/13/2010 04:16 AM, Xiao Guangrong wrote: > > >> btw, what about dirty logging? will prefetch cause pages to be marked >> dirty? >> >> We may need to instantiate prefetched pages with spte.d=0 and examine it >> when tearing down the spte. >> >> > Yeah, all speculative path are broken dirty bit tracking, and i guess it's > need more review, so i plan to do it in the separate patch, i'll post it after > this patchset merged, could you allow it? > > Regressions? no. Or do you mean the problem already exists? Where? -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.