Fix GPU memory recorder's memory leak

deepmodeling / abacus-develop

An electronic structure package based on either plane wave basis or numerical atomic orbitals.

http://abacus.ustc.edu.cn

GNU Lesser General Public License v3.0

174 stars 136 forks source link

Fix GPU memory recorder's memory leak #5491

Closed Critsium-xy closed 1 week ago

Critsium-xy commented 1 week ago

If you are running cpu examples using ABACUS compiled for GPU, then it will cause memory leak and crash the application. But it passed the CI test because CI test will not do tests like this.

Thanks for @dzzz2001 for finding this bug img_v3_02gk_bad01767-7d06-4fc0-a13e-5db49ff5eafg

A-006 commented 1 week ago

Oh, I thought it was a memory issue from FFT,and I looked for a bug in my PR for a long time.Could you add a unittest for this feature?It wouldn't be error next time.