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

[Baesken, Matthias]

>> 1. We already have a ton of cpuid related parsing in
>> ./cpu/x86/vm_version_x86.hpp - can't you hook into that to "check
>> virtualizations" and store the information for later use?

Hi David ,   probably  you mean this one  with   "cpuid logic query "  .

Probably I could move some coding into ./cpu/x86/vm_version_x86.hpp   but I did not like the idea to put more and more coding into the header file .

Best regards, Matthias


> -----Original Message-----
> From: David Holmes <david.holmes at oracle.com>
> Sent: Dienstag, 26. März 2019 13:01
> To: Baesken, Matthias <matthias.baesken at sap.com>; 'hotspot-
> dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>
> Cc: Doerr, Martin <martin.doerr at sap.com>
> Subject: Re: RFR: 8219241: Provide basic virtualization related info in the
> hs_error file on linux/windows x86_64
> 
> Hi Matthias,
> 
> You don't respond on the cpuid logic query.
> 
> Thanks,
> David
> 
> On 26/03/2019 6:32 pm, Baesken, Matthias wrote:
> > Hi David,
> >
> >> This is looking better though I'm struggling to keep all the dots
> >> connected in terms of the APIs - I can't quite see the pattern for what
> >> calls what and what overrides what.
> >
> > I   just  followed  here what is done  at  other places  of    VM_Version  /
> Abstract_VM_Version   , see
> >    for example  :
> >
> > src/hotspot/share/runtime/vm_version.hpp
> >
> >    82   //  .....    Platforms that
> >    83   // need to specialize this define VM_Version::early_initialize().
> >    84   static void early_initialize() { }
> >    85
> >    86   // Called to initialize VM variables needing initialization
> >    87   // after command line parsing. Platforms that need to specialize
> >    88   // this should define VM_Version::init_before_ergo().
> >    89   static void init_before_ergo() {}
> >
> >
> >> 2. os_windows.cpp
> >>
> >> Why the indirection through os::win32::print_virtualization_info instead
> >> of just calling VM_Version::print_detected_virtualization directly?
> >
> > Yes that's a good point - I think I  should change this .
> >
> > Best regards, Matthias
> >
> >
> >
> >
> >> -----Original Message-----
> >> From: David Holmes <david.holmes at oracle.com>
> >> Sent: Mittwoch, 20. März 2019 07:15
> >> To: Baesken, Matthias <matthias.baesken at sap.com>; 'hotspot-
> >> dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>
> >> Cc: Doerr, Martin <martin.doerr at sap.com>
> >> Subject: Re: RFR: 8219241: Provide basic virtualization related info in the
> >> hs_error file on linux/windows x86_64
> >>
> >> Hi Matthias,
> >>
> >> On 19/03/2019 11:53 pm, Baesken, Matthias wrote:
> >>> Hello, here is a new  webrev :
> >>>
> >>> http://cr.openjdk.java.net/~mbaesken/webrevs/8219241.3/
> >>>
> >>> This moved  the ppc/s390  virtualization related info output into  the
> >> platform/cpu  - specific    coding .
> >>
> >> This is looking better though I'm struggling to keep all the dots
> >> connected in terms of the APIs - I can't quite see the pattern for what
> >> calls what and what overrides what. A couple of initial comments:
> >>
> >> 1. We already have a ton of cpuid related parsing in
> >> ./cpu/x86/vm_version_x86.hpp - can't you hook into that to "check
> >> virtualizations" and store the information for later use?
> >>
> >> 2. os_windows.cpp
> >>
> >> Why the indirection through os::win32::print_virtualization_info instead
> >> of just calling VM_Version::print_detected_virtualization directly?
> >>
> >> I'll keep trying to piece this together. :)
> >>
> >> Thanks,
> >> David
> >> -----
> >>
> >>>
> >>> Best regards, Matthias
> >>>
> >>>
> >>>> -----Original Message-----
> >>>> From: David Holmes <david.holmes at oracle.com>
> >>>> Sent: Montag, 11. März 2019 08:36
> >>>> To: Baesken, Matthias <matthias.baesken at sap.com>; 'hotspot-
> >>>> dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>
> >>>> Cc: Doerr, Martin <martin.doerr at sap.com>
> >>>> Subject: Re: RFR: 8219241: Provide basic virtualization related info in the
> >>>> hs_error file on linux/windows x86_64
> >>>>
> >>>> On 7/03/2019 3:13 am, Baesken, Matthias wrote:
> >>>>> Hello ,  could I push the  latest version of   8219241  ?
> >>>>
> >>>> Sorry based on your earlier email I thought you were looking at doing
> >>>> the restructuring. I was then on vacation most of last week.
> >>>>
> >>>>> A  change that moves the coding more into  platform/cpu  - specific
> >> coding
> >>>> (if this is wanted)
> >>>>>     could be  done when bringing in  the AIX  virtualization related info
> in
> >>>> another patch  which I plan to do .
> >>>>
> >>>> If that is imminent then okay - I don't like the code as it is.
> >>>>
> >>>> Thanks,
> >>>> David
> >>>>
> >>>