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

CI test with CUDA 11.0 + Perfetto connector fix (bringing PR 212 to completion) #262

Closed
wants to merge 19 commits into from

Conversation

vlkale
Copy link
Contributor

@vlkale vlkale commented Jul 11, 2024

This PR is a continuation of PR #212, based on review and fixes of that PR and trying to address an associated problem /bug with Perfetto connector that is invoked when using CUDA-11.0, as discussed with @crtrott.

Note that the problem is from perfetto/perfetto.cc and not the Perfetto Kokkos Tools connector.

This PR also may address a Kokkos Tools Issue #259 from a Kokkos Tools user, but more discussion with the user is needed to see if it does fix it for their setup/environment/compilers that they are using for Perfetto.

romintomasetti and others added 14 commits October 6, 2023 15:16
/__w/kokkos-tools/kokkos-tools/profiling/perfetto-connector/perfetto/perfetto.h:7153:35: error: 'BUILTIN_CLOCK_UNKNOWN' was not declared in this scope
 7153 |   BuiltinClock primary_trace_clock_{};
…2 on cicd: test Cuda version supported by Kokkos
/__w/kokkos-tools/kokkos-tools/profiling/perfetto-connector/perfetto/perfetto.h:7153:35: error: 'BUILTIN_CLOCK_UNKNOWN' was not declared in this scope
 7153 |   BuiltinClock primary_trace_clock_{};
@vlkale vlkale added bug Build Building Kokkos tools ci Contiguous Integration for Kokkos Tools (includes external/third-party libraries) labels Jul 11, 2024
@vlkale vlkale requested a review from crtrott July 11, 2024 19:34
@vlkale vlkale changed the title CI test with CUDA 11.0 + Perfetto connector fix (bringing PR 212) CI test with CUDA 11.0 + Perfetto connector fix (bringing PR 212 to completion) Jul 11, 2024
@vlkale vlkale changed the title CI test with CUDA 11.0 + Perfetto connector fix (bringing PR 212 to completion) CI test with CUDA 11.0 + Perfetto connector fix (based on PR 212) Jul 11, 2024
@vlkale vlkale changed the title CI test with CUDA 11.0 + Perfetto connector fix (based on PR 212) CI test with CUDA 11.0 + Perfetto connector fix (bringing PR 212 to completion) Jul 11, 2024
@vlkale vlkale removed the bug label Jul 11, 2024
@crtrott
Copy link
Member

crtrott commented Sep 5, 2024

Closed because part was merged rest with #271

@crtrott crtrott closed this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Building Kokkos tools ci Contiguous Integration for Kokkos Tools (includes external/third-party libraries)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants