am a2340b30: Merge "Don\'t call audio effect process on offloaded playback threads" into klp-dev

* commit 'a2340b3059b9d6e4286d0db7069df477c164df0c':
  Don't call audio effect process on offloaded playback threads

am 0adc67df: Merge "audioflinger: fix crash when starting offload thread" into klp-dev

* commit '0adc67dfc5cedb211c36f06849681a60a32f5805':
  audioflinger: fix crash when starting offload thread

am 1da3b602: Camera2 API: Hack: Always use async mode with GLConsumer

* commit '1da3b602130d71ac3bff1a1fdecdc5e0d7b9d701':
  Camera2 API: Hack: Always use async mode with GLConsumer

Merge "Don't call audio effect process on offloaded playback threads" into klp-dev

Merge "audioflinger: fix crash when starting offload thread" into klp-dev

audioflinger: fix crash when starting offload thread

Passing a sp to parent thread to AsyncCallbackThread() constructor
causes a strong reference to be acquired on the OffloadThread inside
its constructor which causes an early launch of the thread loop
with unpredictable consequences.

Pass a wp to parent thread instead.

Also move the creation of the AsyncCallbackThread to
readOutputParameters() where mUseAsyncWrite is initialized which
makes more sense.

Also change the type of AsyncCallbackThread parent thread to PlaybackThread
instead of OffloadThread to allow a broder use of non blocking write which
in theory is not limited to audio offload use case.

Bug: 8174034.
Change-Id: I4b093b022030cd4f5eb8b8e477333e91098a6549

Camera2 API: Hack: Always use async mode with GLConsumer

Until a cleaner solution is implemented, force buffer queues to
consumers that set USAGE_HW_TEXTURE usage to be asynchronous.

Bug: 10949105
Change-Id: I69e6b02b773831396767c282ce8c5936c5d41f03

Don't call audio effect process on offloaded playback threads

An audio effect process command is not meant to be called for
 playback threads belonging to offloaded playback.

Bug 10933817

Change-Id: Idc7b6a0224791bbf8d61648474d3c288617c498f

am 6914de6d: Merge "Camera: Implement getCameraCharacteristics" into klp-dev

* commit '6914de6dd1d74fcf74e3247c89f9293fbf7bafe5':
  Camera: Implement getCameraCharacteristics

Merge "Camera: Implement getCameraCharacteristics" into klp-dev

Camera: Implement getCameraCharacteristics

Bug: 10904541
Bug: 10360518
Change-Id: Ie9ca6b3b0b5f2fe529e6b0decc193096e770a017

am 1802399c: Merge "fix deadlock in audioflinger::createEffect()" into klp-dev

* commit '1802399c7c4d8557922688f246df292703f417f1':
  fix deadlock in audioflinger::createEffect()

Merge "fix deadlock in audioflinger::createEffect()" into klp-dev

fix deadlock in audioflinger::createEffect()

commit 5baf2af5 introduced a regression by calling
getOutputForEffect() with AudioFLinger main mutex locked.
The locking order must always be AudioPolicyService mutex then
AudioFlinger then ThreadBase mutex.

Bug: 10916796.
Change-Id: Ide34a2d84dbb06dbb35abd0640d91b01b0ac4d40

am 2b890936: Merge "Camera2/3: Support new PASSIVE_UNFOCUSED AF state" into klp-dev

* commit '2b890936e978de0a8d4450ee1aaeda53d22b5f4e':
  Camera2/3: Support new PASSIVE_UNFOCUSED AF state

Merge "Camera2/3: Support new PASSIVE_UNFOCUSED AF state" into klp-dev

am 58dd0786: Merge "Send kWhatConnected in onTimeUpdate() before first access unit" into klp-dev

* commit '58dd07863571951408b67fa0a7f17cb23606fb1c':
  Send kWhatConnected in onTimeUpdate() before first access unit

Merge "Send kWhatConnected in onTimeUpdate() before first access unit" into klp-dev

am 6c63fe0f: am ab70649e: Merge "MediaCodec: Pass the crop rect to SoftwareRenderer"

* commit '6c63fe0fdb33e5e7eb8c89f2420262b072cdd83f':
  MediaCodec: Pass the crop rect to SoftwareRenderer

am ab70649e: Merge "MediaCodec: Pass the crop rect to SoftwareRenderer"

* commit 'ab70649e2b0ea3d917242814417ec5711f4cb811':
  MediaCodec: Pass the crop rect to SoftwareRenderer

Merge "MediaCodec: Pass the crop rect to SoftwareRenderer"

MediaCodec: Pass the crop rect to SoftwareRenderer

Change-Id: I434411516eb5fafe9d7aeac9882bbc7f1a8415d8

am d2ac3144: Merge "Fix log typos in Visualizer effect" into klp-dev

* commit 'd2ac3144e201398340cc5b9bb36e5efe39edd9eb':
  Fix log typos in Visualizer effect

Merge "Fix log typos in Visualizer effect" into klp-dev

am ddfbfaeb: fix oflload effect proxy commmand handling

* commit 'ddfbfaeb00295fff7351711f0f044f17d6c40f3c':
  fix oflload effect proxy commmand handling

Fix log typos in Visualizer effect

Fix errors in logs for Visualizer.
Set loop counters on 32 bits

Bug 8413913

Change-Id: Iad2140d003d15d45be46826a5e89baff14fe9e77

fix oflload effect proxy commmand handling

Implement a more generic command handling in offload effect
proxy. All commands are sent to both sub effects but only the reply
from the active one is returned to the caller.

Bug: 8174034.
Change-Id: Ia45f9933b3bf338257ec70b37732fa1578d26b9f

am 1c7f35d1: Merge "soundpool: allocate shared memory heap by client" into klp-dev

* commit '1c7f35d1f25eb7160314fdef536463fc34deb1ea':
  soundpool: allocate shared memory heap by client

Merge "soundpool: allocate shared memory heap by client" into klp-dev

am 1ecd3dc4: Merge "Camera2/3: Properly disambiguate FPS range vs. single FPS setting" into klp-dev

* commit '1ecd3dc4c779754e9d6603a17b9b3c6b709fb0cc':
  Camera2/3: Properly disambiguate FPS range vs. single FPS setting

Merge "Camera2/3: Properly disambiguate FPS range vs. single FPS setting" into klp-dev

soundpool: allocate shared memory heap by client

Current SoundPool implementation allocates the shared memory heap
containing decoded PCM samples in mediaserver process.
When mediaserver process crashes, the shared memory heaps allocated by
AudioCache cannot be mapped anymore in the new instance of mediaserver.
This causes a silent failure to end playback of new sounds because
AudioFlinger believes the new AudioTracks are opened in streaming mode
and not static mode: it sees a NULL shared memory pointer when the track
is created.

The fix consists in allocating the memory heap in the client process. Thus
the heap is not lost when mediaserver restarts. The global memory usage is
the same as this is shared memory.

Also added a way to detect that a shared memory is passed when the track is
created but cannot be mapped on mediaserver side.

Also fix a crash in SoundPool when ALOGV is enabled.

Bug: 10894793.
Change-Id: Ice6c66ec3b2a409d75dc903a508b6c6fbfb2e8a7

am 664539d2: audioflinger: fix ro.audio.silent in offload again

* commit '664539d25180ab8f77e0521533ea2821cf28985f':
  audioflinger: fix ro.audio.silent in offload again

Send kWhatConnected in onTimeUpdate() before first access unit

Bug: 10642588
Change-Id: If2b4fbbf250d5307e304f31c7aa4ac480e279484

am 402dfba6: Merge "Add support for level measurements in Visualizer" into klp-dev

* commit '402dfba6dcd68f5fd8d8921f9751f3e47eb1449d':
  Add support for level measurements in Visualizer

am 1d2536f4: Merge "AudioTrack: fix music resume" into klp-dev

* commit '1d2536f460d4678770f423f50cbf6a61a13d4d11':
  AudioTrack: fix music resume

am 21e54568: fix incoming call audio start time.

* commit '21e5456821e3c107d09b0629adbf23c9e06e7c0b':
  fix incoming call audio start time.

audioflinger: fix ro.audio.silent in offload again

Commit 1abbdb4 was not working 100% of the times because
it is possible that the offload thread loop never sleeps
after being created in which case the property is never read.

The loop now reads the property once when starting.

Bug: 10899309.
Change-Id: I2e2ca332f2d7086e59d65f6010378c4d9618ba9e

Merge "Add support for level measurements in Visualizer" into klp-dev

Add support for level measurements in Visualizer

New commands to set a measurement mode and perform peak + RMS
 measurements.

Bug 8413913

Change-Id: Ib25254065c79d365ebb34f9dc9caa0490e2d300d

Merge "AudioTrack: fix music resume" into klp-dev

fix incoming call audio start time.

commit cec4abb fixing issue 6720482 had a flaw that
would insert commmands at the wrong position in the
audio policy service command thread queue causing other
commands to be delayed more than expected.

A change in track stop timing introduced by commit bfb1b83
made this issue visible when answering a call.

Bug: 10860012.
Change-Id: Iec769112f14e13d13b64628b808e43663e33771c

AudioTrack: fix music resume

Fix regression introduced by commit 5a6cd22 in AudioTrack resume:
the callback thread was not signaled if paused internaly.

Bug: 10895013.
Change-Id: Ic356b115132d6fccbcee2d9bb855e92671dc20c5

Camera2/3: Support new PASSIVE_UNFOCUSED AF state

Bug: 10860639
Change-Id: I39be64abaf79b49b2d1f74e1c2ab2adf718ae701

Camera2/3: Properly disambiguate FPS range vs. single FPS setting

The camera1 API allows for either setPreviewFrameRate or
setPreviewFPSRange, so both values may or may not change in a single
setParameter call.

The disambiguation of which setting has been changed since the last
setParameter() call was not fully correct, so a sequence of changes
that only changed setPreviewFPSRange or didn't change either could be
interpreted as as a change to setPreviewFrameRate.

Bug: 10842868
Change-Id: I40baeced80a58f09f8a1742ece8dd5e141e9c1e3

am 4b701cc0: Merge "Revert "Workaround slow AudioTrack destruction"" into klp-dev

* commit '4b701cc041d635e5ec56e382043a4c5d01aedd80':
  Revert "Workaround slow AudioTrack destruction"

Merge "Revert "Workaround slow AudioTrack destruction"" into klp-dev

am 3b3cfcfa: Merge "Fix slow AudioTrack and AudioRecord destruction" into klp-dev

* commit '3b3cfcfa272c8e3e16c89765b8817f5a8de0c505':
  Fix slow AudioTrack and AudioRecord destruction

Merge "Fix slow AudioTrack and AudioRecord destruction" into klp-dev

Fix slow AudioTrack and AudioRecord destruction

There were two causes for the slowness:

When thread was paused, it used nanosleep and sleep.  These usually
run to completion (except for POSIX signal, which we avoid because it
is low-level).  Instead, replace the nanosleep and sleep by condition
timed wait, as that can be made to return early by a condition signal.
Another advantage of condition timed wait is that a condition wait was
already being used at top of thread loop, so it is a simpler change.

The AudioRecord destructor was missing a proxy interrupt that was correct
in AudioTrack.  This proxy interrupt is needed in case another thread
is blocked in proxy obtainBuffer.

Does not address the 1 second polling for NS_WHENEVER.

Bug: 10822765
Change-Id: Id665994551e87e4d7da9c7b015f424fd7a0b5560

Revert "Workaround slow AudioTrack destruction"

This reverts commit 8bbbd7da02fac3de40139af19f7cf7a7cc3cc824.

Change-Id: I269a6c445cbce33451b6a9e74223e36e6abbdbe0

am 291a09c3: Merge "Camera2/3: For still captures, use widest FPS range available." into klp-dev

* commit '291a09c3537f2160ec2f1cac16a22212304c2cbf':
  Camera2/3: For still captures, use widest FPS range available.

Merge "Camera2/3: For still captures, use widest FPS range available." into klp-dev

Camera2/3: For still captures, use widest FPS range available.

Instead of using the application-selected preview FPS range for still
captures, select the widest FPS range advertised by the device and use
it for still captures instead, to give the HAL the widest range of exposure
values to choose from possible.

Bug: 10842868
Change-Id: I0e24f5eb713a4f8df116cd5979a84390cc0974d7

am a3b97ad2: Merge "audioflinger: implement getTimestamp() for offloaded tracks" into klp-dev

* commit 'a3b97ad288e6808c54c6275eaac1bd53108d8f11':
  audioflinger: implement getTimestamp() for offloaded tracks

Merge "audioflinger: implement getTimestamp() for offloaded tracks" into klp-dev

audioflinger: implement getTimestamp() for offloaded tracks

Bug: 9587132.
Change-Id: Ie9d5f4cca96306d08bc9a2dbd6edd8953096702d

am fbb2609f: Merge "audioflinger: fix lost offload thread resume event" into klp-dev

* commit 'fbb2609f132f23a983d376dec9ae6bfebfcacc4c':
  audioflinger: fix lost offload thread resume event

Merge "audioflinger: fix lost offload thread resume event" into klp-dev

audioflinger: fix lost offload thread resume event

It was possible that a resume request signaled by addTrack_l()
while waiting for an async write callback is lost. This is because
mSignalPending was not set and waitingAsyncCallback_l() would pause the
thread loop before executing prepareTracks_l().

The fix consists in using signal_l() method to wake the thread
loop o that mSignalPending is set.

Also make sure that sleepTime is reset to 0 when resuming to make sure
that we write any remaining bytes to the HAL.

Bug: 10810347.

Change-Id: If9a3b22cc3b9e6eb384a56c48c40e6258d0896ad

am cfca686a: Merge "Rotate fallback size" into klp-dev

* commit 'cfca686ae8e64bda44a370232f3ba367e6e520d1':
  Rotate fallback size

Merge "Rotate fallback size" into klp-dev

Rotate fallback size

If configuring the encoder for display-size recording fails, we drop
back to 720p.  This was done a bit too literally, and didn't look
good in portrait mode.  Rotate the fallback size as needed.

Bug 10826876

Change-Id: Id3130471fc1467afa8bf91d75c9d4a49245253c3

am b5bca3ff: Merge "camera2: Tell all streams to ignore global device UI rotation" into klp-dev

* commit 'b5bca3ff7784ce95431224482ec42a50261e04b2':
  camera2: Tell all streams to ignore global device UI rotation

Merge "camera2: Tell all streams to ignore global device UI rotation" into klp-dev

am feb6d27b: Merge "Workaround slow AudioTrack destruction" into klp-dev

* commit 'feb6d27bf61cd266cf753215e9cae16b9bc9dbbd':
  Workaround slow AudioTrack destruction

Merge "Workaround slow AudioTrack destruction" into klp-dev

am d1b28d41: audioflniger: fix leaked audiosession references

* commit 'd1b28d41dbda203ffb420ba2e36cbe736b163ff8':
  audioflniger: fix leaked audiosession references

camera2: Tell all streams to ignore global device UI rotation

- Also use android.sensor.orientation to set the right transform
  flags automatically.

Bug: 10804238
Change-Id: I10caf8331f19e107c461696963cc10f597c91d83

audioflniger: fix leaked audiosession references

acquireAudioSessionId() should not create session references
if the caller is not listed in the known notification clients.
It happens when the MediaPlayer or AudioTrack is created by the
mediaserver on behalf of a client (e.g CameraService). In this case
releaseAudioSessionId() can be called from a different pid and
the session reference will not be removed.
Also we will not be able to remove the reference if the client
process dies.

Bug: 10606426.
Change-Id: Ibb5e27518daeb484a2e66302e4c6221cbaca5b70

Workaround slow AudioTrack destruction

Bug: 10809586
Change-Id: I5f30d4deb1233e8ade8967568e40684ef680c395

am 5baf2af5: more support for audio effect offload

* commit '5baf2af52cd186633b7173196c1e4a4cd3435f22':
  more support for audio effect offload

more support for audio effect offload

Offloading of audio effects is now enabled for offloaded
output threads. If an effect not supporting offload is enabled,
the AudioTrack is invalidated so that it can be recreated in PCM
mode.

Fix some issues in effect proxy related to handling of effect
commands to offloaded and non offloaded effects.

Also fixed a bug on capture index in software Visualizer effect.

Bug: 8174034.

Change-Id: Ib23d3c2d5a652361b0aaec7faee09102f2b18fce

am 9a98b6de: Merge "Partial fix for SoundPool not terminating" into klp-dev

* commit '9a98b6de791aeb130192df10744f5b35f8b6ef1a':
  Partial fix for SoundPool not terminating

Merge "Partial fix for SoundPool not terminating" into klp-dev

Partial fix for SoundPool not terminating

SoundPool was waiting for EVENT_UNDERRUN only to indicate end of clip.  In
J, AudioTrack delivered both EVENT_UNDERRUN followed by EVENT_BUFFER_END.
However, as of K, AudioTrack is only delivering EVENT_BUFFER_END (this
lack of EVENT_UNDERRUN is another bug which still needs to be fixed).
The workaround is to also respond to EVENT_BUFFER_END in SoundPool.

Bug: 10787103
Change-Id: Id68a23bddd6dd9df6c49c55138197260d71ca468

am ab6c93b4: Merge "Use changed MediaCodec.PARAMETER_KEY_VIDEO_BITRATE value" into klp-dev

* commit 'ab6c93b4bea96dc4f44a7acfefee1e11a3abda5b':
  Use changed MediaCodec.PARAMETER_KEY_VIDEO_BITRATE value

Merge "Use changed MediaCodec.PARAMETER_KEY_VIDEO_BITRATE value" into klp-dev

Use changed MediaCodec.PARAMETER_KEY_VIDEO_BITRATE value

Change-Id: I157bcafbf705865e66c81517b1eab10c3daa039e
Signed-off-by: Lajos Molnar <lajos@google.com>
Bug: 10461617

am 6cc3a994: LoudnessEnhancer audio effect implementation

* commit '6cc3a9948b51193dfdcb0c3527d7f3d1ca38aa3c':
  LoudnessEnhancer audio effect implementation

LoudnessEnhancer audio effect implementation

Implementation based on DRC effect, controlled by a
 target gain.
 The target gain is used to amplify the signal at
 the input of the DRC, and to compute the knee
 of the DRC.

Bug 8413913

Change-Id: I386d64793a9fa3f7218e053d6f0a99f6836c02bd

am 18f86140: Merge "Fix underruns when fast track denied due to SRC" into klp-dev

* commit '18f861404efc054da0a2ea6c582e293940f63bc8':
  Fix underruns when fast track denied due to SRC

Merge "Fix underruns when fast track denied due to SRC" into klp-dev

am fc270954: am e56f3c96: Merge "strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe."

* commit 'fc270954192ef7e15ac2c88daadd8890d22096e3':
  strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe.

am e56f3c96: Merge "strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe."

* commit 'e56f3c96fa6e7550b67e8b049f999aaa2ada1192':
  strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe.

Merge "strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe."

Fix underruns when fast track denied due to SRC

OpenSL ES requests a fast track.  If sample rate conversion is needed,
the request is denied by server, and a larger client buffer is used
to handle the higher latency of a normal track.  However the client
notification period was calculated based on buffer being divided into
2 sub-buffers.  That resulted in the notification period being too long.
The server pulls chunks that are smaller than half the total buffer.
So now the client uses 3 sub-buffers when there is SRC.

Also removed the 'defer wake' optimization because it was incorrect.
This optimization attempted to reduce the number of wakeups of client,
when server releaseBuffer knows that another releaseBuffer will be
following.  But there is no way for the first releaseBuffer to predict
how soon the second releaseBuffer will occur.  In some cases it was
a long time, and the client underran.  So now the client is woken up
immediately if the total number of available frames to client is >=
the minimum number the client wants to see (the notification period).

Also fix bug where minimum frame count was not being used in the
calculation of notification period.

Bug: 10342804
Change-Id: I3c246f4e7bc3684a344f2cf08268dc082e338e2a

strtok stores its values in thread local storage.
So it can not guarantee works well when multithread environment.
AudioFlinger has multithread.
so strtok_r is more safe.

Change-Id: I6d77ef9cc49a4478dd856dcdca14e4920ce955c6

am bc910e5c: Merge "Fix hang" into klp-dev

* commit 'bc910e5c8d05f24531bd6b11379cf41cc73415ed':
  Fix hang

Merge "Fix hang" into klp-dev

am a099d1f6: Merge "Camera3: Always set a trigger ID for an active trigger" into klp-dev

* commit 'a099d1f6be3fbbb1af4c12b2ab627397f0826982':
  Camera3: Always set a trigger ID for an active trigger

Merge "Camera3: Always set a trigger ID for an active trigger" into klp-dev

Fix hang

Specify that the surface is controlled by the app, to avoid a hang.
b/10531761

Change-Id: Idccc2c73aa3d368d8e7fbdc071ce36e2382efea4

am 72a2f171: Merge "Camera: setCallBackWindow only if callback surface is used" into klp-dev

* commit '72a2f171ddf5c1eab7d6ccd47ac2b7ac4fc9c087':
  Camera: setCallBackWindow only if callback surface is used

Merge "Camera: setCallBackWindow only if callback surface is used" into klp-dev

am a5652e9c: Merge "Add FLAG_CAN_SEEK_BACKWARD and FLAG_CAN_SEEK_FORWARD see flags" into klp-dev

* commit 'a5652e9cdaf6624e03a7e84c40590a42e44c5217':
  Add FLAG_CAN_SEEK_BACKWARD and FLAG_CAN_SEEK_FORWARD see flags

Merge "Add FLAG_CAN_SEEK_BACKWARD and FLAG_CAN_SEEK_FORWARD see flags" into klp-dev

am 66281c3a: Camera: Don\'t overwrite the error status

* commit '66281c3a5d7eea486ddc6ad30088ba92956fd4fd':
  Camera: Don't overwrite the error status

Camera3: Always set a trigger ID for an active trigger

This is to work around HAL implementations that expect to see an
ID for every trigger.

Use a dummy trigger ID of 1, since 0 has special meaning for older
HALs.

Bug: 10720617
Change-Id: I9cb1b8dbcb113cb9e737e5adb04032fd61a54c12

Camera: setCallBackWindow only if callback surface is used

Calling setCallBackWindow without checking the previewCallbackSurface could make
the stream to be deleted during callback stream operation, which is bad because
the preview is still active and the delete fails.

Bug: 10730496
Change-Id: I8addac25e5aa7901bf200d730c749bee2c088090