New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 726484 link

Starred by 3 users

Issue metadata

Status: ExternalDependency
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Fuchsia
Pri: 3
Type: Bug

Blocking:
issue 706592



Sign in to add a comment

No way to get process creation time on Fuchsia

Project Member Reported by scottmg@chromium.org, May 25 2017

Issue description

sys_get_time(MX_CLOCK_THREAD) seems to be the closest, but that's for the thread, not the process, and more importantly, it's only scheduled time, not time-since-creation (so isn't really useful to subtract from the current time).

This is only lightly used, but is in some tracing, metrics, etc. type usages.


 
Upstream MG-800.
Project Member

Comment 2 by bugdroid1@chromium.org, May 25 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e911e117e1312c5d19a9e61b10e68c6b26b36832

commit e911e117e1312c5d19a9e61b10e68c6b26b36832
Author: scottmg <scottmg@chromium.org>
Date: Thu May 25 23:55:06 2017

fuchsia: Don't reference CurrentProcessInfo::CreationTime()

(I thought maybe I could thread time and subtract it from current time,
but thread time is only how long it's been scheduled for, not total wall
time, so it's not really that close:
https://codereview.chromium.org/2909513003/ ).

BUG=706592,726484

Review-Url: https://codereview.chromium.org/2903323002
Cr-Commit-Position: refs/heads/master@{#474846}

[modify] https://crrev.com/e911e117e1312c5d19a9e61b10e68c6b26b36832/base/process/process_info_unittest.cc
[modify] https://crrev.com/e911e117e1312c5d19a9e61b10e68c6b26b36832/base/trace_event/trace_log.cc

Comment 3 by w...@chromium.org, Jul 15 2017

Components: Internals>PlatformIntegration
Cc: w...@chromium.org
 Issue 881333  has been merged into this issue.
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 11

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5f48b80b814b7463b25e94cc352d1e5591b5a3e5

commit 5f48b80b814b7463b25e94cc352d1e5591b5a3e5
Author: Wez <wez@chromium.org>
Date: Tue Sep 11 00:22:49 2018

Add stub base::CurrentProcessInfo::CreationTime() implementation.

Bug: 726484,  881333 
Change-Id: I9b779a087d20dda8f3f0a58ffb70fbbea1e6e601
Reviewed-on: https://chromium-review.googlesource.com/1210382
Reviewed-by: Albert J. Wong <ajwong@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590120}
[modify] https://crrev.com/5f48b80b814b7463b25e94cc352d1e5591b5a3e5/base/BUILD.gn
[add] https://crrev.com/5f48b80b814b7463b25e94cc352d1e5591b5a3e5/base/process/process_info_fuchsia.cc

Sign in to add a comment