Skip to content

[26.04_linux-nvidia-bos] watchdog: sbsa_gwdt: add early_enable module parameter - #558

Open
kbutala-nv wants to merge 1 commit into
NVIDIA:26.04_linux-nvidia-bosfrom
kbutala-nv:sbsa-watchdog-bos
Open

[26.04_linux-nvidia-bos] watchdog: sbsa_gwdt: add early_enable module parameter#558
kbutala-nv wants to merge 1 commit into
NVIDIA:26.04_linux-nvidia-bosfrom
kbutala-nv:sbsa-watchdog-bos

Conversation

@kbutala-nv

Copy link
Copy Markdown

This PR contains backport of change https://patch.msgid.link/[email protected]

We need this patch to support newer firmware revisions.

(backported from commit 11f93e639d51 groeck/linux-staging.git)

@nirmoy nirmoy added the help wanted Extra attention is needed label Sep 1, 2026
@nirmoy

nirmoy commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

BaseOS Kernel Review

Warning

⚠️ Review needs attention

SBSA watchdog early_enable can trigger an unintended reset before userspace when the pre-userspace keepalive handler is disabled.

Findings: Critical 0 · High 0 · Medium 1 · Low 0

🔍 Review artifacts

📦 Kernel deb builds — 🟢 2/2 passed

Note

Build reports and debs are retained for 10 days after the PR closes.

Review metadata
  • Reviewed head: f92bed6ee130
  • Overall status: attention needed
  • Architectures: 2/2 successful

This comment is maintained by BaseOS Reviewer and updated when the GitHub watcher publishes a newer review.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

PR Validation Report

Patchscan ✅ No Missing Fixes

All cherry-picked commits checked — no missing upstream fixes found.

PR Lint ❌ Errors found

Details
Checking 1 commits...

Cherry-pick digest:
┌──────────────┬──────────────────────────────────────────────────────────────────┬────────────┬─────────┬───────────────────────────┐
│ Local        │ Referenced upstream / Patch subject                              │ Patch-ID   │ Subject │ SoB chain                 │
├──────────────┼──────────────────────────────────────────────────────────────────┼────────────┼─────────┼───────────────────────────┤
│ f92bed6ee130 │ [SAUCE] watchdog: sbsa_gwdt: add early_enable module parameter   │ N/A        │ N/A     │ wang, linux, kbutala      │
└──────────────┴──────────────────────────────────────────────────────────────────┴────────────┴─────────┴───────────────────────────┘

Lint results:
E: f92bed6ee130 ("watchdog: sbsa_gwdt: add early_enable module param"): not SAUCE/UBUNTU/Revert but has no upstream reference trailer (cherry picked from commit ... or backported from ...)

PR metadata:
E: PR targets 26.04_linux-nvidia-bos but body has no https://bugs.launchpad.net/... link

On SBSA platforms using standard UEFI firmware (such as EDK II), the
watchdog timer is often enabled during early boot stages but explicitly
disabled by the firmware before handing over control to the OS (e.g.,
during ExitBootServices). This is done to prevent unintended resets
while the OS is loading, assuming the OS watchdog driver will take over.

However, this leaves a protection gap. If the system hangs between the
firmware handover and the userspace watchdog daemon startup, the hardware
watchdog will not fire to recover the system. For safety-critical systems
that require continuous hardware watchdog protection from the earliest
possible moment, this gap is problematic.

Add an 'early_enable' module parameter to allow the kernel driver to
re-enable the watchdog immediately during probe if it was left disabled
by the firmware. By setting the WDOG_HW_RUNNING status bit, the watchdog
core is instructed that the hardware is active. As a result, the core's
pre-userspace handler (controlled by 'handle_boot_enabled') will
automatically issue periodic keepalives until userspace opens the device.

This bridges the protection gap seamlessly without requiring firmware
modifications and without risking unintended resets during kernel boot.

The parameter defaults to false to preserve the traditional behavior.

Signed-off-by: Zexin Wang <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Guenter Roeck <[email protected]>
(backported from commit 11f93e639d51 groeck/linux-staging.git)
Signed-off-by: Kaushal Rajeev Butala <[email protected]>
@kbutala-nv kbutala-nv changed the title [26.04_linux-nvidia-bos] NVIDIA: SAUCE: watchdog: sbsa_gwdt: add early_enable module parameter [26.04_linux-nvidia-bos] watchdog: sbsa_gwdt: add early_enable module parameter Sep 1, 2026
@nirmoy

nirmoy commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Is this patch needed for Vera too @kbutala-nv ?

@nvmochs

nvmochs commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Same comment as in PR 557, please pick with -xs and append "linux-next" after the SHA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed pending_review_comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants