You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2019/07/18 01:22:45 UTC
[GitHub] [incubator-mxnet] ZhennanQin commented on issue #15574: fix naive
engine for multi-threaded inference
ZhennanQin commented on issue #15574: fix naive engine for multi-threaded inference
URL: https://github.com/apache/incubator-mxnet/pull/15574#issuecomment-512627670
I just evaluated the thread-safe condition for `NaiveEngine`, and I agree with @marcoabreu that it's not enough with this point fix. There's too much data race when using MXNet in multi-thread usage, even for `NaiveEngine`, we need to take care all class member not only `req_completed_`. Creating private `NaiveEngine` for each thread is another option. So we need a whole picture plan for MXNet multi-thread support.
----------------------------------------------------------------
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
With regards,
Apache Git Services