areusch commented on PR #11809:
URL: https://github.com/apache/tvm/pull/11809#issuecomment-1167930956
hmm yeah so here's what i know about the history here:
1. originally when we added QEMU to the TVM CI, `USE_MICRO` was an optional
CMake setting. having `ci_qemu` separate from `ci_cpu` provided some diversity
in testing the various config settings. MLF accidentally made `USE_MICRO`
required; however, at this time, we had already completed a refactor that
substantially reduced the code footprint of `USE_MICRO` so i haven't exactly
prioritized fixing it.
- the folks who might care about `USE_MICRO=OFF` still are those who use
`libtvm_runtime.so` without using Python. i'm absolutely happy to prioritize a
fix (or review one) if the runtime impact turns out to be significant for anyone
2. QEMU was pretty large and we do want, operationally, to avoid requiring
any one user to download a huge docker image just to repro the CI. `ci_gpu` is
25GB right now, but I don't want that to become the norm. keeping Zephyr
installed separately was a way to avoid that.
3. i don't think there is a reason why the ci_cpu deps are incompatible with
anything in ci_qemu, but obviously this could be a reason to separate them if
so.
`ci_cpu` is 7GB now, so it's not small but it has a ways to go before it's
the long pole. i could go either way here, though if folks want to continue
adding micro toolchains those could become somewhat sizeable.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
