RFR: 8350813: Rendering of bulky sound bank from MIDI sequence can cause OutOfMemoryError [v4]
Alexander Zuev
kizune at openjdk.org
Tue Mar 18 21:14:09 UTC 2025
On Tue, 18 Mar 2025 17:31:18 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Sounds like (pun intended) my example did not land :)
No, i totally understand your point - it is quite possible that calculating currently available memory size might be detrimental because of the possibility that GC might free up that space when we actually request it - but this is not a critical functionality and the potential damage of suffocating VM with the repeated requests of +1M for potentially 4G sound bank - when we do know that we only have 3G of free memory - heavily outweighs the potential border-line case when we can load the sound bank - maybe - if GC will finds the resources but we were a little paranoid and throwing a checked exception that customer will take into account and the execution of the main program may continue.
Conclusion: i know the risks and i am willing to take them, reliability concern is noted but it is the case where being cautious and a tiny bit paranoid makes more sense. Again, for this exact use case. Plus there is a workaround: user can just increase the maximum heap size if he finds that some specific sound bank causes this kind of a problem. If you find out that you need to generate 4G sound bank - add 4G of heap and you will be fine just fine.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23814#issuecomment-2734741051
More information about the client-libs-dev
mailing list