On Monday 04 August 2008 22:34, Reinhard Tartler <[EMAIL PROTECTED]> wrote:
> Russell Coker <[EMAIL PROTECTED]> writes:
> > The following patch fixes this.
>
> Are you sure that this is the only occurence of text relocations? TBH, I
> doubt that.

I built the package with the patch in question and the text relocations were 
gone.

> I acknowledge that text relocations are a resonable concern, 
> but the patch does not directly address them. It is not clear where the
> textrelocation actually happens, and for what purpose.

It happens in the .c file when it is included in the three ways that I 
disabled.

> A more maintainable approach to this matter would probably be to add
> --disable-mmx to ffmpeg's configure flags, which would in turn disable
> any usage of MMX code.

However there is no need to disable all MMX code, just that one function.

> TBH, If at all, we could build another variant of ffmpeg library
> packages like libavcodec-security or something. However given the
> current complexity of the ffmpeg package, I'm not that keen on
> implementing and maintaining this.

Or we could try and find a way of changing the assembler to need one less 
register.



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to