Skip to content

Commit

Permalink
Advertize paper about traces
Browse files Browse the repository at this point in the history
  • Loading branch information
philippemilink committed Oct 11, 2023
1 parent 4bc94c1 commit de097f5
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,12 @@ be used to generate trace events which describes the lock's behavior during
the execution. It is however very heavy and should not be used unless debugging
StarPU's internal locking.

Be aware that tracing StarPU applications can, in some cases, generate an
overhead impacting application performance. There are several sources of
performance penalty, have a look on [the
paper](https://inria.hal.science/hal-04236246) studying them for more
information.

When the FxT trace file <c>prof_file_something</c> has been generated,
it is possible to generate different trace formats by calling:

Expand Down
8 changes: 8 additions & 0 deletions src/debug/traces/starpu_fxt.c
Original file line number Diff line number Diff line change
Expand Up @@ -620,6 +620,10 @@ static const char *get_state_name(const char *short_name, uint32_t states)

static double compute_time_stamp(double ev_time, struct starpu_fxt_options *options)
{
/* To easily understand what is happening here and have nice pictures, have
* a look on section 5.3 of the paper "Tracing task-based runtime systems:
* Feedbacks from the StarPU case", https://inria.hal.science/hal-04236246
*/
double offset = 0;

if (options->file_offset.nb_barriers < 2)
Expand Down Expand Up @@ -5105,6 +5109,10 @@ void starpu_fxt_generate_trace(struct starpu_fxt_options *options)
* most in the past, so by taking this value, we are sure all events
* in all processes will have a positive timestamp), hence:
* offset[k] += M
*
* If you want nice pictures, have a look on section 5.3 of the paper
* "Tracing task-based runtime systems: Feedbacks from the StarPU
* case", https://inria.hal.science/hal-04236246
*/
for (inputfile = 0; inputfile < options->ninputfiles; inputfile++)
{
Expand Down

0 comments on commit de097f5

Please sign in to comment.