RFR : 8211326 : add OS user related information to hs_err file
Baesken, Matthias
matthias.baesken at sap.com
Thu Nov 15 16:26:13 UTC 2018
Hi Thomas and David,
new webrev :
http://cr.openjdk.java.net/~mbaesken/webrevs/8211326.4/
This includes David's comment
> My only quibble is about the treatment of uid_t
> and gid_t as being the same. It would be marginally cleaner to me if the
> cast to unsigned was done upfront on the library call:
>
and adjusts the STEP-coding (removed else ... part ) .
David - can I add you as reviewer (Thomas was fine with the change already) ?
Thanks, Matthias
> -----Original Message-----
> From: David Holmes <david.holmes at oracle.com>
> Sent: Donnerstag, 15. November 2018 14:38
> To: Baesken, Matthias <matthias.baesken at sap.com>; Thomas Stüfe
> <thomas.stuefe at gmail.com>
> Cc: 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
>
> On 15/11/2018 11:20 pm, Baesken, Matthias wrote:
> > I'll remove the else - but I think I need a second reviewer, isn’t it ?
>
> I hadn't reviewed the actual printing of the user info but now I have
> had a good look at it. My only quibble is about the treatment of uid_t
> and gid_t as being the same. It would be marginally cleaner to me if the
> cast to unsigned was done upfront on the library call:
>
> unsigned id = (unsigned) ::getuid();
> st->print("uid : %u ", id);
> id = (unsigned) ::geteuid();
> ...
> id = (unsigned) ::getgid();
> ...
>
> Thanks,
> David
> -----
>
> > Best regards, Matthias
> >
> >> -----Original Message-----
> >> From: Thomas Stüfe <thomas.stuefe at gmail.com>
> >> Sent: Donnerstag, 15. November 2018 14:15
> >> 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
> >>
> >> On Thu, Nov 15, 2018 at 1:43 PM David Holmes
> <david.holmes at oracle.com>
> >> wrote:
> >>>
> >>> On 15/11/2018 10:33 pm, Thomas Stüfe wrote:
> >>>> I'm fine with this, Matthias.
> >>>>
> >>>> BTW, I looked and the STEP header line is only printed when secondary
> >>>> exceptions happen inside the step, not always as David assumed (?). So
> >>>
> >>> Yes my bad - sorry.
> >>
> >> no problem. I did not see it right away either.
> >>
> >>>
> >>>> your first form
> >>>>
> >>>> STEP("blabla")
> >>>> if (ExtensiveErrorReports && _verbose) {
> >>>> blub
> >>>> }
> >>>>
> >>>> would have been fine too I think. But I do not want to send you
> >>>> through another review iteration, so I am fine with this version.
> >>>
> >>> Deleting the else doesn't need another round of review.
> >>
> >> Same from me. Remove the else path and ship it.
> >>
> >> ..Thomas
> >>
More information about the hotspot-dev
mailing list