From a27eecea75b3858b4052b191143f144a7e966869 Mon Sep 17 00:00:00 2001 From: Beat Bolli Date: Thu, 20 Jul 2023 18:48:23 +0200 Subject: wrapper: use trace2 counters to collect fsync stats As mentioned in the thread starting at [1], trace2 counters should be used to count events instead of ad-hoc static variables. Convert the two fsync static variables to trace2 counters, reducing the coupling between wrapper.c and the trace2 subsystem. Adjust t/t5351 to match the trace2 counter output format. The counters are not per-thread because the ones being replaced also were not. [1] https://lore.kernel.org/git/20230627195251.1973421-2-calvinwan@google.com/ Signed-off-by: Beat Bolli Signed-off-by: Junio C Hamano --- trace2.c | 1 - 1 file changed, 1 deletion(-) (limited to 'trace2.c') diff --git a/trace2.c b/trace2.c index 49c23bfd05..6dc74dff4c 100644 --- a/trace2.c +++ b/trace2.c @@ -276,7 +276,6 @@ void trace2_cmd_exit_fl(const char *file, int line, int code) if (!trace2_enabled) return; - trace_git_fsync_stats(); trace2_collect_process_info(TRACE2_PROCESS_INFO_EXIT); tr2main_exit_code = code; -- cgit v1.2.3