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

Workaround using new/delete in kernel code #925

Merged
merged 2 commits into from
Apr 1, 2021

Conversation

masterleinad
Copy link
Contributor

Using new and delete is not supported by all backends. Hence, this pull request allocates global memory instead. Scratch memory would potentially be more in line with the previous implementation but we can't use that for RangePolicy.

@kokkos-devops-admin
Copy link

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
WARNING: NO REVIEWERS HAVE BEEN REQUESTED FOR THIS PULL REQUEST!

@brian-kelley brian-kelley requested a review from lucbv April 1, 2021 20:25
@kokkos-devops-admin
Copy link

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO REVIEWS HAVE BEEN PERFORMED ON THIS PULL REQUEST!

Copy link
Contributor

@lucbv lucbv left a comment

Choose a reason for hiding this comment

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

Could we have the bannedColors_ be a 2D view, I think it would make things just a little more straightforward and we don't need to use the plain int* pointer inside the functor operator anymore if we go with the view semantic under the hood anyway.

src/graph/impl/KokkosGraph_Distance1Color_impl.hpp Outdated Show resolved Hide resolved
src/graph/impl/KokkosGraph_Distance1Color_impl.hpp Outdated Show resolved Hide resolved
src/graph/impl/KokkosGraph_Distance1Color_impl.hpp Outdated Show resolved Hide resolved
src/graph/impl/KokkosGraph_Distance1Color_impl.hpp Outdated Show resolved Hide resolved
@kokkos-devops-admin
Copy link

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
THE LAST COMMIT TO THIS PULL REQUEST HAS BEEN REVIEWED, BUT NOT ACCEPTED OR REQUIRES CHANGES

@masterleinad masterleinad requested a review from lucbv April 1, 2021 21:20
@kokkos-devops-admin
Copy link

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
THE LAST COMMIT TO THIS PULL REQUEST HAS NOT BEEN REVIEWED YET!

Copy link
Contributor

@lucbv lucbv left a comment

Choose a reason for hiding this comment

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

Thanks, that looks great!

@kokkos-devops-admin
Copy link

Status Flag 'Pre-Test Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ lucbv ]!

@kokkos-devops-admin
Copy link

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: KokkosKernels_PullRequest_GCC720

  • Build Num: 130
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_GCC720

  • Build Num: 123
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_INTEL18

  • Build Num: 109
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_GCC720_Light

  • Build Num: 144
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_CUDA10

  • Build Num: 108
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_CUDA9

  • Build Num: 103
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_GCC720_GCC740

  • Build Num: 102
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Using Repos:

Repo: KOKKOSKERNELS (masterleinad/kokkos-kernels)
  • Branch: dont_use_new_in_kernels
  • SHA: c9cd038
  • Mode: TEST_REPO

Pull Request Author: masterleinad

@kokkos-devops-admin
Copy link

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: KokkosKernels_PullRequest_GCC720

  • Build Num: 130
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_GCC720

  • Build Num: 123
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_INTEL18

  • Build Num: 109
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_GCC720_Light

  • Build Num: 144
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_CUDA10

  • Build Num: 108
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_CUDA9

  • Build Num: 103
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

Build Information

Test Name: KokkosKernels_PullRequest_Tpls_GCC720_GCC740

  • Build Num: 102
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
KOKKOSKERNELS_SOURCE_BRANCH dont_use_new_in_kernels
KOKKOSKERNELS_SOURCE_REPO https://github.com/masterleinad/kokkos-kernels
KOKKOSKERNELS_SOURCE_SHA c9cd038
KOKKOSKERNELS_TARGET_BRANCH develop
KOKKOSKERNELS_TARGET_REPO https://github.com/kokkos/kokkos-kernels
KOKKOSKERNELS_TARGET_SHA 873a19f
PULLREQUESTNUM 925
TEST_REPO_ALIAS KOKKOSKERNELS

@kokkos-devops-admin
Copy link

Status Flag 'Pre-Merge Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ lucbv ]!

@kokkos-devops-admin
Copy link

Status Flag 'Pull Request AutoTester' - Pull Request MUST BE MERGED MANUALLY BY Project Team - This Repo does not support Automerge

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

Successfully merging this pull request may close these issues.

3 participants