James Zern [Sun, 27 Jul 2014 21:57:50 +0000 (14:57 -0700)]
vp9_subexp: remove vp9_compute_update_table()
makes update_bits[] a constant table
Change-Id: If105bf0522487f54a31e12a7cc73aa24671b19d4
Jingning Han [Fri, 25 Jul 2014 22:57:35 +0000 (15:57 -0700)]
Merge "Fix rd_pick_partition search loop for 4x4 blocks"
Jingning Han [Fri, 25 Jul 2014 22:56:53 +0000 (15:56 -0700)]
Merge "Fix potential ioc issue in vp9_get_prob for 4K above sizes"
Jingning Han [Fri, 25 Jul 2014 22:56:31 +0000 (15:56 -0700)]
Merge "Remove unnecessary conditional assignment"
Minghai Shang [Fri, 25 Jul 2014 20:24:27 +0000 (13:24 -0700)]
Merge "[spatial svc]Fix reference issues"
Joshua Litt [Fri, 25 Jul 2014 20:16:36 +0000 (13:16 -0700)]
Merge "Encode perf test"
Joshua Litt [Mon, 21 Jul 2014 17:57:16 +0000 (10:57 -0700)]
Encode perf test
Change-Id: I61993946fe5cbcd7ebeb6302efd9538bff022e5a
Yunqing Wang [Fri, 25 Jul 2014 17:53:13 +0000 (10:53 -0700)]
Merge "Allocate aligned source in variance test"
Alex Converse [Fri, 25 Jul 2014 17:51:48 +0000 (10:51 -0700)]
Merge "Refactor inter/intra_suberblock_yrd."
Yaowu Xu [Fri, 25 Jul 2014 15:49:39 +0000 (08:49 -0700)]
Merge "Fix allocation of context buffers on frame resize"
Yaowu Xu [Fri, 25 Jul 2014 15:48:48 +0000 (08:48 -0700)]
Merge "Changed validation of reference frame size"
Marco Paniconi [Fri, 25 Jul 2014 15:42:41 +0000 (08:42 -0700)]
Merge "vp8: reduce refresh_max_mbs_perframe with fewer layers."
Tim Kopp [Fri, 25 Jul 2014 14:53:51 +0000 (07:53 -0700)]
Merge "s/CONFIG_DENOISING/CONFIG_VP9_TEMPORAL_DENOISING"
Jingning Han [Fri, 25 Jul 2014 14:08:23 +0000 (07:08 -0700)]
Fix rd_pick_partition search loop for 4x4 blocks
The partition search for 4x4 blocks takes unnecessary steps to
reconstruct pixels and an extra partition type update. This commit
removes such operations. No visible compression/speed difference.
Thanks to Yue (yuec@) for finding this issue.
Change-Id: I3f83824aa3fd3717d63be0b280fa57258939a70a
James Zern [Fri, 25 Jul 2014 06:09:35 +0000 (23:09 -0700)]
Merge "rtcd.pl: check for auto_help availability"
Johann [Thu, 24 Jul 2014 16:32:01 +0000 (09:32 -0700)]
Remove neon version of vp8 extend borders
The code fails the unit test. Speed comparisons to the C are invalid
because the code frequently didn't correctly extend the right and
bottom portions of the frame.
Reduce maximum frame size on ARM devices to avoid OOM
Change-Id: Ia664c86406f0bb8120fd7ad401f32d0bd44994fb
Johann [Wed, 16 Jul 2014 10:47:27 +0000 (03:47 -0700)]
Add test for vp8 extend_borders
Change-Id: I5bfc965019815e631a2300eb29aa065fb2bc94aa
Yunqing Wang [Fri, 25 Jul 2014 00:11:58 +0000 (17:11 -0700)]
Allocate aligned source in variance test
The source buffer is an aligned buffer in VP9. Added the alignment
to make it consistent with libvpx.
Change-Id: I3ebb9d2e8555ed532951da479dd5cbbb8812e02d
Marco Paniconi [Thu, 24 Jul 2014 22:44:29 +0000 (15:44 -0700)]
vp8: reduce refresh_max_mbs_perframe with fewer layers.
Change-Id: Ib39cc0218a1f232c0cde8e99ca3319c17274bd48
Jingning Han [Thu, 24 Jul 2014 22:31:32 +0000 (15:31 -0700)]
Fix potential ioc issue in vp9_get_prob for 4K above sizes
This commit turns on the existing vp9_get_prob function using
64 bit in the intermediate step. It fixes the ioc issue for 4K
above frame sizes (issue 828).
Change-Id: I9f627f3beca2c522f73b38fd2a3e7eefdff01a7c
Jingning Han [Thu, 24 Jul 2014 21:43:06 +0000 (14:43 -0700)]
Remove unnecessary conditional assignment
The assignment of the variable mode_excluded in
vp9_rd_pick_inter_mode_sub8x8 takes redundant conditional jump.
This commit removes it.
Change-Id: Ie195fbe6e54ec2ade7093d562c456a2e93143704
Johann [Wed, 16 Jul 2014 09:30:21 +0000 (02:30 -0700)]
Set and use uv_crop_[width|height]
Ensure consistent border extension by rounding uv_crop_* at image
creation time. Where it was rounded problems could arise with the right
and bottom extensions.
When padding = 32, y_width = 64, and y_crop_width = 63:
(padding + width - crop_width + 1) / 2
32 + 64 - 63 + 1 should equal 32 *but*
32 + 1 + 1 equals 34 giving a right buffer of 17 instead of 16.
By calculating uv_crop_* earlier we round up at the appropriate time and
for the same values:
(y_crop_width + 1) / 2
63 + 1 / 2
64
(padding / 2) + uv_width - uv_crop_width
16 + 16 - 16
16
Change-Id: If866cd1b63444771440edb1432280ac83875969b
Yaowu Xu [Thu, 24 Jul 2014 16:12:46 +0000 (09:12 -0700)]
Changed validation of reference frame size
A previous change, https://gerrit.chromium.org/gerrit/#/c/70632,
introduced a size validation for reference frames to insuare the
input stream is a valid VP9 stream. However, the logic requiring
all reference frames have valid size turned out to be too strict.
In this commit, we modify the validation to require one of the
reference frame has valid dimension. In addition, the decoder
reports error whenever it detects the use of reference frame
with invalid scalig ratio.
Change-Id: If8efc312244087556cfe00f1fcbdff811268ebad
James Zern [Thu, 24 Jul 2014 21:55:19 +0000 (14:55 -0700)]
rtcd.pl: check for auto_help availability
'auto_help' was added to Getopt::Long in 2.33
this isn't strictly necessary as an unrecognized option (--help) will
issue a warning and then print the usage
Change-Id: Ia757553a4e19d22a8eb70768a8866ab1a76a0eec
Adrian Grange [Thu, 24 Jul 2014 20:37:47 +0000 (13:37 -0700)]
Fix allocation of context buffers on frame resize
The patch:
https://gerrit.chromium.org/gerrit/#/c/70814/
changed the test that determined whether the context
frame buffers needed to be reallocated or not.
The code checked for a change in total frame area
to signal the need to reallocate context buffers.
However, the above_context buffer needs to be
resized i:xf only the width of the frame has increased.
Change-Id: Ib89d75651af252908144cf662578d84f16cf30e6
Tim Kopp [Thu, 24 Jul 2014 20:30:00 +0000 (13:30 -0700)]
s/CONFIG_DENOISING/CONFIG_VP9_TEMPORAL_DENOISING
This should prevent confusion with the VP8 CONFIG_TEMPORAL_DENOISING and other
flags.
Change-Id: I1fe4e2977895b7966841d861ab74317ad875b6c8
Alex Converse [Wed, 23 Jul 2014 00:57:40 +0000 (17:57 -0700)]
Refactor inter/intra_suberblock_yrd.
Move txfm_rd_in_plane into choose_tx_size_from_rd and cleanup callers.
Change-Id: I1df2d7dc984802bd5e204cbe881ada0d75fbb3f7
James Zern [Thu, 24 Jul 2014 05:45:51 +0000 (22:45 -0700)]
Merge "x86inc: fix compilation with NASM\r \r Change-Id: I5978921ab1ccad6648a5bde6ad023c3e3430f5f4\r"
KO Myung-Hun [Tue, 22 Jul 2014 01:47:20 +0000 (10:47 +0900)]
x86inc: fix compilation with NASM
Change-Id: I5978921ab1ccad6648a5bde6ad023c3e3430f5f4
KO Myung-Hun [Tue, 22 Jul 2014 11:19:52 +0000 (20:19 +0900)]
Makefile: strip debugging informations only
--strip-unneeded causes SIGSEGV when accessing g_executable_path. So
test_libvpx crashes due to SIGSEGV in ::testing::InitGoogleTest().
OS/2, aout, strip v2.23.2
Change-Id: I2718d082447ee0d9ad0c021b9156c50e1ac085a6
Minghai Shang [Wed, 23 Jul 2014 20:52:34 +0000 (13:52 -0700)]
[spatial svc]Fix reference issues
1. Remove last reference flag for first frame upper layers in one pass mode.
2. Disable refresh golden frame flag for key frames.
Change-Id: I44ac1bd2c795169e4fbfdd078ea79a1d33a204d6
Jingning Han [Wed, 23 Jul 2014 22:07:01 +0000 (15:07 -0700)]
Merge "Remove redundant argument entry in handle_inter_mode"
Jingning Han [Wed, 23 Jul 2014 22:06:52 +0000 (15:06 -0700)]
Merge "Use the chessboard pattern pred search in newmv mode"
Marco Paniconi [Wed, 23 Jul 2014 20:58:45 +0000 (13:58 -0700)]
Merge "Fix clang compiler warning in denoising_neon."
Yaowu Xu [Wed, 23 Jul 2014 19:46:11 +0000 (12:46 -0700)]
Merge "Moved call to vp9_clear_system_state() to a proper location"
Jingning Han [Wed, 23 Jul 2014 19:02:52 +0000 (12:02 -0700)]
Remove redundant argument entry in handle_inter_mode
The value of mode_excluded has been properly set in
vp9_rd_pick_inter_mode_sb(). It is redundant to send it in
handle_inter_mode() and re-set the value again.
Change-Id: I408d4731f2f42e0bcf3ae62e85757717bb410471
Jingning Han [Wed, 23 Jul 2014 18:47:56 +0000 (11:47 -0700)]
Use the chessboard pattern pred search in newmv mode
This commit extends the chessboard pattern prediction filter search.
If the above and left blocks have the same prediction filter type,
the encoder will skip the prediction filter type search and use the
reference one.
The overall chessboard pattern prediction filter type search reduces
speed 3 runtime for hard clips. Experiments on park joy at 1080p
and 15000 kbps show that the runtime goes from 723265 ms to 65832 ms,
i.e., about 10% speed-up. Compression performance wise, it affects
the coding quality by
Change-Id: I880975497c7ad166532e9eea9bf46684d77ff327
derf: -0.326%
yt: -0.257%
hd: -0.241%
stdhd: -0.417%
Jingning Han [Wed, 23 Jul 2014 17:34:54 +0000 (10:34 -0700)]
Merge "Remove redundant num_refs definition"
Marco Paniconi [Wed, 23 Jul 2014 16:42:09 +0000 (09:42 -0700)]
Fix clang compiler warning in denoising_neon.
Issue: https://code.google.com/p/webm/issues/detail?id=829
Change-Id: I580308f8aa4af194b5d8990a9692ebd18db68ee8
Jingning Han [Wed, 23 Jul 2014 16:29:04 +0000 (09:29 -0700)]
Remove redundant num_refs definition
Use is_comp_pred to replace the use case of num_refs.
Change-Id: I4d0c1e14d5f728428a2ae3d293cd2b4a8b2f31d8
Jingning Han [Wed, 23 Jul 2014 16:12:56 +0000 (09:12 -0700)]
Merge "Enable chessboard inter prediction filter type search"
James Zern [Wed, 23 Jul 2014 01:40:31 +0000 (18:40 -0700)]
Merge "frame_size_tests: reduce default 'large' frame size"
Jingning Han [Tue, 22 Jul 2014 23:32:20 +0000 (16:32 -0700)]
Enable chessboard inter prediction filter type search
This commit enables a chessboard pattern prediction filter type
search scheme for rate-distortion optimization speed-up. For the
inferred motion vector modes, the encoder can re-use its above/left
neighbor blocks' prediction filter type and skip a full test on
all possible filter types. Such operation is turned on/off
alternatively in a chessboard manner.
It is turned on in speed 3. For test clip pedestrian 1080p, the
runtime is reduced from 231500 ms -> 221700 ms. The compression
performance is changed:
derf: -0.147%
yt: -0.134%
hd: -0.079%
stdhd: -0.220%
Change-Id: I1912f278e7576c2dc632688e3ad7a257410c605a
James Zern [Tue, 22 Jul 2014 22:48:25 +0000 (15:48 -0700)]
frame_size_tests: reduce default 'large' frame size
VP9FrameSizeTestsLarge exposed an integer overflow in the VP9 encoder,
for now reduce the size to allow the tests to clear and prevent further
regressions.
4096x4096 -> 4096x2160
this should be restored after the bug is fixed:
https://code.google.com/p/webm/issues/detail?id=828
Change-Id: I47fdf0648f1d9a3951f731bbf0b727f85ada4fa1
Marco Paniconi [Tue, 22 Jul 2014 22:25:06 +0000 (15:25 -0700)]
Merge "vp8: Set default denoiser_decision to copy for UV channel."
Adrian Grange [Tue, 22 Jul 2014 22:17:27 +0000 (15:17 -0700)]
Merge "Fix get_frame_type function"
Tim Kopp [Tue, 22 Jul 2014 21:48:42 +0000 (14:48 -0700)]
Merge "VP9 denoiser bugfix in debugging code."
Jingning Han [Tue, 22 Jul 2014 21:21:34 +0000 (14:21 -0700)]
Merge "USE local best_filter variable in handle_inter_mode"
Tim Kopp [Tue, 22 Jul 2014 21:00:11 +0000 (14:00 -0700)]
VP9 denoiser bugfix in debugging code.
When OUTPUT_YUV_DENOISED is enabled the encoder outputs the uncompressed,
denoised video to a separate file. Moved the point at which the file is
written to in order to avoid an extra blank frame at the beginning of the video.
Change-Id: I805f6a912b18b3d9cae59b13c5b8108279439ce3
James Zern [Tue, 22 Jul 2014 20:17:40 +0000 (13:17 -0700)]
Merge "vp9_bitstream.c: cosmetics"
Minghai Shang [Tue, 22 Jul 2014 19:43:40 +0000 (12:43 -0700)]
Merge "[spatial svc]Use #if instead of #ifdef on macro CONFIG_SPATIAL_SVC"
Adrian Grange [Tue, 22 Jul 2014 19:12:16 +0000 (12:12 -0700)]
Fix get_frame_type function
Fixed the function get_frame_type to return the correct
frame type for golden and last frames.
Change-Id: I8edddd9aa26cbe7a1de8ff211389410b22b1bd14
James Zern [Tue, 22 Jul 2014 01:27:06 +0000 (18:27 -0700)]
vp9_bitstream.c: cosmetics
fix indent, spelling and drop some vertical whitespace
Change-Id: I722671381a374a24763b07a02805ab1d149ab3f4
Jingning Han [Tue, 22 Jul 2014 18:32:17 +0000 (11:32 -0700)]
USE local best_filter variable in handle_inter_mode
This should be a local variable. Move the definition from
vp9_rd_pick_inter_mode_sb to handle_inter_mode.
Change-Id: I14f4168bb1c896ed04e8f6d4cd89fbf4c9839944
Marco Paniconi [Tue, 22 Jul 2014 18:15:08 +0000 (11:15 -0700)]
Merge "vp8 denoiser fix: Update denoised altref on key frame."
Marco Paniconi [Tue, 22 Jul 2014 18:06:00 +0000 (11:06 -0700)]
vp8: Set default denoiser_decision to copy for UV channel.
Since the UV decision to denoise is based on Y, we need to set
the default/initial denoiser decision_u/v to COPY_BLOCK,
to make sure if no uv_denoiser is applied we still update
(uv)running_avg with source.
Change-Id: I5af1c2afbd40c498cd3de208bea88c837099b24d
Minghai Shang [Tue, 22 Jul 2014 17:05:40 +0000 (10:05 -0700)]
[spatial svc]Use #if instead of #ifdef on macro CONFIG_SPATIAL_SVC
Change-Id: Ifc94377a0d05d66e3d21b007893a985b66db6082
Marco Paniconi [Mon, 21 Jul 2014 16:58:54 +0000 (09:58 -0700)]
vp8 denoiser fix: Update denoised altref on key frame.
On a key frame, the denoised-running_avg for all references
frames should be updated with the source.
The altref denoised-running_avg was not being updated on key frame,
this fixes that.
Change-Id: Ie02cd0ba5383e013af59240e6df7e185d11703f6
Jingning Han [Tue, 22 Jul 2014 16:11:52 +0000 (09:11 -0700)]
Merge "Turn on adaptive pred filter scheme for sub8x8 below 720p"
Jingning Han [Mon, 21 Jul 2014 23:22:56 +0000 (16:22 -0700)]
Turn on adaptive pred filter scheme for sub8x8 below 720p
For sequences of resolution below 720p, the encoder will check
intra prediction modes and inter prediction modes from LAST_FRAME.
This commit turns on adaptive prediction filter scheme for sub8x8
blocks, where inter prediction modes are enabled. For the test
sequence bus at CIF, the speed 2 runtime goes down from 17879 ms
to 16783 ms, i.e., 6% speed up. The compression performance of
derf set is down by -0.128%.
Change-Id: I01d5321a5ceab4e0666ac5be56c52d896c7a8d45
Alex Converse [Thu, 17 Jul 2014 18:29:59 +0000 (11:29 -0700)]
Remove unfinished VP9 alpha channel.
Change-Id: Ic5d3a3a0dac10b49495771886a31e793bb78b5ca
Yaowu Xu [Mon, 21 Jul 2014 22:55:12 +0000 (15:55 -0700)]
Moved call to vp9_clear_system_state() to a proper location
The commit moved a call to vp9_clear_system_state() to a correct
location, i.e. prior function calls using floating point numbers.
This was to fix a mismatch mmx code and sse2 version, where a
floating point number used in adjust_frame_rate(cpi) gets NAN due
to mmx registers being in wrong state.
Change-Id: I40e0a6de98812000ccee6a729badb630604fd7e6
Yunqing Wang [Tue, 15 Jul 2014 23:18:34 +0000 (16:18 -0700)]
Add -DNDEBUG when config option debug is disabled
For gcc, when libvpx config option debug is disabled, added the
flag -DNDEBUG to disable the assertions in libvpx for some speedup.
Change-Id: Ifcb7b9e8ef5cbe5d07a24407b53b9a2923f596ee
Tim Kopp [Mon, 21 Jul 2014 15:28:41 +0000 (08:28 -0700)]
Merge "VP9 denoiser fix: ref frames now updated properly"
James Zern [Thu, 17 Jul 2014 02:03:08 +0000 (19:03 -0700)]
y4m_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I2ede16e0903a96225a342609545281fdbb83d638
James Zern [Thu, 17 Jul 2014 02:02:37 +0000 (19:02 -0700)]
vp9_subtract_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I636012c74f5de91d594d7398e92343f5efe08327
James Zern [Thu, 17 Jul 2014 02:02:16 +0000 (19:02 -0700)]
subtract_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: Ic738b97d492f244c58989a0f73a4d763340ec274
James Zern [Thu, 17 Jul 2014 02:01:46 +0000 (19:01 -0700)]
sixtap_predict_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: Ie59ddc080530d538e4ffb1d862844a84de926d92
James Zern [Thu, 17 Jul 2014 01:59:28 +0000 (18:59 -0700)]
sad_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I60746bc93ba2446c1458a1f09fd1e49cc2e68534
James Zern [Thu, 17 Jul 2014 01:58:53 +0000 (18:58 -0700)]
pp_filter_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: If6cf006a089b0fb42d6bf050570d928d955d658a
James Zern [Thu, 17 Jul 2014 01:58:16 +0000 (18:58 -0700)]
partial_idct_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: Ied220e09fceef53039f234ebbb7e51c8e081c84e
James Zern [Thu, 17 Jul 2014 01:57:42 +0000 (18:57 -0700)]
intrapred_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I74d7489ea35f988fcfec3651bd16f02e688e406a
James Zern [Thu, 17 Jul 2014 01:57:11 +0000 (18:57 -0700)]
idct_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I660a77c0c58ca64075c1051ea440fdf4d7c7e6af
James Zern [Thu, 17 Jul 2014 01:56:22 +0000 (18:56 -0700)]
fdct8x8_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: Id2d43931e9b6e16aa4b8e5e346eee08505bdccfb
James Zern [Thu, 17 Jul 2014 01:55:40 +0000 (18:55 -0700)]
fdct4x4_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I796252a6f7f749b4faa969bbe7fe83eae64fa393
James Zern [Thu, 17 Jul 2014 01:55:08 +0000 (18:55 -0700)]
decode_perf_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I5fdd6319449364f2e930667d3bd54f68ba3cbb2e
James Zern [Thu, 17 Jul 2014 01:54:31 +0000 (18:54 -0700)]
dct32x32_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: Ie1918d93c0a0d7a854e88b5a77b8c21b4cdc28b7
James Zern [Thu, 17 Jul 2014 01:53:33 +0000 (18:53 -0700)]
dct16x16_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I4e6f293790922684402bfa9c22ec1ac2b9981a0e
James Zern [Thu, 17 Jul 2014 01:48:20 +0000 (18:48 -0700)]
convolve_test: drop '_t' from local typenames
_t is reserved by posix
+ switch to camelcase
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Type_Names
Change-Id: I2a22ffc36e9f88781bc7db0d5a28a7ed924bab1a
Adrian Grange [Fri, 18 Jul 2014 23:33:50 +0000 (16:33 -0700)]
Merge "Re-introduce frame size check inadvertantly deleted"
Deb Mukherjee [Fri, 18 Jul 2014 23:30:06 +0000 (16:30 -0700)]
Merge "Use custom mkstemp() to fix Win issue in y4m_test"
Adrian Grange [Fri, 18 Jul 2014 22:40:39 +0000 (15:40 -0700)]
Re-introduce frame size check inadvertantly deleted
This patch adds back in code that checks that the frame
size lies within defined bounds was inadvertantly removed
by a previous patch:
https://gerrit.chromium.org/gerrit/#/c/70814/
Change-Id: If526570ba559260c4b7e98098bc75f7700ae7f97
Deb Mukherjee [Mon, 14 Jul 2014 20:31:29 +0000 (13:31 -0700)]
Use custom mkstemp() to fix Win issue in y4m_test
Uses mkstmp() with directory being the same as the test data
directory to create temporary output file. For Windows
GetTempFileNameA() function is used.
Change-Id: Ie4681b2b4f44f8c22d3b3faf134c44087b484f94
Tim Kopp [Fri, 18 Jul 2014 21:15:30 +0000 (14:15 -0700)]
VP9 denoiser fix: ref frames now updated properly
The ALT_REF_FRAME is now updated in the case of a KEY_FRAME in the VP9 denoiser.
Change-Id: Idf9a9772706f50e774fb240afcc01db38841043c
Johann [Fri, 18 Jul 2014 21:08:52 +0000 (14:08 -0700)]
Disable vp8_sixtap_predict4x4_neon
Crashing on iOS encodes
BUG=817
Change-Id: I2c50f8d359563d15a8b8002b6091184c0f9818af
Yunqing Wang [Fri, 18 Jul 2014 19:29:12 +0000 (12:29 -0700)]
Fix visual studio build failure
Cast frame_size to be int for now to fix build warnings.
Change-Id: I411ba2b1fd429a383c8f6e832b4fa4147914c824
Marco Paniconi [Fri, 18 Jul 2014 17:39:34 +0000 (10:39 -0700)]
temporal_svc sample encoder: setting for denoiser.
Reset it as before commit: https://gerrit.chromium.org/gerrit/#/c/70855/
Change-Id: I93df163c4bb565fb30f0f5283ed71347d01153d3
Deb Mukherjee [Fri, 18 Jul 2014 10:49:30 +0000 (03:49 -0700)]
Merge "Fix FrameSizeTestsLarge unit-test on 32-bit arch."
Deb Mukherjee [Fri, 18 Jul 2014 10:23:51 +0000 (03:23 -0700)]
Merge "Separates profile 2 into 2 profiles 2 and 3"
Deb Mukherjee [Fri, 18 Jul 2014 10:06:07 +0000 (03:06 -0700)]
Fix FrameSizeTestsLarge unit-test on 32-bit arch.
If the img allocation fails the test used to crash before on
32 bit architecture. This patch uses null check on img in
FillFrame. Also, if the first frame initialization has not been
conducted VPX_CODEC_ERROR is expected to return rather than
VPX_CODEC_OK.
Change-Id: I5c4e59c156374009012d280d6ff971a89b43c11f
Deb Mukherjee [Tue, 15 Jul 2014 08:54:29 +0000 (01:54 -0700)]
Separates profile 2 into 2 profiles 2 and 3
Separates HBD profile int two profiles (2 and 3) consistent with the
highbitdepth branch. This patch is ported from the original highbitdepth
branch patch: https://gerrit.chromium.org/gerrit/#/c/70460/
Two of the invalid file tests needed to be updated.
Change-Id: I6a4acd2f7a60b1fb4cbcc8e0dad4eab4248431e3
James Zern [Fri, 18 Jul 2014 01:06:55 +0000 (18:06 -0700)]
Merge "iosbuild.sh: Add VPX_FRAMEWORK_TARGET preproc symbol"
Tom Finegan [Thu, 17 Jul 2014 23:47:31 +0000 (16:47 -0700)]
iosbuild.sh: Add VPX_FRAMEWORK_TARGET preproc symbol
Mainly for determining the actual arch when running performance
tests.
Change-Id: Ia6a1c4fcf83e6db1bf94f3e869d146ec32df02f8
Pengchong Jin [Thu, 17 Jul 2014 21:24:52 +0000 (14:24 -0700)]
Merge "Fixed a bug of setting wrong first pass mb stats pointer"
Adrian Grange [Thu, 17 Jul 2014 19:15:16 +0000 (12:15 -0700)]
Merge "Modified frame buffer handling"
Pengchong Jin [Thu, 17 Jul 2014 18:54:43 +0000 (11:54 -0700)]
Fixed a bug of setting wrong first pass mb stats pointer
The bug sets the wrong pointer to the first pass mb stats
if the encoder does the re-coding in the second pass.
Change-Id: I8a11f45dd7dceb38de814adec24cecccae370d00
Scott LaVarnway [Thu, 17 Jul 2014 18:42:16 +0000 (11:42 -0700)]
Merge "Added vp9_sad64x64_neon(), vp9_sad32x32_neon()"
Adrian Grange [Thu, 10 Jul 2014 22:35:51 +0000 (15:35 -0700)]
Modified frame buffer handling
This patch is the first step toward simplifying the
frame buffer handling.
The final goal is to have a common frame buffer handling
framework for both encoder and decoder that incorporates
the existing ability to use externally allocated memory.
Change-Id: I2c378a4f54a39908915f46c4260e17a080db7ff1
Jim Bankoski [Thu, 17 Jul 2014 13:31:50 +0000 (06:31 -0700)]
allow config options to limit max size of decode
This is a practical concern to allow us to fail in a decoder instance
if the size of a file is bigger than we can reasonably handle.
Change-Id: I0446b5502b1f8a48408107648ff2a8d187dca393
Jim Bankoski [Thu, 17 Jul 2014 13:33:29 +0000 (06:33 -0700)]
fail allocation of buffers if size_t < frame_size
Change-Id: I25c595e8c197ab0a9955d2373f1a74d42fbd1638