[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, which I saw on my local 
NVidia branch. 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