KVM: add vmdos_buslock_de case#573
Merged
xhao22 merged 1 commit intointel:mainfrom Apr 29, 2026
Merged
Conversation
fanchen2
force-pushed
the
vmdos-01
branch
3 times, most recently
from
f54624c to
fc32587
Compare
xhao22
reviewed
Apr 23, 2026
| : "memory"); | ||
| } | ||
|
|
||
| static inline int get_cache_line_size(void) |
Contributor
There was a problem hiding this comment.
These 3 functions above are same as bus_lock_10.c, how about merge them?
| bus_lock_source_file = bus_lock.c | ||
| bus_lock_exec_file = bus_lock.out | ||
| variants: | ||
| - VMDOS_buslock_de_01: |
Contributor
There was a problem hiding this comment.
change the *_01 to a valuable name.
| def _check_host_split_lock_cmdline(): | ||
| """Ensure host kernel cmdline contains split_lock_detect=off.""" | ||
| try: | ||
| result = process.run("cat /proc/cmdline", shell=True) |
|
|
||
| def _check_guest_bus_lock_capability(test, params, vm, session): | ||
| """Ensure guest supports bus lock detection feature.""" | ||
| if session.cmd_status("grep -qw bus_lock_detect /proc/cpuinfo"): |
fanchen2
force-pushed
the
vmdos-01
branch
2 times, most recently
from
d65d110 to
fe79626
Compare
Add buslock_de test case with its QEMU cfg and dependencies. Case names: - buslock_de_warn - buslock_de_off - buslock_de_fatal - buslock_de_ratelimit Case coverage includes: - split_lock_detect mode validation in guest - bus lock trigger execution and bus_error behavior checks - repeated 64-bit bus lock triggering for ratelimit coverage - guest dmesg trap checks with timestamp-format compatibility - ratelimit verification using per-second guest dmesg trap counting for both TDX and non-TDX paths - host-side EXCEPTION_NMI trace verification for non-TDX paths only Signed-off-by: Farrah Chen <farrah.chen@intel.com>
Contributor
Author
|
@xhao22 All comments have been addressed, could you help to review again? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add vmdos_buslock_de test case with its QEMU cfg and dependencies.
Case names:
Case coverage includes: