[FFmpeg-cvslog] xvid: Add C IDCT

Pascal Massimino git at videolan.org
Wed Sep 3 04:10:22 CEST 2014


ffmpeg | branch: master | Pascal Massimino <pascal.massimino at gmail.com> | Wed Aug 27 02:58:10 2014 +0200| [7a1d6ddd2c6b2d66fbc1afa584cf506930a26453] | committer: Diego Biurrun

xvid: Add C IDCT

Thanks to Pascal Massimino and Michael Militzer for relicensing as LGPL.

Signed-off-by: Diego Biurrun <diego at biurrun.de>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=7a1d6ddd2c6b2d66fbc1afa584cf506930a26453
---

 libavcodec/dct-test.c                      |    6 +-
 libavcodec/x86/xvididct_init.c             |    8 +-
 libavcodec/xvididct.c                      |  322 +++++++++++++++++++++++++++-
 libavcodec/xvididct.h                      |    7 +-
 tests/fate/xvid.mak                        |    5 +-
 tests/ref/fate/{xvid-idctmmx => xvid-idct} |    0
 6 files changed, 337 insertions(+), 11 deletions(-)

diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c
index a531c88..fed5ff2 100644
--- a/libavcodec/dct-test.c
+++ b/libavcodec/dct-test.c
@@ -42,6 +42,7 @@
 #include "dct.h"
 #include "idctdsp.h"
 #include "simple_idct.h"
+#include "xvididct.h"
 #include "aandcttab.h"
 #include "faandct.h"
 #include "faanidct.h"
@@ -62,11 +63,14 @@ static const struct algo fdct_tab[4] = {
     { "IJG-LLM-INT", ff_jpeg_fdct_islow_8, FF_IDCT_PERM_NONE },
 };
 
-static const struct algo idct_tab[4] = {
+static const struct algo idct_tab[] = {
     { "FAANI",       ff_faanidct,          FF_IDCT_PERM_NONE },
     { "REF-DBL",     ff_ref_idct,          FF_IDCT_PERM_NONE },
     { "INT",         ff_j_rev_dct,         FF_IDCT_PERM_LIBMPEG2 },
     { "SIMPLE-C",    ff_simple_idct_8,     FF_IDCT_PERM_NONE },
+#if CONFIG_MPEG4_DECODER
+    { "XVID",        ff_xvid_idct,         FF_IDCT_PERM_NONE, 0, 1 },
+#endif /* CONFIG_MPEG4_DECODER */
 };
 
 #if ARCH_ARM
diff --git a/libavcodec/x86/xvididct_init.c b/libavcodec/x86/xvididct_init.c
index ce5f240..e4f7345 100644
--- a/libavcodec/x86/xvididct_init.c
+++ b/libavcodec/x86/xvididct_init.c
@@ -26,10 +26,16 @@
 #include "idctdsp.h"
 #include "xvididct.h"
 
-av_cold void ff_xvid_idct_init_x86(IDCTDSPContext *c)
+av_cold void ff_xvid_idct_init_x86(IDCTDSPContext *c, AVCodecContext *avctx,
+                                   unsigned high_bit_depth)
 {
     int cpu_flags = av_get_cpu_flags();
 
+    if (high_bit_depth ||
+        !(avctx->idct_algo == FF_IDCT_AUTO ||
+          avctx->idct_algo == FF_IDCT_XVID))
+        return;
+
     if (INLINE_MMX(cpu_flags)) {
         c->idct_put  = ff_xvid_idct_mmx_put;
         c->idct_add  = ff_xvid_idct_mmx_add;
diff --git a/libavcodec/xvididct.c b/libavcodec/xvididct.c
index 0562fdb..ca89703 100644
--- a/libavcodec/xvididct.c
+++ b/libavcodec/xvididct.c
@@ -1,4 +1,8 @@
 /*
+ * Xvid MPEG-4 IDCT
+ *
+ * Copyright (C) 2006-2011 Xvid Solutions GmbH
+ *
  * This file is part of Libav.
  *
  * Libav is free software; you can redistribute it and/or
@@ -16,23 +20,331 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
+/**
+ * @file
+ * Walken IDCT
+ * Alternative IDCT implementation for decoding compatibility.
+ *
+ * @author Skal
+ * @note This C version is not the original IDCT, but a modified one that
+ *       yields the same error profile as the MMX/MMXEXT/SSE2 versions.
+ */
+
 #include "config.h"
 #include "libavutil/attributes.h"
 #include "avcodec.h"
 #include "idctdsp.h"
 #include "xvididct.h"
 
+#define ROW_SHIFT 11
+#define COL_SHIFT  6
+
+// #define FIX(x)   (int)((x) * (1 << ROW_SHIFT))
+#define RND0 65536 // 1 << (COL_SHIFT + ROW_SHIFT - 1);
+#define RND1 3597  // FIX (1.75683487303);
+#define RND2 2260  // FIX (1.10355339059);
+#define RND3 1203  // FIX (0.587788325588);
+#define RND4 0
+#define RND5 120   // FIX (0.058658283817);
+#define RND6 512   // FIX (0.25);
+#define RND7 512   // FIX (0.25);
+
+static const int TAB04[] = { 22725, 21407, 19266, 16384, 12873,  8867, 4520 };
+static const int TAB17[] = { 31521, 29692, 26722, 22725, 17855, 12299, 6270 };
+static const int TAB26[] = { 29692, 27969, 25172, 21407, 16819, 11585, 5906 };
+static const int TAB35[] = { 26722, 25172, 22654, 19266, 15137, 10426, 5315 };
+
+static int idct_row(short *in, const int *const tab, int rnd)
+{
+    const int c1 = tab[0];
+    const int c2 = tab[1];
+    const int c3 = tab[2];
+    const int c4 = tab[3];
+    const int c5 = tab[4];
+    const int c6 = tab[5];
+    const int c7 = tab[6];
+
+    const int right = in[5] | in[6] | in[7];
+    const int left  = in[1] | in[2] | in[3];
+    if (!(right | in[4])) {
+        const int k = c4 * in[0] + rnd;
+        if (left) {
+            const int a0 = k + c2 * in[2];
+            const int a1 = k + c6 * in[2];
+            const int a2 = k - c6 * in[2];
+            const int a3 = k - c2 * in[2];
+
+            const int b0 = c1 * in[1] + c3 * in[3];
+            const int b1 = c3 * in[1] - c7 * in[3];
+            const int b2 = c5 * in[1] - c1 * in[3];
+            const int b3 = c7 * in[1] - c5 * in[3];
+
+            in[0] = (a0 + b0) >> ROW_SHIFT;
+            in[1] = (a1 + b1) >> ROW_SHIFT;
+            in[2] = (a2 + b2) >> ROW_SHIFT;
+            in[3] = (a3 + b3) >> ROW_SHIFT;
+            in[4] = (a3 - b3) >> ROW_SHIFT;
+            in[5] = (a2 - b2) >> ROW_SHIFT;
+            in[6] = (a1 - b1) >> ROW_SHIFT;
+            in[7] = (a0 - b0) >> ROW_SHIFT;
+        } else {
+            const int a0 = k >> ROW_SHIFT;
+            if (a0) {
+                in[0] =
+                in[1] =
+                in[2] =
+                in[3] =
+                in[4] =
+                in[5] =
+                in[6] =
+                in[7] = a0;
+            } else
+                return 0;
+        }
+    } else if (!(left | right)) {
+        const int a0 = (rnd + c4 * (in[0] + in[4])) >> ROW_SHIFT;
+        const int a1 = (rnd + c4 * (in[0] - in[4])) >> ROW_SHIFT;
+
+        in[0] = a0;
+        in[3] = a0;
+        in[4] = a0;
+        in[7] = a0;
+        in[1] = a1;
+        in[2] = a1;
+        in[5] = a1;
+        in[6] = a1;
+    } else {
+        const int k  = c4 * in[0] + rnd;
+        const int a0 = k + c2 * in[2] + c4 * in[4] + c6 * in[6];
+        const int a1 = k + c6 * in[2] - c4 * in[4] - c2 * in[6];
+        const int a2 = k - c6 * in[2] - c4 * in[4] + c2 * in[6];
+        const int a3 = k - c2 * in[2] + c4 * in[4] - c6 * in[6];
+
+        const int b0 = c1 * in[1] + c3 * in[3] + c5 * in[5] + c7 * in[7];
+        const int b1 = c3 * in[1] - c7 * in[3] - c1 * in[5] - c5 * in[7];
+        const int b2 = c5 * in[1] - c1 * in[3] + c7 * in[5] + c3 * in[7];
+        const int b3 = c7 * in[1] - c5 * in[3] + c3 * in[5] - c1 * in[7];
+
+        in[0] = (a0 + b0) >> ROW_SHIFT;
+        in[1] = (a1 + b1) >> ROW_SHIFT;
+        in[2] = (a2 + b2) >> ROW_SHIFT;
+        in[3] = (a3 + b3) >> ROW_SHIFT;
+        in[4] = (a3 - b3) >> ROW_SHIFT;
+        in[5] = (a2 - b2) >> ROW_SHIFT;
+        in[6] = (a1 - b1) >> ROW_SHIFT;
+        in[7] = (a0 - b0) >> ROW_SHIFT;
+    }
+    return 1;
+}
+
+#define TAN1  0x32EC
+#define TAN2  0x6A0A
+#define TAN3  0xAB0E
+#define SQRT2 0x5A82
+
+#define MULT(c, x, n)  (((c) * (x)) >> (n))
+// 12b version => #define MULT(c,x, n)  ((((c) >> 3) * (x)) >> ((n) - 3))
+// 12b zero-testing version:
+
+#define BUTTERFLY(a, b, tmp)     \
+    (tmp) = (a) + (b);           \
+    (b)   = (a) - (b);           \
+    (a)   = (tmp)
+
+#define LOAD_BUTTERFLY(m1, m2, a, b, tmp, s)   \
+    (m1) = (s)[(a)] + (s)[(b)];                \
+    (m2) = (s)[(a)] - (s)[(b)]
+
+static void idct_col_8(short *const in)
+{
+    int mm0, mm1, mm2, mm3, mm4, mm5, mm6, mm7, spill;
+
+    // odd
+
+    mm4 = (int) in[7 * 8];
+    mm5 = (int) in[5 * 8];
+    mm6 = (int) in[3 * 8];
+    mm7 = (int) in[1 * 8];
+
+    mm0 = MULT(TAN1, mm4, 16) + mm7;
+    mm1 = MULT(TAN1, mm7, 16) - mm4;
+    mm2 = MULT(TAN3, mm5, 16) + mm6;
+    mm3 = MULT(TAN3, mm6, 16) - mm5;
+
+    mm7 = mm0 + mm2;
+    mm4 = mm1 - mm3;
+    mm0 = mm0 - mm2;
+    mm1 = mm1 + mm3;
+    mm6 = mm0 + mm1;
+    mm5 = mm0 - mm1;
+    mm5 = 2 * MULT(SQRT2, mm5, 16); // 2*sqrt2
+    mm6 = 2 * MULT(SQRT2, mm6, 16); // Watch out: precision loss but done to match
+                                    // the pmulhw used in MMX/MMXEXT/SSE2 versions
+
+    // even
+
+    mm1 = (int) in[2 * 8];
+    mm2 = (int) in[6 * 8];
+    mm3 = MULT(TAN2, mm2, 16) + mm1;
+    mm2 = MULT(TAN2, mm1, 16) - mm2;
+
+    LOAD_BUTTERFLY(mm0, mm1, 0 * 8, 4 * 8, spill, in);
+
+    BUTTERFLY(mm0, mm3, spill);
+    BUTTERFLY(mm0, mm7, spill);
+    in[8 * 0] = (int16_t) (mm0 >> COL_SHIFT);
+    in[8 * 7] = (int16_t) (mm7 >> COL_SHIFT);
+    BUTTERFLY(mm3, mm4, mm0);
+    in[8 * 3] = (int16_t) (mm3 >> COL_SHIFT);
+    in[8 * 4] = (int16_t) (mm4 >> COL_SHIFT);
+
+    BUTTERFLY(mm1, mm2, mm0);
+    BUTTERFLY(mm1, mm6, mm0);
+    in[8 * 1] = (int16_t) (mm1 >> COL_SHIFT);
+    in[8 * 6] = (int16_t) (mm6 >> COL_SHIFT);
+    BUTTERFLY(mm2, mm5, mm0);
+    in[8 * 2] = (int16_t) (mm2 >> COL_SHIFT);
+    in[8 * 5] = (int16_t) (mm5 >> COL_SHIFT);
+}
+
+static void idct_col_4(short *const in)
+{
+    int mm0, mm1, mm2, mm3, mm4, mm5, mm6, mm7, spill;
+
+    // odd
+
+    mm0 = (int) in[1 * 8];
+    mm2 = (int) in[3 * 8];
+
+    mm1 = MULT(TAN1, mm0, 16);
+    mm3 = MULT(TAN3, mm2, 16);
+
+    mm7 = mm0 + mm2;
+    mm4 = mm1 - mm3;
+    mm0 = mm0 - mm2;
+    mm1 = mm1 + mm3;
+    mm6 = mm0 + mm1;
+    mm5 = mm0 - mm1;
+    mm6 = 2 * MULT(SQRT2, mm6, 16); // 2*sqrt2
+    mm5 = 2 * MULT(SQRT2, mm5, 16);
+
+    // even
+
+    mm0 = mm1 = (int) in[0 * 8];
+    mm3 = (int) in[2 * 8];
+    mm2 = MULT(TAN2, mm3, 16);
+
+    BUTTERFLY(mm0, mm3, spill);
+    BUTTERFLY(mm0, mm7, spill);
+    in[8 * 0] = (int16_t) (mm0 >> COL_SHIFT);
+    in[8 * 7] = (int16_t) (mm7 >> COL_SHIFT);
+    BUTTERFLY(mm3, mm4, mm0);
+    in[8 * 3] = (int16_t) (mm3 >> COL_SHIFT);
+    in[8 * 4] = (int16_t) (mm4 >> COL_SHIFT);
+
+    BUTTERFLY(mm1, mm2, mm0);
+    BUTTERFLY(mm1, mm6, mm0);
+    in[8 * 1] = (int16_t) (mm1 >> COL_SHIFT);
+    in[8 * 6] = (int16_t) (mm6 >> COL_SHIFT);
+    BUTTERFLY(mm2, mm5, mm0);
+    in[8 * 2] = (int16_t) (mm2 >> COL_SHIFT);
+    in[8 * 5] = (int16_t) (mm5 >> COL_SHIFT);
+}
+
+static void idct_col_3(short *const in)
+{
+    int mm0, mm1, mm2, mm3, mm4, mm5, mm6, mm7, spill;
+
+    // odd
+
+    mm7 = (int) in[1 * 8];
+    mm4 = MULT(TAN1, mm7, 16);
+
+    mm6 = mm7 + mm4;
+    mm5 = mm7 - mm4;
+    mm6 = 2 * MULT(SQRT2, mm6, 16); // 2*sqrt2
+    mm5 = 2 * MULT(SQRT2, mm5, 16);
+
+    // even
+
+    mm0 = mm1 = (int) in[0 * 8];
+    mm3 = (int) in[2 * 8];
+    mm2 = MULT(TAN2, mm3, 16);
+
+    BUTTERFLY(mm0, mm3, spill);
+    BUTTERFLY(mm0, mm7, spill);
+    in[8 * 0] = (int16_t) (mm0 >> COL_SHIFT);
+    in[8 * 7] = (int16_t) (mm7 >> COL_SHIFT);
+    BUTTERFLY(mm3, mm4, mm0);
+    in[8 * 3] = (int16_t) (mm3 >> COL_SHIFT);
+    in[8 * 4] = (int16_t) (mm4 >> COL_SHIFT);
+
+    BUTTERFLY(mm1, mm2, mm0);
+    BUTTERFLY(mm1, mm6, mm0);
+    in[8 * 1] = (int16_t) (mm1 >> COL_SHIFT);
+    in[8 * 6] = (int16_t) (mm6 >> COL_SHIFT);
+    BUTTERFLY(mm2, mm5, mm0);
+    in[8 * 2] = (int16_t) (mm2 >> COL_SHIFT);
+    in[8 * 5] = (int16_t) (mm5 >> COL_SHIFT);
+}
+
+void ff_xvid_idct(int16_t *const in)
+{
+    int i, rows = 0x07;
+
+    idct_row(in + 0 * 8, TAB04, RND0);
+    idct_row(in + 1 * 8, TAB17, RND1);
+    idct_row(in + 2 * 8, TAB26, RND2);
+    if (idct_row(in + 3 * 8, TAB35, RND3))
+        rows |= 0x08;
+    if (idct_row(in + 4 * 8, TAB04, RND4))
+        rows |= 0x10;
+    if (idct_row(in + 5 * 8, TAB35, RND5))
+        rows |= 0x20;
+    if (idct_row(in + 6 * 8, TAB26, RND6))
+        rows |= 0x40;
+    if (idct_row(in + 7 * 8, TAB17, RND7))
+        rows |= 0x80;
+
+    if (rows & 0xF0) {
+        for (i = 0; i < 8; i++)
+            idct_col_8(in + i);
+    } else if (rows & 0x08) {
+        for (i = 0; i < 8; i++)
+            idct_col_4(in + i);
+    } else {
+        for (i = 0; i < 8; i++)
+            idct_col_3(in + i);
+    }
+}
+
+static void xvid_idct_put(uint8_t *dest, int line_size, int16_t *block)
+{
+    ff_xvid_idct(block);
+    ff_put_pixels_clamped(block, dest, line_size);
+}
+
+static void xvid_idct_add(uint8_t *dest, int line_size, int16_t *block)
+{
+    ff_xvid_idct(block);
+    ff_add_pixels_clamped(block, dest, line_size);
+}
+
 av_cold void ff_xvid_idct_init(IDCTDSPContext *c, AVCodecContext *avctx)
 {
     const unsigned high_bit_depth = avctx->bits_per_raw_sample > 8;
 
-    if (high_bit_depth ||
-        !(avctx->idct_algo == FF_IDCT_AUTO ||
-          avctx->idct_algo == FF_IDCT_XVID))
-        return;
+    if (!high_bit_depth &&
+        (avctx->idct_algo == FF_IDCT_AUTO ||
+         avctx->idct_algo == FF_IDCT_XVID)) {
+        c->idct_put  = xvid_idct_put;
+        c->idct_add  = xvid_idct_add;
+        c->idct      = ff_xvid_idct;
+        c->perm_type = FF_IDCT_PERM_NONE;
+    }
 
     if (ARCH_X86)
-        ff_xvid_idct_init_x86(c);
+        ff_xvid_idct_init_x86(c, avctx, high_bit_depth);
 
     ff_init_scantable_permutation(c->idct_permutation, c->perm_type);
 }
diff --git a/libavcodec/xvididct.h b/libavcodec/xvididct.h
index e6cc69a..499f819 100644
--- a/libavcodec/xvididct.h
+++ b/libavcodec/xvididct.h
@@ -19,11 +19,16 @@
 #ifndef AVCODEC_XVIDIDCT_H
 #define AVCODEC_XVIDIDCT_H
 
+#include <stdint.h>
+
 #include "avcodec.h"
 #include "idctdsp.h"
 
+void ff_xvid_idct(int16_t *const in);
+
 void ff_xvid_idct_init(IDCTDSPContext *c, AVCodecContext *avctx);
 
-void ff_xvid_idct_init_x86(IDCTDSPContext *c);
+void ff_xvid_idct_init_x86(IDCTDSPContext *c, AVCodecContext *avctx,
+                           unsigned high_bit_depth);
 
 #endif /* AVCODEC_XVIDIDCT_H */
diff --git a/tests/fate/xvid.mak b/tests/fate/xvid.mak
index 68f01b5..877b2a1 100644
--- a/tests/fate/xvid.mak
+++ b/tests/fate/xvid.mak
@@ -1,8 +1,7 @@
 fate-xvid-custom-matrix: CMD = framemd5 -flags +bitexact -idct simple  -i $(TARGET_SAMPLES)/mpeg4/xvid_vlc_trac7411.h263
-fate-xvid-idctmmx:       CMD = framemd5 -flags +bitexact -cpuflags all -i $(TARGET_SAMPLES)/mpeg4/xvid_vlc_trac7411.h263
+fate-xvid-idct:          CMD = framemd5 -flags +bitexact -cpuflags all -i $(TARGET_SAMPLES)/mpeg4/xvid_vlc_trac7411.h263
 
-FATE_XVID-$(call DEMDEC, M4V, MPEG4) += fate-xvid-custom-matrix
-FATE_XVID-$(filter $(HAVE_MMX_INLINE), $(call DEMDEC, M4V, MPEG4)) += fate-xvid-idctmmx
+FATE_XVID-$(call DEMDEC, M4V, MPEG4) += fate-xvid-custom-matrix fate-xvid-idct
 
 FATE_SAMPLES_AVCONV += $(FATE_XVID-yes)
 
diff --git a/tests/ref/fate/xvid-idctmmx b/tests/ref/fate/xvid-idct
similarity index 100%
rename from tests/ref/fate/xvid-idctmmx
rename to tests/ref/fate/xvid-idct



More information about the ffmpeg-cvslog mailing list