[FFmpeg-devel] [PATCH] avfilter/vf_rotate: add >8 bit depth support

Paul B Mahol onemda at gmail.com
Sun Jun 26 10:22:25 CEST 2016


On 6/26/16, Muhammad Faiz <mfcc64 at gmail.com> wrote:
> On Sun, Jun 26, 2016 at 2:30 PM, Paul B Mahol <onemda at gmail.com> wrote:
>> On 6/26/16, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
>>> Muhammad Faiz <mfcc64 <at> gmail.com> writes:
>>>
>>>> I think it's not because of bit-exact problem.
>>>> But because fate probes supported formats (with
>>>> libavfilter/tests/filtfmts)
>>>> On BE machine, code with native formats will generate error
>>>> because fate-ref contains yuv*le entries but fate expects yuv*be
>
> Another problem is that drawutils only support LE format.
>
>>>
>>> We control the fate test, so we can require an additional
>>> (bit-exact) conversion from BE to LE to make the fate
>>> test pass.
>>
>> Really, even for pixfmts?
>>
>> I will apply this as is. Feel free to add your hacks if you want, after.
>
> In the perspective of code correctness, this should be OK.
> But performance on BE machine will be unoptimal because:
>  - reading/writing in foreign endian is probably slower
>  - I guess most decoders' output frame is native endian. This will
>    make unnecessary format conversion
>  - some other filters only support native endian formats. This will
>    also make unnecessary format conversion.
>
> Of course, the problem is not in your patch.

Yes. BE is dead.

>
> Thank's
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list