Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Thu, Oct 22, 2015 at 4:51 PM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: > >> To put an end to a long and tortuous thread, and >> due to the lack of relevant outstanding objections, >> pushed. > > To sum it up: > Several developers have explained to you

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Fri, Oct 16, 2015 at 7:53 AM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 7:30 AM, Michael Niedermayer > wrote: >> On Thu, Oct 15, 2015 at 06:38:10AM -0400, Ganesh Ajjanagadde wrote: >>> On Wed, Oct 14, 2015 at 6:53 AM, Hendrik Leppkes

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

Ganesh Ajjanagadde mit.edu> writes: > To put an end to a long and tortuous thread, and > due to the lack of relevant outstanding objections, > pushed. To sum it up: Several developers have explained to you that the numbers you posted show that FFmpeg is now either slower or equally fast,

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Fri, Oct 16, 2015 at 7:30 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 06:38:10AM -0400, Ganesh Ajjanagadde wrote: >> On Wed, Oct 14, 2015 at 6:53 AM, Hendrik Leppkes wrote: >> > On Wed, Oct 14, 2015 at 12:49 PM, Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Thu, Oct 15, 2015 at 06:38:10AM -0400, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 6:53 AM, Hendrik Leppkes wrote: > > On Wed, Oct 14, 2015 at 12:49 PM, Carl Eugen Hoyos wrote: > >> Ganesh Ajjanagadde mit.edu> writes: > >> > >>> What? My numbers

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Thu, Oct 15, 2015 at 12:53 PM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 6:50 AM, Hendrik Leppkes wrote: >> On Thu, Oct 15, 2015 at 12:41 PM, Ganesh Ajjanagadde >> wrote: >>> On Thu, Oct 15, 2015 at 6:37 AM, Hendrik Leppkes

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Thu, Oct 15, 2015 at 6:50 AM, Hendrik Leppkes wrote: > On Thu, Oct 15, 2015 at 12:41 PM, Ganesh Ajjanagadde wrote: >> On Thu, Oct 15, 2015 at 6:37 AM, Hendrik Leppkes wrote: >>> On Thu, Oct 15, 2015 at 12:34 PM, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Wed, Oct 14, 2015 at 6:50 AM, Hendrik Leppkes wrote: > On Wed, Oct 14, 2015 at 12:37 PM, Ganesh Ajjanagadde wrote: >> On Wed, Oct 14, 2015 at 5:01 AM, Matt Oliver wrote: >>> On 14 October 2015 at 09:46, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Thu, Oct 15, 2015 at 6:41 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 6:37 AM, Hendrik Leppkes wrote: >> On Thu, Oct 15, 2015 at 12:34 PM, Ganesh Ajjanagadde >> wrote: >>> On Wed, Oct 14, 2015 at 6:50 AM, Hendrik Leppkes

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Thu, Oct 15, 2015 at 12:41 PM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 6:37 AM, Hendrik Leppkes wrote: >> On Thu, Oct 15, 2015 at 12:34 PM, Ganesh Ajjanagadde >> wrote: >>> On Wed, Oct 14, 2015 at 6:50 AM, Hendrik Leppkes

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Wed, Oct 14, 2015 at 6:53 AM, Hendrik Leppkes wrote: > On Wed, Oct 14, 2015 at 12:49 PM, Carl Eugen Hoyos wrote: >> Ganesh Ajjanagadde mit.edu> writes: >> >>> What? My numbers actually show that the new code may be faster - >> >> No, you are

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Thu, Oct 15, 2015 at 12:34 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 6:50 AM, Hendrik Leppkes wrote: >> On Wed, Oct 14, 2015 at 12:37 PM, Ganesh Ajjanagadde >> wrote: >>> On Wed, Oct 14, 2015 at 5:01 AM, Matt Oliver

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Thu, Oct 15, 2015 at 6:57 AM, Hendrik Leppkes wrote: > On Thu, Oct 15, 2015 at 12:53 PM, Ganesh Ajjanagadde wrote: >> On Thu, Oct 15, 2015 at 6:50 AM, Hendrik Leppkes wrote: >>> On Thu, Oct 15, 2015 at 12:41 PM, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Wed, Oct 14, 2015 at 6:54 AM, Carl Eugen Hoyos wrote: > Hendrik Leppkes gmail.com> writes: > >> One thing I don't understand - why are we bothering >> with something that at best comes out as "same speed" >> from tests performed? (low number of runs are >> irrelevant as they

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

Ganesh Ajjanagadde mit.edu> writes: > >> I think the general case, it'd be nice to figure out > >> why Carl's results are slightly different from yours > > > > Why do you think they are different at all? > > Did you look at the tables? > > They are different, and our conclusions are different

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Wed, Oct 14, 2015 at 2:40 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: > >> >> I think the general case, it'd be nice to figure out >> >> why Carl's results are slightly different from yours >> > >> > Why do you think they are different at all? >> > Did

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Wed, Oct 14, 2015 at 5:01 AM, Matt Oliver wrote: > On 14 October 2015 at 09:46, Ganesh Ajjanagadde wrote: > >> On Tue, Oct 13, 2015 at 9:12 AM, Ganesh Ajjanagadde >> wrote: >> > On Tue, Oct 13, 2015 at 4:02 AM, Clément Bœsch

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Wed, Oct 14, 2015 at 12:37 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 5:01 AM, Matt Oliver wrote: >> On 14 October 2015 at 09:46, Ganesh Ajjanagadde wrote: >> >>> On Tue, Oct 13, 2015 at 9:12 AM, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

Ganesh Ajjanagadde mit.edu> writes: > What? My numbers actually show that the new code may be faster - No, you are misunderstanding the numbers you posted. (Or I misunderstand them but nobody said so yet.) Highest runs are most relevant, skips have to be avoided (afaik). [...] > If you

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Wed, Oct 14, 2015 at 6:49 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: > >> What? My numbers actually show that the new code may be faster - > > No, you are misunderstanding the numbers you posted. > (Or I misunderstand them but nobody said so yet.) > >

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

Ronald S. Bultje gmail.com> writes: > I think the general case, it'd be nice to figure out > why Carl's results are slightly different from yours Why do you think they are different at all? Did you look at the tables? Carl Eugen ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 11:15 PM, Carl Eugen Hoyos wrote: > Ronald S. Bultje gmail.com> writes: > >> I think the general case, it'd be nice to figure out >> why Carl's results are slightly different from yours > > Why do you think they are different at all? > Did you look at

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh Ajjanagadde wrote: > On Tue, Oct 13, 2015 at 12:26 AM, Ganesh Ajjanagadde wrote: > > On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos wrote: > >> Ganesh Ajjanagadde mit.edu> writes: > >> > >>> Bench from

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On 10/13/15, Clement Boesch wrote: > On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh Ajjanagadde wrote: >> On Tue, Oct 13, 2015 at 12:26 AM, Ganesh Ajjanagadde >> wrote: >> > On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos >> > wrote: >> >>

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 09:25:03AM +0200, Paul B Mahol wrote: [...] > What about fmax/FFMAX? Feel free to try that out (it looks OT regarding the patch), but fmax() looks glibc specific -- Clément B. signature.asc Description: PGP signature ___

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 2:45 AM, Clément Bœsch wrote: > On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh Ajjanagadde wrote: >> On Tue, Oct 13, 2015 at 12:26 AM, Ganesh Ajjanagadde >> wrote: >> > On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 9:12 AM, Ganesh Ajjanagadde wrote: > On Tue, Oct 13, 2015 at 4:02 AM, Clément Bœsch wrote: >> On Tue, Oct 13, 2015 at 09:25:03AM +0200, Paul B Mahol wrote: >> [...] >>> What about fmax/FFMAX? >> >> Feel free to try that out (it looks OT

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 8:58 PM, Ronald S. Bultje wrote: > Hi, > > On Tue, Oct 13, 2015 at 8:09 PM, Ganesh Ajjanagadde > wrote: > >> On Tue, Oct 13, 2015 at 2:45 AM, Clément Bœsch wrote: >> > On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh

[FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

Hi all, Once again, gmail is misbehaving with my patches. This one is a ~50 line diff, so there should be no reason at all for it. I am trying to switch to my university's mail smtp for this send-email as I am tired of this nonsense (note: this is after allowing gmail access to "less secure"

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Mon, Oct 12, 2015 at 7:59 AM, Ganesh Ajjanagadde wrote: > On Mon, Oct 12, 2015 at 7:46 AM, Carl Eugen Hoyos wrote: >> Ganesh Ajjanagadde mit.edu> writes: >> >>> It is well known that fabs and fabsf are at least as fast and usually >>> faster than the FFABS

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

Ganesh Ajjanagadde mit.edu> writes: > On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos wrote: > > Ganesh Ajjanagadde mit.edu> writes: > > > >> Bench from libavfilter/astats on a 15 min clip. > > > > I believe that your test would indicate that the > > old variant is faster or that no result

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Mon, Oct 12, 2015 at 4:57 PM, Ganesh Ajjanagadde wrote: > On Mon, Oct 12, 2015 at 7:59 AM, Ganesh Ajjanagadde wrote: >> On Mon, Oct 12, 2015 at 7:46 AM, Carl Eugen Hoyos wrote: >>> Ganesh Ajjanagadde mit.edu> writes: >>> It is well

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: > >> Bench from libavfilter/astats on a 15 min clip. > > I believe that your test would indicate that the > old variant is faster or that no result can be > given which is what my

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 12:26 AM, Ganesh Ajjanagadde wrote: > On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos wrote: >> Ganesh Ajjanagadde mit.edu> writes: >> >>> Bench from libavfilter/astats on a 15 min clip. >> >> I believe that your test would indicate

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 12:44 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: >> On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos wrote: >> > Ganesh Ajjanagadde mit.edu> writes: >> > >> >> Bench from libavfilter/astats on a 15 min clip. >> > >> > I believe

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

On Tue, Oct 13, 2015 at 1:03 AM, Ganesh Ajjanagadde wrote: > On Tue, Oct 13, 2015 at 12:44 AM, Carl Eugen Hoyos wrote: >> Ganesh Ajjanagadde mit.edu> writes: >>> On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos wrote: >>> > Ganesh Ajjanagadde mit.edu>