RFR (S): Recognize windows 8

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Jun 1 10:39:43 PDT 2012


I agree here with Mikael. These changes are going back into 7u6, as I 
understand, so they should be small.

Vladimir

Mikael Vidstedt wrote:
> 
> I fully agree with you that that is a good idea and I actually started 
> doing that first, I still have that change in another sandbox. It became 
> rather big and scary in the end though so I'm proposing that I do that 
> as a separate change after 7u6 to reduce the risk. Ok?
> 
> Cheers,
> Mikael
> 
> On 2012-06-01 07:32, Dmitry Samersoff wrote:
>> Mikael,
>>
>> Is it possible to refactor this code to get rid of
>> multiple
>>
>> if (si.wProcessorArchitecture == PROCESSOR_ARCHITECTURE_AMD64)
>>                st->print(" , 64 bit");
>>
>> lines.
>>
>> -Dmitry
>>
>> On 2012-05-31 19:28, Mikael Vidstedt wrote:
>>> This change adds support for recognizing Windows 8 in
>>> os::print_windows_version, inspired by the corresponding changes in the
>>> libraries [1][2].
>>>
>>> Verified on a Windows 8 machine and passes JPRT.
>>>
>>> http://cr.openjdk.java.net/~mikael/7170275/webrev.00
>>>
>>> Please help me review, and I'll also be needing a sponsor.
>>>
>>> Thanks,
>>> Mikael
>>>
>>> [1] http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7170169
>>> [2] http://cr.openjdk.java.net/~khazra/7170169/7u6/webrev.00
>>>
>>
> 


More information about the hotspot-runtime-dev mailing list