RFR: 8219241: Provide basic virtualization related info in the hs_error file on linux/windows x86_64

Baesken, Matthias matthias.baesken at sap.com
Thu Apr 4 12:35:19 UTC 2019


> 
> On 4/04/2019 8:54 pm, Baesken, Matthias wrote:
> >>
> >> My remaining query is why we need the OS specific checks and code in
> >> os-cpu files? Isn't this just cpu specific? Even if a particular OS
> >> doesn't support virtualization, won't the cpuid query simply report "no
> >> virtualization"?
> >>
> >
> > Hi David,  if you are referring to  these  os-cpu files  :
> >
> > src/hotspot/os_cpu/linux_x86/vm_version_linux_x86.cpp
> > src/hotspot/os_cpu/windows_x86/vm_version_windows_x86.cpp
> >
> > we have different coding available  because  I am not aware  of a cross OS
> API/coding to get   the cpu id info.
> > That’s why different coding for LINUX/Windows.
> 
> ?? But there's nothing in those added functions that is Linux specific
> or Windows specific! The only difference is the mechanism for doing
> "inline assembly" and that's compiler specific. So this should just use
> compiler based ifdefs in vm_version_x86.cpp.
> 

Hi David, I see your point.  But I thought we want to reduce the  ifdefs and favor  putting code into the os or os/cpu  specific  files .
But it is true , from a technical point  we could do using the compiler macros .

Best regards, Matthias
 


More information about the hotspot-dev mailing list