Bug#842513: vlc: immediate crash on launch on powerpc

[luigi burdo]

hi all just to notice

kodi on e5500 2 ghz play without problems and without frame dropping

mkv & mp4 at 1080p without vdpau accelerations.


i suggest to use kodi if vlc crash fo altivec.


Luigi



________________________________
Da: Lennart Sorensen <lsorense at csclub.uwaterloo.ca>
Inviato: martedì 1 novembre 2016 14.56
A: Adrian Bunk
Cc: James Cowgill; Robert Ou; debian-powerpc at lists.debian.org; 842513 at bugs.debian.org
Oggetto: Re: Bug#842513: vlc: immediate crash on launch on powerpc

On Mon, Oct 31, 2016 at 11:13:00PM +0200, Adrian Bunk wrote:
> This actually looks like a bug in upstream configure.ac to me:
>     VLC_ADD_CFLAGS([libvlccore],[${ac_cv_c_altivec}])
>     ALTIVEC_CFLAGS="$ALTIVEC_FLAGS ${ac_cv_c_altivec} ${ac_cv_c_altivec_abi}"
>     VLC_ADD_CFLAGS([deinterlace],[${ac_cv_c_altivec} ${ac_cv_c_altivec_abi}])
>
> It is correct that this adds -maltivec to AltiVec-specific code,
> and vlc has proper autodetection to run this only when AltiVec
> is actually present.
>
> The VLC_ADD_CFLAGS here look just wrong - it is not the job of
> configure.ac to add such flags to generic code (whatever march
> and hardware features are present is defined through compiler
> defaults and the CFLAGS passed by the user).

Actually what looks really wrong is that debian/rules builds ffmpeg with
--disable-altivec and then builds vlc with --enable-altivec.

Since powerpc can't assume you have altivec, perhaps vlc shouldn't be
built with it, or at least ought to have justification for why it is
done that way.  Maybe it was thought that no CPU without altivec would
be powerful enough to bother using vlc, but I think that may not in fact
be a valid assumption these days.

Certainly a 2.5GHz e5500 core is pretty decent performance, but has
no altivec.

--
Len Sorensen

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20161101/46c5ebd8/attachment-0001.html>