summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2016-06-24cosmetics: Change few types to their posix versionYury Gitman
Change-Id: I6d7bc9ed7396e7b0d63ee97bfa473fdea002f9ee
2016-06-24cosmetics: Make few conditions clearerYury Gitman
Change-Id: Ib024b3e42efc7ce1af56824a4644fdefcd45b215
2016-06-14Active map and ROI map use unsigned rows/colsJohann
The vpx_roi_map_t and vpx_active_map_t structures use unsigned rows and cols but VP8_COMMON uses signed values for mb_rows and mb_cols. Cleans warning in Android build: comparison of integers of different signs: 'int' and 'unsigned int' if (cpi->common.mb_rows != rows || cpi->common.mb_cols != cols) ~~~~~~~~~~~~~~~~~~~ ^ ~~~~ comparison of integers of different signs: 'int' and 'unsigned int' if (cpi->common.mb_rows != rows || cpi->common.mb_cols != cols) ~~~~~~~~~~~~~~~~~~~ ^ ~~~~ comparison of integers of different signs: 'unsigned int' and 'int' if (rows == cpi->common.mb_rows && cols == cpi->common.mb_cols) ~~~~ ^ ~~~~~~~~~~~~~~~~~~~ comparison of integers of different signs: 'unsigned int' and 'int' if (rows == cpi->common.mb_rows && cols == cpi->common.mb_cols) Change-Id: If1f118c20ffefd2530fbd371e6787cc8a6c31f0a
2016-06-14Make new_qc signedJohann
Mode is signed Cleans warning in Android build: comparison of integers of different signs: 'int' and 'unsigned int' if (ctx->oxcf.Mode != new_qc) ~~~~~~~~~~~~~~ ^ ~~~~~~ Change-Id: I5cf81c40b103e688a31e1339511f5c9eb27edd38
2016-06-14neon hadamard 16x16Johann
Runs about twice as fast as C BUG=webm:1027 Change-Id: I6760d99f4e22259439ca35d746194b12a81bfa71
2016-06-14Merge "hadamard 16x16 test"Johann Koenig
2016-06-14Merge "vp9: Encoding cycle reduction for speed 8."Jacky Chen
2016-06-14Merge "Small re-arrangement."Paul Wilkins
2016-06-13hadamard 16x16 testJohann
BUG=webm:1027 Change-Id: Ibe58781905b372b9fe29dace39b4bfdd33fd0f83
2016-06-13vp9: Encoding cycle reduction for speed 8.JackyChen
1. Skip golden non-zeromv and newmv-last for bsize >= 16x16 if the temporal variance obtained from choose_partitioning is very low. 2. Skip horz and vert INTRA mode for speed 8. This change works best on the clips with little noise and with some motion (e.g. gips_motion which has > 5% speed up). PSNR drop is 1.78% on rtc test set, no obvious visual quality regression found. Change-Id: Ib43b5b20e67809d03c5a6890818ddff59e1fc94a
2016-06-10Merge "vp9: Put lower limit on frames_to_key adjustment to gf interval."Marco Paniconi
2016-06-10Merge "Add a couple of missing WRAPLOW checks"Debargha Mukherjee
2016-06-10Merge "vp9_diamond_search_sad_avx cosmetics"James Zern
2016-06-10vp9: Put lower limit on frames_to_key adjustment to gf interval.Marco
For 1 pass vbr. Change-Id: I6c1c1cca3944150140e896e175b07ed518641b9b
2016-06-10vp9_diamond_search_sad_avx cosmeticsScott LaVarnway
Fixed cosmetic issues noted in Change 349854. Change-Id: I1d94070e4066fa920173013c5a36a30dd1cb357d
2016-06-09Merge changes I58081309,I502fd707James Zern
* changes: fdct8x8_test: fix unsigned overflow fdct4x4_test: fix unsigned overflow
2016-06-09Add a couple of missing WRAPLOW checksDebargha Mukherjee
To make coefficient checking consistent with the VP9 spec sections 8.7.1.6 and 8.7.1.1. Change-Id: I92e38e89a41d1e482317bb478c48ffa608d2d6ee
2016-06-09Small re-arrangement.paulwilkins
Move initialization of a some new "twopass" values to the function vp9_init_second_pass() and some other small changes. Remove #if GROUP_ADAPTIVE_MAXQ as this is always enabled now. Change-Id: I1dbec2fd7c419779848aa987c4cd7824d4df8456
2016-06-08vp9_subexp.h: normalize function prototypeJames Zern
vp9_prob_diff_update_savings_search_model updated in: 19e0b40 Refactor probability savings search. Change-Id: I13ae3106961f0271f51df17ee1c70c99f2d75f8f
2016-06-09Merge "vp9: Use nonrd_pick_partition on scene-cut, for speed 5 vbr mode."Marco Paniconi
2016-06-08fdct8x8_test: fix unsigned overflowJames Zern
the difference between src and dst will be signed, the error will be unsigned. quiets -fsanitize=integer: unsigned integer overflow: 4294967295 * 4294967295 Change-Id: I580813093ee46284fde7954520dfcb1188f79268
2016-06-08fdct4x4_test: fix unsigned overflowJames Zern
the difference between src and dst will be signed, the error will be unsigned. quiets -fsanitize=integer: unsigned integer overflow: 4294967295 * 4294967295 Change-Id: I502fd707823c4faaa7f587c9cc0312f057e04904
2016-06-08vp9: Use nonrd_pick_partition on scene-cut, for speed 5 vbr mode.Marco
On scene-cut detected frames (i.e., high_source_sad = 1), use nonrd_pick_partition (over choose_part + select_part), as the nonrd_pick partitioning is generally better. Small positive increase in metrics on ytlive set (~0.5 - 1%). Negligle overall speed decrease, as its only used on scene-cut frames. Only affects 1 pass vbr mode, speed = 5. Change-Id: I07c89cbdc75f5bb16eb8e0e2773ead0980d2de5c
2016-06-08vp9: Reduce overshoot factor for setting longer gf interval.Marco
Only affects 1 pass vbr mode. Change-Id: Ifd7ff3771eabe322a371b35b159cace6baa3b070
2016-06-08Merge "Refactor probability savings search."Alex Converse
2016-06-08Merge "Revert "remove vp9_diamond_search_sad_avx.c""James Zern
2016-06-08Merge "Avoid a potential assertion fail in optimize_b()"Hui Su
2016-06-07Revert "remove vp9_diamond_search_sad_avx.c"Scott LaVarnway
This reverts commit be12fefa4b7d224e9f39275a6bb4fab01b8bae3b and commit 057c1c4034ba5b9bf360c5c1f600ebc6d0718c3a. Also, the mismatch between the avx version and the c version has been fixed. BUG=https://bugs.chromium.org/p/webm/issues/detail?id=1168 For a rt encode using 1080p@60fps material, up to 11% performance improvement overall was seen. Change-Id: Icd1f216209ebc6fc0b8da885f32f356fa4355ed0
2016-06-07Avoid a potential assertion fail in optimize_b()hui su
The eob of a block is not perperly set when skip_recode is true, thus triggering assert(eob <= default_eob) to fail. Change-Id: Ifecbe33dce2dc4903e0a80bd384dc09bf0dd8a44
2016-06-07Merge "ivfdec: tolerate invalid framerates (2)"James Zern
2016-06-07Merge "vp9: Replace the avg_size_inter metric, for 1 pass vbr."Marco Paniconi
2016-06-07Merge changes I999ef597,Ic6dc9f53Johann Koenig
* changes: configure: Add -mstackrealign flags to CFLAGS on OS/2 vpx: Add OS/2-specific threading codes
2016-06-07vp9: Replace the avg_size_inter metric, for 1 pass vbr.Marco
Code cleaup, use existing rolling_actual/target metrics instead, set threshold to get same/similar effect. Little/no change in metrics on ytlive set. Change-Id: I74f3c3d0a143a9cf20dc9c3dee54c0f7e6a97a51
2016-06-07vp9: Small ajustment to settings gf_interval, 1 pass vbr.Marco
Add a max condition and lower the min value. No change in behavior (metrics for yt live set) for the default min/max_gf_interval=4/16 settings. Small positive change when min/max_gf_interval=7/16 (for 60fps clips on ytlive set). Change-Id: I1c1d72425c86c69419ea43fb9730130e81062f91
2016-06-06Merge "Move range checks into WRAPLOW"Debargha Mukherjee
2016-06-03ivfdec: tolerate invalid framerates (2)James Zern
add an upper bound to the framerate denominator above which 30fps will be reported; fixes warning in corrupt / fuzzed files Change-Id: I46a6a6f34ab756535cd009fe12273d83dcc1e9f1
2016-06-04Merge changes If31d36c8,I10b947e7James Zern
* changes: vpx_dsp,add_noise: remove mmx implementation vpx_dsp: remove mmx variance implementations
2016-06-03Merge "Fix Visual Studio build failure in filter_selectively_vert_row2() calls"James Zern
2016-06-03Move range checks into WRAPLOWDebargha Mukherjee
Provides more comprehensive coverage for --enable-coefficient-checking. The intent is to make the --enable-coefficient-checking option consistent with the VP9 spec. Change-Id: I12d0120756d17572ca2b2d7e6a2ab9d8071d8d58
2016-06-03Merge "Slightly more damped VBR adjustment."Paul Wilkins
2016-06-03Merge "Change to get_twopass_worst_quality()"Paul Wilkins
2016-06-03Merge "Removed unused data structure."Paul Wilkins
2016-06-03Merge "Adjustment to VBR rate correction."Paul Wilkins
2016-06-03Merge "Adjustment calculation of active worst quality."Paul Wilkins
2016-06-03Merge "Remove gf_zeromotion_pct."Paul Wilkins
2016-06-03Fix Visual Studio build failure in filter_selectively_vert_row2() callsLinfeng Zhang
Error messages: ..\vp9\common\vp9_loopfilter.c(1312): warning C4244: 'function' : conversion from 'uint64_t' to 'unsigned int', possible loss of data [.build-x86_64-win64-vs10\vpx.vcxproj] ..\vp9\common\vp9_loopfilter.c(1313): warning C4244: 'function' : conversion from 'uint64_t' to 'unsigned int', possible loss of data [.build-x86_64-win64-vs10\vpx.vcxproj] ..\vp9\common\vp9_loopfilter.c(1312): error C2220: warning treated as error - no 'object' file generated [.build-x86_64-win64-vs10\vpx.vcxproj] Change-Id: Ia69260611997cd2ba41c7184a85ecead740a7c07
2016-06-03Merge "Slow pshufb removal in 3 intra prediction functions."Linfeng Zhang
2016-06-03Merge "vp9: Fix valgrind failure for short circuit on low temporal vaiance ↵Jacky Chen
block."
2016-06-03Slightly more damped VBR adjustment.paulwilkins
Increase in the damping used in adjusting the active Q range. This does hurt rate accuracy a little in a few extreme cases especially if the clip is very short*, but helps metrics. * Note that the adjustment is applied at the GF/ARF group level based on what happened in the last group. Hence for very short clips where the length of a single group may be a significant % of the clip length there is still scope for some drift that cannot be accommodated. In practice most data points in our test sets are now much closer to target than was previously the case with default settings and in some cases are better even than they were with the command line undershoot and overshoot parameter was set very low (e.g. 2%). For example in bridge_close at high rates the old mechanism was unable to adapt enough to prevent extreme overshoot. Change-Id: I634f8f0e015b5ee64a9f0ccaa2bcfdbc1d360489
2016-06-03Change to get_twopass_worst_quality()paulwilkins
Change to the calculation of the error divisor used in get_twopass_worst_quality(). This follows on from other changes to the rate control that impact the output of this function. Change-Id: I414fa9aa1e6a68a64dccea17c3712f44b8a0c10c