[GitHub] [tvm] areusch commented on pull request #6953: Add retry to sockets on EINTR error

[GitBox <gi...@apache.org>]

areusch commented on pull request #6953:
URL: https://github.com/apache/tvm/pull/6953#issuecomment-736707997


   @rkimball thanks for testing this. can you modify your test as follows, which would replicate the way python does Ctrl+C handling:
   
   1. override the SIGINT handle using signal.
   2. return from the signal handler without exiting the program (you can set some global flag if you want to, which is what the python signal handler does)
   3. now see if you get EINTR from the network call.
   
   if so, you need to check the global flag between retries. unfortunately, in the typical TVM use case, that global flag is in python land, and merely says "please jump the python interpreter to the python-land signal handler at the next instruction"


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