lmxyy commented on pull request #8203:
URL: https://github.com/apache/tvm/pull/8203#issuecomment-856402725


   > @lmxyy thanks for fixing this! I'm not quite sure I understand the order 
of events that leads to the incorrect port, though. could you explain for my 
understanding?
   
   Sure. Previously, when you set up a server on your device side, the device 
would open a new connection to the tracker. When the tracker accepts the 
connection, it will register the device and record where the connection comes 
from 
([code](https://github.com/apache/tvm/blob/64a8e81eeeefee0b11669dc83ee0d9ec871270f7/python/tvm/rpc/tracker.py#L311-L312)).
 However, this connection is built on a dummy port on the device instead of the 
server port, so the address is not correct. Moreover, this address will not be 
updated since the device only sends the device key to the tracker 
([code](https://github.com/apache/tvm/blob/64a8e81eeeefee0b11669dc83ee0d9ec871270f7/python/tvm/rpc/server.py#L219)).


-- 
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:
us...@infra.apache.org


Reply via email to