Skip to content

[Test] Port roi_pool (MPS) to stable ABI. - #9583

Draft
adabeyta wants to merge 1 commit into
adabeyta/roi-pool-cuda-stablefrom
adabeyta/roi-pool-mps-stable
Draft

[Test] Port roi_pool (MPS) to stable ABI.#9583
adabeyta wants to merge 1 commit into
adabeyta/roi-pool-cuda-stablefrom
adabeyta/roi-pool-mps-stable

Conversation

@adabeyta

@adabeyta adabeyta commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Ports roi_pool's MPS kernels to the stable ABI.

Notes

  • Requires [Stable ABI] Add torch_mps_set_arg_bytes to the MPS shim. pytorch#190932, which lands torch_mps_set_arg_bytes.
  • Shader source moves to roi_pool_metal_shader.h. Once every MPS op migrates, the per-op headers can consolidate to a mps_stable_kernels.h and mps_kernels.h gets dropped.
  • Determinism contract. The MPS backward's alert can't call from a stable kernel, so the Python backward's gate becomes grad_output.device.type in ("cuda", "mps").

@pytorch-bot

pytorch-bot Bot commented Aug 4, 2026

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/vision/9583

Note: Links to docs will display an error until the docs builds have been completed.

❌ 24 New Failures, 4 Pending, 1 Unclassified Failure

As of commit 1981680 with merge base 9f1d924 (image):

NEW FAILURES - The following jobs have failed:

UNCLASSIFIED FAILURE - DrCI could not classify the following job because the workflow did not run on the merge base. The failure may be pre-existing on trunk or introduced by this PR:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the cla signed label Aug 4, 2026
@adabeyta
adabeyta force-pushed the adabeyta/roi-pool-mps-stable branch from 99f447f to 4fd4d29 Compare August 4, 2026 23:10
@NicolasHug
NicolasHug force-pushed the adabeyta/roi-pool-mps-stable branch from 4fd4d29 to 1981680 Compare August 5, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant