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

[Coleen Phillimore]

Thank you George!
Coleen

On 6/16/15 9:49 AM, George Triantafillou wrote:
> Hi Coleen,
>
> This change looks good.
>
> -George
>
> On 6/15/2015 8:10 PM, Coleen Phillimore wrote:
>>
>> Hi, I've added Chris's requested flags line but this made me change 
>> more lines of code because the flags line can be pretty long and we 
>> don't want to allocate more stack space during error handling.
>>
>> http://cr.openjdk.java.net/~coleenp/8085865.03/
>>
>> Tested on my machine.  It's a little bit longer but nowhere nearly as 
>> long as printing the 4 processors (or 32 on some other people's 
>> machines).
>>
>> CPU Model and flags from /proc/cpuinfo:
>> model name      : Intel(R) Core(TM) i5-3550 CPU @ 3.30GHz
>> 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 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 sse4_1 
>> sse4_2 x2apic popcnt tsc_deadline_timer aes xsave avx f16c rdrand 
>> lahf_lm ida arat epb xsaveopt pln pts dtherm tpr_shadow vnmi 
>> flexpriority ept vpid fsgsbase smep erms
>>
>> Also tested with JPRT.
>>
>> Thanks,
>> Coleen
>>
>> On 6/15/15 1:39 PM, Coleen Phillimore wrote:
>>>
>>>
>>> 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
>>>>>>>
>>>>>>>
>>>
>>
>