csullivan commented on code in PR #11022:
URL: https://github.com/apache/tvm/pull/11022#discussion_r851551944
##########
src/runtime/hexagon/rpc/simulator/session.cc:
##########
@@ -1311,6 +1311,8 @@ detail::Optional<HEXAPI_Nullptr>
SimulatorRPCChannel::to_nullptr(const detail::M
TVM_REGISTER_GLOBAL("tvm.contrib.hexagon.create_hexagon_session")
Review Comment:
Can we include a test that we expect to fail when the stack size is too
small, and expect to pass when the stack size is set larger?
##########
src/runtime/hexagon/rpc/hexagon/rpc_server.cc:
##########
@@ -196,10 +195,16 @@ class HexagonRPCServer {
} // namespace tvm
namespace {
-tvm::runtime::hexagon::HexagonRPCServer* get_hexagon_rpc_server() {
- static tvm::runtime::hexagon::HexagonRPCServer g_hexagon_rpc_server(
- new uint8_t[TVM_HEXAGON_RPC_BUFF_SIZE_BYTES],
TVM_HEXAGON_RPC_BUFF_SIZE_BYTES);
- return &g_hexagon_rpc_server;
+static tvm::runtime::hexagon::HexagonRPCServer* g_hexagon_rpc_server;
+tvm::runtime::hexagon::HexagonRPCServer* get_hexagon_rpc_server(
+ uint32_t rpc_receive_buff_size_bytes =
TVM_HEXAGON_RPC_BUFF_DEFAULT_SIZE_BYTES) {
Review Comment:
How about we set the default value to 0, delete the
TVM_HEXAGON_RPC_BUFF_DEFAULT_SIZE_BYTES define, and then CHECK that
`rpc_receive_buff_size_bytes` is non-zero when `g_hexagon_rpc_server` is null?
--
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]
