You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/06/21 22:44:37 UTC
[GitHub] [tvm] mehrdadh opened a new issue, #11810: [CMSIS-NN][ARM] ARM_MATH_ARGUMENT_ERROR Error due to not setting context buffer.
mehrdadh opened a new issue, #11810:
URL: https://github.com/apache/tvm/issues/11810
Using TVM main to build keyword spotting model with CMSIS NN (same version used on ci_qemu) would lead to error ARM_MATH_ARGUMENT_ERROR. Here are some details:
calling `arm_avgpool_s8` function would result in calling [this line](https://github.com/ARM-software/CMSIS_5/blob/develop/CMSIS/NN/Source/PoolingFunctions/arm_avgpool_s8.c#L273) and here since the context buffer is null and `arm_avgpool_s8_get_buffer_size` returns non-zero, it results in ARM_MATH_ARGUMENT_ERROR error.
cc @Mousius @ashutosh-arm @areusch
--
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: commits-unsubscribe@tvm.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] mehrdadh commented on issue #11810: [CMSIS-NN][ARM] ARM_MATH_ARGUMENT_ERROR due to not setting the context buffer in `arm_avgpool_s8`
Posted by GitBox <gi...@apache.org>.
mehrdadh commented on issue #11810:
URL: https://github.com/apache/tvm/issues/11810#issuecomment-1230975186
@PhilippvK thanks for reporting this! I checked today and the issue seems to be resolved!
I add some missing CMSIS NN source files to cmake: https://github.com/apache/tvm/pull/12642
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] PhilippvK commented on issue #11810: [CMSIS-NN][ARM] ARM_MATH_ARGUMENT_ERROR due to not setting the context buffer in `arm_avgpool_s8`
Posted by GitBox <gi...@apache.org>.
PhilippvK commented on issue #11810:
URL: https://github.com/apache/tvm/issues/11810#issuecomment-1221592770
@mehrdadh I would guess, that this is related to #11702 which was merged one month ago. You you please try again on the latest TVM commit?
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] ashutosh-arm commented on issue #11810: [CMSIS-NN][ARM] ARM_MATH_ARGUMENT_ERROR due to not setting the context buffer in `arm_avgpool_s8`
Posted by GitBox <gi...@apache.org>.
ashutosh-arm commented on issue #11810:
URL: https://github.com/apache/tvm/issues/11810#issuecomment-1164549167
Thanks @mehrdadh for putting together the script to reproduce this issue. I will get to this early next week.
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] mehrdadh closed issue #11810: [CMSIS-NN][ARM] ARM_MATH_ARGUMENT_ERROR due to not setting the context buffer in `arm_avgpool_s8`
Posted by GitBox <gi...@apache.org>.
mehrdadh closed issue #11810: [CMSIS-NN][ARM] ARM_MATH_ARGUMENT_ERROR due to not setting the context buffer in `arm_avgpool_s8`
URL: https://github.com/apache/tvm/issues/11810
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] mehrdadh commented on issue #11810: [CMSIS-NN][ARM] ARM_MATH_ARGUMENT_ERROR due to not setting the context buffer in `arm_avgpool_s8`
Posted by GitBox <gi...@apache.org>.
mehrdadh commented on issue #11810:
URL: https://github.com/apache/tvm/issues/11810#issuecomment-1163459852
Here is a gist to reproduce this error:
https://gist.github.com/mehrdadh/2f3aa844e2df21e22e5ccf9708a56e0c#file-tvm_kws_cmsis-py
Note:
1. You don't see the error unless you use `west debug` to step by step go through the code. In the generated code `ARM_MATH_ARGUMENT_ERROR` would result in returning -1 and code execution just fails.
2. If you run this with older CMSIS version like (5.8.0 which we used before on ci_qemu) it will show wrong output. The reason is that previous CMSIS didn't check for if `context->buff` is not `NULL` and it would just overwrite some memory.
cc @ashutosh-arm
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] mehrdadh commented on issue #11810: [CMSIS-NN][ARM] ARM_MATH_ARGUMENT_ERROR due to not setting the context buffer in `arm_avgpool_s8`
Posted by GitBox <gi...@apache.org>.
mehrdadh commented on issue #11810:
URL: https://github.com/apache/tvm/issues/11810#issuecomment-1162449654
@grant-arm
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org