Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memory leaks? #178

Open
isuruf opened this issue Jun 11, 2023 · 4 comments
Open

Memory leaks? #178

isuruf opened this issue Jun 11, 2023 · 4 comments

Comments

@isuruf
Copy link
Collaborator

isuruf commented Jun 11, 2023

Running an FMM with,

for _ in range(2):
    timing_data = {}
    drive_fmm(sumpy_wrangler, (charge_device,), timing_data=timing_data)
    queue.finish()
    gc.collect()

leads to a memory allocation failure with cuda in the second loop.

@inducer
Copy link
Owner

inducer commented Jun 26, 2023

When you say "memory allocation", what precisely do you mean? How do you measure that?

@inducer
Copy link
Owner

inducer commented Jul 19, 2023

Possibly related: inducer/pymbolic#125 (comment)

@inducer
Copy link
Owner

inducer commented Jul 19, 2023

Running

 pycl test_fmm.py 'test_sumpy_fmm(_acf, LaplaceKernel(3), LinearPDEConformingVolumeTaylorLocalExpansion, LinearPDEConformingVolumeTaylorMultipoleExpansion, False, False, False, None)'

with this patch applied leads to no appreciable memory growth. 🤔

Could you provide a reproducer?

@inducer
Copy link
Owner

inducer commented Jul 19, 2023

Similarly, no appreciable memory growth when run using mprof:

grafik

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants