gromero commented on pull request #9068:
URL: https://github.com/apache/tvm/pull/9068#issuecomment-929191794


   > @gromero I just ran this PR on RVM with log-level=DEBUG and I didn't see 
`IsContiguous` warning. Maybe you're using a different zephyr branch?
   
   hmr I got that warning on my local environment, however I'm also able to 
reproduce it in the RVM, look:
   
   ```
   (microtvm-Y3ym_Klm-py3.6) vagrant@microtvm:/home/gromero/git/tvm$ 
./tests/scripts/task_python_microtvm.sh
   + source tests/scripts/setup-pytest-env.sh
   ++ set +u
   ++ [[ ! -z '' ]]
   ++ export 'PYTEST_ADDOPTS=-s -v '
   ++ PYTEST_ADDOPTS='-s -v '
   ++ set -u
   +++ pwd
   ++ export TVM_PATH=/home/gromero/git/tvm
   ++ TVM_PATH=/home/gromero/git/tvm
   ++ export PYTHONPATH=/home/gromero/git/tvm/python
   ++ PYTHONPATH=/home/gromero/git/tvm/python
   ++ export TVM_PYTEST_RESULT_DIR=/home/gromero/git/tvm/build/pytest-results
   ++ TVM_PYTEST_RESULT_DIR=/home/gromero/git/tvm/build/pytest-results
   ++ mkdir -p /home/gromero/git/tvm/build/pytest-results
   + make cython3
   cd python; python3 setup.py build_ext --inplace
   WARNING: Cython is not installed, will compile without cython module
   running build_ext
   + run_pytest ctypes python-microtvm-zephyr tests/micro/zephyr 
--zephyr-board=mimxrt1050_evk
   + local ffi_type=ctypes
   + shift
   + local test_suite_name=python-microtvm-zephyr
   + shift
   + '[' -z ctypes -o -z python-microtvm-zephyr ']'
   + TVM_FFI=ctypes
   + python3 -m pytest -o junit_suite_name=python-microtvm-zephyr-ctypes 
--junit-xml=/home/gromero/git/tvm/build/pytest-results/python-microtvm-zephyr-ctypes.xml
 --junit-prefix=ctypes tests/micro/zephyr --zephyr-board=mimxrt1050_evk
   enabled targets: llvm; llvm -device=arm_cpu
   pytest marker: 
   ============================================================================ 
test session starts 
============================================================================
   platform linux -- Python 3.6.9, pytest-5.4.3, py-1.10.0, pluggy-0.13.1 -- 
/home/vagrant/.cache/pypoetry/virtualenvs/microtvm-Y3ym_Klm-py3.6/bin/python3
   cachedir: .pytest_cache
   rootdir: /home/gromero/git/tvm
   collected 12 items                                                           
                                                                                
               
   
   tests/micro/zephyr/test_zephyr.py::test_add_uint[mimxrt1050_evk] 
INFO:__main__:run (in cwd: 
/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/build):
 cmake .. -DWEST=west -DBOARD:STRING=mimxrt1050_evk
   Including boilerplate (Zephyr base): 
/home/vagrant/zephyr/zephyr/cmake/app/boilerplate.cmake
   -- Application: 
/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project
   -- Zephyr version: 2.5.0 (/home/vagrant/zephyr/zephyr)
   -- Found Python3: 
/home/vagrant/.cache/pypoetry/virtualenvs/microtvm-Y3ym_Klm-py3.6/bin/python3.6 
(found suitable exact version "3.6.9") found components: Interpreter 
   -- Found west (found suitable version "0.11.1", minimum required is "0.7.1")
   -- Board: mimxrt1050_evk
   -- Cache files will be written to: /home/vagrant/.cache/zephyr
   -- ZEPHYR_TOOLCHAIN_VARIANT not set, trying to locate Zephyr SDK
   -- Using toolchain: zephyr 0.12.3 (/home/vagrant/zephyr-sdk)
   -- Found dtc: 
/home/vagrant/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/dtc (found 
suitable version "1.5.0", minimum required is "1.4.6")
   -- Found BOARD.dts: 
/home/vagrant/zephyr/zephyr/boards/arm/mimxrt1050_evk/mimxrt1050_evk.dts
   -- Generated zephyr.dts: 
/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/build/zephyr/zephyr.dts
   -- Generated devicetree_unfixed.h: 
/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/build/zephyr/include/generated/devicetree_unfixed.h
   -- Generated device_extern.h: 
/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/build/zephyr/include/generated/device_extern.h
   Parsing /home/vagrant/zephyr/zephyr/Kconfig
   Loaded configuration 
'/home/vagrant/zephyr/zephyr/boards/arm/mimxrt1050_evk/mimxrt1050_evk_defconfig'
   Merged configuration 
'/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/prj.conf'
   Configuration saved to 
'/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/build/zephyr/.config'
   Kconfig header saved to 
'/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/build/zephyr/include/generated/autoconf.h'
   -- The C compiler identification is GNU 10.2.0
   -- The CXX compiler identification is GNU 10.2.0
   -- The ASM compiler identification is GNU
   -- Found assembler: 
/home/vagrant/zephyr-sdk/arm-zephyr-eabi/bin/arm-zephyr-eabi-gcc
   CMake Warning at /home/vagrant/zephyr/zephyr/subsys/random/CMakeLists.txt:11 
(message):
     
   
         Warning: CONFIG_TEST_RANDOM_GENERATOR is not a truly random generator.
         This capability is not secure and it is provided for testing purposes 
only.
         Use it carefully.
   
   
   -- Configuring done
   -- Generating done
   -- Build files have been written to: 
/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/build
   INFO:__main__:run (in cwd: 
/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/build):
 make -j2
   make: Warning: File 'Makefile' has modification time 49 s in the future
   make[1]: Warning: File 'CMakeFiles/Makefile2' has modification time 51 s in 
the future
   make[2]: Warning: File 
'zephyr/CMakeFiles/parse_syscalls_target.dir/progress.make' has modification 
time 51 s in the future
   Scanning dependencies of target parse_syscalls_target
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   make[2]: Warning: File 
'zephyr/CMakeFiles/parse_syscalls_target.dir/progress.make' has modification 
time 51 s in the future
   [  1%] Generating misc/generated/syscalls.json, 
misc/generated/struct_tags.json
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   [  2%] Built target parse_syscalls_target
   make[2]: Warning: File 
'zephyr/CMakeFiles/kobj_types_h_target.dir/progress.make' has modification time 
51 s in the future
   make[2]: Warning: File 
'zephyr/CMakeFiles/syscall_list_h_target.dir/progress.make' has modification 
time 51 s in the future
   Scanning dependencies of target syscall_list_h_target
   Scanning dependencies of target kobj_types_h_target
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   make[2]: Warning: File 
'zephyr/CMakeFiles/syscall_list_h_target.dir/progress.make' has modification 
time 51 s in the future
   make[2]: Warning: File 
'zephyr/CMakeFiles/kobj_types_h_target.dir/progress.make' has modification time 
50 s in the future
   [  3%] Generating include/generated/syscall_dispatch.c, 
include/generated/syscall_list.h
   [  4%] Generating include/generated/kobj-types-enum.h, 
include/generated/otype-to-str.h, include/generated/otype-to-size.h
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   [  4%] Built target kobj_types_h_target
   make[2]: Warning: File 
'zephyr/CMakeFiles/driver_validation_h_target.dir/progress.make' has 
modification time 50 s in the future
   Scanning dependencies of target driver_validation_h_target
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   make[2]: Warning: File 
'zephyr/CMakeFiles/driver_validation_h_target.dir/progress.make' has 
modification time 50 s in the future
   [  5%] Generating include/generated/driver-validation.h
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   [  5%] Built target driver_validation_h_target
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   [  5%] Built target syscall_list_h_target
   make[2]: Warning: File 'CMakeFiles/tvm_model.dir/flags.make' has 
modification time 48 s in the future
   make[2]: Warning: File 'CMakeFiles/common.dir/flags.make' has modification 
time 48 s in the future
   Scanning dependencies of target common
   Scanning dependencies of target tvm_model
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   make[2]: Warning: File 'CMakeFiles/tvm_model.dir/flags.make' has 
modification time 48 s in the future
   [  5%] Building C object 
CMakeFiles/tvm_model.dir/model/codegen/host/src/lib0.c.obj
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   make[2]: Warning: File 'CMakeFiles/common.dir/flags.make' has modification 
time 48 s in the future
   [  6%] Building C object 
CMakeFiles/common.dir/crt/src/runtime/crt/common/crt_backend_api.c.obj
   [  7%] Building C object 
CMakeFiles/tvm_model.dir/model/codegen/host/src/lib1.c.obj
   [  7%] Building C object 
CMakeFiles/common.dir/crt/src/runtime/crt/common/crt_runtime_api.c.obj
   [  8%] Linking C static library libtvm_model.a
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   [  8%] Built target tvm_model
   make[2]: Warning: File 'CMakeFiles/microtvm_rpc_common.dir/flags.make' has 
modification time 48 s in the future
   Scanning dependencies of target microtvm_rpc_common
   make[2]: warning:  Clock skew detected.  Your build may be incomplete.
   make[2]: Warning: File 'CMakeFiles/microtvm_rpc_common.dir/flags.make' has 
modification time 48 s in the future
   [  8%] Building C object 
CMakeFiles/microtvm_rpc_common.dir/crt/src/runtime/crt/microtvm_rpc_common/crcccitt.c.obj
   [  9%] Building CXX object 
CMakeFiles/microtvm_rpc_common.dir/crt/src/runtime/crt/microtvm_rpc_common/frame_buffer.cc.obj
   
/home/gromero/git/tvm/tests/micro/zephyr/workspace_conftest_mimxrt1050_evk/2021-09-27T21-54-04/project/crt/src/runtime/crt/common/crt_runtime_api.c:108:13:
 warning: 'IsContiguous' defined but not used [-Wunused-function]
     108 | static bool IsContiguous(const DLTensor* arr) {
         |             ^~~~~~~~~~~~
   [...]
   ```
   
   I'm of course passing `--zephyr-board=mimxrt1050_evk`  to test the RT1050 
board here.
   
   > as far as the test-config.json I recommend that we reuse board.json file 
and add extra information such as vid/pid to board.json file to have a single 
source. We could do that in a separate PR.
   
   All right. I was just concerned about not having a way to test the board 
with the RVM - I had to take care of it myself to test that PR in the RVM.


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


Reply via email to