Skip to content

ci: append efi=runtime to kernel cmdline for RT builds#2333

Open
apateriy-qcom wants to merge 1 commit into
qualcomm-linux:masterfrom
apateriy-qcom:master
Open

ci: append efi=runtime to kernel cmdline for RT builds#2333
apateriy-qcom wants to merge 1 commit into
qualcomm-linux:masterfrom
apateriy-qcom:master

Conversation

@apateriy-qcom
Copy link
Copy Markdown
Contributor

@apateriy-qcom apateriy-qcom commented Jun 1, 2026

Without efi=runtime, EFI runtime services are not available in the RT kernel, which prevents efivars updates (e.g. VendorDtbOverlays), breaks watchdog/dump collection, and causes remoteproc/PIL cores to show as offline on RT-KVM builds.

Append efi=runtime to KERNEL_CMDLINE_EXTRA in both RT kas configs (linux-qcom-next-rt.yml and linux-qcom-rt-6.18.yml) . EFI runtime services are already enabled for non-RT builds.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

Test run workflow

Test jobs for commit 59c581d

qcom-distro_linux-qcom-6.18
Pass: 219 | Fail: 3 | Total: 244
qcom-distro
Pass: 249 | Fail: 2 | Total: 277
nodistro
Pass: 9 | Fail: 0 | Total: 9

@test-reporting-app
Copy link
Copy Markdown

test-reporting-app Bot commented Jun 1, 2026

Test Results

  101 files  +   42    622 suites  +341   6h 27m 31s ⏱️ + 3h 35m 48s
  129 tests +   24    115 ✅ +   18   0 💤  -  2  14 ❌ +8 
5 922 runs  +3 276  5 871 ✅ +3 250  36 💤 +17  15 ❌ +9 

For more details on these failures, see this check.

Results for commit 4b91374. ± Comparison against base commit 79d0169.

This pull request removes 2 and adds 26 tests. Note that renamed tests count towards both.
lava ‑ auto-login-action
lava ‑ minimal-boot
10_Kubernetes_Kernel_Config ‑ Kubernetes_Kernel_Config
11_Logging_Journalctl_Validation ‑ Logging_Journalctl_Validation
12_Partition_PostBoot_Validation ‑ Boot_Systemd_KPI_Loop
12_Partition_PostBoot_Validation ‑ Partition_PostBoot_Validation
13_Boot_Systemd_KPI_Loop ‑ Boot_Systemd_KPI_Loop
14_Kubernetes_Kerne[ ‑ Kubernetes_Kernel_Config
5_OpenCV ‑ OpenCV
6_OpenCV ‑ OpenCV
6_irq ‑ irq
7_BT_ON_OFF ‑ BT_ON_OFF
…

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Contributor

@lumag lumag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From: Anurag Pateriya anurag.pateriya@oss.qualcomm.com
Signed-off-by: Anurag Pateriya apateriy@qti.qualcomm.com

Which breaks DCO check.

Also, you are just stating that EFI Runtime services require a bootparam on RT kernels. Please point out why does it affect only RT kernels.

@apateriy-qcom apateriy-qcom force-pushed the master branch 2 times, most recently from 2ee249f to 4703df8 Compare June 2, 2026 08:47
RT kas configs were missing the efi=runtime boot parameter that are
preesnt in non-RT builds. Without it, EFI runtime services (efivars)
are not accessible, preventing KVM boot (enabling virtualization
with el2kvm) on kodiak-el2 and breaking watchdog/dump collection and
remoteproc/PIL cores on RT builds.

Append efi=runtime to KERNEL_CMDLINE_EXTRA in both RT kas configs
(linux-qcom-next-rt.yml and linux-qcom-rt-6.18.yml).

Signed-off-by: Anurag Pateriya <apateriy@qti.qualcomm.com>
Copy link
Copy Markdown
Contributor

@lumag lumag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RT kas configs were missing the efi=runtime boot parameter that are
preesnt in non-RT builds

I don't see efi=runtime being set in any of the non-RT builds. Pleae point out, where they are set.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

Test run workflow

Test jobs for commit 4b91374

qcom-distro_linux-qcom-6.18
Pass: 221 | Fail: 4 | Total: 244
qcom-distro
Pass: 267 | Fail: 3 | Total: 291
nodistro
Pass: 9 | Fail: 0 | Total: 9

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