Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce an event pair class? #402

Open
eyalroz opened this issue Sep 9, 2022 · 0 comments
Open

Introduce an event pair class? #402

eyalroz opened this issue Sep 9, 2022 · 0 comments
Labels

Comments

@eyalroz
Copy link
Owner

eyalroz commented Sep 9, 2022

CUDA has almost now APIs which regard pairs of events. There's certainly no CUDA driver or runtime struct fort representing event pairs. Consequently, I have no event pair wrapper.

... and yet... event pairs do show up. Timing using events is only defined on pairs of events. Which also motivated #399. In other words, an event pair type has already snuck in to our codebase. So, perhaps we should flesh it out into a full-fledged class?

  • Let the user create an event pair rather than creating two events separately
  • Offer some kind of wrapped enqueue mechanism for streams, where the two events are used to denote the start and end of the enqueued work
  • ... what else?

I have a single decent usage scenario, a second scenario of questionable generality, and... what else? Is this enough for a full-fledged class?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant