Skip to content

[CUB] Refactor DevicePartition::If to always take an environment#9464

Open
miscco wants to merge 1 commit into
device_partition_flaggedfrom
device_partition_flagged_env
Open

[CUB] Refactor DevicePartition::If to always take an environment#9464
miscco wants to merge 1 commit into
device_partition_flaggedfrom
device_partition_flagged_env

Conversation

@miscco

@miscco miscco commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Description

closes

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@copy-pr-bot

copy-pr-bot Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@cccl-authenticator-app cccl-authenticator-app Bot moved this from Todo to In Progress in CCCL Jun 15, 2026
@miscco miscco force-pushed the device_partition_flagged_env branch from 89ae2b2 to b6d1633 Compare June 15, 2026 09:51
@miscco miscco force-pushed the device_partition_flagged branch 2 times, most recently from 04194e2 to 558503f Compare June 15, 2026 12:38
@miscco miscco force-pushed the device_partition_flagged_env branch 2 times, most recently from 48a510a to bd7b7d9 Compare June 16, 2026 10:49
@miscco miscco force-pushed the device_partition_flagged branch from 003860d to 214516b Compare June 22, 2026 10:17
@miscco miscco force-pushed the device_partition_flagged_env branch from bd7b7d9 to 1fd1bb1 Compare June 22, 2026 10:17
@copy-pr-bot

copy-pr-bot Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@miscco miscco marked this pull request as ready for review June 22, 2026 10:19
@miscco miscco requested review from a team as code owners June 22, 2026 10:19
@cccl-authenticator-app cccl-authenticator-app Bot moved this from In Progress to In Review in CCCL Jun 22, 2026
@miscco miscco force-pushed the device_partition_flagged branch from 214516b to ef6e110 Compare June 22, 2026 11:15
@miscco miscco force-pushed the device_partition_flagged_env branch from 1fd1bb1 to 459699b Compare June 22, 2026 11:15
@miscco miscco force-pushed the device_partition_flagged branch from ef6e110 to 5fc7e96 Compare June 22, 2026 11:20
@miscco miscco force-pushed the device_partition_flagged_env branch from 459699b to 4e582bc Compare June 22, 2026 11:20
@github-actions

This comment has been minimized.

@miscco miscco force-pushed the device_partition_flagged_env branch from 4e582bc to 1abbfcf Compare June 23, 2026 07:21
@miscco miscco force-pushed the device_partition_flagged branch from 5fc7e96 to 7252c69 Compare June 23, 2026 07:21
@miscco miscco force-pushed the device_partition_flagged_env branch from 1abbfcf to d1954a8 Compare June 23, 2026 08:46
@miscco miscco force-pushed the device_partition_flagged branch from 7252c69 to 45c5b58 Compare June 23, 2026 08:46

@Jacobfaib Jacobfaib left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

libcu++ LGTM


// Copy the result back from storage
_CCCL_TRY_CUDA_API(
::cudaMemcpyAsync,

@Jacobfaib Jacobfaib Jun 23, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated to the current PR, but we should try and use the driver wherever possible so that we are explicit about the CUDA contexts. Especially for multi-GPU, the fact that runtime API leaves its the context on the stack is very problematic because some APIs (like cuMemcpy() actually) will behave differently if there is a context on the stack already.

@github-actions

This comment has been minimized.

@miscco miscco force-pushed the device_partition_flagged branch from 45c5b58 to e2fea4c Compare June 25, 2026 08:26
@miscco miscco force-pushed the device_partition_flagged_env branch from d1954a8 to 901cf2f Compare June 25, 2026 08:26
@miscco miscco force-pushed the device_partition_flagged branch from e2fea4c to a910255 Compare June 25, 2026 11:01
@miscco miscco force-pushed the device_partition_flagged_env branch from 901cf2f to 58ecf4a Compare June 25, 2026 11:01
@github-actions

This comment has been minimized.

@miscco miscco force-pushed the device_partition_flagged branch from a910255 to eedc632 Compare June 25, 2026 14:39
@miscco miscco force-pushed the device_partition_flagged_env branch 2 times, most recently from 885dd2d to f802c13 Compare June 25, 2026 15:24
@miscco miscco force-pushed the device_partition_flagged branch from eedc632 to 2b07d4f Compare June 25, 2026 15:24
@miscco miscco force-pushed the device_partition_flagged_env branch from f802c13 to 7edce6b Compare June 25, 2026 17:52
@miscco miscco force-pushed the device_partition_flagged branch from 2b07d4f to 9417767 Compare June 25, 2026 17:52
@github-actions

This comment has been minimized.

@miscco miscco force-pushed the device_partition_flagged branch from 9417767 to c94525f Compare June 26, 2026 06:36
@miscco miscco force-pushed the device_partition_flagged_env branch 2 times, most recently from 5425b25 to d03bf64 Compare June 26, 2026 08:13
@miscco miscco force-pushed the device_partition_flagged branch from c94525f to 01959c0 Compare June 26, 2026 08:13
We want to be able to pass tunings to the APIs that take user provided memory

Make sure we can pass any environment or stream type to them
@miscco miscco force-pushed the device_partition_flagged branch from 01959c0 to 9e01dd2 Compare June 26, 2026 08:13
@miscco miscco force-pushed the device_partition_flagged_env branch from d03bf64 to 614dd6d Compare June 26, 2026 08:13
@github-actions

Copy link
Copy Markdown
Contributor

😬 CI Workflow Results

🟥 Finished in 2h 49m: Pass: 97%/345 | Total: 14d 11h | Max: 2h 47m | Hits: 38%/1705248

See results here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants