8234397: add OS uptime information to os::print_os_info output
Baesken, Matthias
matthias.baesken at sap.com
Thu Nov 28 09:02:01 UTC 2019
Hi Lutz, I adjusted the days calculation, new webrev :
http://cr.openjdk.java.net/~mbaesken/webrevs/8234397.2/
Best regards, Matthias
> Matthias,
>
> your change looks good to me overall. Please note: I'm not a Reviewer!
>
> I feel the urge to complain about one thing, though:
> When calculating the uptime in days, you divide the time retrieved from the
> system (usually seconds or milliseconds) by a large number. Why do you
> force that number to be a float? I would prefer the denominator to be an
> "int" value.
>
> Rationale: floats (32bits) are very limited in precision, only y6 to 7 decimal
> digits. At least in the windows case, where you obtain milliseconds from the
> system, your denominator is 86,400,000. At first glance, that does not fit into
> a float mantissa. What saves you here are the prime factors "2" (10 in total).
> As a result, you only need 17 mantissa bits to represent the denominator.
>
> Thanks,
> Lutz
>
> On 25.11.19, 09:06, "hotspot-dev on behalf of Baesken, Matthias" <hotspot-
> dev-bounces at openjdk.java.net on behalf of matthias.baesken at sap.com>
> wrote:
>
> >
> > The comment in the posix code mentions that it doesn't work on macOS
> but
> > doesn't say anything about Linux. Has it been tested on Solaris?
> >
>
> Hi David, it works on Solaris .
> I think I should adjust the comment (saying macOS AND Linux) .
>
> Best regards, Matthias
>
>
> > >
> > > One example that occurred last week - my colleague Christoph and me
> > were browsing through an hs_err file of a crash on AIX .
> > > When looking into the hs_err we wanted to know the uptime because
> > our latest fontconfig - patches (for getting rid of the crash) needed a
> > reboot too to really work .
> > > Unfortunately we could not find the info , and we were disappointed
> (
> > then we noticed the crash is from OpenJDK and not our internal JVM ).
> > >
> > >
> > >>> Bug/webrev :
> > >>> https://bugs.openjdk.java.net/browse/JDK-8234397
> > >>> http://cr.openjdk.java.net/~mbaesken/webrevs/8234397.1/
> > >>
> > >> Can Linux not use the POSIX version?
> > >>
> > >
> > > Unfortunately the posix code does not give the desired result on Linux
> (at
> > least on my test machines).
> >
> > The comment in the posix code mentions that it doesn't work on macOS
> but
> > doesn't say anything about Linux. Has it been tested on Solaris?
> >
> > I'm really unsure about this code and am hoping someone more
> > knowledgeable in this areas can chime in. I'd be less concerned if there
> > was a single POSIX implementation that worked everywhere. :( Though I
> > have my general concern about adding yet another potential point of
> > failure in the error reporting logic.
> >
> > Thanks,
> > David
> >
> > > Best regards, Matthias
> > >
>
More information about the hotspot-dev
mailing list