Update running avg for VP9 denoiser

Change-Id: I9577d648542064052795bf5770428fbd5c276b7b

Implemented vp9_denoiser_{alloc,free}()

Change-Id: I79eba79f7c52eec19ef2356278597e06620d5e27

Merge "Improve vp9_rb_bytes_read"

Improve vp9_rb_bytes_read

Change-Id: I69eba120eb3d8ec43b5552451c8a9bd009390795

Merge "gen_msvs_proj: fix in tree configure under cygwin"

Merge "Moving RD-opt related code from vp9_encoder.h to vp9_rdopt.h."

Merge "skip the un-necessary motion search in the first pass"

Merge "iosbuild.sh: Add missing function comments."

Merge "vp8 denoising: add bias factor to zero_mv sse."

Merge "Fix C versions of DC calculation functions"

Merge "vp9_rtcd: correct avx2 references"

iosbuild.sh: Add missing function comments.

Change-Id: Ib23a59475d566a7b7f44071614d730ceecfcfa60

Merge "example tests: Make failures due to incorrect bin path easier to diagnose."

vp8 denoising: add bias factor to zero_mv sse.

Change-Id: I95818754424e89f0d56c6d9c0c5709e6f84fa46a

gen_msvs_proj: fix in tree configure under cygwin

strip trailing '/' from paths, this is later converted to '\' which
causes execution errors for obj_int_extract/yasm. vs10+ wasn't affected
by this issue, but make the same change for consistency.

gen_msvs_proj:
+ add missing '"' to obj_int_extract call
  unlike gen_msvs_vcproj, the block is duplicated
  missed in: 1e3d9b9 build/msvs: fix builds in source dirs with spaces

Change-Id: I76208e6cdc66dc5a0a7ffa8aa1edbefe31e4b130

Merge "vp9_pickmode.c: fix vs12 compiler warnings"

example tests: Make failures due to incorrect bin path easier to diagnose.

Add elog() to tools_common, and checks for the executable in each test.

Change-Id: I3c6334eca62c0b56040d91145abbe1bf5f9c763c

Merge "Add non420 support to vp9_extend_frame_borders."

Merge "Fix out of boundary memory read in fuzz test on vpxdec"

skip the un-necessary motion search in the first pass

This patch allows the VP9 encoder to skip the un-necessary
motion search in the first pass. It computes the motion error
of 0,0 motion using the last source frame as the reference,
and skips the further motion search if this error is small.

Borg test shows overall the patch gives PSNR gain (derf -0.001%,
yt 0.341%, hd 0.282%). Individual clips may have PSNR gain or
loss. The best PSNR performance is 7.347% and the worst is -0.662%.
The first pass encoding speedup for slideshow clips is over 30%.

Change-Id: I4cac4dbd911f277ee858e161f3ca652c771344fe

vp9_pickmode.c: fix vs12 compiler warnings

Change-Id: I5042b76a7050c121bf960ecb20c79d35adcc4cd5

Merge "iosbuild.sh: Move to build/make and tidy up a bit."

Merge "test/tools_common.sh: Log all shared variables in verbose mode."

Fix C versions of DC calculation functions

This commit fixes the scaling factors used in the C versions of the
DC calculation functions.

Change-Id: Iab41108c2bb93c2f2e78667214f3a772a2b707b5

Merge "Delay decreasing reference count in frame-parallel decoding."

Merge "Revert "skip un-neccessary motion search in the first pass""

Merge "Allow for deblocking temporal-denoised signal."

Allow for deblocking temporal-denoised signal.

Allow for an option to selectively apply the deblocking loop filter to the denoised
raw block, based on the denoised state (no-filter, filter with zero motion, or filter with non-zero motion)
of the current block and its upper and left denoised block.
This helps to reduce some blocking artifacts from the motion-compensated denoising.

Change-Id: I0ac4e70076df69a98c5391979e739a2681e24ae6

Moving RD-opt related code from vp9_encoder.h to vp9_rdopt.h.

Change-Id: I8fab776c8801e19d3f5027ed55a6aa69eee951de

Replacing RC_MODE with vpx_rc_mode.

Both enums are identical.
Change-Id: I06653f9c90a2d3a2dd5c741e75b17ee7d066a56f

test/tools_common.sh: Log all shared variables in verbose mode.

Several variables were not being logged.

