Pull requests / #2611

#2611 sycl: strata-sycl.sh passes EnableRecoverablePageFaults on to the container, 0 included (#2157)

closed · @qiuzan2001 · 0 comentarios · En GitHub

Setup & installServer & APIMulti-GPUDocumentation

Descripción

**What it changes.** `sycl/serve/strata-sycl.sh` (the Docker launcher `setup.py --backend sycl` installs, via `sycl/setup_intel.py`) forwards a fixed set of variables into the container and drops any value of `0` (for the engine's switches, 0 means "not set"). #2157's fix for a lost Arc card is `NEOReadDebugKeys=1` with `EnableRecoverablePageFaults=0` in the engine's environment (2 of 3 runs lost the card without, 0 of 4 with, on the reporter's 2x Arc Pro B70); the second key never reached a setup install: its name was not on the list, and its value is 0. Now `EnableRecoverablePageFaults` is forwarded with any non-empty value, 0 included. The docs side is #2602.

**What it leaves alone.** Every other variable is forwarded or dropped exactly as before (the old zero rule is unchanged in the `else` branch), and setup does not write the key into the config; a user adds it to `"env"` by hand.

**How it was checked.** Not run on Intel hardware (there is none here). `bash -n` and `shellcheck` (clean, as before). A dry run of the old and the new script with a stub `docker` on `PATH` that prints the `-e` arguments:

| Environment | old | new |
| --- | --- | --- |
| `EnableRecoverablePageFaults=0 NEOReadDebugKeys=1` | `NEOReadDebugKeys=1` + the port defaults | the same + `EnableRecoverablePageFaults=0` |
| `EnableRecoverablePageFaults=1 NEOReadDebugKeys=1` | `NEOReadDebugKeys=1` + defaults | the same + `EnableRecoverablePageFaults=1` |
| `EnableRecoverablePageFaults=` `IGC_X=0` `STRATA_VERIFY_NO_HOST=0` `UR_Y=a=0` `ONEAPI_DEVICE_SELECTOR=level_zero:0` | `ONEAPI_DEVICE_SELECTOR=level_zero:0 STRATA_STAGER_THREADS=12 STRATA_VERIFY_DEVICE_PLAN=1 UR_Y=a=0` | identical |

A sub-agent reviewer re-ran the comparison with more variables (zeros, empties, quoted values, no environment) and checked `set -euo pipefail` behaviour: only the new key differs.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

En el sitio

Enlaces a install, modelos, releases.