RFR: 8153658: Redundant memory copy in LogStreamNoResourceMark

Jon Masamitsu jon.masamitsu at oracle.com
Thu Apr 7 17:41:22 UTC 2016


Stefan,

Change looks good.

Jon

On 04/06/2016 10:30 AM, Stefan Karlsson wrote:
> Hi all,
>
> Please review this patch to remove a redundant memory copy in the UL 
> log stream classes.
>
> http://cr.openjdk.java.net/~stefank/8153658/webrev.01
> https://bugs.openjdk.java.net/browse/JDK-8153658
>
> LogStreamNoResourceMark copies the resource allocated string buffer 
> into a new resource allocated string buffer before copying the data to 
> UL.
>
> Moreover, this also causes problem when implementing a log stream 
> class using CHeap memory instead of Resource memory. Even though the 
> first allocation is done from CHeap the second copy comes from 
> Resource memory.
>
> Thanks,
> StefanK



More information about the hotspot-dev mailing list