RFR: JDK-8189429: SA: MacOSX: Replace the deprecated PT_ATTACH with PT_ATTACHEXC

[Jini George]

Hi Erik,

Thank you very much for looking into this. I have addressed your 
comments. The latest webrev is at:

http://cr.openjdk.java.net/~jgeorge/8189429/webrev.06/

Thank you,
Jini.


On 6/23/2018 3:31 AM, Erik Joelsson wrote:
> Hello Jini,
> 
> In general this looks pretty good, but it's also breaking some new 
> ground as it's adding generation of native source in the java gensrc 
> step. Mixing native code with the java code that the genrcs targets and 
> gensrc output directories are meant for seems ok for now, but may cause 
> trouble in the future. I'm going to accept it for now though.
> 
> In Gensrc-jdk.hotspot.agent.gmk: Please put the new macosx stuff in its 
> own section (as delimited by the 80x # lines) and put that whole thing 
> inside a conditional for macosx. Also please break line 47 (for recipe 
> lines, indent with tab and 4 additional spaces for continuation [1]).
> 
> In Lib-jdk.hotspot.agent.gmk: I believe adding extra src should be 
> enough as that will implicitly add the same directories as header dirs 
> by default. At least that's the intention.
> 
> /Erik
> 
> [1] http://openjdk.java.net/groups/build/doc/code-conventions.html
> 
> On 2018-06-22 11:11, Jini George wrote:
>> Hi all,
>>
>> [Including build-dev also since this includes build related changes].
>>
>> Requesting reviews for:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8189429 (SA: MacOSX: Replace 
>> the deprecated PT_ATTACH with PT_ATTACHEXC)
>>
>> Webrev: http://cr.openjdk.java.net/~jgeorge/8189429/webrev.04/
>>
>> This is the follow-up solution for the temporary restoration of 
>> PT_ATTACH to fix https://bugs.openjdk.java.net/browse/JDK-8184042 
>> (several serviceability/sa tests timed out on MacOS X), which was done 
>> in Oct 2017. The mails related to this are at:
>>
>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2017-August/021702.html 
>>
>>
>> Changes as compared to the patch sent last year 
>> (http://cr.openjdk.java.net/~jgeorge/8184042/webrev.00/):
>>
>> * Addressed the review comments which were provided by Poonam, Dan, 
>> Dmitry.
>> * A major change as compared to what was done last year is that the 
>> MIG generated files have been included as a part of the JDK build 
>> process.
>> * The test case which was provided in the patch last year is no longer 
>> required since we have ClhsdbAttach.java testing the same 
>> functionality as a part of the SA testsuite now.
>> * Other than that, some minor improvements have been done wrt error 
>> handling.
>>
>> The steps for the proposed solution have been provided in JBS.
>>
>> Testing: ALL the SA tests pass on MacOSX and the other Mach5 platforms.
>>
>> Thanks to Sharath, Robin, Gerard and Dan for looking into the changes 
>> and providing valuable comments.
>>
>> Thanks,
>> Jini.
>