[PATCH bpf-next 2/2] Added selftests to check deadlocks in queue and stack map

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Siddharth Chintamaneni <sidchintamaneni@xxxxxx>

 Added selftests to check for nested deadlocks in queue
 and stack maps.

Signed-off-by: Siddharth Chintamaneni <sidchintamaneni@xxxxxx>
---
 .../prog_tests/test_queue_stack_nested_map.c  | 48 ++++++++++++++
 .../bpf/progs/test_queue_stack_nested_map.c   | 62 +++++++++++++++++++
 2 files changed, 110 insertions(+)
 create mode 100644 tools/testing/selftests/bpf/prog_tests/test_queue_stack_nested_map.c
 create mode 100644 tools/testing/selftests/bpf/progs/test_queue_stack_nested_map.c

diff --git a/tools/testing/selftests/bpf/prog_tests/test_queue_stack_nested_map.c b/tools/testing/selftests/bpf/prog_tests/test_queue_stack_nested_map.c
new file mode 100644
index 000000000000..731e958419eb
--- /dev/null
+++ b/tools/testing/selftests/bpf/prog_tests/test_queue_stack_nested_map.c
@@ -0,0 +1,48 @@
+// SPDX-License-Identifier: GPL-2.0
+#include <test_progs.h>
+#include <network_helpers.h>
+
+#include "test_queue_stack_nested_map.skel.h"
+
+
+static void test_map_queue_stack_nesting_success(bool is_map_queue)
+{
+	struct test_queue_stack_nested_map *skel;
+	int err;
+	int prog_fd;
+
+	LIBBPF_OPTS(bpf_test_run_opts, ropts);
+
+	skel = test_queue_stack_nested_map__open_and_load();
+	if (!ASSERT_OK_PTR(skel, "test_queue_stack_nested_map__open_and_load"))
+		goto out;
+
+	err = test_queue_stack_nested_map__attach(skel);
+	if (!ASSERT_OK(err, "test_queue_stack_nested_map__attach"))
+		goto out;
+
+	if (is_map_queue) {
+		prog_fd = bpf_program__fd(skel->progs.test_queue_nesting);
+		err = bpf_prog_test_run_opts(prog_fd, &ropts);
+		ASSERT_OK(err, "test_nested_queue_map_run");
+	} else {
+		prog_fd = bpf_program__fd(skel->progs.test_stack_nesting);
+		err = bpf_prog_test_run_opts(prog_fd, &ropts);
+		ASSERT_OK(err, "test_nested_stack_map_run");
+	}
+
+
+
+out:
+	test_queue_stack_nested_map__destroy(skel);
+}
+
+void test_test_queue_stack_nested_map(void)
+{
+	if (test__start_subtest("map_queue_nesting"))
+		test_map_queue_stack_nesting_success(true);
+	if (test__start_subtest("map_stack_nesting"))
+		test_map_queue_stack_nesting_success(false);
+
+}
+
diff --git a/tools/testing/selftests/bpf/progs/test_queue_stack_nested_map.c b/tools/testing/selftests/bpf/progs/test_queue_stack_nested_map.c
new file mode 100644
index 000000000000..6d22016b1709
--- /dev/null
+++ b/tools/testing/selftests/bpf/progs/test_queue_stack_nested_map.c
@@ -0,0 +1,62 @@
+// SPDX-License-Identifier: GPL-2.0
+#include "vmlinux.h"
+#include <bpf/bpf_helpers.h>
+#include <bpf/bpf_tracing.h>
+
+struct {
+	__uint(type, BPF_MAP_TYPE_STACK);
+	__uint(max_entries, 32);
+	__uint(map_flags, 0);
+	__uint(key_size, 0);
+	__uint(value_size, sizeof(__u32));
+} map_stack SEC(".maps");
+
+struct {
+	__uint(type, BPF_MAP_TYPE_QUEUE);
+	__uint(max_entries, 32);
+	__uint(map_flags, 0);
+	__uint(key_size, 0);
+	__uint(value_size, sizeof(__u32));
+} map_queue SEC(".maps");
+
+SEC("fentry/_raw_spin_unlock_irqrestore")
+int BPF_PROG(test_stack_nesting2, raw_spinlock_t *lock, unsigned long flags)
+{
+	__u32 value = 1;
+
+	bpf_map_push_elem(&map_stack, &value, 0);
+
+	return 0;
+}
+
+SEC("fentry/bpf_fentry_test1")
+int BPF_PROG(test_stack_nesting, int a)
+{
+	__u32 value = 1;
+
+	bpf_map_push_elem(&map_stack, &value, 0);
+
+	return 0;
+}
+
+SEC("fentry/_raw_spin_unlock_irqrestore")
+int BPF_PROG(test_queue_nesting2, raw_spinlock_t *lock, unsigned long flags)
+{
+	__u32 value = 1;
+
+	bpf_map_pop_elem(&map_queue, &value);
+
+	return 0;
+}
+
+SEC("fentry/bpf_fentry_test1")
+int BPF_PROG(test_queue_nesting, int a)
+{
+	__u32 value = 1;
+
+	bpf_map_push_elem(&map_queue, &value, 0);
+
+	return 0;
+}
+
+char _license[] SEC("license") = "GPL";
-- 
2.44.0





[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux