| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
moved surfaceflinger, audioflinger, cameraservice
all native services should now reside in this location.
Change-Id: Iee42b83dd2a94c3bf5107ab0895fe2dfcd5337a8
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
leaking a reference." into froyo
Merge commit '7df7447112371fb5e46f6084b55ac2ccdfde139d' into gingerbread
* commit '7df7447112371fb5e46f6084b55ac2ccdfde139d':
Flush binder buffer after setting raw heap to avoid leaking a reference.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The problem was:
1. In handleShutter(), thread A in CameraService calls
registerBuffers(IMemoryHeap) and it's received by thread B
in system_server. [transaction 1]
2. While thread A is waiting for the reply, thread B calls
back to thread A to get the id of the heap
(IMemoryHeap.getHeapID). [transaction 2]
3. Thread A replies transaction 2 and is preemptied in kernel.
Thread B gets the reply and finishes registerBuffers and send
reply for transaction 1.
4. When thread A runs again, it gets the reply for transaction 1
and returns to handleShutter().
5. At this point the transaction buffer for transaction 2 (which
holds a reference to IMemoryHeap) is not freed because the
BC_FREE_BUFFER command is kept in thread A's local command
queue and not sent to the kernel.
6. Normally when thread A makes next transaction, the
BC_FREE_BUFFER command will be sent together (piggyback) with
the commands for that transaction. But in this case thread A
is a callback thread from camera driver, so it does not make
any binder calls afterwards, and the IMemoryHeap is never freed
(until the next time handleShutter is called).
Change-Id: I435a258187509bdbbaf353339eb9ea577610cbd2
|
| |
| |
| |
| | |
Change-Id: Ie88fe706d2278acf762eca87780de349434778a4
|
| |
| |
| |
| |
| |
| | |
bug - 2677772
Change-Id: I7c996daad8b3bfc89e9fa2d655dc18c9a5c68f2f
|
|\ \ |
|
| | |
| | |
| | |
| | | |
Change-Id: I18dfcb073e6a8e72ad35f47d6884f5f2f62d3dca
|
| | |
| | |
| | |
| | | |
Change-Id: Ice7459b30e1d7dbfe400b68d27ebc04fcca78bae
|
|/ /
| |
| |
| | |
Change-Id: Ibd814d894ff0eba26b01932b9865098b7bb26f95
|
| |
| |
| |
| | |
Change-Id: I80b498d1bbb8f9ffcec4f27802be7ae9977f70ac
|
| |
| |
| |
| | |
Change-Id: Ie3251e3801cd09b3f373bbd4afb5fda82a0c5818
|
| |
| |
| |
| | |
Change-Id: Ib5a0b58ea9e02ae1df9d258de541726189147567
|
| |
| |
| |
| | |
Change-Id: I081f0fbdca4b633715ea7c3b3d42f8662d27598a
|
|/
|
|
| |
Change-Id: I09ddec69997c43b8f17fdd21304c76cb4c5ab8cf
|
| |
|
|
|
|
| |
original version, but it should be ok as the original Y,Cb,Cr are all incorrect.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
headers when specifying the uri of media data to be played.
related-to-bug: 2393577
Original change by Andrei Popescu <andreip@google.com>
|
| |
|
|
|
|
| |
portrait mode.
|
| |
|
| |
|
|
|
|
|
| |
executable but not specified. It is included via dependency of another shared
object.
|
|\
| |
| |
| |
| | |
* changes:
Add CameraServiceTest.
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Merge commit 'bb3bb57a6330f71323fcd7e93e88dbdab55daec3' into eclair-mr2
* commit 'bb3bb57a6330f71323fcd7e93e88dbdab55daec3':
Fix issue 2192673: Music Pausing Even when notifications are set to silent.
|
| |/
| |
| |
| | |
Do not play ringtones, notifications or camera sounds if ringer mode is silent.
|
|/
|
|
|
|
|
| |
We will need those values for new camera framework. And change the canned jpeg
image to match the new width and height setting.
Change-Id: I49f8fb63d2b859b9e9f1c5d27657a10203315bb6
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some camera HALs spin up a preview thread and need to wait for
the thread to exit. This can create a potential deadlock. In
stopPreview, we take the main lock. If a preview callback occurs
while the lock is held, the preview thread will block. If the
camera HAL is waiting for the preview thread to exit, this will
cause a deadlock.
This patch breaks out the preview buffer heap into a separate
mutex. This mutex is never held when the main lock is held, thus
preventing the deadlock from occuring.
|
|
|
|
|
|
|
|
|
|
| |
copyFrameAndPostCopiedFrame was not holding a lock while it accessed
the preview heap. If the client process is torn down while the heap
is accessed, the memcpy could access memory that was deallocated.
This patch creates a local sp reference to the preview heap while
holding the lock, then releases the lock. This should prevent the
heap from being pulled out from underneath us.
|
| |
|
|
|
|
|
|
|
|
| |
We weren't checking to see if there was a valid camera client when
calling the notify callback function. Now we grab a strong pointer
before the callback to guarantee that the client is not destroyed
before we complete the callback. This change also fixes other
places in the code where we weren't holding a local strong pointer.
|
|
|
|
|
|
|
|
| |
Occasionally we see references to the overlay hanging around long
enough to cause problems in applications when they tried to destroy
the overlay and re-create it. This patch causes the camera HAL to
retry the overlay creation call if it fails every 20ms up to 50
times before it gives up.
|
|
|
|
| |
b2060030
|
|
|
|
| |
Change-Id: I13bda991b32aee47e82b5cf9d43b3021c416a9a2
|
|
|
|
| |
bug 2116866
|
|
|
|
| |
Originally from: https://partner.source.android.com/g/#change,829
|
|
|
|
| |
architecture as camera services
|
|
|
|
| |
since we won't be going through the binder in single process mode.
|
|
|
|
|
|
|
| |
Initial commit for review.
Integrated comments after patch set 1 review.
Fixed lockup in AudioFlinger::ThreadBase::exit()
Fixed lockup when playing tone with AudioPlocyService startTone()
|
|
|
|
| |
Enable hardware overlay support for camera and video playback use cases
|
|\ |
|
| |
| |
| |
| | |
Bug 1927069.
|
|\ \
| |/
| |
| |
| |
| |
| | |
Merge commit 'b8a10fe45657f2dcc50cae8a06805f8438a6937e'
* commit 'b8a10fe45657f2dcc50cae8a06805f8438a6937e':
Allow setPreviewDisplay after startPreview.
|
| | |
|
|\ \
| |/
| |
| |
| |
| |
| | |
Merge commit 'c6a482e778e7b5fc5790edf22e554c93f53b1112'
* commit 'c6a482e778e7b5fc5790edf22e554c93f53b1112':
Only remove client after the hardware is teared down, so a new client
|
| |
| |
| |
| |
| |
| | |
is rejected before old client is done.
Also check mUsers in all cases to make sure there are no existing client.
|
|\ \
| |/
| |
| |
| |
| |
| | |
Merge commit '82a32714f07d25259b91163d383ccdb74a166d2d'
* commit '82a32714f07d25259b91163d383ccdb74a166d2d':
Fix 1933269: startPreview failed.
|