code review for second hotspot FDS gobjcopy work around (7165598)
Daniel D. Daugherty
daniel.daugherty at oracle.com
Thu May 24 13:04:45 UTC 2012
On 5/24/12 1:52 AM, serguei.spitsyn at oracle.com wrote:
> I do not see any issues with that webrev.
> Looks good.
> On 5/23/12 12:59 PM, Daniel D. Daugherty wrote:
>> This is a hotspot code review request for the second of a pair of
>> Full Debug Symbols gobjcopy work arounds on Solaris. The first
>> hotspot FDS gobjcopy work around was reviewed using bug 7165060
>> and that fixed the dtrace test failures.
>> The gobjcopy utility also crashes due to empty sections with the
>> SHF_ALLOC flagset on Solaris X64 objects. This causes build
>> The first new temporary work around tool is add_gnu_debuglink
>> and it was added by 7165060.The second new temporary work around
>> tool is:
>> fix_empty_sec_hdr_flags - removes the SHF_ALLOC flag from empty
>> sections in ELF objects.
>> These temporary work arounds are only needed until the proper
>> Solaris 10 Update 6 patches are made available. The two patches
>> are independent of one another which is why there are two
>> separate temporary work arounds. However, we're putting the
>> temporary work arounds in place because the 7u6/HSX-23.2 project
>> window is closing fast.
>> Here is the webrev URL for the HSX-24 version:
>> This fix will also be backported to 7u6/HSX-23.2 and I expect the
>> changes to virtually identical.
>> Thanks, in advance, for any reviews!
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the build-dev