Re: [Xen-devel] [PATCH v2 13/16] xenalyze: handle Credit1 scheduler events

[Dario Faggioli <dario.faggioli@xxxxxxxxxx>]

On Thu, 2016-02-18 at 15:31 +0000, George Dunlap wrote:
> On 16/02/16 18:12, Dario Faggioli wrote:
> >Â
> > diff --git a/tools/xentrace/xenalyze.c b/tools/xentrace/xenalyze.c
> > index 17021f1..4ab2dba 100644
> > --- a/tools/xentrace/xenalyze.c
> > +++ b/tools/xentrace/xenalyze.c
> > @@ -7673,6 +7673,63 @@ void sched_process(struct pcpu_info *p)
> > ÂÂÂÂÂÂÂÂÂdefault:
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂprocess_generic(&p->ri);
> > ÂÂÂÂÂÂÂÂÂ}
> > +ÂÂÂÂ} else if(ri->evt.sub == 2) {
> > +ÂÂÂÂÂÂÂÂ/* TRC_SCHED_CLASS */
> > +ÂÂÂÂÂÂÂÂswitch(ri->event)
> > +ÂÂÂÂÂÂÂÂ{
> > +ÂÂÂÂÂÂÂÂ/* CREDIT (TRC_CSCHED_xxx) */
> > +ÂÂÂÂÂÂÂÂcase TRC_SCHED_CLASS_EVT(CSCHED, 1): /* SCHED_TASKLET */
> 
> Sorry, just one more comment:
> 
> It would probably be good at some point to find a way to expose these
> in
> a public header, rather than having to manually keep this file in
> sync
> with the values in sched_*.c.ÂÂBut that's a nice-to-have for another
> day. :-)
> 
Indeed. I can add this to my TODO list, although...

> (Maybe a cleanup for a GSoC / OPW student...?)
> 
... that's actually a good idea (and we should add this to the list of
similar projects, if we keep one).

Regards,
Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel