Opened 20 years ago

Closed 19 years ago

Last modified 19 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: no Analyzed by developer: no

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@… 19 years ago.
mplayer pic patch
ffmpeg_pie.patch (4.3 KB ) - added by basic@… 19 years ago.
ffmpeg libavcodec pic patch
mplayer_pic.2.patch (8.2 KB ) - added by basic@… 19 years ago.
mplayer pic patch v2
mplayer_pic.3.patch (8.4 KB ) - added by basic@… 19 years ago.
mplayer pic patch v3
ffmpeg_pie.2.patch (5.4 KB ) - added by basic@… 19 years ago.
ffmpeg libavcodec pic patch v2

Download all attachments as: .zip

Change History (15)

comment:1 by webmaster@…, 20 years ago

Version: unspecified1.0pre5

A version would help.

comment:2 by basic@…, 19 years ago

Cc: basic@… added

by basic@…, 19 years ago

Attachment: mplayer_pic.patch added

mplayer pic patch

comment:3 by basic@…, 19 years ago

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.

by basic@…, 19 years ago

Attachment: ffmpeg_pie.patch added

ffmpeg libavcodec pic patch

comment:4 by basic@…, 19 years ago

similar patch for ffmpeg libavcodec

by basic@…, 19 years ago

Attachment: mplayer_pic.2.patch added

mplayer pic patch v2

comment:5 by basic@…, 19 years ago

attachments.isobsolete: 01

missed a register

by basic@…, 19 years ago

Attachment: mplayer_pic.3.patch added

mplayer pic patch v3

comment:6 by basic@…, 19 years ago

comment:7 by basic@…, 19 years ago

attachments.isobsolete: 01

by basic@…, 19 years ago

Attachment: ffmpeg_pie.2.patch added

ffmpeg libavcodec pic patch v2

comment:8 by basic@…, 19 years ago

attachments.isobsolete: 01

missed a file

comment:9 by reimar, 19 years ago

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 by makovick@…, 19 years ago

Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.