From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Yang, Sheng" Subject: Re: [PATCH 2/5] kvmtrace: make cycle calculation architecture aware Date: Thu, 10 Jul 2008 18:24:26 +0800 Message-ID: <200807101824.27570.sheng.yang@intel.com> References: <1215439013-11480-1-git-send-email-ehrhardt@linux.vnet.ibm.com> <1215615799.22935.1.camel@localhost.localdomain> <200807101822.16163.sheng.yang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Cc: Hollis Blanchard , Christian Ehrhardt , cotte@de.ibm.com, xiantao.zhang@intel.com, avi@qumranet.com, kvm-ppc@vger.kernel.org To: kvm@vger.kernel.org Return-path: Received: from mga02.intel.com ([134.134.136.20]:16400 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754094AbYGJKXM (ORCPT ); Thu, 10 Jul 2008 06:23:12 -0400 In-Reply-To: <200807101822.16163.sheng.yang@intel.com> Content-Disposition: inline Sender: kvm-owner@vger.kernel.org List-ID: On Thursday 10 July 2008 18:22:15 Yang, Sheng wrote: > On Wednesday 09 July 2008 23:03:19 Hollis Blanchard wrote: > > On Wed, 2008-07-09 at 11:17 +0200, Christian Ehrhardt wrote: > > > So the question that is left before changing that is, if the > > > original author had something special in mind chosing cycles > > > here. I added Eric on CC for that. > > > > > > I wait with my resubmission of the patch series until all > > > architectures agree *hope* on using getnstimeofday() - after an > > > ack from all sides I would revise my patch series and submit > > > that changes alltogether. > > > > I got an email bounce from Eric the last time I tried to email > > him, so I'm not sure he's still with Intel. > > > > However, I don't think he had any special intention; I think he > > was just porting xentrace to KVM. > > Eric had completed his internship in Intel, so... Drop Eric here... -- Thanks Yang, Sheng > > I like the term "timestamp" too. I think he used "cycles" only > because there is a function called get_cycles(). > > But instead of getnstimeofday(), I suggest using ktime_get() here. > It's little more precise than getnstimeofday(), and ktime_t is more > easily to be handled. And I think the overhead it brought can be > ignored too. > > -- > Thanks > Yang, Sheng > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html