Use steady clock for all millis bench logging #25499

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2206-steady-🏑 changing 8 files +18 −20
  1. MarcoFalke commented at 7:48 am on June 29, 2022: member

    Currently GetTimeMillis is used for bench logging in milliseconds integral precision. Replace it to use a steady clock that is type-safe and steady.

    Microsecond or float precision can be done in a follow-up.

  2. DrahtBot added the label Refactoring on Jun 29, 2022
  3. DrahtBot commented at 11:56 am on June 29, 2022: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #25830 (refactor: Replace m_params with chainman.GetParams() by aureleoules)
    • #25665 (refactor: Add util::Result failure values, multiple error and warning messages by ryanofsky)
    • #25284 ([WIP] consensus: Remove dependency on net (BIP 155 / ADDRV2_FORMAT) by MarcoFalke)
    • #18904 (Don’t call lsn_reset in periodic flush by bvbfan)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. MarcoFalke force-pushed on Jul 19, 2022
  5. DrahtBot added the label Needs rebase on Jul 20, 2022
  6. MarcoFalke force-pushed on Jul 20, 2022
  7. DrahtBot removed the label Needs rebase on Jul 20, 2022
  8. DrahtBot added the label Needs rebase on Jul 29, 2022
  9. Use steady clock for all millis bench logging fa521c9603
  10. MarcoFalke force-pushed on Jul 30, 2022
  11. DrahtBot removed the label Needs rebase on Jul 30, 2022
  12. fanquake approved
  13. fanquake commented at 9:26 am on September 16, 2022: member
    ACK fa521c960337a65d4ce12cd1ef009c652ffe57e6 - started making the same change.
  14. fanquake merged this on Sep 16, 2022
  15. fanquake closed this on Sep 16, 2022

  16. MarcoFalke deleted the branch on Sep 16, 2022
  17. sidhujag referenced this in commit 335124cf84 on Sep 20, 2022
  18. bitcoin locked this on Sep 16, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-07-03 13:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me