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

Reenable Guass-Labotto Unit Test on Lassen #2493

Closed
wrtobin opened this issue May 30, 2023 · 4 comments
Closed

Reenable Guass-Labotto Unit Test on Lassen #2493

wrtobin opened this issue May 30, 2023 · 4 comments
Assignees
Labels
type: bug Something isn't working

Comments

@wrtobin
Copy link
Collaborator

wrtobin commented May 30, 2023

Currently on Lassen clang10 cuda11 testQ5_Hexahedron_Lagrange_GaussLobatto will not compile due to kernel parameter size issues:

testQ5_Hexahedron_Lagrange_GaussLobatto.cpp

ptxas /var/tmp/tobin6/tmpxft_000284c9_00000000-6_testQ5_Hexahedron_Lagrange_GaussLobatto.ptx, line 7580; error   : Entry function '_ZN4RAJA6policy4cuda4impl18forall_cuda_kernelILm32ELm1ENS_9Iterators16numeric_iteratorIiiPiEEZ16testKernelDriverINS1_18cuda_exec_explicitILm32ELm1ELb0EEEEvvEUliE1_iEEvT2_T1_T3_' uses too much parameter space (0x14c0 bytes, 0x1100 max).
ptxas fatal   : Ptx assembly aborted due to errors
make[2]: *** [coreComponents/finiteElement/unitTests/CMakeFiles/testQ5_Hexahedron_Lagrange_GaussLobatto.dir/build.make:76: coreComponents/finiteElement/unitTests/CMakeFiles/testQ5_Hexahedron_Lagrange_GaussLobatto.dir/testQ5_Hexahedron_Lagrange_GaussLobatto.cpp.o] Error 255
make[1]: *** [CMakeFiles/Makefile2:7498: coreComponents/finiteElement/unitTests/CMakeFiles/testQ5_Hexahedron_Lagrange_GaussLobatto.dir/all] Error 2

Resolving this is beyond the scope of the unit-test fix PR, so it should be resolved in a follow-up bugfix PR.

@wrtobin wrtobin added type: bug Something isn't working type: new A new issue has been created and requires attention labels May 30, 2023
@wrtobin wrtobin self-assigned this May 30, 2023
@wrtobin
Copy link
Collaborator Author

wrtobin commented May 30, 2023

also testQ3_Hexahedron_Lagrange_GaussLobatto.cpp on the CI.

Both are suppressed for the moment and need to be fixed / reactivated.

@acitrain
Copy link
Contributor

acitrain commented Jun 1, 2023

Hi @wrtobin,
Thank you for pointing out this issue those test are for the high order wave solver mainly i'll fix them today and test it on Lassen and Pangea3

PR should be open today !

@TotoGaz TotoGaz removed the type: new A new issue has been created and requires attention label Jun 22, 2023
@TotoGaz
Copy link
Contributor

TotoGaz commented Jun 22, 2023

hello @acitrain, is this solved with the PR you mentioned in your comment?

@acitrain
Copy link
Contributor

hello @acitrain, is this solved with the PR you mentioned in your comment?

Hi @TotoGaz
Ho yes the PR solve the issue, maybe I should have closed it when the PR was merged sorry...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants