Conversation
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
Hardware-use freeze: independent review found four high-severity fail-closed defects in watchdog head Written by GPT-5.6 Sol. |
|
STOP - superseded for hardware use: |
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
Corrected replacement pushed at |
|
Independent exact-SHA rereview of Written by GPT-5.6 Sol. |
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
Replacement candidate: This supersedes Finding -> invariant -> regression mapping:
Additional lifecycle coverage confirms signal-mask restoration, delayed SIGTERM handler exit without escalation, direct-leader descendant cleanup, soft-limit descendant Validation:
Hosted CI is running for this exact SHA. Do not consume this candidate until independent review closes all six findings. |
|
Independent exact-SHA review of Written by GPT-5.6 Sol. |
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
New exact review candidate:
Fix invariant:
Regression: Full validation for this exact tree:
Hosted CI and independent exact-SHA review are pending. No hardware or integration authorization is granted. |
|
Independent exact-SHA review of Written by GPT-5.6 Sol. |
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GPT-5.6 Sol Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Overview
Add an independent external Linux host-memory watchdog for the DeepSeek V4.1 Strix Halo validation stack.
meminfoandswapsfrom a configurable procfs root, default/proc/proc/swapscontains any configured swap entryMemTotal - MemAvailableusing integer bytesSIGTERMat the default 116 GiB soft thresholdSIGKILLat the default 118 GiB emergency threshold or after the bounded grace periodSIGHUP,SIGINT, andSIGTERM, terminate remaining descendants, reap the child, emit an authoritative final record, and return shell signal conventionsecondary_errorsThe 118 GiB emergency threshold leaves a documented 2 GiB sampling margin below the strict 120 GiB ceiling. Grace is capped at 30 seconds, sampling at one second, and heartbeat age at five seconds.
This is the independent watchdog slice from halo-box#48. It is stacked on the corrected halo-box#49 branch head
65fdc19831d30b12f06b981c3ca74dd18731f90e. After ggml-org#49 merges, this PR will be recreated or retargeted tohalo-box/strix-llama.cppwith the merged base.Frozen candidate:
bbf182a48123f64c65d56ba48c0fc0c2b1d375f7.This head retains independently approved watchdog ancestry
778db6f50eae04e6c232c69b9575bdbd0747962b, normally merges the corrected ggml-org#49 base, and makes real-process test fixtures wait for complete parseable PID/JSON state instead of racing file creation. It fixes later exact-head review findings: lease validation closes its held pidfd on every success or failure path, parent-signal graceful cleanup refreshes the payload-visible heartbeat while it continues private guardian pulses, and cleanup rechecks the deadline after each synchronous heartbeat attempt so delayed success or failure cannot cause a negative sleep or bypass SIGKILL/reap. Production script SHA-256:98313ee0edabe42455f103f68bcbc6cba0e8e117afae8db658e17a18b87038f9.It deliberately does not implement model admission, the DeepSeek V4.1 graph, Engram, expert caching, or model execution. No model/GGUF was loaded, no inference or model server was started, and no Strix runtime or ROCm package was changed.
Validation
Model-free validation for exact head
bbf182a48123f64c65d56ba48c0fc0c2b1d375f7:python3 -m unittest tests.test_strix_memory_watchdog- 37 discovered, 29 passed, 8 Linux-only skippedpython3 -W error::ResourceWarning -m unittest tests.test_strix_memory_watchdog- 37 discovered, 29 passed, 8 Linux-only skippedResourceWarningpromoted to error, repeated twice - 37/37 each runctest --test-dir /tmp/watchdog-build -R '^test-strix-memory-watchdog$' --output-on-failure- 1/1 passedPYTHONPATH=gguf-py python3 -m unittest gguf-py/tests/test_deepseek41_schema.py- 4/4 passedpython3 -m py_compile scripts/strix_memory_watchdog.py tests/test_strix_memory_watchdog.py- passedflake8-no-print0.1.1 on the watchdog script and tests - passedty0.0.78 on the watchdog script and tests - passedgit diff --check, ASCII-only scan, and leftover-process inspection - passedThe deterministic suite covers parsing, swap gates, thresholds, grace and emergency escalation, malformed/missing procfs, launch failures, child exit propagation, wrapper signals, real child/grandchild cleanup, guardian failure/stall, live launch identity, artifact failure precedence, all closed/blocked guardian-control FD x TERM/KILL/final/lease failure combinations, and the documented
setsid()boundary limitation. The corrective regressions run a real guarded payload whose SIGTERM handler completes after heartbeat max age but before grace, a guarded unresponsive payload that remains alive until grace escalation, repeated successful/rejected lease validation while asserting every injected pidfd is closed and the process descriptor set is unchanged, and successful plus ArtifactError heartbeat writes that cross the grace deadline while proving TERM, SIGKILL, reap, and authoritative final classification still execute without a negative sleep.Independent exact-SHA review approved
bbf182a48123f64c65d56ba48c0fc0c2b1d375f7with no remaining blockers. The reviewer rechecked all nine earlier lifecycle/evidence areas, the complete eight-cell closed/blocked guardian-control FD x TERM/KILL/final/lease matrix, pidfd closure, guarded long-handler and unresponsive-child grace behavior, and both heartbeat deadline-overrun cases. Earlier restacked heads061389280e8df1926f89d51596caeceb41dc2720andc3ddb694db9f94d0108cb937071af07f6daefadcremain rejected and must not be used.Hosted Server CI passed on Ubuntu and Windows and the labeler passed for exact head
bbf182a48123f64c65d56ba48c0fc0c2b1d375f7. Hosted flake8 and Python type-check jobs remain queued because this fork has no[self-hosted, fast]runner; equivalent targeted local checks pass. No runner was installed or registered.Requirements