Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45059 )
Change subject: soc/amd/picasso: pass verstage timestamps to x86
......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45059/3/src/soc/amd/picasso/bootblo...
File src/soc/amd/picasso/bootblock/bootblock.c:
https://review.coreboot.org/c/coreboot/+/45059/3/src/soc/amd/picasso/bootblo...
PS3, Line 146: tse->entry_stamp = abs_stamp * tick_freq_mhz;
If we use COLLECT_TIMESTAMPS_NO_TSC we need to supply monotonic timer instead of TSC. […]
That or use tsc as the implementation for monotonic timer. One still needs to figure out the offset issue (when the TSC actually started, if using TSC) and carry it forward through the rest of the stages so the timestamps make sense.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/45059
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5e89bb54f478153fb40ba51b5ab61fa20af3b99a
Gerrit-Change-Number: 45059
Gerrit-PatchSet: 5
Gerrit-Owner: Kangheui Won
khwon@chromium.org
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Tue, 22 Sep 2020 19:53:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Kangheui Won
khwon@chromium.org
Comment-In-Reply-To: Aaron Durbin
adurbin@chromium.org
Gerrit-MessageType: comment