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

[Coleen Phillimore]

Thanks, Mikael.
Coleen

On 6/12/15 7:10 AM, Mikael Gerdin wrote:
>
>
> On 2015-06-12 12:07, Coleen Phillimore 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?
>
> Oh, I must've overlooked that. That's fine then.
> Thanks
> /Mikael
>
>>
>> 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
>>>>
>>>>
>>