likwid: Add variant for Nvidia GPU profiling feature#20337
likwid: Add variant for Nvidia GPU profiling feature#20337alalazo merged 7 commits intospack:developfrom TomTheBear:develop
Conversation
Would implementing a: @property
def headers(self):
...similarly to what is done e.g. in files = find(spec['cuda'].prefix.extras, 'cupti.h', recursive=True) |
|
Thanks for the hints. A |
Sure, what I meant was either:
As a further hit: do you think this package would benefit from inheriting also from class Likwid(Package, CudaPackage):
...? You can check what $ spack edit -b cuda |
|
IMHO: Since Cuda is only required when building with Nvidia GPU profiling support (default: off), a general inheritance of the Cuda class is not required. Moreover, But Thanks. Every time I add something to LIKWID's What's the problem with the |
I think it's a spurious failure that will be fixed soon. @zackgalbreath or @scottwittenburg may know more than I do. |
|
While we're still ironing out the kinks in the gitlab ci check, an actual breakage of the |
With version 5.0, LIKWID got a profiling interface for Nvidia GPUs (based on CUPTI).
With 5.1, the interface was extended to work with the new CUpti Profiling API.
This PR adds a variant switch to activate that feature.
Single issue left: How to find the subfolder below
spec['cuda'].prefixwhich includescupti.h? It's not inspec['cuda'].prefix.include. Same forlibcupti.sowhich is not inspec['cuda'].prefix.lib. The hardcoded pathsspec['cuda'].prefix + "extras/CUPTI/{include,lib64}"probably work but might break in the future. Is there a spack-way to do it or should the standard library be used for this?