[GitHub] spidyDev edited a comment on issue #9649: TEST(MEMORY_TEST, MemsetAndMemcopyPerformance) sometimes failing with timer failures

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

spidyDev edited a comment on issue #9649: TEST(MEMORY_TEST, MemsetAndMemcopyPerformance) sometimes failing with timer failures
URL: https://github.com/apache/incubator-mxnet/issues/9649#issuecomment-400426575
 
 
   @KellenSunderland  @cjolivier01 
   
   I am looking at this flaky test, had some queries and comments:
   - The test compares  normal memcpy/memset against with using openmp, and expects normal memcpy to be always slower . Although its will be true for small data size ( coz of overhead of creating threads with Openmp), but Openmp will be faster for larger data size , as with multiple threads utilizes greater memory bandwidth.
   Could you please let me know if I am missing something here. (reference to [code](https://github.com/apache/incubator-mxnet/blob/master/tests/cpp/misc/memory_test.cc#L123) 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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