Change-Id: I2f2ded19470a73c4551bbb1abbd6e4d27bd59fac

Fix out of boundary memory read in fuzz test on vpxdec

This commit fixes frame header decoding for superframe index, to
prevent out of boundary memory read triggered by fuzz test
vector. It resolves a chromium security violation issue
crbug.com/376802.

The issue was introduced in the change:

Add VPXD_SET_DECRYPTOR support to the VP9 decoder.
cl-id I88f86c8ff9af34e0b6531028b691921b54c2fc48

where the buffer was read before validation check on index offset
applied.

A test vector is added accordingly.

Change-Id: I41c988e776bbdd1033312a668e03a3dbcf44ca99

iosbuild.sh: Move to build/make and tidy up a bit.

Change-Id: I625beea9db4b5f6db8f177c580a3adeac760a662

Revert "skip un-neccessary motion search in the first pass"

This patch appears to have introduced non-determinism and/or
mismatch from debug vs release.

This reverts commit 5daef90efc4613efd7e7ee80ba4e1ecf9a57a966.

Change-Id: I80081e55cfeaaa821b510b58a4e6e6328003c7da

Delay decreasing reference count in frame-parallel decoding.

The current decoding scheme will decrease the reference count
of the output frame when finish decoding. Then the application
could copy the frame from the decoder buffer to application buffer.
In frame-parallel decoding, a decoded frame will not be outputted
until several frames later which depends on thread numbers. So
the decoded frame's reference count should be decreased only
after application finish copying the frame out. But due to the
limitation of vpx_codec_get_frame, decoder could not know when
application finish decoding. So use a index last_show_frame to
release the last output frame's reference count.

Change-Id: I403ee0d01148ac1182e5a2d87cf7dcc302b51e63

Merge "Use lrand48 on Android"

Merge "Add VPX.framework built script."

Merge "Added skeleton for VP9 denoiser"

Merge "Cleaning up accumulate_frame_motion_stats()."

Add VPX.framework built script.

Builds a framework for use on iOS targets.

Change-Id: I2e42d77d1d83e15dc7e84232255da26809ee31bb

Use lrand48 on Android

When building x86 assembly use lrand48 instead of the
undocumented inlined _rand function.

Android now supports rand()
https://android-review.googlesource.com/97731
but only for new versions. Original workaround:
https://gerrit.chromium.org/gerrit/15744

Change-Id: I130566837d5bfc9e54187ebe9807350d1a7dab2a

Merge "Adding MV_SPEED_FEATURES struct."

Added skeleton for VP9 denoiser

Change-Id: Iccf6ede4c4f85646b0f8daec47050ce93e267c90

Merge "Added OUTPUT_YUV_DENOISED CFLAG to VP8 encoder"

Merge "Initially add frame_parallel_decode flag."

Merge "Removing unused ssim_weighted_pred_err field from FIRSTPASS_STATS."

Adding MV_SPEED_FEATURES struct.

Moving all motion vector related speed parameters from SPEED_FEATURES to
MV_SPEED_FEATURES.

Change-Id: I3e9af0039c7162f8671878c5920bce3cb256a84e

Merge "Moving full_pixel_search() to vp9_mcomp.c."

Merge "Adding is_altref_enabled() function."

Merge "Replacing txfm_size with tx_size."

Moving full_pixel_search() to vp9_mcomp.c.

Change-Id: I12389f801ebd3bd2ae3bf31e125433bfb429ee65

Adding is_altref_enabled() function.

Change-Id: I54cdb4ce11590511e6f86bc2fd55771f1c18a20a

Merge "Fast computation path for forward transform and quantization"

Replacing txfm_size with tx_size.

Change-Id: Ifa6374e9db5919322733b656e0865f5f19ee6f2c

Removing unused ssim_weighted_pred_err field from FIRSTPASS_STATS.

Change-Id: Ia8c7e3905ac21732cb6b8099eaf8df72c7e36b73

Fast computation path for forward transform and quantization

This commit enables a fast path computational flow for forward
transformation. It checks the sse and variance of prediction
residuals and decides if the quantized coefficients are all
zero, dc only, or more. It then selects the corresponding coding
path in the forward transformation and quantization stage.

It is currently enabled in rtc coding mode. Will do it for rd
coding mode next.

In speed -6, the runtime for pedestrian_area 1080p at 1000 kbps
goes down from 14234 ms to 13704 ms, i.e., about 4% speed-up.
Overall coding performance for rtc set is changed by -0.18%.

Change-Id: I0452da1786d59bc8bcbe0a35fdae9f623d1d44e1

Merge "Fix SEG_LVL_SKIP in non-RD inter mode selection."

Merge "Fix SEG_LVL_SKIP in RD inter mode selection."

Merge "skip un-neccessary motion search in the first pass"

skip un-neccessary motion search in the first pass

This patch allows the encoder to skip the
un-neccessary motion search in the first pass. It
calculates the error of the zero motion vector using
the last source frame as reference and skips the
further motion search in the first pass if the error
is small.

The encoding speedup of the first pass for slideshow
videos is over 30%. Borg test shows the overall PSNR
performance remain approximately the same (derf -0.009,
hd 0.387, yt 0.021, stdhd 0.065). Individual clips may
have either PSNR gain or loss. The worst PSNR perfomance
is from yt set, with a PSNR loss of -1.1.

Change-Id: I08b2ab110b695e4689573b2567fa531b6457616e

Merge "[spatial svc]Combine first and second pass test to keep stats data in memory."

Fix SEG_LVL_SKIP in non-RD inter mode selection.

Add a set_mode_info_seg_skip function that fills the requisite mode info.

Change-Id: I460b1b6845d720d9b09ed5b64df0ea0aac443f62

Fix SEG_LVL_SKIP in RD inter mode selection.

* Only use ZEROMV, disalowing the intra modes that were previously
  tested.
* Score rate and distortion as zero.

Change-Id: Ifcf99e272095725f11da1dcd26bd0f850683e680

Initially add frame_parallel_decode flag.

Stub flag temporarily set to 0 until frame parallel
decoding implementations are finished.

Change-Id: I8ab768138e8f8f8eb809875703b2502ea0fe7cea

Merge "Add target armv7s-darwin-gcc."

Add non420 support to vp9_extend_frame_borders.

Fixes an encoder/decoder mismatch problem.

Change-Id: I573b3a2b7ba2171a1a380ff201b082b084e7ade1

[spatial svc]Combine first and second pass test to keep stats data in memory.

Change-Id: Idccbfe35bebe6f05655bd54da7d8b616b1bffe03

Add target armv7s-darwin-gcc.

Really just armv7. This is a convenience target intended to make iOS
development with libvpx easier. Xcode projects with default settings
will fail to build when a framework lacks armv7s support when targetting
iOS7.

Change-Id: I7eb80d52eec25501febc0d2c3c0b4ed964b8ed5b

Merge "Cleaning up vp9_variance_mmx.c."

Merge "Removing two unused TX_SIZE_SEARCH_METHOD members."

vp9_rtcd: correct avx2 references

s/"\$avx2_x86inc"/"avx2"/

avx2 code is all intrinsics and as a result doesn't rely on x86inc.asm

Change-Id: I76ad39474d8a00658f3e43131830ef0f4f34772a

Merge changes I6abc0657,I8224fba2,I04f64a45,I5d49d119,I76b4d171,I88c11ac3

* changes:
  vp9_sub_pixel_*variance*: disable avx2 variants
  vp9_sad*x4d: disable avx2 variants
  vp9_f(dct|ht): disable avx2 variants
  convolve: disable avx2 variants
  fdct8x8_test: add missing avx2 functions
  dct4x4_test: add missing avx2 functions

vp9_sub_pixel_*variance*: disable avx2 variants

tests failing under Win32/Win64

+ variance_test: add missing avx2 functions (partially disabled)

Change-Id: I6abc0657ea076379ab9ca65c12678b9ea199849d

vp9_sad*x4d: disable avx2 variants

tests failing under Win32/Win64

+ sad_test: add missing avx2 functions (disabled)

Change-Id: I8224fba2b270f6039ab1877d71e1e512f0081856

Merge "Makefile: skip .d inclusion for 'testdata' target"

Merge "Add mode info arrays and mode info index."

Add mode info arrays and mode info index.

In non frame-parallel decoding, this works the same way as
current decoding scheme. Every time after decoder finish
decoding a frame, it will swap the current mode info pointer
and  previous mode info pointer if the decoded frame needs
to be shown. Both mode info pointer and previous mode info
pointer are from mode info arrays.

