[foreign-memaccess+abi] RFR: 8262118: Specialize upcalls
mcimadamore at openjdk.java.net
Mon Feb 22 13:24:54 UTC 2021
On Mon, 22 Feb 2021 13:19:32 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> I've tried to explain this in the PR body. Testing the current configuration already takes 15 minutes for TestUpcall. Adding the new configurations would multiply the number of configurations by 4. This seems excessive, since we mostly only use the default configuration any ways. The full matrix can be tested periodically (maybe as part of a higher tier, or CI), but for regular testing runs we could only run the default configuration?
> Oops - I missed that part. On linux it doesn't take long to execute (1 min maybe) so I was surprised. But I understand it might be an issue in other platforms. That said, in the recent ResourceScope rewrite, executing using all modes has been helpful in shaking out issues, so I'm a bit worried to see this going completely. I'm pretty sure in a month I'll forget that I have to go in a manually edit to add the missing combo...
One wacky idea: I assume these properties are not read-only - what if, the tests had an additional parameter which dictated the specific setup for generating upcall/downcalls? That way, we can decide how many stuff should be included when generating the various test combinations (and maybe just keep the main mode for standard executions).
More information about the panama-dev