Re: RFR(S): 8026949: -Xint flag prints wrong warning: Initialization of C1 thread failed (no space to run compilers)
igor.ignatyev at oracle.com
Tue Oct 22 11:34:32 PDT 2013
Albert, please add a jtreg test for this change. I think it is very easy: just start 'java -version' as a external process and check its output in 'Xint' and '-Xcomp' cases. If you don't think so, please file a RFE to add regression test and assign it to me.
----- Reply message -----
From: "Albert Noll" <albert.noll at oracle.com>
To: "Igor Veresov" <igor.veresov at oracle.com>
Cc: "hotspot-compiler-dev at openjdk.java.net" <hotspot-compiler-dev at openjdk.java.net>
Subject: RFR(S): 8026949: -Xint flag prints wrong warning: Initialization of C1 thread failed (no space to run compilers)
Date: Tue, Oct 22, 2013 22:03
Roland, Igor, thanks for looking at
I tested the patch as follows:
java -Xint -version .. make sure that output does not appear
java -Xcomp -version make sure that compiler(s) are working
Execution of nashorn octane + some DaCapo benchmarks. (with -Xint
and -Xcomp and -Xmixed)
Seems to work fine.
On 22.10.2013 19:12, Igor Veresov wrote:
On Oct 22, 2013, at 6:15 AM, Albert Noll <albert.noll at oracle.com>
Thanks Igor. This is a good
Here is the webrev:
On 22.10.2013 08:41, Igor Veresov wrote:
Another trivial fix option is to just bail out
of CompileBroker::compilation_init() is UseCompiler is
false. To avoid initializing the compiler objects and
On Oct 21, 2013, at 11:15 PM, Albert Noll
<albert.noll at oracle.com>
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
is set correctly.
The solution proposed by Igor is simpler.
If the above issues do not matter, I would
also opt for the
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 ==
On Oct 21, 2013, at 2:54
AM, Albert Noll <albert.noll at oracle.com>
could I have reviews for
this small patch?
Problem: 8023014 introduced
a wrong warning:
Initialization of C1 thread
failed (no space to run
compilers) when the VM is
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:
-Xint (-server / -client)
-Xint (-server / -client)
Many thanks in advance,
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-compiler-dev