Merge remote branch 'internal/upstream-experimental' into HEAD

Merge remote branch 'origin/master' into experimental

Change-Id: I993dbef81ca3d1638e16c4134aa8dc177e57875c

Merge remote branch 'internal/upstream' into HEAD

Merge remote branch 'internal/upstream-experimental' into HEAD

Merge "Bugfix in vp8dx_set_reference"

Bugfix in vp8dx_set_reference

The fb_idx_ref_cnt book-keeping was in error. Added an assert to
prevent future errors in the reference count vector. Also fixed a
pointer syntax error.

Change-Id: I563081090c78702d82199e407df4ecc93da6f349

Merge "Fix code under #if CONFIG_INTERNAL_STATS."

Fix code under #if CONFIG_INTERNAL_STATS.

Change-Id: Iccbd78d91c3071b16fb3b2911523a22092652ecd

remove some magic weights associated with sad_per_bit

sad_per_bit has been used for a number of motion vector search routines
with different magic weights: 1, 1/2 and 1/4. This commit remove these
magic numbers and use 1/2 for all motion search routines, also reformat
a number of source code lines to within 80 column limit.

Test on cif set shows overall effect is neutral on all metrics. <=0.01%

Change-Id: I8a382821fa4cffc9c0acf8e8431435a03df74885

Merge remote branch 'origin/master' into experimental

Change-Id: I1e7ce466bc01e380eb392b964ba677f0bb8cd13b

Merge remote branch 'internal/upstream' into HEAD

Merge "vp8_pick_inter_mode code cleanup"

vp8_pick_inter_mode code cleanup

Small code cleanups before attempting to reduce the size
of bmi found in BLOCKD.

Change-Id: Ie9c14adb53afd847716a75bcce067d0e6c04f225

Initialize first_time_stamp_ever

Misplaced #endif caused first_time_stamp_ever to only be initialized if
CONFIG_INTERNAL_STATS was set.

Change-Id: I2296a4ab00f7dfb767583edcc5d59b94f48c0621

Merge remote branch 'internal/upstream-experimental' into HEAD

Merge remote branch 'origin/master' into experimental

Change-Id: Ie2a4927754a9c220b30a84fc7e1372e565fe9eec

Merge remote branch 'internal/upstream' into HEAD

Merge remote branch 'internal/upstream-experimental' into HEAD

Merge "bug fix check frame buffer index before copy"

bug fix check frame buffer index before copy

in onyx_if.c update_reference_frames() make
sure that frame buffer indexes are not equal
before preforming a buffer copy.  If two frames
share the same buffer the flags will already be
set correctly.

Change-Id: Ida9b5516d08e3435c90f131d2dc19d842cfb536e

Merge "Use hex search for realtime mode speed>4"

Use hex search for realtime mode speed>4

Test showed using hex search in realtime mode largely speed up
encoding process, and still achieves similar quality like the
diamond search we have. Therefore, removed the diamond search
option.

Change-Id: I975767d0ec0539f9f6ed7fdfc09506e39761b66c

Merge "Broken EC after MODE_INFO size reduction"

Merge "Remove unused code"

Remove unused code

Hex search is not called in rdopt.c

Change-Id: I67347f03e13684147a7c77fb9e9147e440bb5e8e

Merge remote branch 'origin/master' into experimental

Change-Id: I6d6692418eecf54e23e00a08394b0b37d6e7682b

Merge remote branch 'internal/upstream' into HEAD

Broken EC after MODE_INFO size reduction

This patch fixes the compiler errors and the seg fault
when running decode_with_partial_drops.

Change-Id: I7c75369e2fef81d53b790d5dabc327218216838b

Merge "Do not copy data between encoder reference buffers."

Merge "fix the mix use of errorperbit and sadperbit"

Merge remote branch 'internal/upstream' into HEAD

Merge remote branch 'internal/upstream-experimental' into HEAD

Merge remote branch 'internal/upstream' into HEAD

Merge remote branch 'internal/upstream-experimental' into HEAD

Conflicts:
vp8/encoder/encodeframe.c
vp8/encoder/ethreading.c

Change-Id: I4becf6f101756923de6b98ca6a2132c3605c6ea5

Merge "Use int_mv instead of MV in vp8_mv_cont"

Merge remote branch 'origin/master' into experimental

Change-Id: Ica721b36ffaa6c4c02e1cf82850496c7063ce577

fix the mix use of errorperbit and sadperbit

error_per_bit and sad_per_bit were designed as estimates of a bit worth
of sum squared error and sum absolute difference respectively. Under
this assumption, error_per_bit should be used in combination with 2nd
order errors (variance or sum squared error) while sad_per_bit should
be used in combination with 1st order SADs in motion estimation. There
were a few places where sad_per_bit has been misused with variances,
this commit changes to use error_per_bit for those places, also changes
parameter names to properly indicate which constant is being used.

On cif set, the change has a universal gain by all metrics: 0.13% by
average/overall psnr and 0.1% by ssim.

Change-Id: I4850fdcc3fd6886b30f784bd843f13dd401215fb

Merge "  Use var8x8 instead of get8x8var in VP8_UVSSE"

Merge "Return sse value in vp8_variance SSE2 functions"

remove code not in use

Change-Id: I6e5e86235d341cce3b02abda26dbeb71940ed955

Return sse value in vp8_variance SSE2 functions

Minor modification.

Change-Id: I09511d38fd1451d5c4106a48acdb3f766ce59cb7

  Use var8x8 instead of get8x8var in VP8_UVSSE

  'sum' returned by get8x8var is not used and var8x8 has optimizations
  for more platforms.

Change-Id: I4a907fb1a05f285669fb0b95dc71d42182c980f6

Merge remote branch 'origin/master' into experimental

Change-Id: I9e5c28f898d92091e39f62193f6329b593968819

Fix a bug happening while encoding at profile=3

While profile=3, there is no sub-pixel search. Distortion and SSE
have to calculated using get_inter_mbpred_error().

Change-Id: Ifb36e17eef7750af93efa7d0e2870142ef540184

Use int_mv instead of MV in vp8_mv_cont

Less operations.

Change-Id: Ibb9cd5ae66b8c7c681c9a654d551c8729c31c3ae

Removed unused variable warnings

Change-Id: I6e5e921f03dc15a72da89a457848d519647677a3

Merge "MODE_INFO size reduction"

MODE_INFO size reduction

Declared the bmi in MODE_INFO as a union instead of B_MODE_INFO.
This reduced the memory footprint by 518,400 bytes for 1080
resolutions.  The decoder performance improved by ~4% for the
clip used and the encoder showed very small improvements. (0.5%)
This reduction was first mentioned to me by John K. and in a
later discussion by Yaowu.
This is WIP.

Change-Id: I8e175fdbc46d28c35277302a04bee4540efc8d29

Merge "Fixing bug in VP8_SET_REFERENCE decoder control command"

Merge "Rewrite hex search function"

Fixing bug in VP8_SET_REFERENCE decoder control command

In vp8dx_set_reference, the new reference image is written to an
unused reference frame buffer.

Change-Id: I9e4f2cef5a011094bb7ce7b2719cbfe096a773e8

Merge remote branch 'origin/master' into experimental

Change-Id: I56a5665a5d4e2ed590d75a5ad49e8feb54393f6e

Merge remote branch 'internal/upstream' into HEAD

Merge "use get8x8var directly for non-subpixel motion case in VP8_UVSSE"

Rewrite hex search function

Reduced some bound checks in hex search function.

Change-Id: Ie5f73a6c227590341c960a74dc508cff80f8aa06

use get8x8var directly for non-subpixel motion case in VP8_UVSSE

VP8_UVSSE mistakenly used subpixvar8x8 to calculate SSE for non-subpixl
motion cases.

Change-Id: I4a5398bb9ef39c211039f6af4540546d4972e6a9

Merge remote branch 'internal/upstream-experimental' into HEAD

Merge remote branch 'origin/master' into experimental

Change-Id: I90a1d0095712e0474b0c03773b57376911027fc6

Merge remote branch 'internal/upstream' into HEAD

Merge remote branch 'internal/upstream-experimental' into HEAD

Merge "bug fix active_worst_quality set below active_best_quality"

Merge "cleanup: collect twopass variables"

Merge "Fixed iwalsh_neon build problems with RVDS4.1"

Merge "revise two function definitions with less parameters"

Merge "Remove unused members of VP8_COMP"

Merge remote branch 'origin/master' into experimental

Conflicts:
configure

Change-Id: I91b9059e5b724a96368c7765c147fdf5a5ce03f2

Merge remote branch 'internal/upstream' into HEAD

Merge remote branch 'internal/upstream-experimental' into HEAD

revise two function definitions with less parameters

Change-Id: Ia96e5bf915e4d3c0ac9c1795114bd9e5dd07327a

Merge "disable trellis optimization for first pass"

disable trellis optimization for first pass

also remove 2 #defines and 1 function declaration that are not in use.

Change-Id: I8f743d0e3dd9ebf1de24a8b0c30ff09f29b00c53

bug fix active_worst_quality set below active_best_quality

fixed a bug where active_worst_quality could be set
below active_best_quality which could result in an
infinite loop.

Change-Id: I93c229c3bc5bff2a82b4c33f41f8acf4dd194039

cleanup: collect twopass variables

This patch collects the twopass specific memebers of VP8_COMP into a
dedicated struct. This is a first step towards isolating the two pass
rate control and aids readability by decorating these variables with
the 'twopass.' namespace. This makes it clear to the reader in what
contexts the variable will be valid, and is a hint that a section of
code might be a good candidate to move to firstpass.c in later
refactoring. There likely will be other rate control modes that need
their own specific data as well.

This notation is probably overly verbose in firstpass.c, so an
alternative would be to access this struct through a pointer like
'rc->' instead of 'cpi->firstpass.' in that file. Feel free to make
a review comment to that effect if you prefer.

Change-Id: I0ab8254647cb4b493a77c16b5d236d0d4a94ca4d

Merge "Using partition_info instead of blockd info for splitmv"

Remove unused members of VP8_COMP

Various members that were either completely unreferenced or written
and not read.

Change-Id: Ie41ebac0ff0364a76f287586e4fe09a68907806e

Using partition_info instead of blockd info for splitmv

The partition_info struct contains info just for SPLITMV,
so it should be used instead of BLOCKD.  Eventually, I want
to reduce the size of B_MODE_INFO struct found in BLOCKD, so
this is the first step toward that goal.
Also, since SPLITMV is not supported in vp8_pick_inter_mode(),
the unnecessary mem copies and checks were removed.  For rt
encodes, this gave a slight performance improvement.

Change-Id: I5585c98fa9d5acbde1c7e0f452a01d9ecc080574

Merge "Make hor UV predict ~2x faster (73 vs 132 cycles) using SSSE3."

Merge "changed configure option name to reduce confusion"

Merge "Make activity masking functions static"

Merge "Fix segv without --enable-error-concealment"

Fix segv without --enable-error-concealment

Missed wrapping one function call in #if CONFIG_ERROR_CONCEALMENT.

Change-Id: I5746b1e6e4531670dbed1130467331fe309bdcae

Merge "Adding error-concealment to the decoder."

Adding error-concealment to the decoder.

The error-concealer is plugged in after any motion vectors have been
decoded. It tries to estimate any missing motion vectors from the
motion vectors of the previous frame. Intra blocks with missing
residual are replaced with inter blocks with estimated motion vectors.

This feature was developed in a separate sandbox
(sandbox/holmer/error-concealment).

Change-Id: I5c8917b031078d79dbafd90f6006680e84a23412

Make activity masking functions static

These don't need extern linkage.

Change-Id: I21220ada926380a75ff654f24df84376ccc49323

Move quantizer init functions to quantize.c

Group related functions together.

Change-Id: I92fd779225b75a7204650f1decb713142c655d71

Fixed iwalsh_neon build problems with RVDS4.1

rvct 4.1 was complaining about vstmia.16, store multiple expects 64 data type.
optimized the implementation.

Change-Id: I0701052cabd685c375637bbc3796ff6d88f5972c

Merge remote branch 'origin/master' into experimental

Change-Id: I2f9fd68d7fd52e0aebc57e561c77ebe99e9c33e4

Merge remote branch 'internal/upstream' into HEAD

Merge "Modify MVcount in pick_inter_mode to eliminate calling of vp8_find_near_mvs"

Fix a bug in vp8_clamp_mv function

Scott fixed the bug in MV clamping function in encoder, which
could cause artifacts.

Change-Id: Id05f2794c43c31cdd45e66179c8811f3ee452cb9

Modify MVcount in pick_inter_mode to eliminate calling of vp8_find_near_mvs

Moved MVcount modification in pick_inter_mode, and eliminated
calling of vp8_find_near_mvs.

Change-Id: Icd47448a1dfc8fdf526f86757d0e5a7f218cb5e8

Merge remote branch 'internal/upstream-experimental' into HEAD

Merge remote branch 'origin/master' into experimental

Conflicts:
vp8/encoder/rdopt.c

Change-Id: I85275aab07625bd30bbef16a752b08b18f4451ab

Merge remote branch 'internal/upstream' into HEAD

Conflicts:
vp8/encoder/encodeframe.c
vp8/encoder/rdopt.c

Change-Id: I3c66714e704b22569aff701cc5b9b2a5b70989f3

Merge remote branch 'internal/upstream-experimental' into HEAD

Merge "Improve framerate adaptation"

Merge "adjusting rd constant slightly by ~10%"

Merge "Restructure of activity masking code."

Merge remote branch 'internal/upstream' into HEAD

Conflicts:
vp8/common/blockd.h
vp8/decoder/decodemv.c

Change-Id: Ib97c226d5b33b1ac1675d9c96eac1986af4dd579

Restructure of activity masking code.

This commit restructures the mb activity masking code
to better facilitate experimentation using different metrics
etc. and also allows for adjustment of the zero bin either
for encode only or both the encode and mode selection
stages

It also uses information from the current frame rather than
the previous frame and the default strength has been
reduced.

Change-Id: Id39b19eace37574dc429f25aae810c203709629b