Skip to content

Update buffer ownership documentation in AIDL interfaces for Audio/Video HAL services #238

@shafi12

Description

@shafi12

Problem/Opportunity

The documentation for the following AIDL interface methods needs to be updated to clarify buffer ownership responsibilities:

IAudioDecoderController.aidl::decodeBuffer
IAudioSinkController.aidl::queueAudioFrame
IVideoDecoderController.aidl::decodeBuffer

Steps to reproduce

Update the method descriptions to include the following line:

For the Audio Sink
All buffers passed into queueAudioFrame() are the responsibility of the Audio Sink HAL service to free.

For the Audio decoder
All buffers passed into decodeBuffer() are the responsibility of the Audio Decoder HAL service to free.

For the Video Sink
All buffers passed into queueVideoFrame() are the responsibility of the Video Sink HAL service to free.

For the Video decoder

All buffers passed into decodeBuffer() are the responsibility of the Video Decoder HAL service to free.

Expected Behavior

Update the method descriptions to include the following line:

For the Audio Sink
All buffers passed into queueAudioFrame() are the responsibility of the Audio Sink HAL service to free.

For the Audio decoder
All buffers passed into decodeBuffer() are the responsibility of the Audio Decoder HAL service to free.

For the Video Sink
All buffers passed into queueVideoFrame() are the responsibility of the Video Sink HAL service to free.

For the Video decoder

All buffers passed into decodeBuffer() are the responsibility of the Video Decoder HAL service to free.

Actual Behavior

Once the decoder has finished processing the buffer, it is automatically released
* and returned to the AV Buffer Manager. The caller must not modify or free the
* buffer after submission.

Notes (Optional)

No response

Metadata

Metadata

Assignees

Labels

Initial_IssuesInitial issues raised during the VTS development work.documentationImprovements or additions to documentation

Type

Projects

Status

Under Review

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions