git: e189296153a2 - main - tests: Add buf_ring overflow checks
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Mon, 30 Sep 2024 12:23:44 UTC
The branch main has been updated by andrew:
URL: https://cgit.FreeBSD.org/src/commit/?id=e189296153a232e6066af7e4ddab0430d3b6677d
commit e189296153a232e6066af7e4ddab0430d3b6677d
Author: Andrew Turner <andrew@FreeBSD.org>
AuthorDate: 2024-09-27 15:13:53 +0000
Commit: Andrew Turner <andrew@FreeBSD.org>
CommitDate: 2024-09-30 12:04:24 +0000
tests: Add buf_ring overflow checks
Check enqueue works correctly when the offsets overflow
Reviewed by: imp
Sponsored by: Arm Ltd
Differential Revision: https://reviews.freebsd.org/D46156
---
tests/sys/sys/buf_ring_test.c | 54 +++++++++++++++++++++++++++++++++++++++++++
1 file changed, 54 insertions(+)
diff --git a/tests/sys/sys/buf_ring_test.c b/tests/sys/sys/buf_ring_test.c
index 5ee1b1cc094b..611fb23788cc 100644
--- a/tests/sys/sys/buf_ring_test.c
+++ b/tests/sys/sys/buf_ring_test.c
@@ -10,6 +10,7 @@
#include <machine/cpufunc.h>
#include <errno.h>
+#include <stdint.h>
#include <atf-c.h>
@@ -116,11 +117,64 @@ MC_SC_TEST(buf_ring_dequeue_mc)
MC_SC_TEST(buf_ring_dequeue_peek)
MC_SC_TEST(buf_ring_dequeue_peek_clear_sc)
+ATF_TC_WITHOUT_HEAD(overflow);
+ATF_TC_BODY(overflow, tc)
+{
+ struct buf_ring *br;
+
+ br = buf_ring_alloc(4);
+ ATF_REQUIRE_MSG(br != NULL, "buf_ring_alloc returned NULL");
+
+ br->br_prod_head = br->br_cons_head = br->br_prod_tail =
+ br->br_cons_tail = UINT32_MAX - 1;
+ ATF_REQUIRE(buf_ring_count(br) == 0);
+ ATF_REQUIRE(!buf_ring_full(br));
+ ATF_REQUIRE(buf_ring_empty(br));
+
+ ATF_REQUIRE(buf_ring_enqueue(br, (void *)1) == 0);
+ ATF_REQUIRE(buf_ring_count(br) == 1);
+ ATF_REQUIRE(!buf_ring_full(br));
+ ATF_REQUIRE(!buf_ring_empty(br));
+
+ ATF_REQUIRE(buf_ring_enqueue(br, (void *)2) == 0);
+ ATF_REQUIRE(buf_ring_count(br) == 2);
+ ATF_REQUIRE(!buf_ring_full(br));
+ ATF_REQUIRE(!buf_ring_empty(br));
+
+ ATF_REQUIRE(buf_ring_enqueue(br, (void *)3) == 0);
+ ATF_REQUIRE(buf_ring_count(br) == 3);
+ ATF_REQUIRE(buf_ring_full(br));
+ ATF_REQUIRE(!buf_ring_empty(br));
+
+ ATF_REQUIRE(br->br_prod_head == 1);
+ ATF_REQUIRE(br->br_prod_tail == 1);
+ ATF_REQUIRE(br->br_cons_head == UINT32_MAX - 1);
+ ATF_REQUIRE(br->br_cons_tail == UINT32_MAX - 1);
+
+ ATF_REQUIRE(buf_ring_dequeue_sc(br) == (void *)1);
+ ATF_REQUIRE(buf_ring_count(br) == 2);
+ ATF_REQUIRE(!buf_ring_full(br));
+ ATF_REQUIRE(!buf_ring_empty(br));
+
+ ATF_REQUIRE(buf_ring_dequeue_sc(br) == (void *)2);
+ ATF_REQUIRE(buf_ring_count(br) == 1);
+ ATF_REQUIRE(!buf_ring_full(br));
+ ATF_REQUIRE(!buf_ring_empty(br));
+
+ ATF_REQUIRE(buf_ring_dequeue_sc(br) == (void *)3);
+ ATF_REQUIRE(buf_ring_count(br) == 0);
+ ATF_REQUIRE(!buf_ring_full(br));
+ ATF_REQUIRE(buf_ring_empty(br));
+
+ buf_ring_free(br);
+}
+
ATF_TP_ADD_TCS(tp)
{
ATF_TP_ADD_TC(tp, buf_ring_dequeue_sc);
ATF_TP_ADD_TC(tp, buf_ring_dequeue_mc);
ATF_TP_ADD_TC(tp, buf_ring_dequeue_peek);
ATF_TP_ADD_TC(tp, buf_ring_dequeue_peek_clear_sc);
+ ATF_TP_ADD_TC(tp, overflow);
return (atf_no_error());
}