-
Notifications
You must be signed in to change notification settings - Fork 99
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
BLAS: fix assignable check in gemv and gemm #1914
Conversation
This is actually not a required pre-requiste we only need to check that the views are accessible.
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
Status Flag 'Pull Request AutoTester' - Failure: Timed out waiting for job KokkosKernels_PullRequest_GNU1021 to start: Total Wait = 3603
|
These changes look fine. Was the issue the fact that it's Passing two memory spaces definitely seems weird and there are few other places that do it, like scal and ger |
@brian-kelley the |
Status Flag 'Pull Request AutoTester' - User Requested Retest - Label AT: RETEST will be reset after testing. |
Status Flag 'Pull Request AutoTester' - Failure: Timed out waiting for job KokkosKernels_PullRequest_GNU1021 to start: Total Wait = 3603
|
@lucbv Ohh, thanks for explaining that. I was thinking 'assignable' meant that the memory was writable (probably because of the way it was used in these KK files). It looks like the only other places |
Status Flag 'Pull Request AutoTester' - User Requested Retest - Label AT: RETEST will be reset after testing. |
Status Flag 'Pull Request AutoTester' - Failure: Timed out waiting for job KokkosKernels_PullRequest_GNU1021 to start: Total Wait = 3603
|
Status Flag 'Pull Request AutoTester' - User Requested Retest - Label AT: RETEST will be reset after testing. |
Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: KokkosKernels_PullRequest_CUDA11_CUDA11_LayoutRight
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_GCC930_Light_Tpls_GCC930_Tpls_CLANG13CUDA10
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_GNU1021
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_GNU1021_Light_LayoutRight
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_Tpls_GNU1021
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_Tpls_INTEL19_solo
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_CLANG1001_solo
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_A64FX_Tpls_ARMPL2110
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_A64FX_GCC1020
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_VEGA908_ROCM520
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_VEGA908_Tpls_ROCM520
Jenkins Parameters
Using Repos:
Pull Request Author: lucbv |
Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand)Build InformationTest Name: KokkosKernels_PullRequest_CUDA11_CUDA11_LayoutRight
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_GCC930_Light_Tpls_GCC930_Tpls_CLANG13CUDA10
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_GNU1021
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_GNU1021_Light_LayoutRight
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_Tpls_GNU1021
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_Tpls_INTEL19_solo
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_CLANG1001_solo
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_A64FX_Tpls_ARMPL2110
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_A64FX_GCC1020
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_VEGA908_ROCM520
Jenkins Parameters
Build InformationTest Name: KokkosKernels_PullRequest_VEGA908_Tpls_ROCM520
Jenkins Parameters
|
Status Flag 'Pre-Merge Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ e10harvey brian-kelley ]! |
Status Flag 'Pull Request AutoTester' - Pull Request MUST BE MERGED MANUALLY BY Project Team - This Repo does not support Automerge |
@brian-kelley thanks I will issue an other PR for the remaining kernels. |
…s:develop' (1dd3da3). * trilinos-develop: Fix logic error when parsing --kokkos-develop flag Correct spec inheritance for CXX20 build Panzer q2 mesh support (trilinos#11364) Skip PR merge when using --kokkos-develop flag. Add flag to set Kokkos and KokkosKernels to develop Tpetra_Kokkos_Compat_KokkosSYCLWrapperNode->Tpetra_KokkosCompat_KokkosSYCLWrapperNode typo Cherry-pick kokkos/kokkos-kernels#1914 Update GDSW_Proxy_def.hpp Tpetra: Oops Tpetra: Adding docs Tpetra: Adding unit labels Tpetra: Adding GDSW memory diagnostic XML support Kokkos: Disable default oneDPL support Tpetra: Adding GDSWStyle perf tests Tpetra: GDSW Fixes Tpetra: GDSW test mods Tpetra: Modifying test Tpetra: Modifying test Tpetra: More mods to the test Tpetra: Initial commit for GDSW Proxy
…s:develop' (1dd3da3). * trilinos-develop: (22 commits) Fix logic error when parsing --kokkos-develop flag Correct spec inheritance for CXX20 build Tpetra: Use Kokkos::Experimental::SYCLDeviceUSMSpace for KokkosSYCLWrapperNode Panzer q2 mesh support (trilinos#11364) Skip PR merge when using --kokkos-develop flag. Add flag to set Kokkos and KokkosKernels to develop Tpetra_Kokkos_Compat_KokkosSYCLWrapperNode->Tpetra_KokkosCompat_KokkosSYCLWrapperNode typo Cherry-pick kokkos/kokkos-kernels#1914 Update GDSW_Proxy_def.hpp Tpetra: Oops Tpetra: Adding docs Tpetra: Adding unit labels Tpetra: Adding GDSW memory diagnostic XML support Kokkos: Disable default oneDPL support Tpetra: Adding GDSWStyle perf tests Tpetra: GDSW Fixes Tpetra: GDSW test mods Tpetra: Modifying test Tpetra: Modifying test Tpetra: More mods to the test ...
…s:develop' (1dd3da3). * trilinos-develop: (22 commits) Fix logic error when parsing --kokkos-develop flag Correct spec inheritance for CXX20 build Tpetra: Use Kokkos::Experimental::SYCLDeviceUSMSpace for KokkosSYCLWrapperNode Panzer q2 mesh support (trilinos#11364) Skip PR merge when using --kokkos-develop flag. Add flag to set Kokkos and KokkosKernels to develop Tpetra_Kokkos_Compat_KokkosSYCLWrapperNode->Tpetra_KokkosCompat_KokkosSYCLWrapperNode typo Cherry-pick kokkos/kokkos-kernels#1914 Update GDSW_Proxy_def.hpp Tpetra: Oops Tpetra: Adding docs Tpetra: Adding unit labels Tpetra: Adding GDSW memory diagnostic XML support Kokkos: Disable default oneDPL support Tpetra: Adding GDSWStyle perf tests Tpetra: GDSW Fixes Tpetra: GDSW test mods Tpetra: Modifying test Tpetra: Modifying test Tpetra: More mods to the test ...
…s:develop' (1dd3da3). * trilinos-develop: (22 commits) Fix logic error when parsing --kokkos-develop flag Correct spec inheritance for CXX20 build Tpetra: Use Kokkos::Experimental::SYCLDeviceUSMSpace for KokkosSYCLWrapperNode Panzer q2 mesh support (trilinos#11364) Skip PR merge when using --kokkos-develop flag. Add flag to set Kokkos and KokkosKernels to develop Tpetra_Kokkos_Compat_KokkosSYCLWrapperNode->Tpetra_KokkosCompat_KokkosSYCLWrapperNode typo Cherry-pick kokkos/kokkos-kernels#1914 Update GDSW_Proxy_def.hpp Tpetra: Oops Tpetra: Adding docs Tpetra: Adding unit labels Tpetra: Adding GDSW memory diagnostic XML support Kokkos: Disable default oneDPL support Tpetra: Adding GDSWStyle perf tests Tpetra: GDSW Fixes Tpetra: GDSW test mods Tpetra: Modifying test Tpetra: Modifying test Tpetra: More mods to the test ...
…s:develop' (1dd3da3). * trilinos-develop: (22 commits) Fix logic error when parsing --kokkos-develop flag Correct spec inheritance for CXX20 build Tpetra: Use Kokkos::Experimental::SYCLDeviceUSMSpace for KokkosSYCLWrapperNode Panzer q2 mesh support (trilinos#11364) Skip PR merge when using --kokkos-develop flag. Add flag to set Kokkos and KokkosKernels to develop Tpetra_Kokkos_Compat_KokkosSYCLWrapperNode->Tpetra_KokkosCompat_KokkosSYCLWrapperNode typo Cherry-pick kokkos/kokkos-kernels#1914 Update GDSW_Proxy_def.hpp Tpetra: Oops Tpetra: Adding docs Tpetra: Adding unit labels Tpetra: Adding GDSW memory diagnostic XML support Kokkos: Disable default oneDPL support Tpetra: Adding GDSWStyle perf tests Tpetra: GDSW Fixes Tpetra: GDSW test mods Tpetra: Modifying test Tpetra: Modifying test Tpetra: More mods to the test ...
This is actually not a required pre-requisite we only need to check that the views are accessible.
@vbrunini