Tpetra: Add "Error 127" CUDA build work-around to FAQ #2352

mhoemmen · 2018-03-08T06:28:12Z

Add work-around for "Error 127" kokkos-kernels CUDA build link error to Tpetra's FAQ.
See Trilinos/packages/tpetra/doc/FAQ.txt.
This pull request ONLY affects Tpetra's documentation.
It does not touch source code files.

@trilinos/tpetra

See the following GitHub issue:

#2141

and the following detailed instructions:

https://tribits.org/doc/TribitsBuildReference.html#enabling-the-usage-of-resource-files-to-reduce-length-of-build-lines

Thanks to @bartlettroscoe for the instructions!

@bartlettroscoe

@trilinos/tpetra See the following GitHub issue: trilinos#2141 and the following detailed instructions: https://tribits.org/doc/TribitsBuildReference.html#enabling-the-usage-of-resource-files-to-reduce-length-of-build-lines Thanks to @bartlettroscoe for the instructions!

trilinos-autotester · 2018-03-08T06:43:29Z

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request.

trilinos-autotester · 2018-03-08T06:45:30Z

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

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: Trilinos_autotester_test

Build Num: 463
Status: STARTED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.8.4
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	2352
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	Tpetra-FAQ-Error-127-workaround
TRILINOS_SOURCE_REPO	https://github.com/mhoemmen/Trilinos
TRILINOS_SOURCE_SHA	`44d886b`
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	`797671c`

Build Information

Test Name: Trilinos_pullrequest_gcc_4.9.3

Build Num: 328
Status: STARTED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.9.3
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	2352
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	Tpetra-FAQ-Error-127-workaround
TRILINOS_SOURCE_REPO	https://github.com/mhoemmen/Trilinos
TRILINOS_SOURCE_SHA	`44d886b`
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	`797671c`

Build Information

Test Name: Trilinos_pullrequest_gcc_4.8.4

Build Num: 43
Status: STARTED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.8.4
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	2352
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	Tpetra-FAQ-Error-127-workaround
TRILINOS_SOURCE_REPO	https://github.com/mhoemmen/Trilinos
TRILINOS_SOURCE_SHA	`44d886b`
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	`797671c`

Using Repos:

Repo: TRILINOS (mhoemmen/Trilinos)

Pull Request Author: mhoemmen

trilinos-autotester · 2018-03-08T09:17:05Z

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

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: Trilinos_autotester_test

Build Num: 463
Status: PASSED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.8.4
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	2352
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	Tpetra-FAQ-Error-127-workaround
TRILINOS_SOURCE_REPO	https://github.com/mhoemmen/Trilinos
TRILINOS_SOURCE_SHA	`44d886b`
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	`797671c`

Build Information

Test Name: Trilinos_pullrequest_gcc_4.9.3

Build Num: 328
Status: PASSED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.9.3
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	2352
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	Tpetra-FAQ-Error-127-workaround
TRILINOS_SOURCE_REPO	https://github.com/mhoemmen/Trilinos
TRILINOS_SOURCE_SHA	`44d886b`
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	`797671c`

Build Information

Test Name: Trilinos_pullrequest_gcc_4.8.4

Build Num: 43
Status: PASSED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.8.4
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	2352
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	Tpetra-FAQ-Error-127-workaround
TRILINOS_SOURCE_REPO	https://github.com/mhoemmen/Trilinos
TRILINOS_SOURCE_SHA	`44d886b`
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	`797671c`

trilinos-autotester · 2018-03-08T09:17:24Z

Status Flag 'Pre-Merge 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!

trilinos-autotester · 2018-03-08T09:17:30Z

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

ibaned

Looks fine to me.

bartlettroscoe

You change the URL to point to:

http://trilinos.org/docs/files/TrilinosBuildReference.html#enabling-the-usage-of-resource-files-to-reduce-length-of-build-lines

to not confuse people.

bartlettroscoe · 2018-03-08T13:10:00Z

packages/tpetra/doc/FAQ.txt

+
+Short answer: Try setting the following CMake option:
+#+BEGIN_EXAMPLE
+CMAKE_CXX_USE_RESPONSE_FILE_FOR_OBJECTS:BOOL=ON


You might want to add the -D prefix as:

-D CMAKE_CXX_USE_RESPONSE_FILE_FOR_OBJECTS:BOOL=ON

Just a suggestion for people who might actually try to just pass in CMAKE_CXX_USE_RESPONSE_FILE_FOR_OBJECTS:BOOL=ON to the cmake comamnd.

done (locally)

bartlettroscoe · 2018-03-08T13:11:38Z

packages/tpetra/doc/FAQ.txt

+
+and the following instructions:
+
+https://tribits.org/doc/TribitsBuildReference.html#enabling-the-usage-of-resource-files-to-reduce-length-of-build-lines


Also, once TriBITS is snapshotted into Trilinos, you should point into the Trilinos Build reference document instead of this generic one.

