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

Issue 661266 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

how much boot speed can we gain by changing usleep_range(a, b) to usleep_range(a, a)?

Project Member Reported by diand...@chromium.org, Nov 1 2016

Issue description

The usleep_range() function takes a min, max parameter and it's supposed to sleep for between min and max microseconds.

In practice, it usually sleeps for "max" microseconds.  This is for power optimization purposes so that CPUs need to wake up less often (the idea is that if it sleeps for longer there's a better chance that another wakeup will happen and wake us up anyway).

At boot time we really care about speed, not about power consumption.  Also: there are a lot of delays (some using usleep_range()) at bootup.  In the past, we saved hundreds of milliseconds by tuning usleep_range() in dwc2, though that was probably an extreme case.


It would be interesting to see how much boot speed would improve if we changed usleep_range() to always use the minimum during bootup (maybe stop doing it at late_initcall, like clk_disable_unused?).

Note: with async probe the benefit might be less than it was historically, but it still might be interesting?

Hrmmm.  Maybe not useful on kevin, though.  Doing some quick checking w/ printouts shows a maximum of 6 ms gained on my kevin.  :(  I guess there's nobody doing things as stupid as:

  usleep_range(100000, 150000);



 
Project Member

Comment 1 by sheriffbot@chromium.org, Nov 2 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: WontFix (was: Untriaged)

Sign in to add a comment