Skip to content

Scheduling related tracing improvements

Dario Faggioli xen-devel 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest16 patches 34 repliesIn review
  2. v114 patches 16 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/16xen: sched: __runq_tickle takes a useless cpu parameterxen/common/sched_credit.c+4 -61 reply
2/16xen: sched: move up the trace record for vcpu_wake and vcpu_sleepxen/common/schedule.c+10 -61 reply
3/16xen: sched: improve domain creation tracingxen/common/domain.c xen/common/schedule.c xen/include/public/trace.h+12 -34 replies
4/16xen: credit2: pack trace data better for xentrace_formatxen/common/sched_credit2.c+8 -82 replies
5/16xen: RTDS: pack trace data better for xentrace_formatxen/common/sched_rt.c+4 -232 replies
6/16xen: sched: tracing: enable TSC tracing for all eventsxen/common/sched_credit.c xen/common/sched_credit2.c xen/common/sched_rt.c+5 -56 replies
7/16xentrace: formats: update format of scheduling eventstools/xentrace/formats+9 -91 reply
8/16xentrace: formats: add events from Credit schedulertools/xentrace/formats+7 -00 replies
9/16xentrace: formats: add events from Credit2 schedulertools/xentrace/formats+13 -00 replies
10/16xentrace: formats: add events from RTDS schedulertools/xentrace/formats+6 -00 replies
11/16xentrace: formats: add domain create and destroy events.tools/xentrace/formats+3 -00 replies
12/16xenalyze: handle scheduling eventstools/xentrace/xenalyze.c+126 -301 reply
13/16xenalyze: handle Credit1 scheduler eventstools/xentrace/xenalyze.c+57 -03 replies
14/16xenalyze: handle Credit2 scheduler eventstools/xentrace/xenalyze.c+101 -01 reply
15/16xenalyze: handle RTDS scheduler eventstools/xentrace/xenalyze.c+59 -04 replies
16/16xenalyze: handle DOM0 operaions eventstools/xentrace/xenalyze.c+26 -00 replies

Reviews trailers per patch

12 of 16 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchKonrad Rzeszutek WilkMeng XuGeorge DunlapJan Beulich
SeriesR-b R-b R-b A-b A-b A-b A-b
1/16R-b A-b
2/16R-b A-b
3/16R-b R-b A-b
4/16R-b A-b
5/16R-b R-b A-b
6/16R-b A-b
7/16R-b A-b
8/16R-b
9/16
10/16
11/16
12/16R-b A-b
13/16R-b A-b
14/16R-b A-b
15/16R-b
16/16

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 4 reviewers.

Read the v2 thread

Cover letter [PATCH v2 00/16] Scheduling related tracing improvements

Hi,

Take 2 of this series. v1 is here:

http://lists.xenproject.org/archives/html/xen-devel/2016-02/msg01016.html

Konrad's feedback taken into acount, which basically meant modifying a little bit patch 3 ("xen: sched: improve domain creation tracing"), and adding two simple patches to xentrace formats file, and to xenalyze (patch 11 and 16).

Thanks and Regards,
Dario
---
Dario Faggioli (16):
xen: sched: __runq_tickle takes a useless cpu parameter
xen: sched: move up the trace record for vcpu_wake and vcpu_sleep
xen: sched: improve domain creation tracing
xen: credit2: pack trace data better for xentrace_format
xen: RTDS: pack trace data better for xentrace_format
xen: sched: tracing: enable TSC tracing for all events
xentrace: formats: update format of scheduling events
xentrace: formats: add events from Credit scheduler
xentrace: formats: add events from Credit2 scheduler
xentrace: formats: add events from RTDS scheduler
xentrace: formats: add domain create and destroy events.
xenalyze: handle scheduling events
xenalyze: handle Credit1 scheduler events
xenalyze: handle Credit2 scheduler events
xenalyze: handle RTDS scheduler events
xenalyze: handle DOM0 operaions events

 tools/xentrace/formats     |   47 ++++-
 tools/xentrace/xenalyze.c  |  399 +++++++++++++++++++++++++++++++++++++++++---
 xen/common/domain.c        |    5 -
 xen/common/sched_credit.c  |   12 +
 xen/common/sched_credit2.c |   22 +-
 xen/common/sched_rt.c      |   29 +--
 xen/common/schedule.c      |   20 +-
 xen/include/public/trace.h |    6 +
 8 files changed, 450 insertions(+), 90 deletions(-)

--
<<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)