Skip to content

add caml_get_wall_clock -- similar to caml_get_monotonic_time#31

Merged
hannesm merged 2 commits into
mirage:masterfrom
hannesm:clock
Jan 11, 2021
Merged

add caml_get_wall_clock -- similar to caml_get_monotonic_time#31
hannesm merged 2 commits into
mirage:masterfrom
hannesm:clock

Conversation

@hannesm
Copy link
Copy Markdown
Member

@hannesm hannesm commented Nov 25, 2020

The motivation is to avoid lots of conversions in mirage-clock just to retrieve
the current wall clock time. This is one part to solve
mirage/mirage-clock#23

The unix_gettimeofday is still useful for the OCaml runtime system - which uses
it to initialize its RNG (the alternative is it being initialized with 0s which
is obviously worse) - that's why that symbol is kept as well.

The motivation is to avoid lots of conversions in mirage-clock just to retrieve
the current wall clock time. This is one part to solve
mirage/mirage-clock#23

The unix_gettimeofday is still useful for the OCaml runtime system - which uses
it to initialize its RNG (the alternative is it being initialized with 0s which
is obviously worse) - that's why that symbol is kept as well.
Copy link
Copy Markdown
Contributor

@mato mato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This prepares the integration of OCaml-CI which executes the above for linting.
@hannesm hannesm merged commit a079a3c into mirage:master Jan 11, 2021
@hannesm hannesm deleted the clock branch January 11, 2021 12:56
hannesm added a commit to hannesm/opam-repository that referenced this pull request Nov 18, 2021
CHANGES:

* Fix OCaml documentation strings (mirage/mirage-xen#30 @hannesm)
* Add caml_get_wall_clock to retrieve the wall clock as int64 (mirage/mirage-xen#31 @hannesm)
* Don't require opam for the bindings compilation (mirage/mirage-xen#32 @sternenseemann)
* Remove checksum_stubs and alloc_pages_stubs (available in mirage-tcpip
  and io-page), to keep in sync with mirage-solo5 (mirage/mirage-xen#33 @dinosaure, adapted
  by @hannesm to mirage-xen)
arbipher pushed a commit to arbipher/opam-repository that referenced this pull request Nov 21, 2021
CHANGES:

* Fix OCaml documentation strings (mirage/mirage-xen#30 @hannesm)
* Add caml_get_wall_clock to retrieve the wall clock as int64 (mirage/mirage-xen#31 @hannesm)
* Don't require opam for the bindings compilation (mirage/mirage-xen#32 @sternenseemann)
* Remove checksum_stubs and alloc_pages_stubs (available in mirage-tcpip
  and io-page), to keep in sync with mirage-solo5 (mirage/mirage-xen#33 @dinosaure, adapted
  by @hannesm to mirage-xen)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants