add sbc_primitives_mmx.c for x86

reconcile main tree with open-source eclair

reconcile android-2.1_r1 snapshot

android-2.1_r1 snapshot

Use monotonically increasing clock for A2DP timing.

This is to avoid skips when the system time changes.

Stop trying to send A2DP packets quickly to catch-up on missed write()'s after
falling behind by 200ms, or on poll() timeout.

This allows us to recover more quickly from a situation where an A2DP headset
rejects A2DP packets for a short time. We now reset to regular packet metering
faster.

Changes flush timeout from 60 ms to 120 ms.

Less A2DP drop-outs due to unnecessary flushing.

BCM recommend 120-150ms for lazy flushing.

Fix liba2dp.c compiler warnings.

Set A2DP L2CAP socket to flushable while streaming A2DP, and set the automatic
flush timeout.

Remove the (ahead < 0) logic that was designed to skip forwards when streaming
was lagging. This logic was not working properly, and ACL flushing does its
job.

Add L2CAP_LM_FLUSHABLE to Bluez header.

Add ACL flow control state (MTU, packets available, packets total) for debug.

Accessed with the debug tool 'hcitool con'

eclair snapshot

Send SBC frames if output buffer is nearly full.

This is to address an A2DP issue seen with the Mazda carkit, where A2DP audio
cannot be played. The MTU size (8200) requested by this carkit is much greater
than the output buffer size (2048) allocated by the Bluez A2DP profile. The
output buffer being full is giving an SBC encode error. This patch checks for
the buffer full condition.

Change-Id: Idded504d085c760baef15cd3cef84fcf57165e4f
Signed-off-by: Yu Kang Ku <kang.ku@motorola.com>
Signed-off-by: Nick Pelly <npelly@google.com>

Add support for sending Broadcom coexistance setup commands for A2DP links.

Enabled on platforms with BOARD_HAVE_BLUETOOTH_BCM.

Change-Id: Ic223955ac3580e5a9430f96e069f686a7afdbef8

Fix handling of SDP records.

Delete SDP records when device is deleted.
Update cache whenever SDP is performed.
Remove SDP records even when there is no driver associated with the UUID.

Bug: 2173752
Dr No: Eastham

do not merge - Fix for issue 2184627 cherry picked from eclair-mr2

Bluetooth A2DP suspend-resume improvements.

This change will reduce the occurence rate of A2DP sink suspend resume failures observed in issues 2184627, 2181005 and possibly 2189628.

Avoid lockups in case of BT device disconnection during the A2DP start process by using a timeout when reading from bluetooth command socket.
Correct a typo causing potential deadlock in wait_for_start().

Add DBUS interfaces AddRfcommServiceRecord() and RemoveServiceRecord().

org.bluez.Adapter.AddRfcommServiceRecord takes a name, uuid and channel
and creates a record like:

Service Name: MyApplication
Service RecHandle: 0x10009
Service Class ID List:
  UUID 128: 42999bc0-ac5e-11de-8a39-0800200c9a66
Protocol Descriptor List:
  "L2CAP" (0x0100)
  "RFCOMM" (0x0003)
    Channel: 26

It returns the integer Record Handle, that can be used with
org.bluez.Adapter.RemoveServiceRecord to remove that record.

Bug: 2158900
DrNo: eastham

Change-Id: Ie375d5ccd24b5faf067a26b8a04a1449b3f56ce2

Remove redundant call to bluetooth_init().

This is a cleanup to Ian Kent's preceeding patchset.

Change-Id: I8ff00530fdd337873ff898b7fd9009b0fc13f0b4

Add Bluetooth A2DP suspend/resume to DBUS API.

Change-Id: Icef1729957ed55e491e63d74d14a2c0c93984c4d

bluez a2dp - use a sensible write timeout

The 500 msec write timeout is extremely optimistic, a 1000 msec
timeout is closer to reality.

bluez a2dp - fix wait_for_start() spurious wakeup

When waiting on a condition is is possble to receive spurious
wake ups. Deal with this in wait_for_start().

Change-Id: I678e9d7831333d9b8baa31e9b0ec2597ca9263cd

bluez a2dp - fix wait_for_start() synchronization

On entry to wait_for_start() it's possible we are in state
A2DP_STATE_NONE if a2dp_thread() has not completed bluetooth_init().
Also, we call the pthread_mutex_*() functions more frequently than
we really need to.

Move the mutex locking outside the loop and let condition wait
take care of the locking. Also move the A2DP_STATE_NONE check
below the condition wait so we can be sure the state machine
has been poked before we test it.

Change-Id: I020bea365a623e88cb1a5f7e5fccd8f8aa948518

bluez a2dp - destroy thread attr after thread create

Pthread resources may be used by the thread attributes object.
We should destroy this after thread creation.

bluez a2dp - destroy thread resources at exit

The dynamically created pthread mutex and condition variables
consume pthread resources. They should be destroyed prior to
freeing the structure containing them.

bluez a2dp - fix a2dp_thread() spurious wakeup

When waiting on a condition is is possble to receive spurious
wake ups. Use the fact that when we poke the state machine the
command must have changed to deal with this.

Change-Id: I0ed2f2a15c15dea30cb1d5cf3b67c8549d6802d4

bluez a2dp - fix state machine synchronization

Using a worker thread implementation for A2DP means that if we don't
have some way to syncroniize state transitions we can get overlap of
requested command functions or lost signals. In order for the A2DP
state machine to function each command operation must complete before
the next is initiated or we will get errors caused by these out of
sequence commands. And if we signal the thread when it is not waiting
on the condition the signal will be missed.

This could be partly resolved by implementing a queue but then there
is an overhead with also implementing a wait mechamism for state
transitions. It's much easier and simpler to just hold the mutex
during command processing which also deals with the lost signals issue.
This may well not achieve the original goal of the worker thread
implementation but neither would a queue implementation. It looks
like this is just the way things are!

bluez a2dp - syncronize worker thread start

There is an order of execution assumption made when starting the
a2dp worker thread. Add an additional condition variable to allow
us to make sure we have reached the thread_wait condition wait
before returning from a2dp_init().

Rename audio_device_is_connected more appropriately

Fix AVDTP Abort when transport closes before response

Change-Id: I5d2bced52514eb14fd8236a9262ae5c71e68f524

Fix agent_free call due to the picking up of upmerge fixes.

Change-Id: Ia0a12ec45dd498572e4a4f2354d383ba5ea436ce

Fix AVDTP suspend while closing

We should reject any suspend request if we've already started the close
procedure for a stream.

Fix command timeout handling when there's no stream

The media stream could be gone by the time that the AVDTP command timeout
expires (e.g. if the media transport channel gets closed prematurely). In
such a case there's no need to send a separate ABORT command but we can
directly proceed with closing the signalling channel.

Fix AVDTP ABORTING state transition

The transition to ABORTING state when acting as initiator should happen
when we receive the ABORT reply and not when we send the initial ABORT
command. The reply handling is already correct but since there's a state
change also in the sending part the later state change request would fail.
This patch fixes the issue by removing the state change when sending the
ABORT command.

Change-Id: I9cf9f27924b7d348523125ddccff7c8ecebacf93

Fix stream handling when the media transport disconnects prematurely

We might not have a stream anymore when receiving the reply to a AVDTP
command that assumes that there is an existing stream. So ignore the
replies to these commands if we don't have a stream anymore.

Fixed duplicate attempt from device to connect 'signalling' channel

A device may attempt to initiate a duplicate avdtp channel while
a host-initiated channel connection is still pending. This
situation is referred to as 'XCASE connect:connect' in sink.c and
source.c. This fix rejects the device-initiated connection in this
circumstance.

Fix agent destroy callback handling for errors

The destroy callback should not be called if calling the agent fails and
an error is returned to the caller of the agent.c public function.

Fix agent_authorize error return

Fix audio authorization cancelation on the same mainloop iteration

If the device is trusted and the authorization is canceled before the idle
callback takes place the idle callback should be removed. This patch adds
tracking of the idle id and removes it in the case of cancelation.

Change-Id: I7a22176bb9c881349cde0730e196378c94665ecb

Make dev->authorized private to audio/device.c

There's no need to expose the authorized setting outside of device.c.

Fix AVDTP stream leak for invalid media transport config

Fix race condition in authorizing audio connections

The current check for implicit authorization is if any other audio profile
is already connected. However, a the second profile might try to connect
right after we've gotten a positive authorization reply for the first one
but before the first profile has reached "connected" state. So just by
checking the connected state of other profiles we might get a false
negative for the decision of doing implicit authorization.

This patch adds a variable to the audio_device struct for keeping track of
if we've gotten a positive authorization reply that can be used in
addition to the "any other profiles" connected check. This variable gets
cleared when the (global audio) device state goes back to disconnected.

Fix a few more ref-leaks with AVDTP failures and unix socket clients

If any of the IPC messages cause us to create a brand new AVDTP session
and then there's failure caused by disconnection, we have to unref the
session since there will not be a unix.c callback to do it when avdtp.c
wants to clean up in the next mainloop iteration.

Fix potential AVDTP reference leak

If the AVDTP connection has just recently disconnected but not yet been
notified to the mainloop then we need to clean the unix.c avdtp session
reference within start_discovery() instead of doing it when the unix
client disconnects. This is because when the AVDTP disconnection finally
gets reported in the next mainloop iteration the unix.c reference wouldn't
get cleared as is intended (since unix.c doesn't have any callbacks at
this point).

Fix authorization when the audio profiles are slow to connect

