Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Wed, 2016-06-22 at 20:33 -0300, James Almer wrote: > On 6/22/2016 8:15 PM, Dan Parrot wrote: > > On Thu, 2016-06-23 at 01:03 +0200, Michael Niedermayer wrote: > >> On Tue, Jun 21, 2016 at 12:04:42AM -0500, Dan Parrot wrote: > >>> On Tue, 2016-06-21 at 02:22 +0200, Michael Niedermayer wrote: >

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On 6/22/2016 8:15 PM, Dan Parrot wrote: > On Thu, 2016-06-23 at 01:03 +0200, Michael Niedermayer wrote: >> On Tue, Jun 21, 2016 at 12:04:42AM -0500, Dan Parrot wrote: >>> On Tue, 2016-06-21 at 02:22 +0200, Michael Niedermayer wrote: On Mon, Jun 20, 2016 at 06:38:18PM -0500, Dan Parrot wrote:

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

Dan Parrot mail.com> writes: > > Did you already test the TIMER macros? > > No I did not test with the TIMER macros. If you refuse to run the requested performance test, please at least post your patch, maybe others find the necessary time. Carl Eugen

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Thu, 2016-06-23 at 01:03 +0200, Michael Niedermayer wrote: > On Tue, Jun 21, 2016 at 12:04:42AM -0500, Dan Parrot wrote: > > On Tue, 2016-06-21 at 02:22 +0200, Michael Niedermayer wrote: > > > On Mon, Jun 20, 2016 at 06:38:18PM -0500, Dan Parrot wrote: > > > > On Tue, 2016-06-21 at 01:06 +0200,

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On 6/22/2016 7:36 PM, Carl Eugen Hoyos wrote: > Dan Parrot mail.com> writes: > > [...] > > Did you already test the TIMER macros? > > I don't know if they work on ppc64le but we won't know > until you tested them. > > Thank you, Carl Eugen If this patch works (FATE passes on ppc64) and is

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Tue, Jun 21, 2016 at 12:04:42AM -0500, Dan Parrot wrote: > On Tue, 2016-06-21 at 02:22 +0200, Michael Niedermayer wrote: > > On Mon, Jun 20, 2016 at 06:38:18PM -0500, Dan Parrot wrote: > > > On Tue, 2016-06-21 at 01:06 +0200, Michael Niedermayer wrote: > > > > On Mon, Jun 20, 2016 at 05:55:47PM

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Wed, 2016-06-22 at 22:36 +, Carl Eugen Hoyos wrote: > Dan Parrot mail.com> writes: > > [...] > > Did you already test the TIMER macros? No I did not test with the TIMER macros. I don't see what that has to do with Trac ticket #5570. > I don't know if they work on ppc64le but we won't

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

Dan Parrot mail.com> writes: [...] Did you already test the TIMER macros? I don't know if they work on ppc64le but we won't know until you tested them. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Wed, 2016-06-22 at 21:02 +, Carl Eugen Hoyos wrote: > Dan Parrot mail.com> writes: > > > Could I get a yes or no answer on whether the patch will be applied? > > Please comment on my email: time make fate can be used to show > large performance changes (although it isn't optimal), I

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

Dan Parrot mail.com> writes: > Could I get a yes or no answer on whether the patch will be applied? Please comment on my email: time make fate can be used to show large performance changes (although it isn't optimal), I don't think it can show the difference between a division and a shift.

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

Could I get a yes or no answer on whether the patch will be applied? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Tue, 2016-06-21 at 00:04 -0500, Dan Parrot wrote: > On Tue, 2016-06-21 at 02:22 +0200, Michael Niedermayer wrote: > > On Mon, Jun 20, 2016 at 06:38:18PM -0500, Dan Parrot wrote: > > > On Tue, 2016-06-21 at 01:06 +0200, Michael Niedermayer wrote: > > > > On Mon, Jun 20, 2016 at 05:55:47PM -0500,

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

Dan Parrot mail.com> writes: > "/usr/bin/time -p make -j 4 fate SAMPLES=fate-suite/" I would expect that this depends on many factors outside of your patch. You could test with the START_TIMER and STOP_TIMER macros. (Or look at the asm code that is produced...) Carl Eugen

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Tue, 2016-06-21 at 02:22 +0200, Michael Niedermayer wrote: > On Mon, Jun 20, 2016 at 06:38:18PM -0500, Dan Parrot wrote: > > On Tue, 2016-06-21 at 01:06 +0200, Michael Niedermayer wrote: > > > On Mon, Jun 20, 2016 at 05:55:47PM -0500, Dan Parrot wrote: > > > > On Tue, 2016-06-21 at 00:45 +0200,

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Tue, 2016-06-21 at 02:22 +0200, Michael Niedermayer wrote: > On Mon, Jun 20, 2016 at 06:38:18PM -0500, Dan Parrot wrote: > > On Tue, 2016-06-21 at 01:06 +0200, Michael Niedermayer wrote: > > > On Mon, Jun 20, 2016 at 05:55:47PM -0500, Dan Parrot wrote: > > > > On Tue, 2016-06-21 at 00:45 +0200,

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Mon, Jun 20, 2016 at 06:38:18PM -0500, Dan Parrot wrote: > On Tue, 2016-06-21 at 01:06 +0200, Michael Niedermayer wrote: > > On Mon, Jun 20, 2016 at 05:55:47PM -0500, Dan Parrot wrote: > > > On Tue, 2016-06-21 at 00:45 +0200, Michael Niedermayer wrote: > > > > On Sun, Jun 19, 2016 at 09:57:42PM

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Tue, 2016-06-21 at 01:06 +0200, Michael Niedermayer wrote: > On Mon, Jun 20, 2016 at 05:55:47PM -0500, Dan Parrot wrote: > > On Tue, 2016-06-21 at 00:45 +0200, Michael Niedermayer wrote: > > > On Sun, Jun 19, 2016 at 09:57:42PM +, Dan Parrot wrote: > > > > First commit addressing Trac

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Mon, Jun 20, 2016 at 05:55:47PM -0500, Dan Parrot wrote: > On Tue, 2016-06-21 at 00:45 +0200, Michael Niedermayer wrote: > > On Sun, Jun 19, 2016 at 09:57:42PM +, Dan Parrot wrote: > > > First commit addressing Trac ticket #5570. Functions defined in > > > libswscale/input.c > > > have

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Tue, 2016-06-21 at 00:45 +0200, Michael Niedermayer wrote: > On Sun, Jun 19, 2016 at 09:57:42PM +, Dan Parrot wrote: > > First commit addressing Trac ticket #5570. Functions defined in > > libswscale/input.c > > have corresponding SIMD definitions in libswscale/ppc/input_vsx.c > > --- > >

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Sun, Jun 19, 2016 at 09:57:42PM +, Dan Parrot wrote: > First commit addressing Trac ticket #5570. Functions defined in > libswscale/input.c > have corresponding SIMD definitions in libswscale/ppc/input_vsx.c > --- > libswscale/ppc/Makefile |1 + > libswscale/ppc/input_vsx.c|

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

Could a PPC maintainer verify that this patch integrates cleanly into the design? I would like to proceed with the remaining changes to close out ticket #5570 but since the first patch was rejected, I am unsure on whether I'll have to rewrite the code. Thanks. Dan. On Sun, 2016-06-19 at 21:57

[FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

First commit addressing Trac ticket #5570. Functions defined in libswscale/input.c have corresponding SIMD definitions in libswscale/ppc/input_vsx.c --- libswscale/ppc/Makefile |1 + libswscale/ppc/input_vsx.c| 1070 + libswscale/swscale.c

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Wed, 2016-06-15 at 16:51 +0200, Hendrik Leppkes wrote: > On Wed, Jun 15, 2016 at 6:25 AM, Dan Parrot wrote: > > This is the first commit addressing Trac ticket #5570. Functions defined in > > libswscale/input.c have corresponding definitions in > >

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Wed, 2016-06-15 at 11:19 +, Carl Eugen Hoyos wrote: > Dan Parrot mail.com> writes: > > [...] > > I know this is isn't completely related but do you have time > to look at ticket #5508? > https://trac.ffmpeg.org/ticket/5508 > No active developer has hardware and knowledge to look into >

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Wed, Jun 15, 2016 at 6:25 AM, Dan Parrot wrote: > This is the first commit addressing Trac ticket #5570. Functions defined in > libswscale/input.c have corresponding definitions in > libswscale/ppc/input_vsx.h > The corresponding function names in the latter contain the

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Wed, 2016-06-15 at 10:15 +0200, Michael Niedermayer wrote: > On Wed, Jun 15, 2016 at 04:25:11AM +, Dan Parrot wrote: > > This is the first commit addressing Trac ticket #5570. Functions defined in > > libswscale/input.c have corresponding definitions in > > libswscale/ppc/input_vsx.h > >

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

Dan Parrot mail.com> writes: [...] I know this is isn't completely related but do you have time to look at ticket #5508? https://trac.ffmpeg.org/ticket/5508 No active developer has hardware and knowledge to look into this issue;-( Carl Eugen ___

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

On Wed, Jun 15, 2016 at 04:25:11AM +, Dan Parrot wrote: > This is the first commit addressing Trac ticket #5570. Functions defined in > libswscale/input.c have corresponding definitions in > libswscale/ppc/input_vsx.h > The corresponding function names in the latter contain the suffix "_vsx".

[FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

This is the first commit addressing Trac ticket #5570. Functions defined in libswscale/input.c have corresponding definitions in libswscale/ppc/input_vsx.h The corresponding function names in the latter contain the suffix "_vsx". --- libswscale/input.c | 44 +--