-
Notifications
You must be signed in to change notification settings - Fork 572
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
Trilinos Master Merge PR Generator: Auto PR created to promote from master_merge_20200825_000614 branch to master #7899
Conversation
putScalar will by default write to device. replaceGlobalValue will always write to host. So we can sync_host like this after putScalar to make sure the device write is synched for UVM to host before replaceGlobalValue. Then since replaceGlobalValue does not call modify_host, I think we need to call this afterwards. That won't be necessary for UVM, but later if device is CudaSpace, then skipping that call means the export will be confused about where the data is and try to pull it from device. An alternative is to call modify_host before the putScalar calls. Then the putScalar will write to host and we can skip the modify_host calls after the replaceGlobalValue. This might be confusing as I'm not sure calling modify_host to force putScalar to host is a normal pattern.
These should be resolved with the changes in 7856
These loops will eventually need to be fixed. Either host mirrors or I expect kernels to do the remappings.
This solution would work for pure CudaSpace and Host when UVM is gone but not sure if we want to have MDField have some kind of built in support for host access.
When using more than two ranks, as the mesh is elongated in the y axis only, middle ranks do not have boundary conditions and yield a different number of aggregates compare to rank 0 and n-1.
Automatically Merged using Trilinos Pull Request AutoTester PR Title: MueLu: quick fix for NotayAggregates unit-test PR Author: lucbv
Panzer: Fix PanzerDofMgr_tFilteredUGI_MPI_2 test for CUDA_LAUNCH_BLOCKING=0
Panzer: Fix MDField reads for host
Panzer: Fix basis_times_vector.cpp for CUDA_LAUNCH_BLOCKING=0
Ifpack hypre
Panzer: Add fences for BasisValues2
Automatically Merged using Trilinos Pull Request AutoTester PR Title: Re-enable the three tempus tests listed in #7861 PR Author: prwolfe
…aluator Panzer: Fix point_basis_values_evaluator for CUDA_LAUNCH_BLOCKING=0
Panzer: Fix PanzerDiscFE_LinearObjFactory_Tests_MPI_2 for CUDA_LAUNCH…
Automatically Merged using Trilinos Pull Request AutoTester PR Title: STK snapshot 08-24-20 PR Author: tasmith4
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request. |
Status Flag 'Master Merge AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: Trilinos_pullrequest_gcc_4.8.4
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_intel_17.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_4.9.3_SERIAL
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_8.3.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_7.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_9.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_10.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_cuda_9.2
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0_debug
Jenkins Parameters
Using Repos:
Pull Request Author: trilinos-autotester |
Status Flag 'Master Merge AutoTester' - Jenkins Testing: 1 or more Jobs FAILED Note: Testing will normally be attempted again in approx. 15 Mins. If a change to the PR source branch occurs, the testing will be attempted again on next available autotester run. Pull Request Auto Testing has FAILED (click to expand)Build InformationTest Name: Trilinos_pullrequest_gcc_4.8.4
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_intel_17.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_4.9.3_SERIAL
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_8.3.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_7.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_9.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_10.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_cuda_9.2
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0_debug
Jenkins Parameters
Console Output (last 100 lines) : Trilinos_pullrequest_gcc_4.8.4 # 7656 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_intel_17.0.1 # 7466 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_gcc_4.9.3_SERIAL # 5891 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_gcc_7.2.0 # 5650 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_gcc_8.3.0 # 1927 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_clang_7.0.1 # 328 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_clang_9.0.0 # 1284 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_clang_10.0.0 # 372 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_cuda_9.2 # 5223 (click to expand)
Console Output (last 100 lines) : Trilinos_pullrequest_gcc_7.2.0_debug # 111 (click to expand)
|
Status Flag 'Master Merge AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: Trilinos_pullrequest_gcc_4.8.4
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_intel_17.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_4.9.3_SERIAL
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_8.3.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_7.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_9.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_10.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_cuda_9.2
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0_debug
Jenkins Parameters
Using Repos:
Pull Request Author: trilinos-autotester |
Status Flag 'Master Merge AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand)Build InformationTest Name: Trilinos_pullrequest_gcc_4.8.4
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_intel_17.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_4.9.3_SERIAL
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_8.3.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_7.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_9.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_clang_10.0.0
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_cuda_9.2
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_7.2.0_debug
Jenkins Parameters
|
Status Flag 'Pre-Merge Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request. |
Status Flag 'Master Merge AutoTester' - Pull Request will be Automerged |
Merge on Pull Request# 7899: IS A SUCCESS - Pull Request successfully merged |
Auto PR created to promote from master_merge_20200825_000614 branch to master