On Thu, 27 Feb 2025 15:33:47 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:

>> No, there is nothing for that. There is a conceptual leap between 
>> determining how to compile hotspot and how to decide which modules to 
>> include in the image, and is not at all clear how you would want to 
>> integrate these two. Like, should you exclude models by saying a hotspot 
>> feature is not needed? Or, should you modify how hotspot is compiled by 
>> saying that you want to exclude a model?
>> 
>> I think it is better left to whoever configures the build to figure such 
>> things out.
>
>> No, there is nothing for that. There is a conceptual leap between 
>> determining how to compile hotspot and how to decide which modules to 
>> include in the image, and is not at all clear how you would want to 
>> integrate these two. Like, should you exclude models by saying a hotspot 
>> feature is not needed? Or, should you modify how hotspot is compiled by 
>> saying that you want to exclude a model?
>> 
>> I think it is better left to whoever configures the build to figure such 
>> things out.
> 
> I was just following up to the idea stated above 
> 'If the only VM to be built is the minimal VM then maybe it should filter 
> down the set of modules to include'
> but this is not so easy it seems.
> Regarding - 'should you exclude models (modules?) by saying a hotspot feature 
> is not needed?'  excluding the JVM feature  JFR means that the jfr related 
> modules are broken/not (fully) working.
> Excluding them would probably be logical (and also help the tests because 
> quite a lot of tests do not have requires vm.hasJFR or requires vm.jvmti to 
> check if the test makes sense on some JVM with limited features) .

If you want to see that implemented, the first step is to open an enhancement 
issue on JBS.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23805#discussion_r1974374119

Reply via email to