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 2021/06/14 21:33:06 UTC

[GitHub] [tvm] mehrdadh opened a new issue #8255: [microTVM] RPCSession Device Type Bug

mehrdadh opened a new issue #8255:
URL: https://github.com/apache/tvm/issues/8255


   While running graph executor debugger with micro target (e.g. qemu_x86), [device](https://github.com/apache/tvm/blob/1c251f50ee616507bdfd8866408e7acf9888cc3f/python/tvm/rpc/client.py#L75) function generates invalid device_type.
   In this case we have:
   ```
   -> return dev
   (Pdb) self._tbl_index
   0
   (Pdb) base.RPC_SESS_MASK
   128
   (Pdb) encode
   128
   (Pdb) dev.device_type
   129
   ``` 
   I think `dev.device_type` is expected to be 1 since this is a `cpu` type. 
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] mehrdadh commented on issue #8255: [microTVM] RPCSession Device Type Bug

Posted by GitBox <gi...@apache.org>.
mehrdadh commented on issue #8255:
URL: https://github.com/apache/tvm/issues/8255#issuecomment-868727201


   @tqchen I think the device type is correct here. I added a test [here](https://github.com/apache/tvm/pull/8259/files#diff-1f8002fa397ffc8695c94af887bebe36dc2367c1b5655350d87cf660119d5bc7R193)
   And the `sess.device` is equal to `remote[0]:cpu(0)`


-- 
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 #8255: [microTVM] RPCSession Device Type Bug

Posted by GitBox <gi...@apache.org>.
mehrdadh commented on issue #8255:
URL: https://github.com/apache/tvm/issues/8255#issuecomment-868834705


   https://github.com/apache/tvm/pull/8345


-- 
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 #8255: [microTVM] RPCSession Device Type Bug

Posted by GitBox <gi...@apache.org>.
mehrdadh commented on issue #8255:
URL: https://github.com/apache/tvm/issues/8255#issuecomment-861098268


   Update:
   device_type should be converted before passing to DeviceName function. PR #8259 will fix this issue.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] tqchen commented on issue #8255: [microTVM] RPCSession Device Type Bug

Posted by GitBox <gi...@apache.org>.
tqchen commented on issue #8255:
URL: https://github.com/apache/tvm/issues/8255#issuecomment-867620092


   In this particular case, it might be related to the fact that we are running graph runtime locally while the computation are on the remote. @areusch might have more background. In such cases, `remote.cpu(0)` is actually the correct device type.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org