<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#CCCCCC" text="#000000">
Dan,<br>
<br>
I've reviewed the fixes as did not notice any problems.<br>
Looks good.<br>
<br>
Thanks,<br>
Serguei<br>
<br>
On 5/23/12 12:50 PM, Daniel D. Daugherty wrote:
<blockquote cite="mid:4FBD3F7A.4030501@oracle.com" type="cite">Greetings,
<br>
<br>
This is a JDK code review request for a pair of Full Debug Symbols
<br>
gobjcopy work arounds on Solaris. The gobjcopy utility on Solaris
10
<br>
corrupts the SUNW_* sections on objects. This has caused dtrace
test
<br>
failures and Monitoring & Management test failures. The
gobjcopy
<br>
utility crashes due to empty sections with the SHF_ALLOC flagset
on
<br>
Solaris X64 objects. This causes build failures.
<br>
<br>
There are two new temporary work around tools:
<br>
<br>
add_gnu_debuglink - adds the ".gnu_debuglink" section to an ELF
<br>
object without corrupting the other sections in the object.
<br>
<br>
fix_empty_sec_hdr_flags - removes the SHF_ALLOC flag from empty
<br>
sections in ELF objects.
<br>
<br>
These temporary work arounds are only needed until the proper
<br>
Solaris 10 Update 6 patches are made available. The two patches
<br>
are independent of one another which is why there are two
<br>
separate temporary work arounds. However, we're putting the
<br>
temporary work arounds in place because the 7u6 project window
<br>
is closing fast.
<br>
<br>
Here is the webrev URL for the JDK8-T&L version:
<br>
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~dcubed/fds_revamp/7170449-webrev/0/">http://cr.openjdk.java.net/~dcubed/fds_revamp/7170449-webrev/0/</a>
<br>
<br>
This fix will also be backported to 7u6-T&L and I expect the
<br>
changes to virtually identical.
<br>
<br>
Thanks, in advance, for any reviews!
<br>
<br>
Dan
<br>
<br>
<br>
</blockquote>
<br>
</body>
</html>