Skip to content

io/iommu/amd: Update kernel config and dmesg check methods#2944

Merged
narasimhan-v merged 2 commits intoavocado-framework-tests:masterfrom
dhsrivas:master
Mar 4, 2025
Merged

io/iommu/amd: Update kernel config and dmesg check methods#2944
narasimhan-v merged 2 commits intoavocado-framework-tests:masterfrom
dhsrivas:master

Conversation

@dhsrivas
Copy link
Contributor

@dhsrivas dhsrivas commented Mar 4, 2025

This patch series

  1. [Patch 1] Update check_kernelconf_5lvl() to usem "/lib/modules/<kernel_version>/build/.config" to check for kernel config
    file making test work for both ubuntu and rhel.
  2. [Patch 2] Add addtional journalctl check for kernel logs in addition to dmesg.

Run
avocado-misc-tests# avocado run io/iommu/amd/iommu_v2pgmode_test.py
JOB ID : 3cc7db3ef40f8195a00951bf6c7f6556d79beefc
JOB LOG : /root/tests/results/job-2025-03-04T08.04-3cc7db3/job.log
(1/1) io/iommu/amd/iommu_v2pgmode_test.py:IommuPageTable.test: STARTED
(1/1) io/iommu/amd/iommu_v2pgmode_test.py:IommuPageTable.test: PASS (0.21 s)
RESULTS : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML : /root/tests/results/job-2025-03-04T08.04-3cc7db3/results.html
JOB TIME : 1.91 s

dhsrivas added 2 commits March 4, 2025 13:26
Currently "check_kernelconf_5lvl()" checks "/boot/" directory for kernel
config files. Ubuntu based system have config files for distro as well
as custom build kernel in "/boot/" directory. But on rhel based systems,
custom build kernel's config are not present in /boot/ directory.

Use /lib/modules/<kernel_version>/build/.config to check for kernel config
file making test work for both ubuntu and rhel.

Signed-off-by: Dheeraj Kumar Srivastava <dheerajkumar.srivastava@amd.com>
Current code checks dmesg alone for kernel logs. But for cases where
dmesg is cleared, checking journalctl adds more confidence on the outcome
of kernel logs check.

Signed-off-by: Dheeraj Kumar Srivastava <dheerajkumar.srivastava@amd.com>
@narasimhan-v narasimhan-v self-assigned this Mar 4, 2025
@narasimhan-v narasimhan-v merged commit 3994e78 into avocado-framework-tests:master Mar 4, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants