RFR: JDK-8143255: Remove debug logging from SymbolTable::unlink() and SymbolTable::possibly_parallel_unlink()

Thomas Schatzl thomas.schatzl at oracle.com
Thu Nov 19 09:00:30 UTC 2015


Hi Bengt,

On Thu, 2015-11-19 at 09:24 +0100, Bengt Rutisson wrote:
> Hi Thomas,
> 
> Thanks for looking at this!
> 
> On 2015-11-18 21:36, Thomas Schatzl wrote:
> > Hi Bengt,
> >
> > On Wed, 2015-11-18 at 20:47 +0100, Bengt Rutisson wrote:
> >> Hi everyone,
> >>
> >> Could I have a couple of reviews for this patch to remove some debugging
> >> code?
> >>
> >> http://cr.openjdk.java.net/~brutisso/8143255/webrev.00/
> >> https://bugs.openjdk.java.net/browse/JDK-8143255
> >    afaik the memory_total parameter to the various methods is only every
> > consumed by the now removed printout.
> >
> > Can you please remove that too?
> 
> Good point. Here's an updated webrev:
> http://cr.openjdk.java.net/~brutisso/8143255/webrev.01/

  looks good.

Thomas




More information about the hotspot-runtime-dev mailing list