[FFmpeg-devel] [PATCH] Rewrite emu_edge functions to have separate src/dst_stride arguments.

[Ronald S. Bultje]

Hi,

On Sun, Sep 22, 2013 at 4:19 PM, Michael Niedermayer <michaelni at gmx.at>wrote:

> On Sun, Sep 22, 2013 at 03:50:41PM -0400, Ronald S. Bultje wrote:
> > Hi,
> >
> >
> > On Sun, Sep 22, 2013 at 3:25 PM, Michael Niedermayer <michaelni at gmx.at
> >wrote:
> >
> > > On Fri, Sep 20, 2013 at 08:03:00AM -0400, Ronald S. Bultje wrote:
> > > > From: "Ronald S. Bultje" <rsbultje at gmail.com>
> > > >
> > > > This allows supporting files for which the image stride is smaller
> than
> > > > the max. block size + number of subpel mc taps, e.g. a 64x64 VP9 file
> > > > or a 16x16 VP8 file with -fflags +emu_edge.
> > > > ---
> > > >  libavcodec/cavs.c              |   7 +-
> > > >  libavcodec/diracdec.c          |   3 +-
> > > >  libavcodec/h264.c              |  10 +-
> > > >  libavcodec/mpegvideo.c         |  17 +-
> > > >  libavcodec/mpegvideo_enc.c     |   6 +-
> > > >  libavcodec/mpegvideo_motion.c  |  74 ++--
> > > >  libavcodec/rv34.c              |  15 +-
> > > >  libavcodec/snow.c              |   4 +-
> > > >  libavcodec/svq3.c              |   6 +-
> > > >  libavcodec/vc1dec.c            |  47 +--
> > > >  libavcodec/videodsp.h          |  20 +-
> > > >  libavcodec/videodsp_template.c |  33 +-
> > > >  libavcodec/vp3.c               |   5 +-
> > > >  libavcodec/vp56.c              |   2 +-
> > > >  libavcodec/vp8.c               |  38 +-
> > > >  libavcodec/vp8.h               |   2 +-
> > > >  libavcodec/wmv2.c              |  16 +-
> > > >  libavcodec/x86/dsputil_mmx.c   |   7 +-
> > > >  libavcodec/x86/videodsp.asm    | 840
> > > ++++++++++++++++-------------------------
> > > >  libavcodec/x86/videodsp_init.c | 182 +++++++--
> > > >  20 files changed, 658 insertions(+), 676 deletions(-)
> > >
> > > fails fate on mips-qem
> > >
> >
> > Uh? I have no friggin' clue :)
>
> > Does it fail only on vp8, or on other stuff
> > as well?
>
> grep Err fatelist
> make: *** [fate-vp8-test-vector-004] Error 1
> make: *** [fate-vp8-test-vector-003] Error 1
> make: *** [fate-vp8-test-vector-007] Error 1
> make: *** [fate-vp8-test-vector-emu-edge-004] Error 1
> make: *** [fate-vp8-test-vector-emu-edge-003] Error 1
> make: *** [fate-vp8-test-vector-emu-edge-007] Error 1
>
>
> > Did it fail before this patch?
>
> no it works without the patch


Apply the other patch to fix vp8's bilinear C code and PPC assembly before
this patch to fix assorted fate failures...

Ronald