RFR(XXS) 8201321: NMT: Unnecessary re-recording thread stack and size when attaching listener to JavaThread

[David Holmes]

Hi Zhengyu,

On 10/04/2018 5:07 AM, Zhengyu Gu wrote:
> Hi,
> 
> Please review this trivial change, that removed unnecessary call to 
> record Java thread, which should already be done. Also, added assertion 
> to ensure it is the case.

Minor nit:

"Should already setup" => "Should already be setup"

Otherwise looks fine.

Thanks,
David


> This line was added during early NMT work, and it is redundant.
> 
> Bug:  https://bugs.openjdk.java.net/browse/JDK-8201321
> Webrev: http://cr.openjdk.java.net/~zgu/8201321/webrev.00/
> 
> Test:
> 
>   hotspot_runtime on Linux x64 (release and fastdebug)
>                      Windows x64 fastdebug
> 
>   Submit-hs tests.
> 
> Thanks,
> 
> -Zhengyu