Changeset 246809 in webkit

Timestamp:

Jun 25, 2019 2:35:03 PM (5 years ago)

Author:

abarth@webkit.org

Message:

[fuchsia] Update to newer zx_clock_get syscall
​https://bugs.webkit.org/show_bug.cgi?id=199191

Reviewed by Sam Weinig.

Fuchsia has changed the zx_clock_get syscall to return the clock value
via an out parameter rather than via its return value. This change
makes zx_clock_get consistent with all the other syscalls.

This patch updates our use of zx_clock_get to use the new syscall. The
old syscall is no longer supported by Fuchsia.

• wtf/fuchsia/CPUTimeFuchsia.cpp:

(WTF::CPUTime::get): Switch to using the out parameter.
(WTF::CPUTime::forCurrentThread): Switch to using the out parameter.

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/fuchsia/CPUTimeFuchsia.cpp (modified) (2 diffs)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2019-06-25  Adam Barth  <abarth@webkit.org>
+
+        [fuchsia] Update to newer zx_clock_get syscall
+        https://bugs.webkit.org/show_bug.cgi?id=199191
+
+        Reviewed by Sam Weinig.
+
+        Fuchsia has changed the zx_clock_get syscall to return the clock value
+        via an out parameter rather than via its return value. This change
+        makes zx_clock_get consistent with all the other syscalls.
+
+        This patch updates our use of zx_clock_get to use the new syscall. The
+        old syscall is no longer supported by Fuchsia.
+
+        * wtf/fuchsia/CPUTimeFuchsia.cpp:
+        (WTF::CPUTime::get): Switch to using the out parameter.
+        (WTF::CPUTime::forCurrentThread): Switch to using the out parameter.
+
 2019-06-25  Michael Catanzaro  <mcatanzaro@igalia.com>
 

trunk/Source/WTF/wtf/fuchsia/CPUTimeFuchsia.cpp

@@ -42 +42 @@
     // times atomically and being able to separate ZX_CLOCK_THREAD into user and
     // kernel time.
-    zx_time_t thread = zx_clock_get(ZX_CLOCK_THREAD);
+    zx_time_t thread = 0;
+    zx_clock_get(ZX_CLOCK_THREAD, &thread);
 
     return CPUTime { MonotonicTime::now(), timeToSeconds(thread), Seconds() };
@@ -49 +50 @@
 Seconds CPUTime::forCurrentThread()
 {
-    return timeToSeconds(zx_clock_get(ZX_CLOCK_THREAD));
+    zx_time_t thread = 0;
+    zx_clock_get(ZX_CLOCK_THREAD, &thread)
+    return timeToSeconds(thread);
 }