Opened 15 years ago

Closed 14 years ago

Last modified 14 years ago

#47 closed defect (wontfix)

Getting PIC to work with mplayer

Reported by: webmaster@… Owned by: alex@…
Priority: normal Component: core
Version: 1.0pre5 Severity: normal
Keywords: Cc: basic@…
Blocked By: Blocking:
Reproduced by developer: Analyzed by developer:

Description

There's a couple of issues with mplayer and PIC. First is:

719:#if defined ARCH_X86 && !defined PIC

of libavcodecs/msmpeg4.c

I believe someone mentioned it was PIC and not PIC. Another workaround was
to add -DPIC to the CFLAGS for the makefile. That worked.

Then I ran into this:

imdct.c: In function `imdct_do_512_sse':
imdct.c:937: warning: use of memory input without lvalue in asm operand 1 is
deprecated
imdct.c:937: warning: use of memory input without lvalue in asm operand 2 is
deprecated
imdct.c:784: error: can't find a register in class BREG' while reloading asm'
make[1]: * [imdct.o] Error 1

my first thought was "Ok, disable sse and sse2, and that should work". I also
disabled mmx, because there was a couple of vf_ source files with mmx code that
were messing with the ebx register (what PIC uses). So I did:

./configure --prefix=/usr --disable-mmx --disable-mmx2 --disable-sse
--disable-sse2

I still get the same error as before. So, if this is sse code, shouldn't
--disable-sse and --disable-sse2 cause it to not compile? Apparently it
didn't, so I'm stuck here.

Attachments (5)

mplayer_pic.patch (7.7 KB) - added by basic@… 14 years ago.
mplayer pic patch
ffmpeg_pie.patch (4.3 KB) - added by basic@… 14 years ago.
ffmpeg libavcodec pic patch
mplayer_pic.2.patch (8.2 KB) - added by basic@… 14 years ago.
mplayer pic patch v2
mplayer_pic.3.patch (8.4 KB) - added by basic@… 14 years ago.
mplayer pic patch v3
ffmpeg_pie.2.patch (5.4 KB) - added by basic@… 14 years ago.
ffmpeg libavcodec pic patch v2

Download all attachments as: .zip

Change History (15)

comment:1 Changed 15 years ago by webmaster@…

  • Version changed from unspecified to 1.0pre5

A version would help.

comment:2 Changed 14 years ago by basic@…

  • Cc basic@… added

Changed 14 years ago by basic@…

mplayer pic patch

comment:3 Changed 14 years ago by basic@…

here's a patch that tries to get mplayer compiling with pic. It doesn't go all
the way though. mp3lib causes mplayer to crash with pic on, so I didn't include
that. This patch just replace uses of "b" with "r" in inline assembly. I
haven't really tested it though.

Changed 14 years ago by basic@…

ffmpeg libavcodec pic patch

comment:4 Changed 14 years ago by basic@…

similar patch for ffmpeg libavcodec

Changed 14 years ago by basic@…

mplayer pic patch v2

comment:5 Changed 14 years ago by basic@…

  • attachments.isobsolete changed from 0 to 1

missed a register

Changed 14 years ago by basic@…

mplayer pic patch v3

comment:6 Changed 14 years ago by basic@…

comment:7 Changed 14 years ago by basic@…

  • attachments.isobsolete changed from 0 to 1

Changed 14 years ago by basic@…

ffmpeg libavcodec pic patch v2

comment:8 Changed 14 years ago by basic@…

  • attachments.isobsolete changed from 0 to 1

missed a file

comment:9 Changed 14 years ago by reimar

Maybe you should explain why you want MPlayer to compile with -fPIC. I up to now
heard only very convincing reasons against it.

comment:10 Changed 14 years ago by makovick@…

  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.