I was going to add another thing to TriBITS before I snaphsotted it into Trilinos but I will go ahead and do that now.

I just pushed the commit afed21c that should bring this documentation into the document:

http://trilinos.org/docs/files/TrilinosBuildReference.html

the next time the Trilinos Framework script runs that rebuilds this document. (You will have to ask @jwillenbring when that occurs.)

So you should likely change this to point to:

http://trilinos.org/docs/files/TrilinosBuildReference.html#enabling-the-usage-of-resource-files-to-reduce-length-of-build-lines

instead.

@bartlettroscoe Cool, thanks Ross! I'll make those changes.

@bartlettroscoe btw, trilinos.org gives me the nastygram about having a bad certificate; may I also point to the other link as a back-up?

trilinos.org gives me the nastygram about having a bad certificate; may I also point to the other link as a back-up?

Ya, I know. I told the @trilinos/framework team about this a while back. You can check with @jwillenbring about the progress on fixing that.

done (locally)

trilinos-autotester · 2018-03-08T15:44:15Z

Status Flag 'Pre-Merge 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

trilinos-autotester · 2018-03-08T15:44:22Z

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

mhoemmen · 2018-03-08T16:53:37Z

I'll mark this as [WIP] temporarily so the autotester doesn't bother y'all.

@bartlettroscoe

@trilinos/tpetra I implemented @bartlettroscoe's suggested changes.

mhoemmen · 2018-03-09T00:19:43Z

Thanks all!

@bartlettroscoe

Add work-around for "Error 127" kokkos-kernels CUDA build link error to Tpetra's FAQ. See Trilinos/packages/tpetra/doc/FAQ.txt. This commit ONLY affects Tpetra's documentation. It does not touch source code files. @trilinos/tpetra See the following GitHub issue: trilinos#2141 and the following detailed instructions: https://tribits.org/doc/TribitsBuildReference.html#enabling-the-usage-of-resource-files-to-reduce-length-of-build-lines Thanks to @bartlettroscoe for the instructions!

mhoemmen added pkg: Tpetra TpetraRF labels Mar 8, 2018

mhoemmen self-assigned this Mar 8, 2018

mhoemmen added the stage: in progress Work on the issue has started label Mar 8, 2018

mhoemmen requested review from bartlettroscoe, ibaned, crtrott, william76 and tjfulle and removed request for crtrott March 8, 2018 06:28

ibaned approved these changes Mar 8, 2018

View reviewed changes

bartlettroscoe requested changes Mar 8, 2018

View reviewed changes

mhoemmen changed the title ~~Tpetra: Add "Error 127" CUDA build work-around to FAQ~~ [WIP] Tpetra: Add "Error 127" CUDA build work-around to FAQ Mar 8, 2018

mhoemmen added the AT: WIP Causes the PR autotester to not test the PR. (Remove to allow testing to occur.) label Mar 8, 2018

Tpetra: Improved FAQ

3778ff9

@trilinos/tpetra I implemented @bartlettroscoe's suggested changes.

mhoemmen removed the AT: WIP Causes the PR autotester to not test the PR. (Remove to allow testing to occur.) label Mar 9, 2018

mhoemmen changed the title ~~[WIP] Tpetra: Add "Error 127" CUDA build work-around to FAQ~~ Tpetra: Add "Error 127" CUDA build work-around to FAQ Mar 9, 2018

mhoemmen merged commit 1e1255a into trilinos:develop Mar 9, 2018

mhoemmen removed the stage: in progress Work on the issue has started label Mar 9, 2018


		and the following instructions:

		https://tribits.org/doc/TribitsBuildReference.html#enabling-the-usage-of-resource-files-to-reduce-length-of-build-lines

Tpetra: Add "Error 127" CUDA build work-around to FAQ #2352

Tpetra: Add "Error 127" CUDA build work-around to FAQ #2352

Conversation

mhoemmen commented Mar 8, 2018

trilinos-autotester commented Mar 8, 2018

trilinos-autotester commented Mar 8, 2018

Build Information

Test Name: Trilinos_autotester_test

Jenkins Parameters

Build Information

Test Name: Trilinos_pullrequest_gcc_4.9.3

Jenkins Parameters

Build Information

Test Name: Trilinos_pullrequest_gcc_4.8.4

Jenkins Parameters

Using Repos:

trilinos-autotester commented Mar 8, 2018

Build Information

Test Name: Trilinos_autotester_test

Jenkins Parameters

Build Information

Test Name: Trilinos_pullrequest_gcc_4.9.3

Jenkins Parameters

Build Information

Test Name: Trilinos_pullrequest_gcc_4.8.4

Jenkins Parameters

trilinos-autotester commented Mar 8, 2018

trilinos-autotester commented Mar 8, 2018

ibaned left a comment

Choose a reason for hiding this comment

bartlettroscoe left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

trilinos-autotester commented Mar 8, 2018

trilinos-autotester commented Mar 8, 2018

mhoemmen commented Mar 8, 2018

mhoemmen commented Mar 9, 2018