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

Issue 832318 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jul 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Volume latency when having slow sink

Project Member Reported by sonnysasaka@chromium.org, Apr 12 2018

Issue description

If the sink has high latency AVRCP processing, volume up/down at high rate results in lag and volume level skipping around.
 

Comment 1 by r...@chromium.org, Apr 12 2018

Components: -IO>Bluetooth OS>Systems>Bluetooth
Description: Show this description
Description: Show this description
Project Member

Comment 4 by bugdroid1@chromium.org, May 4 2018

Labels: merge-merged-chromeos-5.44
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/bluez/+/a9eb09caec039254824b0cba3042995c16d205c8

commit a9eb09caec039254824b0cba3042995c16d205c8
Author: Sonny Sasaka <sonnysasaka@chromium.org>
Date: Fri May 04 10:02:18 2018

CHROMIUM: Queue SetAbsoluteVolume if there is an in-progress one.

SetAbsoluteVolume command may receive late response for Target Device
that have high latency processing. In that case we may send the next
SetAbsoluteVolume commands before the previous SetAbsoluteVolume
response is received. This causes the media transport volume to jitter.

The solution in this patch is to not send any SetAbsoluteVolume command
if there is an in-progress one. Instead we should queue this command to
be executed after the in-progress one receives the response.

BUG= chromium:832318 ,b:74008707
TEST=Tested with simulated high latency AVRCP

Change-Id: I8f79bc08fa74d51070e1f85f20f6e13286118015
Reviewed-on: https://chromium-review.googlesource.com/1013382
Commit-Ready: Sonny Sasaka <sonnysasaka@chromium.org>
Tested-by: Sonny Sasaka <sonnysasaka@chromium.org>
Reviewed-by: Miao-chen Chou <mcchou@chromium.org>

[modify] https://crrev.com/a9eb09caec039254824b0cba3042995c16d205c8/profiles/audio/avrcp.c

Status: Fixed (was: Started)
Status: Verified (was: Fixed)

Sign in to add a comment