]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: activity: don't report the lat_tot column for show profiling tasks
authorWilly Tarreau <w@1wt.eu>
Thu, 11 Sep 2025 08:33:50 +0000 (10:33 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 11 Sep 2025 14:32:34 +0000 (16:32 +0200)
This column is pretty useless, as the total latency experienced by tasks
is meaningless, what matters is the average per call. Since we'll add more
columns and we need to keep all of this readable, let's get rid of this
column.

src/activity.c

index 1efd6281a154f4ff475420ffdb5f7faf4472a3a7..b06b48d6865292976b4c0b923bd6cc8297deef31 100644 (file)
@@ -990,7 +990,7 @@ static int cli_io_handler_show_profiling(struct appctx *appctx)
 
        if (!ctx->linenum)
                chunk_appendf(&trash, "Tasks activity over %.3f sec till %.3f sec ago:\n"
-                                     "  function                      calls   cpu_tot   cpu_avg   lat_tot   lat_avg\n",
+                                     "  function                      calls   cpu_tot   cpu_avg   lat_avg\n",
                              (prof_task_start_ns ? (prof_task_stop_ns ? prof_task_stop_ns : now_ns) - prof_task_start_ns : 0) / 1000000000.0,
                              (prof_task_stop_ns ? now_ns - prof_task_stop_ns : 0) / 1000000000.0);
 
@@ -1027,7 +1027,6 @@ static int cli_io_handler_show_profiling(struct appctx *appctx)
 
                print_time_short(&trash, "   ", tmp_activity[i].cpu_time, "");
                print_time_short(&trash, "   ", tmp_activity[i].cpu_time / tmp_activity[i].calls, "");
-               print_time_short(&trash, "   ", tmp_activity[i].lat_time, "");
                print_time_short(&trash, "   ", tmp_activity[i].lat_time / tmp_activity[i].calls, "");
 
                if (caller && !ctx->aggr && caller->what <= WAKEUP_TYPE_APPCTX_WAKEUP)