[MPlayer-users] compiling on FC4 ppc test

Bret Hughes bhughes at elevating.com
Sun Mar 20 05:53:34 CET 2005


Forgot the patch. left all the text so a search would find this message
also.

Bret

On Sat, 2005-03-19 at 22:48, Bret Hughes wrote:
> On Sat, 2005-03-19 at 10:33, Dominik 'Rathann' Mierzejewski wrote:
> > On Saturday, 19 March 2005 at 05:05, Bret Hughes wrote:
> > > On Fri, 2005-03-18 at 21:22, Bret Hughes wrote:
> > > > On Fri, 2005-03-18 at 18:18, Dominik 'Rathann' Mierzejewski wrote:
> > > > > On Friday, 18 March 2005 at 07:24, Bret Hughes wrote:
> > > > > > I am experimenting with a mac mini and trying to get hardware decoding
> > > > > > of high res(1024X768) mpeg2 video working.  After dinking with the OS X
> > > > > > side for a while (almost maxes the cpu about 97%) and FC3 ppc (100% cpu
> > > > > > usage) I thought I would try FC4 test1 but I can't get it to pass the
> > > > > > compiler test during configure.
> 
> <snip >
> 
> > Post your command line, build log and compiler version here (actually,
> > follow bugreports.html) or file a report in MPlayer's bugzilla:
> > http://bugzilla.mplayerhq.hu/ . We'll take it from there.
> 
> 
> 
> Thanks for the reply. Here is the scoop.  I am doing this on a Fedora 4
> test 1 box on a mac mini.
> 
> I got pissed last night and dug through the ffmpeg
> code that was puking and with some help from a guy on the fedora-devel
> irc and the ffmpeg-devel list I found the issue.  The primary deal is
> that gcc 4 has deprecated the cast as lvalue extension.  The culprit was
> libavcodec/ppc/fdct_altivec.c and I submitted a patch to the
> ffmpeg-devel list that performs explict casts for all the lines in
> error.  Fron a post after my patch was sent it looks like I was close
> enough that they are trying to find someone to test it by conparing
> md5sums on files encoded before and after the patch.  I guess compiling
> with a supported compiler.
> 
> I am attaching the ffmeg patch in case someone else wants to do this
> before this stuff gets committed.
> 
> I am no C coder so if anyone has comments on what should be done
> differently I am all ears.
> 
> FYI. All this was done against yesterdays mplayer cvs snapshot
> (MPlayer-20050318).
> fdct_altivec.c had not been touched for 16 months so the patch should
> work against any recent version.
> 
> 
> There was one other fix that it took to compile, an incomplete array
> definition in libmpdemux/frequencies.h.
> 
> I commented out line 107:
> //extern struct STRTAB chanlist_names[];
> 
> I could not find a reference to chanlist_names or STRTAB by greping the
> code so I just got rid of it.
> 
> The compilation went fine after that using 
>  ./configure --disable-gcc-checking
> make
> make install
> 
> > 
> > As for hardware decoding, I assume you mean hardware scaling and
> > colourspace conversion. It should work with Xvideo extension on your
> > Radeon (it is supported on ppc, isn't it?). We need more details here,
> > too.
> > 
> 
> Now you are asking me to comment on stuff that is over my head.  I
> assumed that there was some motion compensation or some such stuff built
> into chips that claim hardware decoding of mpeg2 and or mpeg4 files.  My
> experience with this is limited to the tremendous gains I got on a
> M10000 mini-itx system when I finally got -vo xv -fs -zoom to work on a
> 1024 X 768 mpeg2 30 second clip with 60% cpu usage. This was compared to
> -vo -x11 that took 47 seconds at 100% cpu.  I was hoping for something
> similar on the mac since it is a well know chip (Radeon 9200) on a
> faster processor and risc based to boot. I should probably start another
> thread for these questions since it appears that my compilation issues
> are overcome.
> 
> Thanks and feel free to correct anything I may have mis stated since I
> don't know my rear from a hole in the ground when it comes to this
> stuff. I am slowly gaining knowledge but man is it painful.
> 
> Bret
> 
> _______________________________________________
> MPlayer-users mailing list
> MPlayer-users at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-users

-------------- next part --------------
--- libavcodec/ppc/fdct_altivec.c	2003-10-26 04:14:05.000000000 -0600
+++ ../mplayer.cvs/MPlayer-20050318/libavcodec/ppc/fdct_altivec.c	2005-03-19 01:42:42.000000000 -0600
@@ -214,8 +214,8 @@
 
     /* setup constants {{{ */
     /* mzero = -0.0 */
-    vu32(mzero) = vec_splat_u32(-1);
-    vu32(mzero) = vec_sl(vu32(mzero), vu32(mzero));
+    mzero = ((vector float)vec_splat_u32(-1));
+    mzero = ((vector float)vec_sl(vu32(mzero), vu32(mzero)));
     cp = fdctconsts;
     cnsts0 = vec_ld(0, cp); cp++;
     cnsts1 = vec_ld(0, cp); cp++;
@@ -227,43 +227,43 @@
 #define MERGE_S16(hl,a,b) vec_merge##hl(vs16(a), vs16(b))
 
     bp = (vector signed short*)block;
-    vs16(b00) = vec_ld(0,    bp);
-    vs16(b40) = vec_ld(16*4, bp);
-    vs16(b01) = MERGE_S16(h, b00, b40);
-    vs16(b11) = MERGE_S16(l, b00, b40);
+    b00 = ((vector float)vec_ld(0,    bp));
+    b40 = ((vector float)vec_ld(16*4, bp));
+    b01 = ((vector float)MERGE_S16(h, b00, b40));
+    b11 = ((vector float)MERGE_S16(l, b00, b40));
     bp++;
-    vs16(b10) = vec_ld(0,    bp);
-    vs16(b50) = vec_ld(16*4, bp);
-    vs16(b21) = MERGE_S16(h, b10, b50);
-    vs16(b31) = MERGE_S16(l, b10, b50);
+    b10 = ((vector float)vec_ld(0,    bp));
+    b50 = ((vector float)vec_ld(16*4, bp));
+    b21 = ((vector float)MERGE_S16(h, b10, b50));
+    b31 = ((vector float)MERGE_S16(l, b10, b50));
     bp++;
-    vs16(b20) = vec_ld(0,    bp);
-    vs16(b60) = vec_ld(16*4, bp);
-    vs16(b41) = MERGE_S16(h, b20, b60);
-    vs16(b51) = MERGE_S16(l, b20, b60);
+    b20 = ((vector float)vec_ld(0,    bp));
+    b60 = ((vector float)vec_ld(16*4, bp));
+    b41 = ((vector float)MERGE_S16(h, b20, b60));
+    b51 = ((vector float)MERGE_S16(l, b20, b60));
     bp++;
-    vs16(b30) = vec_ld(0,    bp);
-    vs16(b70) = vec_ld(16*4, bp);
-    vs16(b61) = MERGE_S16(h, b30, b70);
-    vs16(b71) = MERGE_S16(l, b30, b70);
-
-    vs16(x0) = MERGE_S16(h, b01, b41);
-    vs16(x1) = MERGE_S16(l, b01, b41);
-    vs16(x2) = MERGE_S16(h, b11, b51);
-    vs16(x3) = MERGE_S16(l, b11, b51);
-    vs16(x4) = MERGE_S16(h, b21, b61);
-    vs16(x5) = MERGE_S16(l, b21, b61);
-    vs16(x6) = MERGE_S16(h, b31, b71);
-    vs16(x7) = MERGE_S16(l, b31, b71);
-
-    vs16(b00) = MERGE_S16(h, x0, x4);
-    vs16(b10) = MERGE_S16(l, x0, x4);
-    vs16(b20) = MERGE_S16(h, x1, x5);
-    vs16(b30) = MERGE_S16(l, x1, x5);
-    vs16(b40) = MERGE_S16(h, x2, x6);
-    vs16(b50) = MERGE_S16(l, x2, x6);
-    vs16(b60) = MERGE_S16(h, x3, x7);
-    vs16(b70) = MERGE_S16(l, x3, x7);
+    b30 = ((vector float)vec_ld(0,    bp));
+    b70 = ((vector float)vec_ld(16*4, bp));
+    b61 = ((vector float)MERGE_S16(h, b30, b70));
+    b71 = ((vector float)MERGE_S16(l, b30, b70));
+
+    x0 = ((vector float)MERGE_S16(h, b01, b41));
+    x1 = ((vector float)MERGE_S16(l, b01, b41));
+    x2 = ((vector float)MERGE_S16(h, b11, b51));
+    x3 = ((vector float)MERGE_S16(l, b11, b51));
+    x4 = ((vector float)MERGE_S16(h, b21, b61));
+    x5 = ((vector float)MERGE_S16(l, b21, b61));
+    x6 = ((vector float)MERGE_S16(h, b31, b71));
+    x7 = ((vector float)MERGE_S16(l, b31, b71));
+
+    b00 = ((vector float)MERGE_S16(h, x0, x4));
+    b10 = ((vector float)MERGE_S16(l, x0, x4));
+    b20 = ((vector float)MERGE_S16(h, x1, x5));
+    b30 = ((vector float)MERGE_S16(l, x1, x5));
+    b40 = ((vector float)MERGE_S16(h, x2, x6));
+    b50 = ((vector float)MERGE_S16(l, x2, x6));
+    b60 = ((vector float)MERGE_S16(h, x3, x7));
+    b70 = ((vector float)MERGE_S16(l, x3, x7));
 
 #undef MERGE_S16
     /* }}} */
@@ -275,32 +275,32 @@
  */
 #if 1
     /* fdct rows {{{ */
-    vs16(x0) = vec_add(vs16(b00), vs16(b70));
-    vs16(x7) = vec_sub(vs16(b00), vs16(b70));
-    vs16(x1) = vec_add(vs16(b10), vs16(b60));
-    vs16(x6) = vec_sub(vs16(b10), vs16(b60));
-    vs16(x2) = vec_add(vs16(b20), vs16(b50));
-    vs16(x5) = vec_sub(vs16(b20), vs16(b50));
-    vs16(x3) = vec_add(vs16(b30), vs16(b40));
-    vs16(x4) = vec_sub(vs16(b30), vs16(b40));
+    x0 = ((vector float)vec_add(vs16(b00), vs16(b70)));
+    x7 = ((vector float)vec_sub(vs16(b00), vs16(b70)));
+    x1 = ((vector float)vec_add(vs16(b10), vs16(b60)));
+    x6 = ((vector float)vec_sub(vs16(b10), vs16(b60)));
+    x2 = ((vector float)vec_add(vs16(b20), vs16(b50)));
+    x5 = ((vector float)vec_sub(vs16(b20), vs16(b50)));
+    x3 = ((vector float)vec_add(vs16(b30), vs16(b40)));
+    x4 = ((vector float)vec_sub(vs16(b30), vs16(b40)));
 
-    vs16(b70) = vec_add(vs16(x0), vs16(x3));
-    vs16(b10) = vec_add(vs16(x1), vs16(x2));
+    b70 = ((vector float)vec_add(vs16(x0), vs16(x3)));
+    b10 = ((vector float)vec_add(vs16(x1), vs16(x2)));
 
-    vs16(b00) = vec_add(vs16(b70), vs16(b10));
-    vs16(b40) = vec_sub(vs16(b70), vs16(b10));
+    b00 = ((vector float)vec_add(vs16(b70), vs16(b10)));
+    b40 = ((vector float)vec_sub(vs16(b70), vs16(b10)));
 
 #define CTF0(n) \
-    vs32(b##n##1) = vec_unpackl(vs16(b##n##0)); \
-    vs32(b##n##0) = vec_unpackh(vs16(b##n##0)); \
+    b##n##1 = ((vector float)vec_unpackl(vs16(b##n##0))); \
+    b##n##0 = ((vector float)vec_unpackh(vs16(b##n##0))); \
     b##n##1 = vec_ctf(vs32(b##n##1), 0); \
     b##n##0 = vec_ctf(vs32(b##n##0), 0);
 
     CTF0(0);
     CTF0(4);
 
-    vs16(b20) = vec_sub(vs16(x0), vs16(x3));
-    vs16(b60) = vec_sub(vs16(x1), vs16(x2));
+    b20 = ((vector float)vec_sub(vs16(x0), vs16(x3)));
+    b60 = ((vector float)vec_sub(vs16(x1), vs16(x2)));
 
     CTF0(2);
     CTF0(6);
@@ -321,8 +321,8 @@
     b61 = vec_madd(cnst, b61, x1);
 
 #define CTFX(x,b) \
-    vs32(b##0) = vec_unpackh(vs16(x)); \
-    vs32(b##1) = vec_unpackl(vs16(x)); \
+    b##0 = ((vector float)vec_unpackh(vs16(x))); \
+    b##1 = ((vector float)vec_unpackl(vs16(x))); \
     b##0 = vec_ctf(vs32(b##0), 0); \
     b##1 = vec_ctf(vs32(b##1), 0); \
 
@@ -473,9 +473,9 @@
 #define CTS(n) \
     b##n##0 = vec_round(b##n##0); \
     b##n##1 = vec_round(b##n##1); \
-    vs32(b##n##0) = vec_cts(b##n##0, 0); \
-    vs32(b##n##1) = vec_cts(b##n##1, 0); \
-    vs16(b##n##0) = vec_pack(vs32(b##n##0), vs32(b##n##1)); \
+    b##n##0 = ((vector float)vec_cts(b##n##0, 0)); \
+    b##n##1 = ((vector float)vec_cts(b##n##1, 0)); \
+    b##n##0 = ((vector float)vec_pack(vs32(b##n##0), vs32(b##n##1))); \
     vec_st(vs16(b##n##0), 0, bp);
 
     bp = (vector signed short*)block;


More information about the MPlayer-users mailing list