VTA build failure

yzhliu · November 15, 2019, 6:26am

if we turn on USE_VTA_FSIM and USE_VTA_TSIM,

/home/ubuntu/tvm/vta/include/vta/hw_spec.h:57:28: error: ‘VTA_LOG_BLOCK_IN’ was not declared in this scope
 #define VTA_BLOCK_IN (1 << VTA_LOG_BLOCK_IN)

@thierry is it a bug? I search the code base not see any definition, so as others like VTA_LOG_BLOCK_OUT, VTA_LOG_WGT_WIDTH etc.

thierry · November 15, 2019, 6:55am

These are generated by this file: https://github.com/apache/incubator-tvm/blob/master/vta/config/vta_config.py#L133

If you dive through the layers, you’ll see that they’re generated from the config.json file corresponding to your target.

Hope this helps

yzhliu · November 15, 2019, 8:07am

Got it, thanks @thierry for pointing

I guess I know the problem. in following files VTA uses python, if host only has python3 it will fail silently.

github.com

apache/incubator-tvm/blob/master/vta/hardware/xilinx/scripts/hls.tcl

# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements.  See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership.  The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License.  You may obtain a copy of the License at
# 
#   http://www.apache.org/licenses/LICENSE-2.0
# 
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied.  See the License for the
# specific language governing permissions and limitations
# under the License.

# Command line arguments:
# Arg 1: path to vta root
# Arg 2: path of config param script

This file has been truncated. show original

github.com

apache/incubator-tvm/blob/master/vta/hardware/xilinx/scripts/vivado.tcl

# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements.  See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership.  The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License.  You may obtain a copy of the License at
# 
#   http://www.apache.org/licenses/LICENSE-2.0
# 
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied.  See the License for the
# specific language governing permissions and limitations
# under the License.

# Check if script is running in correct Vivado version.
set scripts_vivado_version 2018.3
set current_vivado_version [version -short]

This file has been truncated. show original

github.com

apache/incubator-tvm/blob/master/vta/hardware/xilinx/Makefile

# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements.  See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership.  The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License.  You may obtain a copy of the License at
#
#   http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied.  See the License for the
# specific language governing permissions and limitations
# under the License.

# Directories
ROOTDIR = $(CURDIR)
VTA_DIR = $(CURDIR)/../..

This file has been truncated. show original

I think one solution is to use what VTA.cmake detected.

thierry · November 15, 2019, 5:51pm

Ack, very good catch. I’ll issue a patch then since this can end up being very annoying in the long run.

Terex12 · November 17, 2019, 3:20am

When I built TVM with python 3.5, I got the exact VTA error as @yzhliu had. But, when I built TVM with python 2.7, I was able to build it successfully. It is strange though I do not know the reason.

thierry · November 18, 2019, 2:05am

Thanks @Terex12, I’ll try reproducing this bug with pyenv on my machine.

leeexyz · December 22, 2019, 10:40am

I got the same error when i built TVM with python3.5.2 on Ubuntu16.04.