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

Diagnosing memory leaks with native leaks. #2906

Open
mikelle-rogers opened this issue Mar 4, 2022 · 6 comments
Open

Diagnosing memory leaks with native leaks. #2906

mikelle-rogers opened this issue Mar 4, 2022 · 6 comments
Labels
documentation Documentation related issue enhancement New feature or request
Milestone

Comments

@mikelle-rogers
Copy link
Member

mikelle-rogers commented Mar 4, 2022

Walkthrough: see #515

@mikelle-rogers mikelle-rogers added the documentation Documentation related issue label Mar 4, 2022
@mikelle-rogers mikelle-rogers added this to the .NET 7.0 milestone Mar 4, 2022
@mikelle-rogers mikelle-rogers assigned mikem8361 and hoyosjs and unassigned hoyosjs Mar 4, 2022
@mikem8361 mikem8361 assigned noahfalk and unassigned mikem8361 Apr 6, 2022
@mikem8361
Copy link
Member

Not sure we need to document how to debug native leaks.

@noahfalk
Copy link
Member

noahfalk commented Apr 6, 2022

I'll take it and figure out what to do with it.

@noahfalk
Copy link
Member

noahfalk commented May 3, 2022

I've been thinking on this one a bit and I think we should mark this enhancement and move it to the Future milestone. While I don't doubt some users of .NET would appreciate a guide on diagnosing native memory leaks, I am only aware of about ~4 cases over the past 4 years. There are also a variety of pre-existing web articles and the methods to diagnose these issues vary based on OS and C runtime implementation. In total I expect we'd have to spend a non-trivial effort to research and write something substantial enough that it would be an improvement over users just doing a web search with the particulars of their issue and environment. It doesn't feel like a good return on our time unless we got a lot more feedback that native leaks are an important pain point.

Thoughts?

@mikelle-rogers
Copy link
Member Author

I've been thinking on this one a bit and I think we should mark this enhancement and move it to the Future milestone. While I don't doubt some users of .NET would appreciate a guide on diagnosing native memory leaks, I am only aware of about ~4 cases over the past 4 years. There are also a variety of pre-existing web articles and the methods to diagnose these issues vary based on OS and C runtime implementation. In total I expect we'd have to spend a non-trivial effort to research and write something substantial enough that it would be an improvement over users just doing a web search with the particulars of their issue and environment. It doesn't feel like a good return on our time unless we got a lot more feedback that native leaks are an important pain point.

Thoughts?

That sounds good to me.

@mikelle-rogers mikelle-rogers added the enhancement New feature or request label May 3, 2022
@mikelle-rogers mikelle-rogers modified the milestones: .NET 7.0, Future May 3, 2022
@jerviscui
Copy link

I'd like to see it.

1 similar comment
@botinko
Copy link

botinko commented Jun 15, 2022

I'd like to see it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation related issue enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

7 participants