RFR: 8219241: Provide basic virtualization related info in the hs_error file on linux/windows x86_64
Baesken, Matthias
matthias.baesken at sap.com
Mon Apr 8 08:07:47 UTC 2019
Hi David,
> >>
> >> So compiler ifdefs in the cpu files is the way to handle this.
> >>
> >
> > Hello, I moved the coding from
> >
> > src/hotspot/os_cpu/linux_x86/vm_version_linux_x86.cpp
> > src/hotspot/os_cpu/windows_x86/vm_version_windows_x86.cpp
> >
> >
> > Into src/hotspot/cpu/x86/vm_version_x86.cpp and guarded it with the
> compiler macros :
> >
> > http://cr.openjdk.java.net/~mbaesken/webrevs/8219241.5/
>
> I'm not clear why we are still using os-guards as well?
>
> + #elif defined(__GNUC__) && defined(__linux__)
>
> + #if defined(_LP64) && (defined(__linux__) || defined(_WIN64))
>
> won't this code still execute okay on Solaris/BSD/MacOS ?
>
It executes okay on MacOS .
Most likely it works also on Solaris x86, not sure ; only Solaris Sparc is in the list of Solaris build platforms supported in JDK13 :
https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms
I would prefer to keep the os-guards .
Best regards, Matthias
More information about the hotspot-dev
mailing list