[GitHub] [tvm] areusch commented on a diff in pull request #11644: [CI] fix ci_gpu dockerfile

[GitBox <gi...@apache.org>]

areusch commented on code in PR #11644:
URL: https://github.com/apache/tvm/pull/11644#discussion_r893859797


##########
docker/install/ubuntu_install_papi.sh:
##########
@@ -23,11 +23,15 @@ set -o pipefail
 apt-get update --fix-missing
 
 # deps
-apt-get install -y linux-tools-common linux-tools-generic
+apt-get install -y linux-tools-common linux-tools-generic kmod
 
 cd /
 git clone https://bitbucket.org/icl/papi.git
-cd papi/src
+# Pulling the latest version of this has broken the images before. Checkout the commit below for now.
+cd papi
+git checkout e3c263b6fb578695085d0ab093e4fd1ee5aeac23

Review Comment:
   did we want to pull a specific version rather than a sha1 that looks random to a new user? it would simplify install instructions.
   
   can you also update the docs in docs/install/from_source.rst to include this version constraint?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org