|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [xen master] symbols: add minimal self-test
commit 1f09240beac7222d94c4241eb10068e7553578d7
Author: Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Thu Aug 28 13:34:21 2025 +0200
Commit: Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Thu Aug 28 13:34:21 2025 +0200
symbols: add minimal self-test
... before making changes to the involved logic.
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Reviewed-by: Jason Andryuk <jason.andryuk@xxxxxxx>
---
xen/common/symbols.c | 35 +++++++++++++++++++++++++++++++++++
1 file changed, 35 insertions(+)
diff --git a/xen/common/symbols.c b/xen/common/symbols.c
index f8a7bde37b..1bc7ce7e05 100644
--- a/xen/common/symbols.c
+++ b/xen/common/symbols.c
@@ -260,6 +260,41 @@ unsigned long symbols_lookup_by_name(const char *symname)
return 0;
}
+#ifdef CONFIG_SELF_TESTS
+
+static void __init test_lookup(unsigned long addr, const char *expected)
+{
+ char buf[KSYM_NAME_LEN + 1];
+ const char *name, *symname;
+ unsigned long size, offs;
+
+ name = symbols_lookup(addr, &size, &offs, buf);
+ if ( !name )
+ panic("%s: address not found\n", expected);
+ if ( offs )
+ panic("%s: non-zero offset (%#lx) unexpected\n", expected, offs);
+
+ /* Cope with static symbols, where varying file names/paths may be used. */
+ symname = strchr(name, '#');
+ symname = symname ? symname + 1 : name;
+ if ( strcmp(symname, expected) )
+ panic("%s: unexpected symbol name: '%s'\n", expected, symname);
+
+ offs = symbols_lookup_by_name(name);
+ if ( offs != addr )
+ panic("%s: address %#lx unexpected; wanted %#lx\n",
+ expected, offs, addr);
+}
+
+static void __init __constructor test_symbols(void)
+{
+ /* Be sure to only try this for cf_check functions. */
+ test_lookup((unsigned long)dump_execstate, "dump_execstate");
+ test_lookup((unsigned long)test_symbols, __func__);
+}
+
+#endif /* CONFIG_SELF_TESTS */
+
/*
* Local variables:
* mode: C
--
generated by git-patchbot for /home/xen/git/xen.git#master
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |