Re: [Bug 383240] Re: Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk for armel

2009-10-11 Thread Reinhard Tartler
Mans Rullgard m...@mansr.com writes:

 dsputil.h is not part of the public API, so changes there should not
 affect anything outside libavcodec.

Unfortunately mplayer does #inlcude dsputil.h, and we carefully need to
make sure that changes to that file do not destabilize mplayer. I've
also checked gstreamer0.10-ffmpeg, but that package does not seem to use
dsputil.h.

If in doubt, we should probably reupload mplayer with the changed
dsputil.h together with the new mplayer.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

-- 
Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk 
for armel
https://bugs.launchpad.net/bugs/383240
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


Re: [Bug 383240] Re: Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk for armel

2009-06-29 Thread Reinhard Tartler
Dave Martin dave.mar...@arm.com writes:

 Having played a bit, --extra-cflags=-mfpu=neon -mfloat-abi=softfp
 seems to be _required_ for the neon flavour in order for the NEON
 optimisations to get built.

Is Mans aware of that? If yes, then the upstream configure file should
be patched to automatically add these settings to CFLAGS.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

-- 
Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk 
for armel
https://bugs.launchpad.net/bugs/383240
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


Re: [Bug 383240] Re: Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk for armel

2009-06-08 Thread Reinhard Tartler
excellent, thanks for clarification Loïc!
-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

-- 
Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk 
for armel
https://bugs.launchpad.net/bugs/383240
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Re: [Bug 383240] Re: Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk for armel

2009-06-07 Thread Reinhard Tartler
Loïc Minier l...@dooz.org writes:

 I think there are various different things here:
 - ffmpeg has a mechanism to do multiple passes (multiple builds) and produce 
 multiple shared libs which can be loaded by glibc's hwcaps-based selection 
 mechanism automatically
 - ffmpeg has grown some runtime detection of the feature (instead of build 
 time selection / autodetection on the buildd)
 - NEON was broken in hardware on imx51 TO1 which is all hardware which was 
 around for jaunty

perhaps the kernel shouldn't advertise NEON hwcaps in this case then?

 What we want:
 i) karmic should use the latest ffmpeg
 ii) the latest ffmpeg should use NEON automatically at runtime
 iii) we want to backport key stuff to some PPA to show the NEON stuff based 
 on jaunty (as it's impractical to play with karmic as it's still quite 
 unstable)
 iv) keep the Debian and Ubuntu packaging as identical as possible

 I guess i) will happen as ffmpeg updates over the karmic cycle.

Debian and ubuntu track the FFmpeg 0.5 release. From what I get of this
thread of discussion, I believe the arm related changes have been
implemented in the trunk branch of ffmpeg after 0.5 was branched of. I'd
be against switching back to trunk because of instabilities it would
cause in gstreamer0.10-ffmpeg, mplayer and vlc. You could literally feel
the rate of bugreports dropping down after these package finally agreed
on a common ffmpeg version to use.

I'd rather suggest to backport the changes to the 0.5 release branch
instead of tracking again ffmpeg trunk.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

-- 
Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk 
for armel
https://bugs.launchpad.net/bugs/383240
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Re: [Bug 383240] Re: Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk for armel

2009-06-05 Thread Reinhard Tartler
Dave Martin dave.mar...@arm.com writes:

 It should be safe to uncomment the FLAVORS += neon line now, because we
 deliberately disable NEON in the kernel for Jaunty to avoid applications
 and libraries accidentally believing it works.  (This also means neon is
 not present in the reported hwcaps).

So you're suggesting we should deliberatly and knowingly waste buildd
time for a flavor nobody is technically able to use?

Well, we could of course do that, but I'd rather prefer we wouldn't.


-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

-- 
Integrate and enable ARMv5TE/v6/VFP and NEON optimisations from ffmpeg trunk 
for armel
https://bugs.launchpad.net/bugs/383240
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs