RFR: 8181313: SA: Remove libthread_db dependency on Linux

Jini George jini.george at oracle.com
Thu Jan 10 06:10:12 UTC 2019


Thank you for implementing this change, Yasumasa. It looks good to me. 
Just a nit.

==> linux/native/libsaproc/libproc_impl.c
Could you please remove these lines ? I don't think these are needed 
anymore.

421
422 // new libthread_db of NPTL seem to require this symbol
423 JNIEXPORT ps_err_e JNICALL
424 ps_get_thread_area() {
425   print_debug("ps_get_thread_area not implemented\n");
426   return PS_OK;
427 }

==> linux/native/libsaproc/proc_service.h
Could you please remove these lines too ?
  79
  80 // new libthread_db of NPTL seem to require this symbol
  81 JNIEXPORT ps_err_e JNICALL
  82 ps_get_thread_area();

Thanks,
Jini.



On 1/9/2019 6:53 AM, Yasumasa Suenaga wrote:
> Hi all,
> 
> Please review this change:
> 
>    JBS: https://bugs.openjdk.java.net/browse/JDK-8181313
>    webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8181313/webrev.00/
> 
> It has passed all tests on submit repo, and serviceability/sa tests on
> Linux x64.
> 
> 
> Thanks,
> 
> Yasumas
> 


More information about the serviceability-dev mailing list