Skip to content

Commit c1ad2e4

Browse files
Martin Kellyanakryiko
authored andcommitted
selftests/bpf: Add tests for ring_buffer__ring
Add tests for the new API ring_buffer__ring. Signed-off-by: Martin Kelly <martin.kelly@crowdstrike.com> Signed-off-by: Andrii Nakryiko <andrii@kernel.org> Link: https://lore.kernel.org/bpf/20230925215045.2375758-5-martin.kelly@crowdstrike.com
1 parent 1c97f6a commit c1ad2e4

1 file changed

Lines changed: 15 additions & 0 deletions

File tree

tools/testing/selftests/bpf/prog_tests/ringbuf_multi.c

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,8 @@ void test_ringbuf_multi(void)
4242
{
4343
struct test_ringbuf_multi *skel;
4444
struct ring_buffer *ringbuf = NULL;
45+
struct ring *ring_old;
46+
struct ring *ring;
4547
int err;
4648
int page_size = getpagesize();
4749
int proto_fd = -1;
@@ -84,11 +86,24 @@ void test_ringbuf_multi(void)
8486
if (CHECK(!ringbuf, "ringbuf_create", "failed to create ringbuf\n"))
8587
goto cleanup;
8688

89+
/* verify ring_buffer__ring returns expected results */
90+
ring = ring_buffer__ring(ringbuf, 0);
91+
if (!ASSERT_OK_PTR(ring, "ring_buffer__ring_idx_0"))
92+
goto cleanup;
93+
ring_old = ring;
94+
ring = ring_buffer__ring(ringbuf, 1);
95+
ASSERT_ERR_PTR(ring, "ring_buffer__ring_idx_1");
96+
8797
err = ring_buffer__add(ringbuf, bpf_map__fd(skel->maps.ringbuf2),
8898
process_sample, (void *)(long)2);
8999
if (CHECK(err, "ringbuf_add", "failed to add another ring\n"))
90100
goto cleanup;
91101

102+
/* verify adding a new ring didn't invalidate our older pointer */
103+
ring = ring_buffer__ring(ringbuf, 0);
104+
if (!ASSERT_EQ(ring, ring_old, "ring_buffer__ring_again"))
105+
goto cleanup;
106+
92107
err = test_ringbuf_multi__attach(skel);
93108
if (CHECK(err, "skel_attach", "skeleton attachment failed: %d\n", err))
94109
goto cleanup;

0 commit comments

Comments
 (0)