In frame-parallel decoding, this will become more complicated
as current frame's mode info pointer will be shared with next
frame as previous mode info pointer. But when one decoder
thread finishes decoding one frame and starts to work on next
available frame, it needs to retain the decoded frame's mode
info pointers until next frame finishes decoding. The mode info
index will serve this purpose. The decoder will use different
buffer in the mode info arrays and use the other buffer to save
previous decoded frame’s mode info.

Change-Id: If11d57d8eb0ee38c8876158e5482177fcb229428

Removing two unused TX_SIZE_SEARCH_METHOD members.

Change-Id: I33a38bb9f46e7ef509bbbf0cfd7bc3ea5072d022

Merge "Removing chessboard_index from SPEED_FEATURES."

Merge "Removing unused motion_vector_context enum from vp9_encodeframe.c"

Merge "Add x86_64-iphonesimulator-gcc target."

vp9_f(dct|ht): disable avx2 variants

tests failing under Win32/Win64

+ dct16x16_test: add missing avx2 functions (partially disabled)

exercises the forward transforms
no idct/iht implementations, so the c-code is used

Change-Id: I04f64a457fa0828a00f32b5c9fe4f55294f21f61

convolve: disable avx2 variants

tests failing under Win32/Win64

Change-Id: I5d49d11911bcda3a832b14efe5500d22597bedcf

fdct8x8_test: add missing avx2 functions

exercises the forward transforms
no idct/iht implementations, so the c-code is used

Change-Id: I76b4d1712f10225c1ffa5ffb0ed9a551e68b93b4

dct4x4_test: add missing avx2 functions

exercises the forward transforms
no idct/iht implementations, so the c-code is used

Change-Id: I88c11ac37ac6456de9b4b3ec5da059faf4ad6066

Merge "Use small transform size in non-rd real-time mode"

Merge "Removing unused tt_activity_measure()."

Use small transform size in non-rd real-time mode

In non-rd real-time mode, choosing smaller transform size in
encoding gives better video quality and good speed gain than
choosing larger transform size. This patch set tx size search
method to ALLOW_8X8, which is better than using 4x4 or other
larger sizes.

Borg tests on rtc set at speed 6 showed significant gain on quality.
PSNR gain: 11.034% and SSIM gain: 15.466%.

The speed gain is 5% - 12% for <720p clips, and 2% - 7% for
720p clips.

Change-Id: If4dc74ed2df359346b059f47fb73b4a0193ec548

Merge "Fix internal stats printing"

Makefile: skip .d inclusion for 'testdata' target

avoids some unnecessary work when pulling testdata in a freshly
configured tree

Change-Id: Ib43379b0e1549107f3f1b227afca49a899bcc3ab

Merge "gen_msvs_*proj.sh: speed up file generation"

gen_msvs_*proj.sh: speed up file generation

execute fix_path once on the source file list rather than once per entry

Change-Id: Ibc8226e391b3028c1b0bcfeab83c790387c9fe23

Add x86_64-iphonesimulator-gcc target.

macho64 x86_64 with:
- -miphoneos-version-min=5.0
- -isysroot from xcrun --sdk iphonesimulator

Change-Id: I1ba183e16d50c93d48f526e2dc43a7a6040741c5

Merge "Revert "Removing this_frame_stats member from TWO_PASS struct.""

Revert "Removing this_frame_stats member from TWO_PASS struct."

Use of stack frame variable "fps" beyond the lifetime of the function.

fps is sent as a paremeter to output_stats and stored in the
packet holding this encoded frame. This has scope beyond the
lifetime of the calling function.

This reverts commit 3f95a230c7e54104201b3f6c577dfa9251b4642a

Change-Id: Icd8e14b3d7dd733590ada12e619b9dce95b6b0f5

Merge "Adding encode_tiles() function."

Merge "Fixes qindex for first frame in 1-pass cq/q modes"

Merge "README: add a note about the test vectors"

Merge changes I0e4d807f,Ia5ff575c,Ie4a1f313

* changes:
  gen_msvs_*proj.sh: strip SRC_PATH_BARE from obj names
  *.mk: pass SRC_PATH_BARE to all GEN_VCPROJ invocations
  build/msvs: fix builds in source dirs with spaces

Removing chessboard_index from SPEED_FEATURES.

This is not a speed feature, adding inline function instead.

Change-Id: Ia48c41802eec9e92cf990339d724097279695c9a