Skip to content

Commit

Permalink
Timestamp: Fix comments from rakuco's first review
Browse files Browse the repository at this point in the history
Signed-off-by: Arnaud Mandy <arnaud.mandy@intel.com>
  • Loading branch information
arskama committed Jun 7, 2024
1 parent 99912ab commit a54b8ca
Showing 1 changed file with 10 additions and 5 deletions.
15 changes: 10 additions & 5 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -1108,16 +1108,21 @@ <h3>Data delivery</h3>
<li>
Let |state:PressureState| be an [=adjusted pressure state=] given |data| and |source|.
</li>
<li>
Let |timestamp| be an platform-specific timestamp converted in an [=implementation-defined=]
way to an [=monotonic clock/unsafe current time=] using the same [=monotonic clock=]
that is shared by [=environment settings object/time origin=].
<aside class="note">
The |data| sample and mapping between |data| sample, and [=pressure states=],
is [=implementation-defined=] and may use many different metrics. For instance,
for CPU, it might consider processor frequency and utilization, as well
as thermal conditions.
</aside>
<li>
Let |timestamp| be an platform-specific timestamp converted in an [=implementation-defined=]
way to an [=monotonic clock/unsafe current time=] using the same [=monotonic clock=]
that is shared by [=environment settings object/time origin=].
<aside class="note">
The goal of this step is to ensure that a timestamp that may have been relative to
a different time origin is converted to a value that can be used in computations with
the same [=monotonic clock=] used by the operations described in [[HR-TIME]].
</aside>
</li>
<li>
[=list/For each=] |observer:PressureObserver| in |relevantGlobal|'s
Expand All @@ -1140,7 +1145,7 @@ <h3>Data delivery</h3>
{{PressureRecord/[[Source]]}} set to |source|,
{{PressureRecord/[[State]]}} set to |state|
and {{PressureRecord/[[Time]]}} set to [=relative high resolution time=] based on |timestamp|
and a global.
and |relevantGlobal|.
</li>
<li>
If |observer|.{{PressureObserver/[[AfterPenaltyRecordMap]]}}[source] [=map/exists=]:
Expand Down

0 comments on commit a54b8ca

Please sign in to comment.