[GitHub] [incubator-mxnet] andrei5055 edited a comment on pull request #18582: Refactoring of Pooled Storage Manager classes

[GitBox]

andrei5055 edited a comment on pull request #18582:
URL: https://github.com/apache/incubator-mxnet/pull/18582#issuecomment-654937596


   **Please, for now ignore graphs for CPU and CPU_PINNED memory. I am not sure 
why, but I don't see 2x run time improvement for newly implemented Pooled 
Storage Managers with respect to "Unpooled" ones. Perhaps, it's because I was 
using my workstation with limited amount of CPU memory. I will try to run the 
same benchmarks on the machine with more CPU memory and I will update the 
results.**
    
   
   
![image](https://user-images.githubusercontent.com/7293680/86802902-55de1a80-c02a-11ea-9c0c-b8aa6c7d9089.png)
   (Here we do have only three lines, because the Pooled Storage Manager was 
not implemented for CPU_PINNED memory.)


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