-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[CI Image] support CSI-NN2 in ci_qemu #11688
Comments
hi @areusch, the nightly rebuild has passed. So next we need to push the image to tlcpack, right? |
The docker build passed, but it looks like the validate failed, something between commits cf15375 and ca88c52 is causing this error on the QEMU build (example):
and for some reason the job doesn't exit but sits there until it hits the 3 hr timeout. #11689 is the only change to the |
thanks @driazati, It seems to be the problem of environment variables. let me solve it. |
hi @driazati, It is the conflict between Xuantie QEMU and Zephyr QEMU. I have solved it by modifying the script in CSI-NN2 (merged), and it will not involve the content in TVM code. It seems that we need to wait for nightly rebuild before continuing. |
hi @driazati , I use image |
@alter-xp we run these nightly, here's the latest failure. resolving the ARM toolchain seems maybe-related to this...what if instead of trying to fix the two together, we just create ci_riscv now? if you create Dockerfile.ci_riscv, i can help with getting that one built and added to various scripts. |
thanks @areusch, i will create Dockerfile.ci_riscv. and should we need to create a RFC for this? |
i think we're ok w/o RFC for this one as it's just moving around previously-RFC'd functionality. thanks! |
hi @areusch, I created a Dockerfile.ci_riscv (pr), which only retains the basic environment and riscv related content in Dockerfile.ci_qemu. Do we need to use CI to build ci_riscv image next? I'm not familiar with this. So can you help me list the next process and what I need to do? thanks! |
@alter-xp sorry--i've been a bit busy lately. looks like @driazati approved/merged that one, and i created tlc-pack/tlcpack#131 to add that build to our nightly docker image rebuild/test. looks like that pipeline is green now so we should be able to get that image built today or tomorrow and then we can begin using it. i'll update this thread when it's ready. |
S0. Reason: Introduce CSI-NN2 Compute Library into TVM to accelerate the inference performance of RISC-V CPU with Vector Extension
S1. Tag of nightly build: TAG: 20220713-060117-ca88c522f
S2. The nightly is built on TVM commit: [CI Image] support CSI-NN2 in ci_qemu #11689 [ci][docker] fix the path of custom toolchain in ci_qemu for csinn2 #11905 [ci][docker] create Dockerfile.ci_riscv #12230
the ci_riscv image does include the CSI-NN install script.
The text was updated successfully, but these errors were encountered: