am ddfbfaeb: fix oflload effect proxy commmand handling

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

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

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: 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

Camera: Don't overwrite the error status

Bug: 10749544
Change-Id: I6096effde03e18ee1b8c63cf3b36dce0344054c5

am d176ec1e: Merge "Fix SoundPool.play() looping" into klp-dev

* commit 'd176ec1e371a686c58385692a28da677e398c302':
  Fix SoundPool.play() looping

am 7f91e5e0: Merge "audioflinger: fix ro.audio.silent in offload mode" into klp-dev

* commit '7f91e5e09cf3bd8068894b2e54cbdaac70bf53a0':
  audioflinger: fix ro.audio.silent in offload mode

am d4dadf23: Merge "Camera: Limit supported preview sizes" into klp-dev

* commit 'd4dadf23dd2dd8acd514d60d84475954070340b8':
  Camera: Limit supported preview sizes

am 972a173d: audioflinger: longer offload thread standby delay

* commit '972a173d7d1de1a3b5a617aae3e2abb6e05ae02d':
  audioflinger: longer offload thread standby delay

Merge "Fix SoundPool.play() looping" into klp-dev

Merge "audioflinger: fix ro.audio.silent in offload mode" into klp-dev

audioflinger: fix ro.audio.silent in offload mode

Change-Id: I5a1a79000d53146689b0a198cc5419c36509703f

Merge "Camera: Limit supported preview sizes" into klp-dev

Camera: Limit supported preview sizes

The ANDROID_SCALER_AVAILABLE_PROCESSED_SIZES can provide sizes as large
as jpeg sizes, which could cause potential issues when trying to do
full size still capture and full size preview. This is not supported
by many devices due to hardware limitation. This change limits the
preview (as well video) size to no more than 1080p.

Bug: 10625115
Change-Id: I9467ab843553ec06e8249b4a17c0ecf4c6d6f04e

audioflinger: longer offload thread standby delay

- Increase offloaded output thread standby delay to 1 second
to allow transition between tracks with going to stanby
if reusing the same audio track (gapless)

- Make sure pause/flush/resume sequence is sent to the HAL
in the right order

- Fix format display in track dump

Bug: 8174034.
Change-Id: I43ef6f8fdbf7427e4eff6cc2d0665d7d1463ea8a

am 99bf6267: Merge "Fix MediaDrm cts tests" into klp-dev

* commit '99bf6267f0a4e2c3153be49ced59ae164d55ea6d':
  Fix MediaDrm cts tests

Merge "Fix MediaDrm cts tests" into klp-dev

am 877c673c: Merge "ACodec: fix flush/resume for decoder-output-meta-data mode" into klp-dev

* commit '877c673c3d3794e89e4677d448f38f77bdee7fdb':
  ACodec: fix flush/resume for decoder-output-meta-data mode

Merge "ACodec: fix flush/resume for decoder-output-meta-data mode" into klp-dev

ACodec: fix flush/resume for decoder-output-meta-data mode

When in decoder-output-meta-data mode, ACodec does not hold onto
buffers, but they are either with the native window, or with the
component/client.  However, for flushing we did not release the
discarded buffers back to native window (this makes sense because
they will be resubmitted shortly.)  This logic can be handled by
the normal resubmission.

Change-Id: Ic472b386422251515ef12f426e187f208f14decc
Signed-off-by: Lajos Molnar <lajos@google.com>
Bug: 10621959
Bug: 10192533

am dc33c542: Merge "Fix AudioTrack shared memory leak" into klp-dev

* commit 'dc33c542351449ceee95cc50a9009f495e7c72ba':
  Fix AudioTrack shared memory leak

Merge "Fix AudioTrack shared memory leak" into klp-dev

Fix MediaDrm cts tests

Moving the build of the mock MediaDrm plugin back to
frameworks/av since it can't be installed by cts.

b/10668350

Change-Id: Id662bdb0775389b458f86a299392217f2d959422

Add FLAG_CAN_SEEK_BACKWARD and FLAG_CAN_SEEK_FORWARD see flags

Also update seek flag in NuPlayerDriver, otherwise MediaPlayer will
get wrong flags.

Bug: 10676387

Change-Id: Ice30f27a9a04e37b4718d26228a407fea8d9e4fc

am efc4d489: Merge "audioflinger: add HOTWORD audio source." into klp-dev

* commit 'efc4d4899e03fb8464017dd48539106cec2163de':
  audioflinger: add HOTWORD audio source.

Merge "audioflinger: add HOTWORD audio source." into klp-dev

am a911f51c: Pass subtitle track properties to getTrackInfo

* commit 'a911f51c21430ac92f1d796b2338878fd98382e9':
  Pass subtitle track properties to getTrackInfo

Pass subtitle track properties to getTrackInfo

Bug: 10326117

Change-Id: I15fcc49ad02e26d7cc92e82ee670bafca62a09a7

Fix SoundPool.play() looping

This is done by configuring SoundPool for shared memory and fast track.
Previously SoundPool used a streaming track, and looping in streaming
mode relied on the ability to loop the most recently enqueued data.
That 'feature' was lost in the new implementation of streaming, so we're
now switching from streaming mode to shared memory mode.  Shared memory
mode had always been desired, but was blocked by bug 2801375 which is fixed now.

Bug: 10171337
Change-Id: I2a938e3ffafa2a74d5210b4198b50db20ad5da0e

Fix AudioTrack shared memory leak

Bug: 2801375
Change-Id: I50e15164fe310f69ea019dca5b49171a02bc6992

audioflinger: add HOTWORD audio source.

HOTWORD is a special capture audio source that uses
the same audio tuning as VOICE_RECOGNITION but allows an
active capture to be preempted.

Bug: 10640877.

Change-Id: Iea09a11cbcdbacc8d434e5230e7559dcd1b44ac0