[FFmpeg-devel] [PATCH] Revert

Paul B Mahol onemda at gmail.com
Fri Nov 2 22:53:57 CET 2012


On 11/2/12, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Thu, Nov 01, 2012 at 01:55:31AM +0100, Carl Eugen Hoyos wrote:
>> On Thursday 01 November 2012 01:45:28 am Michael Niedermayer wrote:
>>
>> > > The problem is the MMX version of AV_COPY64()
>> >
>> > try adding a emms_c() somewhere, maybe after parse_bintree()
>>
>> Attached patch fixes direct rendering in MPlayer here when I revert my
>> removal
>> of CODEC_CAP_DR1.
>>
>> Thank you, Carl Eugen
>
>>  indeo3.c |    1 +
>>  1 file changed, 1 insertion(+)
>> f958fd52d2eebcf9a0da9411a08cb8b5a3cf4c07  patchiv32dr.patch
>> diff --git a/libavcodec/indeo3.c b/libavcodec/indeo3.c
>> index 126bd2b..9501763 100644
>> --- a/libavcodec/indeo3.c
>> +++ b/libavcodec/indeo3.c
>> @@ -1094,6 +1094,7 @@ static int decode_frame(AVCodecContext *avctx, void
>> *data, int *data_size,
>>
>>      if ((res = decode_plane(ctx, avctx, &ctx->planes[2], ctx->v_data_ptr,
>> ctx->v_data_size, 10)))
>>          return res;
>> +    emms_c();
>>
>>      if (ctx->frame.data[0])
>>          avctx->release_buffer(avctx, &ctx->frame);
>
> this should be applied instead of letting it rot ...

I would prefer if it is first confirmed that inline asm for AV_COPY stuff have
any significant benefits.


More information about the ffmpeg-devel mailing list