RFR: 8181313: SA: Remove libthread_db dependency on Linux

[Yasumasa Suenaga]

Hi Jini,

Thank you for your comment.
I uploaded a new webrev:

   http://cr.openjdk.java.net/~ysuenaga/JDK-8181313/webrev.01/

It passed tests on submit repo.
Could you review again?


Yasumasa


On 2019/01/10 15:10, Jini George wrote:
> 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
>>