bpftool: Fix typo in MAX_NUM_METRICS macro name
authorSwan Beaujard <beaujardswan@gmail.com>
Sun, 2 Jun 2024 22:58:12 +0000 (00:58 +0200)
committerDaniel Borkmann <daniel@iogearbox.net>
Mon, 3 Jun 2024 14:58:27 +0000 (16:58 +0200)
Correct typo in bpftool profiler and change all instances of 'MATRICS' to
'METRICS' in the profiler.bpf.c file.

Signed-off-by: Swan Beaujard <beaujardswan@gmail.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Quentin Monnet <qmo@kernel.org>
Link: https://lore.kernel.org/bpf/20240602225812.81171-1-beaujardswan@gmail.com
tools/bpf/bpftool/skeleton/profiler.bpf.c

index 2f80edc..f48c783 100644 (file)
@@ -40,17 +40,17 @@ struct {
 
 const volatile __u32 num_cpu = 1;
 const volatile __u32 num_metric = 1;
-#define MAX_NUM_MATRICS 4
+#define MAX_NUM_METRICS 4
 
 SEC("fentry/XXX")
 int BPF_PROG(fentry_XXX)
 {
-       struct bpf_perf_event_value___local *ptrs[MAX_NUM_MATRICS];
+       struct bpf_perf_event_value___local *ptrs[MAX_NUM_METRICS];
        u32 key = bpf_get_smp_processor_id();
        u32 i;
 
        /* look up before reading, to reduce error */
-       for (i = 0; i < num_metric && i < MAX_NUM_MATRICS; i++) {
+       for (i = 0; i < num_metric && i < MAX_NUM_METRICS; i++) {
                u32 flag = i;
 
                ptrs[i] = bpf_map_lookup_elem(&fentry_readings, &flag);
@@ -58,7 +58,7 @@ int BPF_PROG(fentry_XXX)
                        return 0;
        }
 
-       for (i = 0; i < num_metric && i < MAX_NUM_MATRICS; i++) {
+       for (i = 0; i < num_metric && i < MAX_NUM_METRICS; i++) {
                struct bpf_perf_event_value___local reading;
                int err;
 
@@ -99,14 +99,14 @@ fexit_update_maps(u32 id, struct bpf_perf_event_value___local *after)
 SEC("fexit/XXX")
 int BPF_PROG(fexit_XXX)
 {
-       struct bpf_perf_event_value___local readings[MAX_NUM_MATRICS];
+       struct bpf_perf_event_value___local readings[MAX_NUM_METRICS];
        u32 cpu = bpf_get_smp_processor_id();
        u32 i, zero = 0;
        int err;
        u64 *count;
 
        /* read all events before updating the maps, to reduce error */
-       for (i = 0; i < num_metric && i < MAX_NUM_MATRICS; i++) {
+       for (i = 0; i < num_metric && i < MAX_NUM_METRICS; i++) {
                err = bpf_perf_event_read_value(&events, cpu + i * num_cpu,
                                                (void *)(readings + i),
                                                sizeof(*readings));
@@ -116,7 +116,7 @@ int BPF_PROG(fexit_XXX)
        count = bpf_map_lookup_elem(&counts, &zero);
        if (count) {
                *count += 1;
-               for (i = 0; i < num_metric && i < MAX_NUM_MATRICS; i++)
+               for (i = 0; i < num_metric && i < MAX_NUM_METRICS; i++)
                        fexit_update_maps(i, &readings[i]);
        }
        return 0;