Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Enable dp4a for the CI #9684

Merged
merged 1 commit into from
Feb 4, 2018
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion tests/ci_build/ci_build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -168,6 +168,10 @@ echo "Running '${COMMAND[@]}' inside ${DOCKER_IMG_NAME}..."
# ${WORKDIR} is the working directory we start the container in. By default this
# is /workspace, but for example when running cmake it is sometimes /workspace/build.

# Currently compiling for sm52 (g3s) and sm61(p3s). sm61 is required to test DP4A support on p3
# instances. In the long term we should enable sm70 for p3 instances, but this will require
# CUDA 9(.1) support with our base AMI drivers.

${DOCKER_BINARY} run --rm --pid=host \
-v ${WORKSPACE}:/workspace \
-w ${WORKDIR} \
Expand All @@ -176,7 +180,7 @@ ${DOCKER_BINARY} run --rm --pid=host \
-e "CI_BUILD_UID=$(id -u)" \
-e "CI_BUILD_GROUP=$(id -g -n)" \
-e "CI_BUILD_GID=$(id -g)" \
-e "CUDA_ARCH=-gencode arch=compute_52,code=[sm_52,compute_52] --fatbin-options -compress-all" \
-e "CUDA_ARCH=-gencode arch=compute_52,code=[sm_52,compute_52] -gencode arch=compute_61,code=sm_61 --fatbin-options -compress-all" \
Copy link
Contributor

Choose a reason for hiding this comment

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

6.1 is Pascal based GPUs and thus would have no effect on our CI. Could you elaborate?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This will produce backwards compatible binaries, including the dp4a instruction, that will work on p3 instances.

-e "MXNET_STORAGE_FALLBACK_LOG_VERBOSE=0" \
${CI_DOCKER_EXTRA_PARAMS[@]} \
${DOCKER_IMG_NAME} \
Expand Down