Skip to content

Benchmark CI: skip unsupported SYCL SinKernelGraph graph-mode cases on PVC - #23046

Closed
lukaszstolarczuk with Copilot wants to merge 3 commits into
syclfrom
copilot/fix-github-actions-benchmark-job
Closed

Benchmark CI: skip unsupported SYCL SinKernelGraph graph-mode cases on PVC#23046
lukaszstolarczuk with Copilot wants to merge 3 commits into
syclfrom
copilot/fix-github-actions-benchmark-job

Conversation

Copilot AI commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

The [PR] Benchmark suite testing / Framework test only: L0, Minimal preset, dry-run job failed because graph_api_benchmark_sycl was invoked with --withGraphs=1 on PVC, where graph record/replay is unsupported and aborts under --exit-on-failure.
This PR narrows benchmark scheduling so unsupported SYCL graph-mode variants are not selected on that platform.

  • Root-cause alignment

    • In ComputeBench.benchmarks(), GraphApiSinKernelGraph(..., with_graphs=1, ...) is now skipped when:
      • runtime is in SYCL_RUNTIMES
      • adapter is level_zero_v2
      • detected device architecture contains pvc
  • Selection logic change (minimal scope)

    • Existing adapter-based SYCL graph gating remains intact.
    • New PVC guard applies only to the failing graph-mode path (with_graphs == 1), leaving non-graph (with_graphs == 0) cases unchanged.
  • Targeted coverage

    • Added devops/scripts/benchmarks/tests/test_compute_bench.py checks for:
      • SYCL graph-mode excluded on PVC
      • SYCLPreview graph-mode excluded on PVC
      • SYCL graph-mode retained on non-PVC
if (
    "pvc" in device_arch
    and options.ur_adapter == "level_zero_v2"
    and runtime in SYCL_RUNTIMES
    and with_graphs == 1
):
    continue

Copilot AI and others added 2 commits August 28, 2026 07:51
Co-authored-by: lukaszstolarczuk <29728779+lukaszstolarczuk@users.noreply.github.com>
Co-authored-by: lukaszstolarczuk <29728779+lukaszstolarczuk@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job for benchmark suite Benchmark CI: skip unsupported SYCL SinKernelGraph graph-mode cases on PVC Aug 28, 2026
Copilot AI requested a review from lukaszstolarczuk August 28, 2026 07:53
@lukaszstolarczuk

Copy link
Copy Markdown
Contributor

merged into: #23033

@bader
bader deleted the copilot/fix-github-actions-benchmark-job branch August 28, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants