[Bugfix][VTA] PkgConfig cause crash in PYNQ board due to link library

[huajsj]

not exist.

Symptom:
When run vta_get_started.py with pynq board, host crash and
complain "cannot find -lsds_lib" and "cannot find -l:libdma.so"

Reproduce:
At pynq board, delete the ./build/vta_config.json, then run rpc
server.
In host machine run vta_get_started.py, issue would reproduce.

Analysis:
This issue caused by 'PkgConfig' function still using pynq2.1
library which not exist in pynq2.3 anymore, when a "reconfig_runtime"
logic of rpc_server.py get triggered , the compile would failed due to
link library not exist.

Solution:
change the link library to libcma.so.

Thanks for contributing to TVM! Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers.

[huajsj]

@tmoreau89 could you help to review this patch?

Thanks
Hua

[tmoreau89]

Thanks for fixing Pynq support for 2.3 image. If we do release 2.3 support, we'll need to update the documentation. Can you also update the doc in the PR? You can find it under tvm/docs/vta/install.md

[tmoreau89]

Please update the doc to reflect the 2.3 pynq image migration

[huajsj]

@tmoreau89 , the content of vta/install.md already reflect pynq 2.3, and a former PR #1945 did such change, please kindly let me know if you need any other change.

[tmoreau89]

Ah good point, what I meant was that the current version of VTA is at 2.4. I believe that there isn't a difference between these two versions in terms of library paths. Would you mind double checking that it works on 2.4, and update the doc accordingly since this is the image that's available at the moment?

[huajsj]

@tmoreau89 , thanks for the clarify, this patch not target for pynq 2.4 and just 2.3 issue fixed, about 2.4 support, how about we use different PR to address? I can work for this 2.4 support issue then use separate PR for doc and any other necessary fix, please kindly let me know how you think.

[tmoreau89]

@huajsj since 2.4 is now the version that can be downloaded from the pynq.io website I believe it would be best to support that now. I believe that if you flash a new SD card, you should be able to get your branch to build - therefore if you could just verify that it works, and update the documentation, we can keep this PR as is.

[tmoreau89]

in other words, no need for a new PR

[huajsj]

@tmoreau89 , sure, get it, verification finished(build pass, vta_get_start.py success), current vta work good at PYNQ2.4, related documents already get updated from 2.3 into 2.4.

[tmoreau89]

LGTM