Page MenuHomeFreeBSD

ping6: Replace gettimeofday() with clock_gettime(CLOCK_MONOTONIC)
ClosedPublic

Authored by jansucan on Aug 12 2019, 11:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 2, 12:32 AM
Unknown Object (File)
Sun, Jun 29, 3:01 PM
Unknown Object (File)
Sat, Jun 28, 10:54 PM
Unknown Object (File)
Sat, Jun 28, 3:08 PM
Unknown Object (File)
Thu, Jun 26, 7:48 PM
Unknown Object (File)
Thu, Jun 26, 5:52 AM
Unknown Object (File)
Thu, Jun 26, 5:24 AM
Unknown Object (File)
Thu, Jun 26, 5:20 AM
Subscribers

Details

Summary

Replace gettimeofday() with clock_gettime(CLOCK_MONOTONIC)

Submitted by: Ján Sučan <sucanjan@gmail.com>
Sponsored by: Google, inc. (Google Summer of Code 2019)

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 25801
Build 24373: arc lint + arc unit

Event Timeline

asomers requested changes to this revision.Aug 12 2019, 7:34 PM

Thanks for fixing this. This will be a good change.

sbin/ping6/ping6.c
2218

You should eliminate this function and use timespecsub(3) instead.

This revision now requires changes to proceed.Aug 12 2019, 7:34 PM

Use timespecsub(3) instead of a private function.

sbin/ping6/ping6.c
2218

Fixed.

Annotate truncating of time values saved in a packet.

asomers requested changes to this revision.Aug 13 2019, 3:10 PM
asomers added inline comments.
sbin/ping6/ping6.c
1152–1154

These 10 lines can be replaced by timespecadd.

This revision now requires changes to proceed.Aug 13 2019, 3:10 PM

Use timespecadd(), timespecsub(), and timespecclear().

sbin/ping6/ping6.c
1152–1154

Fixed.

asomers added inline comments.
sbin/ping6/ping6.c
1152–1154

Thanks. It's much more readable now.

This revision is now accepted and ready to land.Aug 13 2019, 7:22 PM