RFR(S): 8210514: Obsolete SyncVerbose

[coleen.phillimore at oracle.com]

I think this looks good. I think as a future RFE we should add logging 
judiciously to this code and not preserve the logging that's already 
there.  The new logging for this code should look like the log_debug() 
etc calls and not with this distracting TEVENT macro.

Thanks,
Coleen

On 9/7/18 5:14 PM, Mikael Vidstedt wrote:
> Please review this change which obsoletes the SyncVerbose flag.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8210514 <https://bugs.openjdk.java.net/browse/JDK-8210514>
> Webrev: http://cr.openjdk.java.net/~mikael/webrevs/8210514/webrev.00/open/webrev/ <http://cr.openjdk.java.net/~mikael/webrevs/8210514/webrev.00/open/webrev/>
>
> * Background (from bug)
>
> The experimental SyncVerbose flag can in theory be used to produce logging of some synchronization primitives. The flag was convenient when the synchronization implementation was implemented and tuned a long time ago.
>
> The SyncVerbose flag no longer serves the purpose it used to, and is "Unstable" (the documentation of the flag says so explicitly). It should be obsoleted and later removed.
>
>
> Testing: I’m running the normal tier1 testing (still in progress).
>
> Cheers,
> Mikael
>