History log of /frameworks/av/media/libeffects/proxy/EffectProxy.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
32a5f5d53f9fe2ba584d2ecdf3edd487fb4146a7 22-Dec-2016 Mark Salyzyn <salyzyn@google.com> resolve merge conflicts of 4532a36 to master

Test: build
Change-Id: I9b93c4a703c0ee16116be2bea0e6df0606d29363
60d02077d86d2d1092443519290101f503aa6f7a 29-Sep-2016 Mark Salyzyn <salyzyn@google.com> Replace cutils/log.h with either android/log.h or log/log.h

Test: compile
Bug: 31289077
Change-Id: I795a76f978d2213737f1fa908da789c543219b75
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
48946cd0cdaddfd8abcf6827ac4836cc111cd89c 05-Nov-2016 Andy Hung <hunga@google.com> EffectProxy: Check offload reply size

Test: Custom test case
Bug: 32448121
Change-Id: Iacdd84097f897e86a188ab6831fd0e11738973d1
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
bd0f59e493dbd67d1871b243ddad9cce3b50d7ce 19-Oct-2016 Caroline Tice <cmtice@google.com> Merge "Fix potental memory leak (compiler warning)." am: c1184a7991 am: 7a40848e50
am: 46b06cf79e

Change-Id: Ia51169703363cfa7eb488193316b12606097a970
f8dd1bdb12c76822ec1c2ab5d848c31f256cf36a 19-Oct-2016 Caroline Tice <cmtice@google.com> Fix potental memory leak (compiler warning).

This CL fixes the following warning:

EffectProxy.cpp:93:8: warning: Potential leak of memory pointed to by
'pContext'.

BUG=None
Test: WITH_TIDY=1 WITH_STATIC_ANLYZER=1 mm

Change-Id: I7f127042eb50d3d6297473bfd90a5402b182cc85
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
ef225074ea06dc8ac2801b2fb9f4707dcd3f4200 05-Oct-2016 Glenn Kasten <gkasten@google.com> Enable strict warnings and fix unused warnings

Test: builds without warnings
Change-Id: I0701a44cbd9a2adc405a1e253b57df7ee03a90b7
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
f90c7e0bb8d83d8b7f733bdf430d331ea3f221e8 14-Nov-2013 jpadmana <jayashree.r.padmanaban@intel.com> fix deadlock issues that arise when there are simultaneous
effect control interface calls to proxy and to
non sub-effect wrappers(eg., bundlewrapper) from audioflinger
Also, return NO_ERROR when CMD_OFFLOAD succeeds

Whenever there are parallel calls to proxy and non sub-effects wrappers,
some of the calls are not completed. This is due to deadlock arsing out
of Proxy waiting for the subeffect call to return and subeffect waiting
for proxy to release lock.
The call flow is changed to a cleaner and simple one - Proxy gets the
aeli(effect library info) of subeffects during the EffectGetSubEffects()
call. Therby, proxy will manage the sub effects by itself rather than
going through effects factory.

Signed-off-by: jpadmana <jayashree.r.padmanaban@intel.com>
Bug: 12424044
Change-Id: I16852222f1d0e94e433a19177729323a4bb1c090
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
9803acb6b2c1b9c01444e0c8c0124adbe9a7157a 18-Dec-2013 synergy dev <synergye@codefi.re> libeffects: do not use GNU old-style field designators

Avoiding the use of GCC extensions improves code portability

Change-Id: I9edbedc5c8ad4aa46ca54bc2e28280441431a530
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
385e7509eb563c983647e72b1232225c2200435f 04-Oct-2013 Eric Laurent <elaurent@google.com> fix offload audio effect proxy implementation uuid

The proxy implementation UUID should not be the NULL UUID as
AudioFlinger will reject effect creation if the AudioEffect
is constructed by passing the implementation UUID and not the
type UUID.

Bug: 11070481.

Change-Id: Ia9049d974e76303c5b63a607ee594b7dc1f182d4
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
5d6d86a4d102704f49b9235eaf282c428d7100b6 20-Sep-2013 Eric Laurent <elaurent@google.com> 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: I28aa0f0d806e846332bc29801ee40d34e4ea0c43
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
eba9bf72fb5e036bb15ca4a1dc126883a2cb938d 28-Sep-2013 Eric Laurent <elaurent@google.com> fix command handling in effect offload proxy

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: I119458fea597cc3acbc0ef9ec315f67aa211cbd9
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
faca05e96744dfaa2f352e3dbb29eead4e55cfa0 04-Jun-2013 jpadmana <jayashree.r.padmanaban@intel.com> Effect Offload Proxy for effects offload

Effect Proxy abstracts the sub effects to the upper layers.
It has the following functionalities:
- creation and release of sub effects
- routing the effect commands and process to the appropriate sub effect

Bug: 8174034.

Change-Id: Iec34b61104f0bbec4ef67c62f0710a5536dc325b
Signed-off-by: jpadmana <jayashree.r.padmanaban@intel.com>
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
5b8ce24b849f6cd5629b4ba508f7c78d6227d250 27-Sep-2013 Eric Laurent <elaurent@google.com> Revert "Effect Offload Proxy for effects offload"

This reverts commit 60c60df7db278d2fa5c90b0fa14f99a61d50272b.

Change-Id: Iafba9e02a9f3bfde6248d802e96c4e649686a87d
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
ddfbfaeb00295fff7351711f0f044f17d6c40f3c 20-Sep-2013 Eric Laurent <elaurent@google.com> 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
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
5baf2af52cd186633b7173196c1e4a4cd3435f22 13-Sep-2013 Eric Laurent <elaurent@google.com> 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
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp
60c60df7db278d2fa5c90b0fa14f99a61d50272b 04-Jun-2013 jpadmana <rpadmanaban.jayashree@gmail.com> Effect Offload Proxy for effects offload

Effect Proxy abstracts the sub effects to the upper layers.
It has the following functionalities:
- creation and release of sub effects
- routing the effect commands and process to the appropriate sub effect

Bug: 8174034.

Change-Id: I22d8136636048e7fe8f8807cbc6e348ffa200a22
Signed-off-by: jpadmana <rpadmanaban.jayashree@gmail.com>
/frameworks/av/media/libeffects/proxy/EffectProxy.cpp