RFR (s) 8085865: hs_err improvement: Printing /proc/cpuinfo makes too long hs_err files

[Coleen Phillimore]

On 6/15/15 12:04 PM, Christian Thalinger wrote:
>> On Jun 12, 2015, at 3:07 AM, Coleen Phillimore <coleen.phillimore at oracle.com> wrote:
>>
>>
>>
>> On 6/12/15 4:17 AM, Mikael Gerdin wrote:
>>> Hi Coleen,
>>>
>>> On 2015-06-11 18:06, Coleen Phillimore wrote:
>>>> Summary: summarize information from linux-x86; it's too long and redundant
>>>>
>>>> The cpuinfo on one linux/x86 machine now prints:
>>>>
>>>> /proc/cpuinfo:
>>>> model name      : Intel(R) Core(TM) i5-3550 CPU @ 3.30GHz
>>>>
>>>> on 32 bit machine:
>>>> *
>>>> */proc/cpuinfo:
>>>> model name      : Intel(R) Xeon(R) CPU           L5520  @ 2.27GHz
>>>> **
>>>> Other linux platforms are unaffected because they don't print as much
>>>> and information seems not redundant.
>>> Would you be ok with printing one of the "flags" lines as well?
>>>
>>> flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc aperfmperf eagerfpu pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic popcnt tsc_deadline_timer aes xsave avx lahf_lm ida arat epb pln pts dtherm tpr_shadow vnmi flexpriority ept vpid xsaveopt
>>>
>>> It would give some quick insight into what CPU features were available during runtime.
>> We already print
>>
>> CPU:total 4 (4 cores per cpu, 1 threads per core) family 6 model 58 stepping 9, cmov, cx8, fxsr, mmx, sse, sse2, sse3, ssse3, sse4.1, sse4.2, popcnt, avx, aes, clmul, erms, tsc, tscinvbit, tscinv
>>
>> For the flags we use.   This isn't enough?
> I’d prefer to print the flags the CPU supports too since these are not the same.

Okay, sure, I'll add that line.
Coleen

>
>> thanks,
>> Coleen
>>> /Mikael
>>>
>>>> Rather than duplicating the pd_print_cpu_info function for all the
>>>> os_cpu/linux* combinations, I added an ifdef.   In os_aix.cpp there's a
>>>> pd_print_cpu_info, which is called from os::print_cpu_info that
>>>> duplicates the information but I didn't touch it.
>>>>
>>>> open webrev at http://cr.openjdk.java.net/~coleenp/8085865.01/
>>>> bug link https://bugs.openjdk.java.net/browse/JDK-8085865
>>>>
>>>> Thanks,
>>>> Coleen
>>>>
>>>>