WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-ia64-devel

Re: [Xen-ia64-devel] [PATCH][3/3] Steal time accounting forPVdomain/IA64

To: Atsushi SAKAI <sakaia@xxxxxxxxxxxxxx>
Subject: Re: [Xen-ia64-devel] [PATCH][3/3] Steal time accounting forPVdomain/IA64 TAKE2
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Fri, 9 May 2008 16:03:10 +0900
Cc: Aron Griffis <aron@xxxxxx>, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Fri, 09 May 2008 00:03:25 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <200805090648.m496mUE7006103@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20080506033326.GB11275@xxxxxxxxx> <200805090648.m496mUE7006103@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.6i
On Fri, May 09, 2008 at 03:48:24PM +0900, Atsushi SAKAI wrote:

> Isaku Yamahata <yamahata@xxxxxxxxxxxxx> wrote:
> 
> > 
> > >         do_div(stolentick, NS_PER_TICK);
> > >         stolentick++;
> > > 
> > >         do_div(stolen, NS_PER_TICK);
> > > 
> > >         if (stolen > stolentick)
> > >                 stolen = stolentick;
> > > 
> > >         stolentick -= stolen;
> > >         do_div(blocked, NS_PER_TICK);
> > > 
> > >         if (blocked > stolentick)
> > >                 blocked = stolentick;
> > 
> > Could you please explain the above logic?
> > I guess that stolentick should be
> > ia64_get_itc() - (the itc of the last time 
> >                   the timer interrupt handler was invoked)
> > or something like that.
> 
> your suggested value is new_itm.
> That variable keeps as "local_cpu_data->itm_next" in the ia64 time code.

No. local_cpu_data->itm_next doesn't hold such value because
the valuable is updated by consider_steal_time() so that the
wanted value is lost.

-- 
yamahata

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel