[Openmp-commits] [PATCH] D82718: [OpenMP] Use primary context in CUDA plugin

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld added a subscriber: openmp-commits.
Hahnfeld added a comment.

(In general, all patches must be sent to the respective -commits list. This also makes feedback more likely.)

> Retaining per device primary context is preferred to creating a context owned by the plugin.
>  CUDA driver API documentation recommends this.

Do you have a link for this? From a users / admin perspective, my only concern is that libomptarget should only "block" devices that are actually used. This is important for interactive machines that are configured in exclusive mode. It looks like `cuDevicePrimaryCtxRetain` does this, but maybe you can test that it's indeed working this way?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82718/new/

https://reviews.llvm.org/D82718