Merge "vp8:fix threading issues"

vp8:fix threading issues

1 - stops de allocating before threads are closed.
2 - limits threads to mb_rows when mb_rows < partitions

BUG=webm:851

Change-Id: I7ead53e80cc0f8c2e4c1c53506eff8431de2a37e

Add VPX_SWAP macro

Change-Id: I60e233eddef238ad918183392794084673f27d2d

Merge "vp9 1pass-vbr: Adjust gf setting for nonzero-lag case."

vp9 1pass-vbr: Adjust gf setting for nonzero-lag case.

Change-Id: I230c586c6d5ae56ee9a6d37b7d9452351bb4bd80

Merge "Sample points to reduce encode overhead."

Merge "Noise energy Experiment in first pass."

Add back header in threading.h

Added back the header needed in threading.h

Change-Id: I2ce66ad4fe58004997623f6c3f3b8dd11640aa98

Merge "Revert "Amend and improve VP8 multithreading implementation""

Revert "Amend and improve VP8 multithreading implementation"

Reverted the patch because of possible performance issue.

Change-Id: I49944f827ccd38ed194c9f8d9cb9036fa9bf79e1

MinArfFreqTest: Don't leak video on failure.

Change-Id: I250379f0ac8d4929c9032e7343290e2980fc2e77

Make test encoder test driver less likely to leak on failure.

Individual tests still need to be updated.

Change-Id: Ic433d0f742e13560b136f136b72b2a9973970d78

Merge changes from topic 'clang-tidy'

* changes:
  vp8/onyx_if.c: rework #if's to avoid dangling else's
  vp8/bitstream.c: rework #if to avoid dangling else

Merge remote-tracking branch 'origin/khakicampbell'

Release v1.6.0 Khaki Campbell Duck

Change-Id: I08da365dd889093f9919476a02ee96ae9615f140

vp9: Fix the clang warning of unsigned int type.

Change-Id: I6308db16bd626fa5943925471e9171f567669350

Fix encoder crashes for odd size input

(cherry picked from commit 98431cde070a33fe346f1316abd8437cd873dbfa)

Change-Id: Id5c30c419282369cc8c3280d9a70b34a859a71d8

vp8/onyx_if.c: rework #if's to avoid dangling else's

Change-Id: Ieda8958a3da1000424fcff91a1315d0049612202

vp8/bitstream.c: rework #if to avoid dangling else

Change-Id: I9178ae75876f3df3fa3271314db39830552b9549

Merge changes from topic 'clang-tidy'

* changes:
  vp8/{bitstream,rdopt},y4minput: correct break placement
  y4minput.c: correct empty loop formatting
  vp8: simplify a few #if's
  vp8: remove extra semicolons

Change to call vp9_post_proc_frame()

This commit changes the call in vp9 encoder from vp9_deblock() to
vp9_post_proc_frame() to ensure the data structures used in the call
are properly allocated. This fixes an encoder crash when configured
with --enable-internal-stats.

Change-Id: I2393b336c0f566665336df4f1ba91c405eb56764

vp8/{bitstream,rdopt},y4minput: correct break placement

these should be placed within {}s when present

Change-Id: If00e9766fa8cb039cc070467f353a468f99460fb

y4minput.c: correct empty loop formatting

prefer {}s over ';'

Change-Id: I563fc82717e1deb4f42a40e03dca318c6adaa0c1

vp8: simplify a few #if's

bitstream.c: asserts are disabled when CONFIG_DEBUG is unset
vp8_dx_iface.c: split |s into 2 statements across #if bounds

Change-Id: I307d1e969134db5c9c0edd7690589b6b29116cbd

vp8: remove extra semicolons

Change-Id: I84e1a293ee033865f82c244e8aaaadfb2fb27e63

top-level: apply clang-format

Change-Id: Ibd5395bf8956a80f7c0df4d539c7a42c927a1fc7

Merge "vp8: apply clang-tidy google-readability-braces-around-statements"

vp8: apply clang-tidy google-readability-braces-around-statements

applied against an x86_64 configure

clang-tidy-3.7.1 \
  -checks='-*,google-readability-braces-around-statements' \
  -header-filter='.*' -fix
+ clang-format afterward

Change-Id: I6694edeaee89b58b8b3082187e6756561136b459

Merge "examples: apply clang-format"

Merge changes Ia6004c08,I1954f9d6

* changes:
  cosmetics: Add a few explanatory comments
  cosmetics: Correct grammar/spelling in comments

cosmetics: Add a few explanatory comments

Change-Id: Ia6004c08e6f5fd269a1bbd4df51ce9b76345150d

Merge "vp9: Allow usage of lookahead for real-time, 1 pass vbr."

Merge "Change 'git cl upload' default to --no-squash"

Merge changes I18982dbf,I15c8976c

* changes:
  build/make/Makefile: add a 'test_*' default target
  build/make/Makefile: remove default suffix rules

Merge "take II: variance_test partial clean-up"

examples: apply clang-format

Change-Id: Icc3bbb07c99a31a70030baec7e51b881902a7b5e

Merge "configure: turn on all unused warnings by default"

build/make/Makefile: add a 'test_*' default target

allows 'make test_libvpx', etc. some reworking of the makefiles would be
needed to avoid hard coding targets here.

Change-Id: I18982dbf691e7d36ab8bcf5934bab9340687b061

build/make/Makefile: remove default suffix rules

Change-Id: I15c8976c6478bf75ec617398f49461b310ab7569

take II: variance_test partial clean-up

remove some (but not all yet!) tuple mis-use, and revamp the code a lot.
Factorize some common chores into MainTestClass.

Change-Id: Id37b7330eebe80d19b9d12a454f24ff9be6b1116

vp9: Allow usage of lookahead for real-time, 1 pass vbr.

Allow usage of lookahead for VBR in real-time mode, for 1 pass vbr.

Current usage is for fast checking of future scene cuts/changes,
and adjusting rate control (gf interval and active_worst/target size).

Added unittests (datarate) for 1 pass vbr mode, with non-zero lag.

Added an experimental option to limit QP based on lookahead.

Overall positive gain in metrics on ytlive set:
avgPNSR/SSIM up on average ~1-3%; several clips up by 5, 7%.

Change-Id: I960d57dfc89de121c4824b9a9bf88d2814e74b56

Change 'git cl upload' default to --no-squash

Chromium changed the upstream default to --squash but this conflicts
with libvpx historical defaults.

Change-Id: I80f2f2b48e2ba08e02184b50e6d5f8f5e76fec24

cosmetics: Correct grammar/spelling in comments

Change-Id: I1954f9d6e33abff9081fe7a5cf59d5497768e0df

prepend ++ instead of post in for loops.

Applied the following regex  :
search for: (for.*\(.*;.*;) ([a-zA-Z_]*)\+\+\)
replace with: \1 ++\2)

This misses some for loops:
ie : for (mb_col = 0; mb_col < oci->mb_cols; mb_col++, mi++)

Change-Id: Icf5f6fb93cced0992e0bb71d2241780f7fb1f0a8

Merge "Revert "variance_test partial clean-up""

vpx_plane_add_noise_c: normalize int types

quiets signed/unsigned mismatch warning

Change-Id: Iaabd7dfff110ba26056258457541f5635d2e85e6

Revert "variance_test partial clean-up"

This reverts commit f993ed5c86217e31b92ecc7ef58d7d7860b80108.

build warnings under msvc, segfaults with high-bitdepth enabled.

Change-Id: I67502651107830bcadb6ef56d8f2709cccbfdf2b

Merge "sad_test: add some const to methods"

Merge "vp9_error_block_test: simplify fn wrapper generation"

sad_test: add some const to methods

Change-Id: I6f2481509b0aa94338ed6185f80c4a6b65532280

Merge "remove tuple from 'sad_test.cc'"

vp9_error_block_test: simplify fn wrapper generation

Change-Id: I1f1d396b9456e52e863c4c75f23c3d17420668b4

remove tuple from 'sad_test.cc'

+ general clean-up

Change-Id: Ib9dca3d1a3b7f0c1bedef2a26c9ff5ae1c289e8a

vp8: apply clang-format

Change-Id: I7605b6678014a5426ceb45c27b54885e0c4e06ed

add .clang-format, based on Google style

derived from clang-format 3.7.1; same as used in libaom

Change-Id: I8ea915a41d1f2ea3b0d4e4dab9ebc808e9116f11

Merge "addnoise : clear out static size for generated noise"

addnoise : clear out static size for generated noise

Change-Id: I5d4343f2da9cd4b01dd37be7a048d159fec109d1

configure: turn on all unused warnings by default

Change-Id: I7f6cb446cd3ac57ac39835cf065d9501a66acd5b

vp9_postproc.c : unused variable if not vp9_highbitdepth.

Change-Id: Ib89b128f23767934c40b5add3fcf9dbd875e82f9

Merge "postproc : fix function parameters for noise functions."

postproc : fix function parameters for noise functions.

Change-Id: I582b6307f28bfc987dcf8910379a52c6f679173c

Merge "variance_test partial clean-up"

variance_test partial clean-up

remove some (but not all yet!) tuple mis-use, and revamp the code a lot.
Factorize some common chores into MainTestClass.

Change-Id: Ia14f3924140e8545e4f10d0504475681baae8336

Merge "vp9_intrapred_test: follow-up cleanup"

gtest-all.cc: quiet an unused variable warning

under windows / mingw builds

Change-Id: I93f9a5df77cea0c28d4afb272abcde5a9732e355

vp9_intrapred_test: follow-up cleanup

address few comments from ce050afaf3e288895c3bee4160336e2d2133b6ea

Change-Id: I5d8fc9dab35c4ee5ec3671134c4eef4ec241e309

Merge "postproc: noise style fixes."

Merge "Revert "Eliminate isolated and small tail coefficients:""

Merge "clean-up vp9_intrapred_test"

Merge "Fix encoder crashes for odd size input"

postproc: noise style fixes.

Change-Id: Ifdcb36b8e77b65faeeb10644256e175acb32275d

clean-up vp9_intrapred_test

remove tuple and overkill VP9IntraPredBase class.

Change-Id: I85b85bdd33d7fe417895e75f77db219f713dfea3

Revert "Eliminate isolated and small tail coefficients:"

This reverts commit ff19cdafdbb5ee470e4369582b0266f4bc23287d.

Change-Id: I81f68870ca27a1ff683ee22090530b6997815fb2

Merge "Disable trellis optimization when lossless is on"

Merge "postproc - move filling of noise buffer to vpx_dsp."

postproc - move filling of noise buffer to vpx_dsp.

Change-Id: I63ba35dc0ae9286c9812367a531e01d79a4c1635

deblock: missing const on extern const.

Change-Id: I0df08f7c431daf939e266f008bf5158b0c97358b

vp9_postproc.c missing extern.

BUG=webm:1256

Change-Id: I5271e71bc53cce033fb906040643dcdd5ccb2381

Merge changes from topic 'webp-thread-update'

* changes:
  vpx_thread: use CreateThread for windows phone
  vpx_thread: use WaitForSingleObjectEx if available
  vpx_thread: use InitializeCriticalSectionEx if available
  vpx_thread: use native windows cond var if available
  vpx_thread.[hc]: update webp source reference

Fix encoder crashes for odd size input

Change-Id: Id5c30c419282369cc8c3280d9a70b34a859a71d8

Merge "vp9 svc: Reuse scaled_temp in two stage downscaling."

vp9 svc: Reuse scaled_temp in two stage downscaling.

This change eliminates redundant computation in the two stage
downscaling, which saves ~1% encoding time in 3-layer svc encoding.

Change-Id: Ib4b218811b68499a740af1f9b7b5a5445e28d671

Disable trellis optimization when lossless is on

Disable trellis coefficient optimization when the lossless mode
is turned on.

Change-Id: I9001bf626e86dc3c8c32331ede04fd39036e5f7c

deblock filter : moved from vp8 code branch

The deblocking filters used in vp8 have been moved to vpx_dsp for
use by both vp8 and vp9.

Change-Id: I5209d76edafc894b550f751fc76d3aa6799b392d

Merge "remove *debugmodes.c from the default build"

Merge "vp9_rd_pick_intra_mode_sb(): set interp_filter to"

Sample points to reduce encode overhead.

Only noise filter sampled points in first pass to reduce
any first pass speed overhead.

Change-Id: Ic80d4400e59146d1c3332336c4350faf28ff8b17

vp9_rd_pick_intra_mode_sb(): set interp_filter to

SWITCHABLE_FILTERS.  This is a partial fix for the build
issues with Change 357240.

Change-Id: I4e507c196175bae729a4f1397878ec8776b0146c

Fix non-highbitdepth coding path for HBD build

Change-Id: I38eb42b8d051924a7cd1ccc3421a4057cf6e170f

Merge "vp9: Adjustment of gfu_boost and af_ratio for 1 pass vbr."

Merge "Enable HBD support in real time encoding path"

vp9: Adjustment of gfu_boost and af_ratio for 1 pass vbr.

Modify the gfu_boost and af_ratio setting based on the
average frame motion level.

Change only affects 1 pass vbr.

Metrics overall positive on ytlive set.
On average up by ~1%, several clips up by 2-4%.

Change-Id: Ic18c49eb2df74cb4986b63cdb11be36d86ab5e8d

Merge "vp9: Adjustment to mv bias for non-rd pickmode."

Enable coeff optimization for intra modes

This further improves the coding performance by
lowres 0.3%
midres 0.5%
hdres  0.6%

Change-Id: I6a03b6da210b9cbc261474bad4a103e0ba021c68

Use precise context to estimate coeff rate cost

Use the precise context to estimate the zero token cost in trellis
optimization process. This improves the speed 0 coding performance
by 0.15% for lowres and 0.1% for midres. It improves the speed 1
coding performance by 0.2% for midres and hdres.

Change-Id: I59c7c08702fc79dc4f8534b64ca594da909e2c91

Enable uniform quantization with trellis optimization in speed 0

This commit allows the inter prediction residual to use uniform
quantization followed by trellis coefficient optimization in
speed 0. It improves the coding performance by

lowres 0.79%
midres 1.07%
hdres  1.44%

Change-Id: I46ef8cfe042a4ccc7a0055515012cd6cbf5c9619

Refactor coeff_cost() function

Move the operations that update the context buffers outside this
function. The coeff_cost() takes all input as const value and returns
the coefficient cost.

This makes preparation for the next coefficient optimization CLs.

Change-Id: I850eec6e5470b91ea84646ff26b9231b09f70a0c

Merge "Support measure distortion in the pixel domain"

vp9: Adjustment to mv bias for non-rd pickmode.

Replace the existing mv bias with a bias only for
NEWMV, and based on the motion vector difference of
its top/left neighbors.

For cbr non-screen-content mode.

Change-Id: I8a8cf56347cfa23e9ffd8ead69eec8746c8f9e09

Merge "Fix Visual Studio build warning"