[Mesa-dev] [PATCH] st/va: move YUV content to deinterlaced buffer when reallocated for encoder
Leo Liu
leo.liu at amd.com
Fri Aug 25 15:15:07 UTC 2017
On 08/25/2017 10:53 AM, Leo Liu wrote:
>
>
> On 08/25/2017 02:57 AM, Christian König wrote:
>> Am 24.08.2017 um 20:49 schrieb Leo Liu:
>>> v2: use deinterlace common function
>>> v3: make sure deinterlace only
>>>
>>> Signed-off-by: Leo Liu <leo.liu at amd.com>
>>> ---
>>> src/gallium/state_trackers/va/picture.c | 22 ++++++++++++++++------
>>> 1 file changed, 16 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/src/gallium/state_trackers/va/picture.c
>>> b/src/gallium/state_trackers/va/picture.c
>>> index 6c3c4fe..aa4062d 100644
>>> --- a/src/gallium/state_trackers/va/picture.c
>>> +++ b/src/gallium/state_trackers/va/picture.c
>>> @@ -613,17 +613,22 @@ vlVaEndPicture(VADriverContextP ctx,
>>> VAContextID context_id)
>>> mtx_lock(&drv->mutex);
>>> surf = handle_table_get(drv->htab, context->target_id);
>>> context->mpeg4.frame_num++;
>>> -
>>> screen = context->decoder->context->screen;
>>> interlaced = screen->get_video_param(screen,
>>> context->decoder->profile,
>>> context->decoder->entrypoint,
>>> PIPE_VIDEO_CAP_SUPPORTS_INTERLACED);
>>> if (surf->buffer->interlaced != interlaced) {
>>> - surf->templat.interlaced = screen->get_video_param(screen,
>>> context->decoder->profile,
>>> - PIPE_VIDEO_ENTRYPOINT_BITSTREAM,
>>> - PIPE_VIDEO_CAP_PREFERS_INTERLACED);
>>> - realloc = true;
>>> + interlaced = screen->get_video_param(screen,
>>> context->decoder->profile,
>>> + context->decoder->entrypoint,
>>> + PIPE_VIDEO_CAP_PREFERS_INTERLACED);
>>> + if (!interlaced) {
>>> + /* The current cases for buffer reallocation are
>>> + all from the interlaced to the deinterlaced,
>>> + and there is no case for the other way around */
>>> + surf->templat.interlaced = false;
>>> + realloc = true;
>>> + }
>>
>> Should we bail out with an error here when it's the other way around?
> Although I cannot think of any of case that to get buffer Interlaced
> now, It's still a good idea to bail out here when it happnens
> Will add it in v4.
It's not a error when case like buffer is deinterlaced, and interlaced
result from query. What we need to do is to do nothing, just ignores.
I have sent out v4, please ignore it, it won't work.
Leo
>
> Thanks,
> Leo
>
>
>
>>
>> Would be nice if we could at least sanely handle that case.
>>
>> Apart from that it looks good to me,
>> Christian.
>>
>>> }
>>> if (u_reduce_video_profile(context->templat.profile) ==
>>> PIPE_VIDEO_FORMAT_JPEG &&
>>> @@ -640,13 +645,18 @@ vlVaEndPicture(VADriverContextP ctx,
>>> VAContextID context_id)
>>> }
>>> if (realloc) {
>>> - surf->buffer->destroy(surf->buffer);
>>> + struct pipe_video_buffer *old_buf = surf->buffer;
>>> if (vlVaHandleSurfaceAllocate(ctx, surf, &surf->templat)
>>> != VA_STATUS_SUCCESS) {
>>> + old_buf->destroy(old_buf);
>>> mtx_unlock(&drv->mutex);
>>> return VA_STATUS_ERROR_ALLOCATION_FAILED;
>>> }
>>> + if (context->decoder->entrypoint ==
>>> PIPE_VIDEO_ENTRYPOINT_ENCODE)
>>> + vl_compositor_yuv_deint(&drv->cstate, &drv->compositor,
>>> old_buf, surf->buffer);
>>> +
>>> + old_buf->destroy(old_buf);
>>> context->target = surf->buffer;
>>> }
>>
>>
>
More information about the mesa-dev
mailing list