test: cover chunk prefetch in the import-window fixture #488

Merged
monsterlag merged 1 commit from fix/import-prefetch-fixture into main 2026-11-08 08:19:57 +00:00
monsterlag commented 2026-11-08 05:50:19 +00:00 (Migrated from github.com)

pool_lookup now consumes prefetched pools and schedules neighbouring chunks, but its import-window fixture still omits those dependencies. The existing Linux check consequently fails to compile with missing prefetch_chunks, Pool.prefetched and st_prefetch_* declarations.

Compile the real production prefetch helper with an explicit fake device and recording background queue. Keep every existing import-window assertion, and add checks for first use of a prefetched pool, repeated/unprefetched lookups, on-demand neighbour queueing, prefetch disabled by configuration, and unavailable host-memory imports. No production import policy or workflow is changed.

Validation on source ef2e1c4 plus commit 2d3faa3:

  • Audited GNU Windows Clang23.1.5 compiled the fixture and all six cases passed: original16/3/0MB windows, plus three prefetch configurations.
  • The unmodified main fixture reproduced the undeclared-prefetch compile failure. A separate in-memory negative leaving the consumed prefetch flag set fails the new first-use assertion; production source was not modified.
  • git diff --check passes. Existing Linux GCC CI will run the updated file; it has not been run locally. GPU imports, driver timing and asynchronous background-thread completion are outside this fixture.
`pool_lookup` now consumes prefetched pools and schedules neighbouring chunks, but its import-window fixture still omits those dependencies. The existing Linux check consequently fails to compile with missing `prefetch_chunks`, `Pool.prefetched` and `st_prefetch_*` declarations. Compile the real production prefetch helper with an explicit fake device and recording background queue. Keep every existing import-window assertion, and add checks for first use of a prefetched pool, repeated/unprefetched lookups, on-demand neighbour queueing, prefetch disabled by configuration, and unavailable host-memory imports. No production import policy or workflow is changed. Validation on source `ef2e1c4` plus commit `2d3faa3`: - Audited GNU Windows Clang23.1.5 compiled the fixture and all six cases passed: original16/3/0MB windows, plus three prefetch configurations. - The unmodified main fixture reproduced the undeclared-prefetch compile failure. A separate in-memory negative leaving the consumed prefetch flag set fails the new first-use assertion; production source was not modified. - `git diff --check` passes. Existing Linux GCC CI will run the updated file; it has not been run locally. GPU imports, driver timing and asynchronous background-thread completion are outside this fixture.
Commenting is not possible because the repository is archived.
No description provided.