[Webkit-unassigned] [Bug 75522] Enable IPP for FFTFrame

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 5 21:01:09 PST 2012


--- Comment #10 from Chris Rogers <crogers at google.com>  2012-01-05 21:01:09 PST ---
(In reply to comment #9)
> I tried the convolution layout test and failed with the result:
> *******
> Tests ConvolverNode processing a mono channel with mono impulse response.
> On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
> PASS Initial latency of convolver is silent.
> FAIL Triangular portion of convolution is not incorrect.  Max deviation = 2.0867834820236123e-7
> FAIL First part of tail of convolution is not sufficiently small: 0.00244140625
> PASS Rendered signal after tail of convolution is silent.
> FAIL Test signal was not correctly convolved.
> PASS successfullyParsed is true
> ******
> I found the threshold values are experimental, the "Max deviation" and "small" I got are only larger than the values in test script a little. I wonder to know whether such values are tolerable?

I believe this is well within an acceptable tolerance, and the performance improvement is significant.  It would be great to go with the "fast" option.

Ray, can you sanity check this and refine the layout test by re-defining the "Max deviation" and "not sufficiently small" in terms of decibels. I can see that "Max deviation" in terms of dB deviation is very slight.  The "not sufficiently small" can be compared to the "peak" value of the triangle to see how far down "in the noise" we are.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list