RFR : 8211326 : add OS user related information to hs_err file
David Holmes
david.holmes at oracle.com
Wed Nov 14 22:32:29 UTC 2018
Hi Matthias,
On 15/11/2018 3:16 am, Baesken, Matthias wrote:
> Hi Thomas , so are you fine with the latest revision
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8211326.1/
>
> of the patch , can I add you as reviewer ?
>
> David - is the info added by Thomas sufficient for you?
Thomas explained about _verbose. I still have concerns that we will
print the header but then no info - I think that needs to be addressed.
I did not review the actual print_user_info as I'm not familiar with the
actual content.
Thanks,
David
> Any other reviewers ?
>
>
>
>> -----Original Message-----
>> From: Thomas Stüfe <thomas.stuefe at gmail.com>
>> Sent: Samstag, 10. November 2018 14:04
>> To: David Holmes <david.holmes at oracle.com>
>> Cc: Baesken, Matthias <matthias.baesken at sap.com>; Langer, Christoph
>> <christoph.langer at sap.com>; Volker Simonis <volker.simonis at gmail.com>;
>> HotSpot Open Source Developers <hotspot-dev at openjdk.java.net>
>> Subject: Re: RFR : 8211326 : add OS user related information to hs_err file
>>
>> Hi David,
>>
>> On Sat, Nov 10, 2018 at 12:59 PM David Holmes <david.holmes at oracle.com>
>> wrote:
>>>
>>> Hi Matthias,
>>>
>>> On 9/11/2018 3:13 AM, Baesken, Matthias wrote:
>>>> Hello , after the flag "-XX:+-ExtensiveErrorReports" make it into jdk/jdk
>> , I created another webrev :
>>>>
>>>> http://cr.openjdk.java.net/~mbaesken/webrevs/8211326.1/
>>>>
>>>>
>>>> The user info output is now guarded by ExtensiveErrorReports .
>>>
>>> + STEP("printing user info")
>>> + if (ExtensiveErrorReports) {
>>> + if (_verbose) {
>>> + os::Posix::print_user_info(st, buf, sizeof(buf));
>>> + }
>>> + }
>>>
>>> I don't understand why we explicitly need _verbose if we've asked for
>>> ExtensiveErrorreports?
>>
>> That flag has a different purpose: _verbose distinguishes the two
>> calls to VMError::report(): the first one to print a small report to
>> stderr, the second print a large report to the hs-err file. We only
>> want to print to the hs-err file, so _verbose is still needed.
>>
>>>
>>> Also, ideally the STEP would be inside the guard as otherwise we will
>>> just print the step description followed by nothing. If the macro
>>> expansion makes that impossible then we should have an else clause that
>>> prints something like:
>>>
>>> - disabled (use -XX:+ExtensiveErrorReports to see these details)
>>>
>>
>> We may even go a step further (but in a separate patch) and make
>> "ExtensiveErrorReports" a property of the STEP, to be given as
>> parameter of the STEP macro.
>>
>> Thanks, Thomas
>>
>
More information about the hotspot-dev
mailing list