Incoming audio connections should be automatically authorized if either
HFP/HSP or A2DP is already connected for the same device.

Reject AVDTP_DISCOVER when there are no SEPs available

Reject AVDTP START/STOP when disconnecting

Add new Agent API : RequestPairingConsent.

This callback will be called for incoming pairing requests for 2.1
devices only when the remote IO capabilities are NoInputNoOutput
or DisplayOnly and the local IO capability is DisplayYesNo.
Currently, we were silently auto accepting.

Do not advertise Voice Recognition support in HF AG SDP record.

Change UART speed before Bluetooth firmware download.

Don't bail out if Discovery is in progress when getting the rfcomm channel.

We should be having the SDP record in the cache. In the very rare
case, that the remote device's SDP records have changed and if a discovery
is in progress, we would have already updated the cache or if we have not
queried the device yet, we are going to fail anyways.

Wait 5 milliseconds before doing Simple Pairing auto-accept

The BITE tester is again really stupid and it can't handle dongles
that can do the DH key exchange quickly. For example the Broadcom
dongles use a fast processors and an optimized DH algorithm. So in
the end they are way too fast.

To make this work, use a 5 milliseconds delay before replying to
the auto-accept case of Simple Pairing.

Change version number to 4.47.

Don't advertise AVRCP Controller support in SDP records.

We are a AVRCP 1.0 target, not a controller.

This is identical to the change we applied to Bluez 3.x

Don't enable the SCO server socket when not necessary

The SCO server socket shouldn't be enabled when neither Gateway nor
Headset support is enabled.

Upgrade to Bluez 4.47.

Modify PBAP SDP record.

Fix to pass PTS case TC_SDAS_BV_03_I(since PTS3.2 update2).
The code tried to append the UINT type value to the supported
respositories.  However, the wrong function sdp_seq_alloc()
was used. This function adds 2 redundant bytes (SDP_SEQ8 as
0x35, seq) ahead of 0x08, so that latest PTS IOPT will fail.
According to spec, PBAP attributes should report 0x08 0x01
or 0x08 0x03 right after 0x0314. The correct function to use
is sdp_data_alloc(), with clean 0x08 0x01 appended as
SDP_ATTR_SUPPORTED_REPOSITORIES.

Release 4.47

Update library version

Fix calling set_limited_discoverable twice

set_mode doesn't need to call set_limited_discoverable since
adapter_mode_changed will do it.

Use limited discoverable mode only when 0 < discov_timeout <= 60

Cache service class updates during bootup

adapter->svc_cache is used to cache service class updates during
bluetoothd bringup or when adapter is down.adapter_disable_svc_cache() is
used to switch off the cache, so any successive service class updates are
directly written to the device.

Update audio-api.txt including AudioSource documentation.

Remove deprecated method and signals for AudioSource.

When RFKILL is unblocked restore the previous power setting

Add functions for RFKILL and offline mode

This patch adds a couple of helper functions that are needed to cleanly
implement RFKILL and offline mode support.

Fix ipctest and its strict-aliasing problems

Add skeleton for RFKILL support

Fix missing includes from header files

Fix serial proxy record info attributes

Use 16/32-bit UUID if possible in serial proxy records

Track sender of CreateProxy.

Only proxies created by serial.conf should be persistent.

Remove the serial storage functionality

This functionality isn't needed anymore now that we have serial.conf.
Proxies created through the D-Bus interface will always be temporary not
remembered after a bluetoothd restart.

Fix whitespace

Add support for multiple serial proxies configuration

Fix another possible array out of bounds.

Note also that the pppd_opts should be NULL terminated thus there is only
place for 39 options rather 40.

Remove the service_classes_callback mechanism

Fix bug which could cause buffer overruns.

Fix possible array out of bounds.

Fix segfault when a record is updated.

Current code is freeing the new record when it shouldn't as the memory is
reused by extract_pdu_server.

Fix bogus error comparison

There's no "error" variable here so the comparison was always with the
address of the public error function, which will always be != NULL.

Release 4.46

Update library version

Fix possible use of a pointer after free.

Fix possible memory leak.

Fix possible leak or crash on gen_dataseq_pdu in case of out of memory.

Fix unintended fall through.

Cleanup gateway code to make Coverty happy.

Fix unhandled return.

Remove useless instruction.

Fix strict-aliasing rules handling for hciemu and IP addresses

Fix strict-aliasing rules handling of l2test and info requests

Use -fno-strict-aliasing for audio subdirectory for now

Fix another strict-aliasing rules handling with CMSG_DATA

Fix strict-aliasing rules handling with CMSG_DATA

sbc: Enable neon optimizations.

Signed-off-by: Mike Lockwood <lockwood@android.com>

Fix variable typo

Add option to disable optimization independent from debug

disable neon extras until they build properly

Fix ipctest reading socket forever.

Add Sink role support to audio IPC.