Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread u-9iep
On Mon, Oct 03, 2016 at 07:28:08PM +0200, Hendrik Leppkes wrote: > > Again: no offence! Standard libraries are just a quite different area, > > it postulated other skills and presents other implementation challenges > > than multimedia programming. > > Optimized code is the same everywhere, you

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Hendrik Leppkes
On Mon, Oct 3, 2016 at 7:24 PM, wrote: > On Mon, Oct 03, 2016 at 05:27:19PM +0200, wm4 wrote: >> > Musl merely showed you the problem and now you are suggesting to "document >> > that the messenger with his bad news about our health is no longer >> > welcome". >> >> musl could

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread u-9iep
On Mon, Oct 03, 2016 at 05:27:19PM +0200, wm4 wrote: > > Musl merely showed you the problem and now you are suggesting to "document > > that the messenger with his bad news about our health is no longer welcome". > > musl could also choose to abandon its incredibly "clever" hack (that > makes

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread wm4
On Mon, 3 Oct 2016 17:01:12 +0200 u-9...@aetey.se wrote: > Ronald, > > I sincerely appreciate that you are taking the effort to talk to me > and explain your position. > > Unfortunately in your messages you consistently ignored a crucial point > and this is the last time I try to retell it: >

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread u-9iep
Ronald, I sincerely appreciate that you are taking the effort to talk to me and explain your position. Unfortunately in your messages you consistently ignored a crucial point and this is the last time I try to retell it: > On Mon, Oct 3, 2016 at 10:26 AM, wrote: > > What you

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread u-9iep
On Mon, Oct 03, 2016 at 04:26:50PM +0200, u-9...@aetey.se wrote: > With all the competence present here, is it really infeasible to improve > the code structure so that it does not involve the C library in the > bit-crunching performance critical paths?? Bad wording. Should be:

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Ronald S. Bultje
Hi Rune, On Mon, Oct 3, 2016 at 10:26 AM, wrote: > On Mon, Oct 03, 2016 at 08:01:05AM -0400, Ronald S. Bultje wrote: > > > Ping on the patch: > > > The patch is unlikely to assist in fixing the issue and is likely to > cause > > further inflammation. Therefore I would prefer

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread u-9iep
On Mon, Oct 03, 2016 at 08:01:05AM -0400, Ronald S. Bultje wrote: > > Ping on the patch: > The patch is unlikely to assist in fixing the issue and is likely to cause > further inflammation. Therefore I would prefer you did not apply the patch > and also please don't send a new version that is

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Ronald S. Bultje
Hi, On Mon, Oct 3, 2016 at 8:46 AM, Carl Eugen Hoyos wrote: > 2016-10-03 13:57 GMT+02:00 Hendrik Leppkes : > > > The underlying problem is that mmx code is mixed with allocations, > > Definitely. > > > which seems like an unusual case to begin with > > I

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Carl Eugen Hoyos
2016-10-03 13:57 GMT+02:00 Hendrik Leppkes : > The underlying problem is that mmx code is mixed with allocations, Definitely. > which seems like an unusual case to begin with I am not sure if I understand this but one instance is calling radix_sort() in the dnxhd encoder.

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Hendrik Leppkes
On Mon, Oct 3, 2016 at 1:37 PM, Carl Eugen Hoyos wrote: > Hi! > > 2016-10-03 13:05 GMT+02:00 Guillaume POIRIER : >> Hello, >> >> On Mon, Oct 3, 2016 at 11:38 AM, Carl Eugen Hoyos wrote: >>> 2016-10-03 3:52 GMT+02:00 Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Ronald S. Bultje
Hi, On Mon, Oct 3, 2016 at 7:37 AM, Carl Eugen Hoyos wrote: > Hi! > > 2016-10-03 13:05 GMT+02:00 Guillaume POIRIER : > > Hello, > > > > On Mon, Oct 3, 2016 at 11:38 AM, Carl Eugen Hoyos > wrote: > >> 2016-10-03 3:52 GMT+02:00 Carl

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Carl Eugen Hoyos
Hi! 2016-10-03 13:05 GMT+02:00 Guillaume POIRIER : > Hello, > > On Mon, Oct 3, 2016 at 11:38 AM, Carl Eugen Hoyos wrote: >> 2016-10-03 3:52 GMT+02:00 Carl Eugen Hoyos : >> >>> New patch attached. >> >>> +Compilation with

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Guillaume POIRIER
Hello, On Mon, Oct 3, 2016 at 11:38 AM, Carl Eugen Hoyos wrote: > 2016-10-03 3:52 GMT+02:00 Carl Eugen Hoyos : > >> New patch attached. > >> +Compilation with @uref{http://www.musl-libc.org/, musl} on x86-64 is >> supposed >> +to work out-of-the-box. > >

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread u-9iep
On Mon, Oct 03, 2016 at 12:37:29PM +0200, Carl Eugen Hoyos wrote: > 2016-10-03 12:30 GMT+02:00 Hendrik Leppkes : > > > Lets just fix the bugs > > Sorry: Which bugs exactly should we just fix? > > Carl Eugen Please fix the UB in the MMX-code. Rune

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Carl Eugen Hoyos
2016-10-03 12:30 GMT+02:00 Hendrik Leppkes : > Lets just fix the bugs Sorry: Which bugs exactly should we just fix? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Hendrik Leppkes
On Mon, Oct 3, 2016 at 2:36 AM, Ronald S. Bultje wrote: > Hi, > > On Sun, Oct 2, 2016 at 7:51 PM, Carl Eugen Hoyos wrote: > >> Hi! >> >> Attached patch adds a musl section to doc/platform. > > > This is counter-productive... Let's work with the musl-devs and

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Carl Eugen Hoyos
2016-10-03 3:52 GMT+02:00 Carl Eugen Hoyos : > New patch attached. > +Compilation with @uref{http://www.musl-libc.org/, musl} on x86-64 is supposed > +to work out-of-the-box. Is this true or is it just working by accident? Carl Eugen

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Nicolas George
Le duodi 12 vendémiaire, an CCXXV, wm4 a écrit : > You could mention that this is due to FFmpeg violating the ABI > (apparently). Still, using floats in malloc() looks like a very bad idea, borderline recipe for disaster. Especially on x86_32. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread Carl Eugen Hoyos
2016-10-03 2:57 GMT+02:00 wm4 : >> +Do not compile FFmpeg with musl on x86_32, random >> crashes have to be expected. > You could mention that this is due to FFmpeg violating the > ABI (apparently). New patch attached. Carl Eugen From

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread wm4
On Mon, 3 Oct 2016 01:51:25 +0200 Carl Eugen Hoyos wrote: > From 35bf44b909658864c9e8cca8dbc855dbde1e1ef5 Mon Sep 17 00:00:00 2001 > From: Carl Eugen Hoyos > Date: Mon, 3 Oct 2016 01:41:37 +0200 > Subject: [PATCH] doc/platform: Mention musl where x86_32 is

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread Josh de Kock
On 03/10/2016 01:44, Ronald S. Bultje wrote: Hi, On Sun, Oct 2, 2016 at 8:42 PM, Carl Eugen Hoyos wrote: 2016-10-03 2:36 GMT+02:00 Ronald S. Bultje : Hi, On Sun, Oct 2, 2016 at 7:51 PM, Carl Eugen Hoyos wrote: Attached patch

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread Ronald S. Bultje
Hi, On Sun, Oct 2, 2016 at 8:42 PM, Carl Eugen Hoyos wrote: > 2016-10-03 2:36 GMT+02:00 Ronald S. Bultje : > > Hi, > > > > On Sun, Oct 2, 2016 at 7:51 PM, Carl Eugen Hoyos > wrote: > > > >> Attached patch adds a musl section to

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread Carl Eugen Hoyos
2016-10-03 2:36 GMT+02:00 Ronald S. Bultje : > Hi, > > On Sun, Oct 2, 2016 at 7:51 PM, Carl Eugen Hoyos wrote: > >> Attached patch adds a musl section to doc/platform. > > This is counter-productive... Sorry, I just wanted to be productive wrt a bug

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread Ronald S. Bultje
Hi, On Sun, Oct 2, 2016 at 7:51 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch adds a musl section to doc/platform. This is counter-productive... Let's work with the musl-devs and fix their libc-alternative? Ronald ___