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

Issue 763504 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Tune Chameleon Bluetooth Kit Detection Polling

Project Member Reported by alent@google.com, Sep 8 2017

Issue description

As part of adding Bluetooth kit detection to Chameleon, every Bluetooth kit waits a few seconds for detection. When we added more kits, the first detection after enabling the driver would fail with a 2 sec timeout (1 sec interval), so we're bumping this to 5 sec (1 sec interval).

This may make Chameleon startup slower because it will always wait ~10 sec for Bluetooth kits, so let's track tuning this value as a bug.

Ideas:
- Reduce interval
- Tweak timeout to 2-3 seconds
- Only wait longer after the first time we enable the driver, instead of universally bumping the timeout.

See: https://chromium-review.googlesource.com/c/chromiumos/platform/chameleon/+/636218
 
Increasing the chameleon startup time to 10 seconds shouldn't be an issue from the test side. Let's try with this value for now and if we can go lower, great, otherwise it's not too big of a deal if it makes the bluetooth kits more stable.

Comment 2 by alent@google.com, Sep 11 2017

Owner: alent@google.com
Without enabling the driver, does the Bluetooth kit show up in lsusb?

Comment 4 by alent@google.com, Sep 12 2017

As discussed in person, it does not, and it takes approximately 2 seconds (which was likely less before we added more dongles, hence the bug appearing) after enabling the driver for everything to be ready. We might go as low as 3 seconds, and we could lower the interval, which might make detect faster, but for future-proofing as we add more dongles, I'd suggest a lower interval if possible and keeping the timeout at 5 seconds.

We might also want to check if the driver is already ready, in which case no polling is necessary.

Comment 5 by alent@google.com, Sep 15 2017

Owner: josephsih@chromium.org
Status: Assigned (was: Untriaged)

Sign in to add a comment