RFR (XXS): 7130404: [macosx] "os.arch" value should be "x86_64" for compatibility with Apple JDK6
james.melvin at oracle.com
Tue Feb 21 11:36:43 PST 2012
Comments inline... updated webrev a bit later today...
On 2/21/12 5:41 AM, Michael McMahon wrote:
> Thanks for doing this. Some minor comments on the JDK changes.
> 1) the amd64 definitions in make/common/Defs-macosx.gmk are redundant now.
> Maybe we should just delete them (though I agree with keeping the
> runtime checks
> in the .java sources)
Good point. I've deleted the old amd64 references that have been replaced.
> 2) the (Mac) checks for os.name have generally been using
> String.startsWith() instead of equals()
> so they will work with a future "Mac OS X Server"
> I think there are similar checks in hotspot too. I see Mike Swingler is
> String.contains("OS X"). I'd be ok with that too.
For these set of changes, I've switched to Mike's suggestion. Perhaps we
can switch the other callsites (unrelated to this change) as part of a
general cleanup in JDK8.
> 3) one other location where a check needs to be added is:
> Also, I don't see any code in Hotspot that checks for the new "x86_64"
> So, is it actually necessary for the hotspot change to be integrated first?
Yup, still necessary. There is a reference in ...
> - Michael,
> On 20/02/12 21:02, James Melvin wrote:
>> To maintain compatibility with Apple JDKs, a proposal will be made to
>> change the 'os.arch' system property from 'amd64' to 'x86_64' on Mac OS
>> X. Minor changes are required to the following repositories, for which
>> I've provided webrevs...
>> JPRT job (2012-02-20-203901.jmelvin.hotspot)
>> Notepad, SwingSet2, SPECjbb2005
>> This change will also impact a small number of internal tests and RE
>> scripts. The bundle names will also reflect the change amd64 -> x86_64.
>> HotSpot changes can be integrated first, with the JDK changes in the
>> following promotion. Should the proposal be rejected for 7u4, I
>> obviously withdraw the bugfix.
>> Feedback welcome,
More information about the hotspot-dev