From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] KVM: MMU: increment stats after event occurrence Date: Sun, 15 Jan 2012 16:04:10 +0200 Message-ID: <4F12DCDA.7040706@redhat.com> References: <1326416880.3416.2.camel@offbook> <4F0F87BE.2020202@oss.ntt.co.jp> <1326459360.3231.3.camel@offbook> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: Takuya Yoshikawa , Marcelo Tosatti , Xiao Guangrong , lkml , kvm@vger.kernel.org To: dave@gnu.org Return-path: Received: from mx1.redhat.com ([209.132.183.28]:22217 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751219Ab2AOOEU (ORCPT ); Sun, 15 Jan 2012 09:04:20 -0500 In-Reply-To: <1326459360.3231.3.camel@offbook> Sender: kvm-owner@vger.kernel.org List-ID: On 01/13/2012 02:56 PM, Davidlohr Bueso wrote: > On Fri, 2012-01-13 at 10:24 +0900, Takuya Yoshikawa wrote: > > (2012/01/13 10:08), Davidlohr Bueso wrote: > > > From: Davidlohr Bueso > > > > > > It makes more sense to actually increment statistics for tlb flushes and page table entry updates after such _events_ occur, instead of before. > > > > Why? > > > From a logical point of view its better to increase stats once the event > is processed - usually because it can fail and therefore the stats would > be incorrect. > That doesn't hold for the cases you change. -- error compiling committee.c: too many arguments to function