diff mbox series

[bpf] selftests/bpf: fix trampoline_count.c selftest compilation warning

Message ID 20200202065152.2718142-1-andriin@fb.com
State Accepted
Delegated to: BPF Maintainers
Headers show
Series [bpf] selftests/bpf: fix trampoline_count.c selftest compilation warning | expand

Commit Message

Andrii Nakryiko Feb. 2, 2020, 6:51 a.m. UTC
Fix missing braces compilation warning in trampoline_count test:

  .../prog_tests/trampoline_count.c: In function ‘test_trampoline_count’:
  .../prog_tests/trampoline_count.c:49:9: warning: missing braces around initializer [-Wmissing-braces]
  struct inst inst[MAX_TRAMP_PROGS] = { 0 };
         ^
  .../prog_tests/trampoline_count.c:49:9: warning: (near initialization for ‘inst[0]’) [-Wmissing-braces]

Fixes: d633d57902a5 ("selftest/bpf: Add test for allowed trampolines count")
Signed-off-by: Andrii Nakryiko <andriin@fb.com>
---
 tools/testing/selftests/bpf/prog_tests/trampoline_count.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Yonghong Song Feb. 2, 2020, 4:50 p.m. UTC | #1
On 2/1/20 10:51 PM, Andrii Nakryiko wrote:
> Fix missing braces compilation warning in trampoline_count test:
> 
>    .../prog_tests/trampoline_count.c: In function ‘test_trampoline_count’:
>    .../prog_tests/trampoline_count.c:49:9: warning: missing braces around initializer [-Wmissing-braces]
>    struct inst inst[MAX_TRAMP_PROGS] = { 0 };
>           ^
>    .../prog_tests/trampoline_count.c:49:9: warning: (near initialization for ‘inst[0]’) [-Wmissing-braces]
> 
> Fixes: d633d57902a5 ("selftest/bpf: Add test for allowed trampolines count")
> Signed-off-by: Andrii Nakryiko <andriin@fb.com>

Acked-by: Yonghong Song <yhs@fb.com>
Daniel Borkmann Feb. 3, 2020, 11:11 p.m. UTC | #2
On 2/2/20 7:51 AM, Andrii Nakryiko wrote:
> Fix missing braces compilation warning in trampoline_count test:
> 
>    .../prog_tests/trampoline_count.c: In function ‘test_trampoline_count’:
>    .../prog_tests/trampoline_count.c:49:9: warning: missing braces around initializer [-Wmissing-braces]
>    struct inst inst[MAX_TRAMP_PROGS] = { 0 };
>           ^
>    .../prog_tests/trampoline_count.c:49:9: warning: (near initialization for ‘inst[0]’) [-Wmissing-braces]
> 
> Fixes: d633d57902a5 ("selftest/bpf: Add test for allowed trampolines count")
> Signed-off-by: Andrii Nakryiko <andriin@fb.com>

Applied, thanks!
diff mbox series

Patch

diff --git a/tools/testing/selftests/bpf/prog_tests/trampoline_count.c b/tools/testing/selftests/bpf/prog_tests/trampoline_count.c
index 1235f3d1cc50..1f6ccdaed1ac 100644
--- a/tools/testing/selftests/bpf/prog_tests/trampoline_count.c
+++ b/tools/testing/selftests/bpf/prog_tests/trampoline_count.c
@@ -46,7 +46,7 @@  void test_trampoline_count(void)
 	const char *fentry_name = "fentry/__set_task_comm";
 	const char *fexit_name = "fexit/__set_task_comm";
 	const char *object = "test_trampoline_count.o";
-	struct inst inst[MAX_TRAMP_PROGS] = { 0 };
+	struct inst inst[MAX_TRAMP_PROGS] = {};
 	int err, i = 0, duration = 0;
 	struct bpf_object *obj;
 	struct bpf_link *link;