On Fri, 13 Mar 2026 20:10:43 GMT, Artur Barashev <[email protected]> wrote:
>> `r.maxMemory(`) is in bytes, so the extra 10 is to convert that to KiB. So >> combined this w/ 2 (for the 1/4) => 12. Will think about the >> `Long.MAX_VALUE` case. >> I thought about limiting iterations, but not sure what would be a reasonable >> limit, so I have not added it. Any suggestion? > > Oh, I see, we should probably comment it so it's clear. About limiting > iterations: since iterations and memory are connected, I would limit the > product of them (t * m) to something like 32-64 GiB? Should we make it a > security property? Hmm, I have removed the upper limits for now. We can add the restrictions back once we have consensus on how to best handle this. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/29597#discussion_r3184107944
