harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vladimir Beliaev (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-2692) [drlvm] throwing exception in mixed mode leaks memory
Date Thu, 15 Feb 2007 11:02:06 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-2692?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12473346

Vladimir Beliaev commented on HARMONY-2692:

Regarding regression test - I believe there should be no reggression test because:

1. test runs ~7min (about 8'500'000 iterations passed by that time)  - too long for "acceptance"

2. test behaviour depends on current DRLVM "state" in terms of "C heap usage":
    - if DRLVM is optimized to use less c-memory, then number of iterations in test may be
not enough (test execution time is critical too)
    - if we re-develop test to check if the C-Heap-bound is achieved then such a test may
fail due to DRLVM use more C Heap (say, because of new features / optimization are done)...
So such the test would be unstable.

So, I vote for comitting the patch w/o regression test.

Vladimir Beliaev

> [drlvm] throwing exception in mixed mode leaks memory
> -----------------------------------------------------
>                 Key: HARMONY-2692
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2692
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>         Environment: Windows XP i686
>            Reporter: Salikh Zakirov
>         Assigned To: Gregory Shimansky
>         Attachments: 0001-Fix-of-memory-leak-from-rth_throw_lazy-runtime-helper.patch,
BufferWrap.java, ExceptionLeakTest.java
> The function wrap() of nio buffers leaks memory when invoked with illegal arguments.
> When the attached test BufferWrap.java is run on DRLVM, it steadily increases used memory
until crash.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message