[PATCH] lib: add tests for sbi_ecall functionality
Akshay Behl
akshaybehl231 at gmail.com
Tue Feb 18 10:27:50 PST 2025
This patch adds unit tests for verifying the sbi_ecall version,
impid handling, and extension registration functions. The tests
ensure that the extension registration and unregistration work as
expected.
Signed-off-by: Akshay Behl <akshaybehl231 at gmail.com>
---
lib/sbi/tests/objects.mk | 5 +++-
lib/sbi/tests/sbi_ecall_test.c | 49 ++++++++++++++++++++++++++++++++++
2 files changed, 53 insertions(+), 1 deletion(-)
create mode 100644 lib/sbi/tests/sbi_ecall_test.c
diff --git a/lib/sbi/tests/objects.mk b/lib/sbi/tests/objects.mk
index 16608fa..0bf6a83 100644
--- a/lib/sbi/tests/objects.mk
+++ b/lib/sbi/tests/objects.mk
@@ -14,4 +14,7 @@ carray-sbi_unit_tests-$(CONFIG_SBIUNIT) += locks_test_suite
libsbi-objs-$(CONFIG_SBIUNIT) += tests/riscv_locks_test.o
carray-sbi_unit_tests-$(CONFIG_SBIUNIT) += math_test_suite
-libsbi-objs-$(CONFIG_SBIUNIT) += tests/sbi_math_test.o
\ No newline at end of file
+libsbi-objs-$(CONFIG_SBIUNIT) += tests/sbi_math_test.o
+
+carray-sbi_unit_tests-$(CONFIG_SBIUNIT) += ecall_test_suite
+libsbi-objs-$(CONFIG_SBIUNIT) += tests/sbi_ecall_test.o
\ No newline at end of file
diff --git a/lib/sbi/tests/sbi_ecall_test.c b/lib/sbi/tests/sbi_ecall_test.c
new file mode 100644
index 0000000..c555cd4
--- /dev/null
+++ b/lib/sbi/tests/sbi_ecall_test.c
@@ -0,0 +1,49 @@
+#include <sbi/sbi_unit_test.h>
+#include <sbi/sbi_ecall.h>
+
+static void test_sbi_ecall_version(struct sbiunit_test_case *test)
+{
+ SBIUNIT_EXPECT_EQ(test, sbi_ecall_version_major(), SBI_ECALL_VERSION_MAJOR);
+ SBIUNIT_EXPECT_EQ(test, sbi_ecall_version_minor(), SBI_ECALL_VERSION_MINOR);
+}
+
+static void test_sbi_ecall_impid(struct sbiunit_test_case *test)
+{
+ unsigned long old_impid = sbi_ecall_get_impid();
+ sbi_ecall_set_impid(42);
+ SBIUNIT_EXPECT_EQ(test, sbi_ecall_get_impid(), 42);
+ sbi_ecall_set_impid(old_impid);
+}
+
+static int dummy_handler(unsigned long extid, unsigned long funcid,
+ struct sbi_trap_regs *regs,
+ struct sbi_ecall_return *out)
+{
+ return 0;
+}
+
+static void test_sbi_ecall_register_find_extension(struct sbiunit_test_case *test)
+{
+ struct sbi_ecall_extension test_ext = {
+ /*large extid_start for no overlap*/
+ .extid_start = 0x80,
+ .extid_end = 0x80,
+ .name = "TestExt",
+ .handle = dummy_handler,
+ };
+
+ SBIUNIT_EXPECT_EQ(test, sbi_ecall_register_extension(&test_ext), 0);
+ SBIUNIT_EXPECT_EQ(test, sbi_ecall_find_extension(0x80), &test_ext);
+
+ sbi_ecall_unregister_extension(&test_ext);
+ SBIUNIT_EXPECT_EQ(test, sbi_ecall_find_extension(0x80), NULL);
+}
+
+static struct sbiunit_test_case ecall_tests[] = {
+ SBIUNIT_TEST_CASE(test_sbi_ecall_version),
+ SBIUNIT_TEST_CASE(test_sbi_ecall_impid),
+ SBIUNIT_TEST_CASE(test_sbi_ecall_register_find_extension),
+ SBIUNIT_END_CASE,
+};
+
+SBIUNIT_TEST_SUITE(ecall_test_suite, ecall_tests);
\ No newline at end of file
--
2.34.1
More information about the opensbi
mailing list