Add: partial-good-aware AICPU thread resolution (a2a3/a5) - #1521
Open
yanghaoran29 wants to merge 1 commit into
Open
Add: partial-good-aware AICPU thread resolution (a2a3/a5)#1521yanghaoran29 wants to merge 1 commit into
yanghaoran29 wants to merge 1 commit into
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughAICPU thread configuration now uses ChangesAICPU auto-resolution
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant TestCase
participant DeviceRunner
participant TopologyProbe
participant Runtime
TestCase->>DeviceRunner: submit config with aicpu_thread_num=0
DeviceRunner->>TopologyProbe: probe usable AICPU cores
TopologyProbe-->>DeviceRunner: return usable count
DeviceRunner->>Runtime: write resolved thread count
Runtime-->>TestCase: launch with resolved configuration
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
yanghaoran29
force-pushed
the
aicpu-pg-a2a3-a5
branch
3 times, most recently
from
July 27, 2026 07:11
3a46dec to
0f7adad
Compare
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 6
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/a2a3/platform/onboard/host/device_runner.cpp`:
- Around line 330-337: Keep the resolved AICPU count consistent across runtime
and launch/setup state: in src/a2a3/platform/onboard/host/device_runner.cpp
lines 330-337, propagate resolved_aicpu to launch_aicpu_num or use the updated
Runtime getter for subsequent calls; in
src/a2a3/platform/sim/host/device_runner.cpp lines 234-239 and
src/a5/platform/sim/host/device_runner.cpp lines 217-222, write each resolved
fallback value back via runtime.set_aicpu_thread_num(...).
In `@src/a2a3/platform/sim/host/device_runner.cpp`:
- Around line 234-239: After resolving the zero-value fallback in the launch
flow, write the resulting launch_aicpu_num back to the Runtime object before it
is passed to AICPU execution and setup. Update the existing runtime thread-count
state used by get_aicpu_thread_num(), while preserving configured nonzero values
and the PLATFORM_DEFAULT_AICPU_THREAD_NUM fallback.
In `@src/a5/platform/onboard/host/device_runner.cpp`:
- Around line 159-163: Update the launch_aicpu_num normalization in the device
runner to treat only aicpu_thread_num equal to zero as auto; preserve negative
values so validate_launch_aicpu_num() rejects them instead of converting them to
PLATFORM_DEFAULT_AICPU_THREAD_NUM.
- Around line 159-163: After the AICPU resolution logic that computes
resolved_aicpu, update Runtime unconditionally via
runtime.set_aicpu_thread_num(resolved_aicpu). Place this after the allowed-CPU
clamping path so both configured and auto modes publish the effective active
count, including when no adjustment branch is taken.
In `@src/a5/platform/sim/host/device_runner.cpp`:
- Around line 217-222: After resolving the platform default in the launch setup,
write the resulting launch_aicpu_num back through the Runtime setter so
runtime.get_aicpu_thread_num() matches the count used by init_l2_swimlane and
AICPU execution. Keep the existing explicit-count behavior unchanged.
In `@src/common/platform/onboard/host/device_runner_base.cpp`:
- Around line 1168-1174: Update the clamp warning in the thread-count
calculation to log desired instead of requested, so auto mode reports the
architecture default selected by the desired calculation. Preserve the existing
warning text, usable count, and scheduler count.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 8a54c647-7f3b-42da-ab4b-d4cd2bea8eac
📒 Files selected for processing (92)
examples/a2a3/tensormap_and_ringbuffer/benchmark_bgemm/test_benchmark_bgemm.pyexamples/a2a3/tensormap_and_ringbuffer/merge_pipeline_barrier/test_merge_pipeline_barrier.pyexamples/a2a3/tensormap_and_ringbuffer/paged_attention/test_paged_attention.pyexamples/a2a3/tensormap_and_ringbuffer/paged_attention_manual_scope/test_paged_attention.pyexamples/a2a3/tensormap_and_ringbuffer/paged_attention_ringbuffer/test_paged_attention_ringbuffer.pyexamples/a2a3/tensormap_and_ringbuffer/paged_attention_unroll_manual_scope/test_paged_attention_unroll.pyexamples/a2a3/tensormap_and_ringbuffer/qwen3_14b_decode/test_qwen3_14b_decode.pyexamples/a2a3/tensormap_and_ringbuffer/scalar_data_test/test_scalar_data.pyexamples/a2a3/tensormap_and_ringbuffer/vector_example/test_vector_example.pyexamples/a5/tensormap_and_ringbuffer/bgemm/test_bgemm.pyexamples/a5/tensormap_and_ringbuffer/paged_attention/test_paged_attention.pyexamples/a5/tensormap_and_ringbuffer/paged_attention_manual_scope/test_paged_attention.pyexamples/a5/tensormap_and_ringbuffer/paged_attention_unroll_manual_scope/test_paged_attention_unroll.pyexamples/a5/tensormap_and_ringbuffer/vector_example/test_vector_example.pypython/simpler/task_interface.pysimpler_setup/scene_test.pysrc/a2a3/docs/hardware.mdsrc/a2a3/platform/include/common/platform_config.hsrc/a2a3/platform/onboard/host/device_runner.cppsrc/a2a3/platform/sim/host/device_runner.cppsrc/a5/docs/hardware.mdsrc/a5/platform/include/common/platform_config.hsrc/a5/platform/onboard/host/device_runner.cppsrc/a5/platform/sim/host/device_runner.cppsrc/common/platform/onboard/host/device_runner_base.cppsrc/common/platform/onboard/host/device_runner_base.hsrc/common/platform/sim/host/device_runner_base.cppsrc/common/task_interface/call_config.htests/st/a2a3/host_build_graph/available_aicore_counts/test_available_aicore_counts.pytests/st/a2a3/host_build_graph/bgemm/test_bgemm.pytests/st/a2a3/host_build_graph/dfx/dep_gen/test_dep_gen.pytests/st/a2a3/host_build_graph/dump_args/test_dump_args_example.pytests/st/a2a3/host_build_graph/matmul/test_matmul.pytests/st/a2a3/host_build_graph/paged_attention/test_paged_attention.pytests/st/a2a3/host_build_graph/prepared_callable/test_prepared_callable.pytests/st/a2a3/host_build_graph/run_stream_reuse/test_run_stream_reuse.pytests/st/a2a3/host_build_graph/vector_example/test_vector_example.pytests/st/a2a3/tensormap_and_ringbuffer/alternating_matmul_add/test_alternating_matmul_add.pytests/st/a2a3/tensormap_and_ringbuffer/available_aicore_counts/test_available_aicore_counts.pytests/st/a2a3/tensormap_and_ringbuffer/batch_paged_attention/test_batch_paged_attention.pytests/st/a2a3/tensormap_and_ringbuffer/dfx/args_dump/test_args_dump.pytests/st/a2a3/tensormap_and_ringbuffer/dfx/dep_gen/test_dep_gen.pytests/st/a2a3/tensormap_and_ringbuffer/dfx/l2_swimlane/test_l2_swimlane.pytests/st/a2a3/tensormap_and_ringbuffer/dfx/l2_swimlane/test_l2_swimlane_mixed.pytests/st/a2a3/tensormap_and_ringbuffer/dfx/pmu/test_pmu.pytests/st/a2a3/tensormap_and_ringbuffer/dfx/scope_stats/test_scope_stats.pytests/st/a2a3/tensormap_and_ringbuffer/dynamic_register/test_dynamic_register.pytests/st/a2a3/tensormap_and_ringbuffer/fanin_lookup_perf/test_fanin_lookup_perf.pytests/st/a2a3/tensormap_and_ringbuffer/mixed_example/test_mixed_example.pytests/st/a2a3/tensormap_and_ringbuffer/multi_round_paged_attention/test_multi_round_paged_attention.pytests/st/a2a3/tensormap_and_ringbuffer/orch_so_cache/test_orch_so_cache.pytests/st/a2a3/tensormap_and_ringbuffer/paged_attention_unroll/test_paged_attention_unroll.pytests/st/a2a3/tensormap_and_ringbuffer/paged_attention_unroll_4dims/test_paged_attention_unroll_4dims.pytests/st/a2a3/tensormap_and_ringbuffer/prepared_callable/test_prepared_callable.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_basic/test_spmd_basic.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_batch_dispatch_oob/test_spmd_batch_dispatch_oob.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_multiblock_aiv/test_spmd_multiblock_aiv.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_multiblock_mix/test_spmd_multiblock_mix.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_paged_attention/test_spmd_paged_attention.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_paged_attention_highperf/test_spmd_paged_attention_highperf.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_starvation/test_spmd_starvation.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start/test_spmd_sync_start.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start_aiv/test_spmd_sync_start_aiv.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start_early_dispatch/test_spmd_sync_start_early_dispatch.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start_edge/test_spmd_sync_start_edge.pytests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start_stress/test_spmd_sync_start_stress.pytests/st/a2a3/tensormap_and_ringbuffer/test_l3_dependency.pytests/st/a2a3/tensormap_and_ringbuffer/test_l3_group.pytests/st/a5/tensormap_and_ringbuffer/available_aicore_counts/test_available_aicore_counts.pytests/st/a5/tensormap_and_ringbuffer/dfx/args_dump/test_args_dump.pytests/st/a5/tensormap_and_ringbuffer/dfx/dep_gen/test_dep_gen.pytests/st/a5/tensormap_and_ringbuffer/dfx/l2_swimlane/test_l2_swimlane.pytests/st/a5/tensormap_and_ringbuffer/dfx/l2_swimlane/test_l2_swimlane_mixed.pytests/st/a5/tensormap_and_ringbuffer/dfx/pmu/test_pmu.pytests/st/a5/tensormap_and_ringbuffer/dfx/scope_stats/test_scope_stats.pytests/st/a5/tensormap_and_ringbuffer/mixed_example/test_mixed_example.pytests/st/a5/tensormap_and_ringbuffer/orch_so_cache/test_orch_so_cache.pytests/st/a5/tensormap_and_ringbuffer/paged_attention_unroll/test_paged_attention_unroll.pytests/st/a5/tensormap_and_ringbuffer/prepared_callable/test_prepared_callable.pytests/st/a5/tensormap_and_ringbuffer/simt_basic/test_simt_basic.pytests/st/a5/tensormap_and_ringbuffer/spmd_basic/test_spmd_basic.pytests/st/a5/tensormap_and_ringbuffer/spmd_multiblock_aiv/test_spmd_multiblock_aiv.pytests/st/a5/tensormap_and_ringbuffer/spmd_multiblock_mix/test_spmd_multiblock_mix.pytests/st/a5/tensormap_and_ringbuffer/spmd_starvation/test_spmd_starvation.pytests/st/a5/tensormap_and_ringbuffer/spmd_sync_start/test_spmd_sync_start.pytests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_aiv/test_spmd_sync_start_aiv.pytests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_early_dispatch/test_spmd_sync_start_early_dispatch.pytests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_edge/test_spmd_sync_start_edge.pytests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_mix_spill/test_spmd_sync_start_mix_spill.pytests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_stress/test_spmd_sync_start_stress.pytests/st/task_timing/task_timing_slots/test_task_timing_e2e.pytests/ut/py/test_chip_worker.py
💤 Files with no reviewable changes (72)
- tests/st/a5/tensormap_and_ringbuffer/spmd_starvation/test_spmd_starvation.py
- examples/a2a3/tensormap_and_ringbuffer/benchmark_bgemm/test_benchmark_bgemm.py
- tests/st/a2a3/tensormap_and_ringbuffer/alternating_matmul_add/test_alternating_matmul_add.py
- examples/a2a3/tensormap_and_ringbuffer/merge_pipeline_barrier/test_merge_pipeline_barrier.py
- examples/a5/tensormap_and_ringbuffer/vector_example/test_vector_example.py
- tests/st/a2a3/tensormap_and_ringbuffer/dfx/l2_swimlane/test_l2_swimlane.py
- tests/st/a5/tensormap_and_ringbuffer/mixed_example/test_mixed_example.py
- examples/a2a3/tensormap_and_ringbuffer/scalar_data_test/test_scalar_data.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_aiv/test_spmd_sync_start_aiv.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_multiblock_aiv/test_spmd_multiblock_aiv.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_sync_start/test_spmd_sync_start.py
- tests/st/a5/tensormap_and_ringbuffer/available_aicore_counts/test_available_aicore_counts.py
- tests/st/a5/tensormap_and_ringbuffer/dfx/pmu/test_pmu.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start/test_spmd_sync_start.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start_aiv/test_spmd_sync_start_aiv.py
- tests/st/a2a3/tensormap_and_ringbuffer/available_aicore_counts/test_available_aicore_counts.py
- tests/st/a5/tensormap_and_ringbuffer/dfx/args_dump/test_args_dump.py
- tests/st/a5/tensormap_and_ringbuffer/dfx/l2_swimlane/test_l2_swimlane.py
- examples/a2a3/tensormap_and_ringbuffer/paged_attention_ringbuffer/test_paged_attention_ringbuffer.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_starvation/test_spmd_starvation.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_basic/test_spmd_basic.py
- tests/st/a5/tensormap_and_ringbuffer/dfx/l2_swimlane/test_l2_swimlane_mixed.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_multiblock_mix/test_spmd_multiblock_mix.py
- tests/st/a2a3/host_build_graph/bgemm/test_bgemm.py
- tests/st/a5/tensormap_and_ringbuffer/prepared_callable/test_prepared_callable.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_early_dispatch/test_spmd_sync_start_early_dispatch.py
- tests/st/a5/tensormap_and_ringbuffer/dfx/scope_stats/test_scope_stats.py
- tests/st/a2a3/tensormap_and_ringbuffer/dfx/scope_stats/test_scope_stats.py
- tests/st/a2a3/host_build_graph/vector_example/test_vector_example.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start_early_dispatch/test_spmd_sync_start_early_dispatch.py
- examples/a2a3/tensormap_and_ringbuffer/qwen3_14b_decode/test_qwen3_14b_decode.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_mix_spill/test_spmd_sync_start_mix_spill.py
- tests/st/a2a3/tensormap_and_ringbuffer/dfx/l2_swimlane/test_l2_swimlane_mixed.py
- tests/st/a5/tensormap_and_ringbuffer/paged_attention_unroll/test_paged_attention_unroll.py
- tests/st/a2a3/host_build_graph/matmul/test_matmul.py
- examples/a2a3/tensormap_and_ringbuffer/vector_example/test_vector_example.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start_edge/test_spmd_sync_start_edge.py
- tests/st/a2a3/host_build_graph/available_aicore_counts/test_available_aicore_counts.py
- tests/st/a2a3/tensormap_and_ringbuffer/dfx/args_dump/test_args_dump.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_sync_start_stress/test_spmd_sync_start_stress.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_batch_dispatch_oob/test_spmd_batch_dispatch_oob.py
- examples/a2a3/tensormap_and_ringbuffer/paged_attention_unroll_manual_scope/test_paged_attention_unroll.py
- tests/st/a2a3/tensormap_and_ringbuffer/multi_round_paged_attention/test_multi_round_paged_attention.py
- tests/st/a2a3/host_build_graph/prepared_callable/test_prepared_callable.py
- tests/st/a2a3/tensormap_and_ringbuffer/dfx/pmu/test_pmu.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_stress/test_spmd_sync_start_stress.py
- examples/a5/tensormap_and_ringbuffer/paged_attention_manual_scope/test_paged_attention.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_multiblock_mix/test_spmd_multiblock_mix.py
- tests/st/a2a3/host_build_graph/dump_args/test_dump_args_example.py
- tests/st/a2a3/tensormap_and_ringbuffer/paged_attention_unroll/test_paged_attention_unroll.py
- examples/a2a3/tensormap_and_ringbuffer/paged_attention/test_paged_attention.py
- tests/st/a2a3/host_build_graph/paged_attention/test_paged_attention.py
- tests/st/a2a3/tensormap_and_ringbuffer/dfx/dep_gen/test_dep_gen.py
- tests/st/a2a3/tensormap_and_ringbuffer/mixed_example/test_mixed_example.py
- examples/a5/tensormap_and_ringbuffer/bgemm/test_bgemm.py
- tests/st/a2a3/tensormap_and_ringbuffer/paged_attention_unroll_4dims/test_paged_attention_unroll_4dims.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_basic/test_spmd_basic.py
- tests/st/a5/tensormap_and_ringbuffer/spmd_sync_start_edge/test_spmd_sync_start_edge.py
- tests/st/a5/tensormap_and_ringbuffer/orch_so_cache/test_orch_so_cache.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_paged_attention_highperf/test_spmd_paged_attention_highperf.py
- tests/st/a5/tensormap_and_ringbuffer/dfx/dep_gen/test_dep_gen.py
- examples/a5/tensormap_and_ringbuffer/paged_attention_unroll_manual_scope/test_paged_attention_unroll.py
- tests/st/task_timing/task_timing_slots/test_task_timing_e2e.py
- tests/st/a2a3/host_build_graph/dfx/dep_gen/test_dep_gen.py
- examples/a5/tensormap_and_ringbuffer/paged_attention/test_paged_attention.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_multiblock_aiv/test_spmd_multiblock_aiv.py
- tests/st/a2a3/tensormap_and_ringbuffer/orch_so_cache/test_orch_so_cache.py
- tests/st/a2a3/tensormap_and_ringbuffer/fanin_lookup_perf/test_fanin_lookup_perf.py
- tests/st/a2a3/tensormap_and_ringbuffer/spmd_paged_attention/test_spmd_paged_attention.py
- tests/st/a2a3/tensormap_and_ringbuffer/dynamic_register/test_dynamic_register.py
- tests/st/a2a3/tensormap_and_ringbuffer/batch_paged_attention/test_batch_paged_attention.py
- tests/st/a2a3/tensormap_and_ringbuffer/prepared_callable/test_prepared_callable.py
yanghaoran29
force-pushed
the
aicpu-pg-a2a3-a5
branch
3 times, most recently
from
July 27, 2026 11:28
561e54d to
4bb36f8
Compare
aicpu_thread_num is now derived from the probed AICPU usable count and an
arch-default topology instead of a fixed user value that must stay <= usable:
- 0 (the new CallConfig/scene_test default) = auto ->
PLATFORM_DEFAULT_AICPU_THREAD_NUM (a2a3: 1 orch + 3 sched = 4; a5: 1 orch +
4 sched = 5).
- DeviceRunner clamps it to the probed usable pool (PG/OS cores are absent
from the AICPU OCCUPY bitmap), running 1 orch + fewer schedulers on a die
with fewer usable AICPU, and errors if usable < 2 (need >=1 orch + 1 sched).
- orch is always 1 (last thread), sched = total-1 (existing executor split).
validate_launch_aicpu_num allows 0 (auto). Sim resolves 0 -> arch default
(no real OCCUPY/PG) and writes the effective count back to Runtime so AICPU
init / DFX match the launch gate; onboard clamps via
DeviceRunnerBase::resolve_aicpu_thread_num and always publishes the resolved
count (a5 auto no longer leaves Runtime at the 0 sentinel). a5 normalizes
only ==0 as auto so negatives still fail validation. Clamp warnings log the
resolved desired count.
prepared_callable ST helpers use case.get("config", {}) after dropping the
fixed aicpu_thread_num override from CASES.
hardware.md (a2a3/a5) documents the policy and aligns the "user-accessible 6"
note with the active cap (4 a2a3 / 7 a5).
a5 host/device OCCUPY alignment (host 0x1fe=8 vs device 0x1f8=6) is deferred:
the new a5 default 5 <= 6 device-usable covers the common case; deeper
PG-degraded tolerance needs a5 onboard verification.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
aicpu_thread_num is now derived from the probed AICPU usable count and an arch-default topology instead of a fixed user value that must stay <= usable:
validate_launch_aicpu_num allows 0 (auto). Sim resolves 0 -> arch default (no real OCCUPY/PG); onboard clamps via DeviceRunnerBase::resolve_aicpu_thread_num. hardware.md (a2a3/a5) documents the policy and aligns the "user-accessible 6" note with the active cap (4 a2a3 / 7 a5).
a5 host/device OCCUPY alignment (host 0x1fe=8 vs device 0x1f8=6) is deferred: the new a5 default 5 <= 6 device-usable covers the common case; deeper PG-degraded tolerance needs a5 onboard verification.