On Thu, 26 Jan 2023 17:33:28 GMT, Justin King <jck...@openjdk.org> wrote:

> Adds initial LSan (LeakSanitizer) support to Hotspot. This setup has been 
> used to identify multiple leaks so far. It can run most of the test suite 
> except those that also fail with ASan, which is being looked at separately. 
> It is especially useful when combined with ASan, as LSan can use poisoning 
> information to determine what memory to scan or not to scan, making leak 
> detection more accurate and faster.
> 
> **Suppressing:**
> Currently the suppression list is only used to suppress JLI leaks that are 
> known, the rest are done in code. Suppressing needs to identify the source of 
> thet leak. Due to Hotspot's code organization, we would need to suppress 
> `os::malloc` and friends, which would suppress everything. Suppressing in 
> code has the added benefit of being explicit and surviving refactors if 
> methods change.
> 
> **Caveats:**
> - By default ASan enables LSan, however we explicitly disable it unless 
> `--enable-lsan` is given. It is useful to be able to use ASan without LSan. 
> Using LSan by itself is less likely to be useful and will probably not work, 
> but its still possible currently.

This pull request has now been integrated.

Changeset: 27126157
Author:    Justin King <jck...@openjdk.org>
Committer: Magnus Ihse Bursie <i...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/27126157d927c5ec4354cde8f31076899691996b
Stats:     316 lines in 22 files changed: 307 ins; 0 del; 9 mod

8298445: Add LeakSanitizer support in HotSpot

Reviewed-by: erikj, ihse

-------------

PR: https://git.openjdk.org/jdk/pull/12229

Reply via email to