RFR(S): 8026949: -Xint flag prints wrong warning: Initialization of C1 thread failed (no space to run compilers)
igor.veresov at oracle.com
Mon Oct 21 23:41:20 PDT 2013
Another trivial fix option is to just bail out of CompileBroker::compilation_init() is UseCompiler is false. To avoid initializing the compiler objects and all..
On Oct 21, 2013, at 11:15 PM, Albert Noll <albert.noll at oracle.com> wrote:
> Christian, Igor, thanks for the reviews.
> There seems to be an ambiguity between -Xint and -XX:+UseCompiler. Whether we use a compiler or
> not, depends on the order in which the arguments are given. I am not exactly sure why we have
> the -XX:(+/-)UseCompiler flag at all, also as a product flag.
> The reason why I added the checks when we determine the number of compiler threads is that we
> do not generate the compiler objects (they are not needed). Also, the number of compiler threads
> is set correctly.
> The solution proposed by Igor is simpler. If the above issues do not matter, I would also opt for the
> simple solution.
> On 22.10.2013 06:17, Igor Veresov wrote:
>> If think you can just use the existing "UseCompiler" flag, that we set to false when -Xint is specified and just not call init_compiler_threads() if UseCompiler == false.
>> On Oct 21, 2013, at 2:54 AM, Albert Noll <albert.noll at oracle.com> wrote:
>>> could I have reviews for this small patch?
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8026949
>>> webrev: http://cr.openjdk.java.net/~anoll/8026949/webrev.00/
>>> Problem: 8023014 introduced a wrong warning: Initialization of C1 thread failed (no space to run compilers) when the VM is started
>>> with -Xint
>>> Solution: Do not start compiler threads if VM is started with -Xint
>>> Testing: -Xint flag does not print out the false warning. Tested flag combinations:
>>> -XX:+TieredCompilation -Xint (-server / -client)
>>> -XX:-TieredCompilation -Xint (-server / -client)
>>> Many thanks in advance,
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-compiler-dev