JDK-8151514: Remove the noisy NOISY debugging code from parCarTableModRefBS.cpp

Bengt Rutisson bengt.rutisson at oracle.com
Wed Mar 9 12:54:10 UTC 2016


Hi everyone,

Could I have a couple of reviews for this change?

http://cr.openjdk.java.net/~brutisso/8151514/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8151514

In CardTableModRefBSForCTRS::process_chunk_boundaries() there is a NOISY 
definition to log some debug messages.

Several problems with this macro:
* The code inside process_chunk_boundaries() is very hard to read due to 
all the NOISY lines.
* The logging is done to tty and not the logging framework-.
* At some point someone detected a conflict with a preprocessor flag.

It is better to remove this macro and just add relevant logging if we 
every find the need for it.

Thanks,
Bengt


More information about the hotspot-gc-dev mailing list