You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/07/29 17:52:41 UTC

[GitHub] [incubator-tvm] mbaret opened a new pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

mbaret opened a new pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171


   This introduces the necessary changes to docker to support building the Ethos-N driver stack. This is required for subsequent patches which introduce the Ethos-N integration into TVM (see https://discuss.tvm.ai/t/rfc-ethosn-arm-ethos-n-integration/6680 for more information).
   
   Co-authored-by: Leo Blonk <Le...@arm.com>
   Co-authored-by: Leandro Nunes <le...@arm.com>


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] tqchen merged pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171


   


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] mbaret commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
mbaret commented on pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-673030439


   @tqchen when do you expect the docker image to be updated?


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] tqchen commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-668335709


   Thanks @mbaret @zhiics ! This PR is merged. I will send another note when the docker binary get updated.


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] tqchen commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-674342900


   https://github.com/apache/incubator-tvm/pull/6283


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] tqchen commented on a change in pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
tqchen commented on a change in pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171#discussion_r462400065



##########
File path: docker/Dockerfile.ci_gpu
##########
@@ -52,6 +52,10 @@ RUN bash /install/ubuntu_install_nodejs.sh
 COPY install/ubuntu_install_rocm.sh /install/ubuntu_install_rocm.sh
 RUN bash /install/ubuntu_install_rocm.sh
 
+# Github Arm(R) Ethos(TM)-N NPU driver
+COPY install/ubuntu_install_ethosn_driver_stack.sh /install/ubuntu_install_ethosn_driver_stack.sh

Review comment:
       Is it necessary to introduce it to GPU docker? or is CPU suffice




----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] mbaret commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
mbaret commented on pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-666399486


   This has failed to pass CI 4 times now with different errors that don't appear related to this patch. Is there a CI issue at the moment?


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] mbaret commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
mbaret commented on pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-665741725






----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] tqchen commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-665787759


   @mbaret can you also elaborate a bit what is the effect of running the driver on a x86 machine? will it hook up to a simulated backend? or do we need an ARM device to execute the CI


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tvm] mbaret commented on pull request #6171: [CI][ETHOSN] Enable CI for Ethos-N

Posted by GitBox <gi...@apache.org>.
mbaret commented on pull request #6171:
URL: https://github.com/apache/incubator-tvm/pull/6171#issuecomment-668088055


   ping @tqchen, is there anything else that needs to be done here?


----------------------------------------------------------------
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.

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