PING: RFR: 8250598: Hyper-V is detected in spite of running on host OS

Baesken, Matthias matthias.baesken at sap.com
Tue Aug 25 11:22:19 UTC 2020


Hello,  
     the latest webrev  looks good to me .
I tested additionally  on  VMWare / Xen / KVM  and  the detection worked nicely .

You might want to rephrase this comment  (it sounds a little strange to me) 

http://cr.openjdk.java.net/~ysuenaga/JDK-8250598/webrev.02/src/hotspot/cpu/x86/vm_version_x86.cpp.frames.html

1821   // In case of Xen, it might be appear until 0x40010000.


Don't  need to see another webrev for this .

Best regards, Matthias



>On 2020/08/18 23:39, Doerr, Martin wrote:
>> Hi Yasumasa,
>> 
>> thanks a lot for measuring and improving the overhead. This version looks much better.
>> 
>> I still wonder what the reason was for using a stub. There was probably a reason, but I don't remember.
>
>It might be called in many times, so I made change would generate stub in first.
>
>VM_Version::check_virtualizations() would call CPUID 256 times (0x40000000 to 0x40010000 step 0x100),
>and also I changed it is used to call CPUID with EAX = 0x40000007 in Hyper-V.
>
>
>> Hopefully, Matthias can find time for another review when he's back next week.
>
>Yeah, I want to commit this change after his review.





More information about the hotspot-runtime-dev mailing list