[Libva] [PATCH intel-driver 4/5] test: streamable valarray

2016-10-26 Thread U. Artie Eoff
Add stream operators for std::valarray. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_streamable.h | 25 + 1 file changed, 25 insertions(+) diff --git a/test/i965_streamable.h b/test/i965_streamable.h index 4969c1413173..0f7e1291c4c4

[Libva] [PATCH intel-driver 5/5] test: use YUVImage in JPEG encode tests

2016-10-26 Thread U. Artie Eoff
The YUVImage class allows for more efficient (faster) operations on the YUV input/output of these tests. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_encode_test.cpp | 135 +++- test/i965_jpeg_test_data.cpp

[Libva] [PATCH intel-driver 0/5] JPEG Encode Test Speed Optimization

2016-10-26 Thread U. Artie Eoff
Encode test cases. Finally, included is a Timer class for convenience that can be used for future test development. U. Artie Eoff (5): test: add a timer class test: use C random library for random numbers test: add YUVImage class test: streamable valarray test: use YUVImage in JPEG

[Libva] [PATCH intel-driver 3/5] test: add YUVImage class

2016-10-26 Thread U. Artie Eoff
input/output data can eventually converge onto use of this common class instead. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 2 + test/i965_test_image_utils.cpp | 747 + test/i965_test_image_u

[Libva] [PATCH intel-driver 2/5] test: use C random library for random numbers

2016-10-26 Thread U. Artie Eoff
executions. The seed is then recorded in the test results so that the sequence can be reproduced if needed. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_test_environment.cpp | 7 +++ test/object_heap_test.cpp | 4 test/test_utils.h

[Libva] [PATCH intel-driver 1/5] test: add a timer class

2016-10-26 Thread U. Artie Eoff
The timer is useful to quickly instrument various sections of code during test development optimization tasks or other timing needs. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/test_utils.h | 26 ++ 1 file changed, 26 insertions(+) diff --git

[Libva] [PATCH intel-driver 4/4] test: use common I965ConfigTest fixture for jpeg enc/dec

2016-10-05 Thread U. Artie Eoff
Instantiate the JPEG encode/decode Entrypoint tests from the common I965ConfigTest fixture with additional test inputs. Also, separate them into their own files. This changes their test case names, too. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makef

[Libva] [PATCH intel-driver 0/4] Test Environment and some AVC Enc/Dec Tests

2016-10-05 Thread U. Artie Eoff
fixture. U. Artie Eoff (4): test: move vaInitialize/vaTerminate to a global test environment test: add ostream operators for VAProfile and VAEntrypoint test: add avce/avcd create config tests test: use common I965ConfigTest fixture for jpeg enc/dec test/Makefile.am| 10

[Libva] [PATCH intel-driver 3/4] test: add avce/avcd create config tests

2016-10-05 Thread U. Artie Eoff
Add a common I965ConfigTest parameterized test fixture with a i965_CreateConfig test case and add the AVC encode/decode create config test instantiations with associated profile and entrypoint inputs. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makef

[Libva] [PATCH intel-driver 1/4] test: move vaInitialize/vaTerminate to a global test environment

2016-10-05 Thread U. Artie Eoff
be achieved by executing one test case at a time via the --gtest_filter option. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 4 +- test/i965_test_environment.cpp | 86 +++ test/i965_test_environment.h

[Libva] [PATCH intel-driver 2/4] test: add ostream operators for VAProfile and VAEntrypoint

2016-10-05 Thread U. Artie Eoff
Add ostream operators for VAProfile and VAEntrypoint so the testing framework and tests can log them by name. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_streamable.h | 84 ++ 1 file changed, 84 insertions(+) diff

[Libva] [PATCH intel-driver] test: add create surface test

2016-10-03 Thread U. Artie Eoff
Add test to verify correct result for create surfaces with supported and unsupported pixel formats. Currently the IYUV pixel format case fails... see https://bugs.freedesktop.org/show_bug.cgi?id=98033 Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am

[Libva] [PATCH intel-driver 08/11] test: jpeg/enc: improve random YUV data initialization

2016-09-28 Thread U. Artie Eoff
Make initialization of YUV input for jpeg encode a little faster, esp. for larger resolution. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_test_data.cpp | 17 + test/test_utils.h| 2 +- 2 files changed, 10 insertions(+), 9 del

[Libva] [PATCH intel-driver 11/11] test: jpeg/enc: add tests for Y800 inputs

2016-09-28 Thread U. Artie Eoff
Add support for Y800 input data for jpeg encode tests. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_encode_test.cpp | 10 +- test/i965_jpeg_test_data.cpp | 12 test/i965_jpeg_test_data.h | 8 3 files changed, 25 insertions

[Libva] [PATCH intel-driver 01/11] test: move ASSERT_NO_FAILURE macro to test.h

2016-09-28 Thread U. Artie Eoff
Both jpeg decode and encode test files use this macro. So move it to the common test.h header to avoid duplicating it. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_decode_test.cpp | 4 test/i965_jpeg_encode_test.cpp | 4 test/test.h

[Libva] [PATCH intel-driver 03/11] test: jpeg/enc: fix TestInput::SharedConst typedef

2016-09-28 Thread U. Artie Eoff
The ::JPEG::Encode::TestInput::SharedConst typedef should actually use a 'const TestInput' template parameter for the std::shared_ptr. Hence, the name. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_test_data.h | 2 +- 1 file changed, 1 insertion(+), 1 de

[Libva] [PATCH intel-driver 05/11] test: jpeg/enc: move input creators to data file

2016-09-28 Thread U. Artie Eoff
Move the ::JPEG::Encode::TestInputCreator's to the i965_jpeg_test_data[.h|.cpp] files. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_encode_test.cpp | 81 +- test/i965_jpeg_test_data.cpp

[Libva] [PATCH intel-driver 07/11] test: jpeg/enc: move YUV conversion to TestInput class

2016-09-28 Thread U. Artie Eoff
Let the ::JPEG::Encode::TestInput class deal with converting from its input fourcc to its expected output fourcc. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_encode_test.cpp | 26 ++ test/i965_jpeg_test_data.cpp

[Libva] [PATCH intel-driver 09/11] test: jpeg/enc: return empty TestInput::Shared if fourcc not handled

2016-09-28 Thread U. Artie Eoff
Move JPEG::Encode::TestInput member initialization to its static "create" routine so that an empty/invalid Shared can be returned if the input fourcc is not handled/implemented. This will allow a caller/test to react appropriately. Signed-off-by: U. Artie Eoff <ullysses.a.e

[Libva] [PATCH intel-driver 04/11] test: jpeg/enc: move TestInput impl to compilation unit

2016-09-28 Thread U. Artie Eoff
Move the implementation of JPEG::Encode::TestInput to the compilation unit file (.cpp). This helps reduce the number of include headers needed in the header file. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_encode_test.cpp | 2 + test/i965_jpeg_test_da

[Libva] [PATCH intel-driver 06/11] test: jpeg/enc: enable shared_from_this on TestInput

2016-09-28 Thread U. Artie Eoff
...and make the constructor private so that only Shared instances can be created. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_encode_test.cpp | 2 +- test/i965_jpeg_test_data.cpp | 8 +++- test/i965_jpeg_test_data.h | 6 +- 3 files chang

[Libva] [PATCH intel-driver 02/11] test: move generic toString to i965_streamable.h

2016-09-28 Thread U. Artie Eoff
The toString template is generic and should be part of the i965_streamable.h header. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_encode_test.cpp | 10 -- test/i965_streamable.h | 9 + 2 files changed, 9 insertions(+), 10 deletions(-)

[Libva] [PATCH intel-driver v2 6/7] test: add common test utils

2016-09-26 Thread U. Artie Eoff
Common utilities and functions that may be useful for multiple tests. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 1 + test/test_utils.h | 49 + 2 files changed, 50 insertions(+) create mode 10064

[Libva] [PATCH intel-driver v2 4/7] test: add streamable operators for VA structs and other

2016-09-26 Thread U. Artie Eoff
Add convenience streamable operators for VA and other common data structures so that tests can log them as needed. v2: put std::array stream operators into the std namespace otherwise gcc 5.x complains. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_streamable.h

[Libva] [PATCH intel-driver v2 7/7] test: add some jpeg encode tests

2016-09-26 Thread U. Artie Eoff
. It is not clear why only this resolution exhibits this problem. v2: don't create any input data in test fixture if jpeg encoding is not supported. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 1 + test/i965_jpeg_encode_test.cpp

[Libva] [PATCH intel-driver v2 2/7] test: wrap jpeg decode tests and data in namespace

2016-09-26 Thread U. Artie Eoff
Put the JPEG decode tests and test data into its own namespace so that it can be distinguished from encode. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_decode_test.cpp | 8 ++--- test/i965_jpeg_test_data.cpp | 4 +++ test/i965_jpeg_test_data.h

[Libva] [PATCH intel-driver v2 1/7] test: split jpeg test data into header and cpp

2016-09-26 Thread U. Artie Eoff
Move static definitions of JPEG TestPatternData into the .cpp so that multiple files can include the i965_jpeg_test_data.h header file. Otherwise, there will be "multiple definition" compiler errors. Also, change generateSolid to be inline for the same reason. Signed-off-by: U.

[Libva] [PATCH intel-driver 4/7] test: add streamable operators for VA structs and other

2016-09-26 Thread U. Artie Eoff
Add convenience streamable operators for VA and other common data structures so that tests can log them as needed. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_streamable.h | 359 +++ test/i965_test_fixture.h |

[Libva] [PATCH intel-driver 5/7] test: add wrapper for SyncSurface

2016-09-26 Thread U. Artie Eoff
Add a wrapper that calls the driver i965_SyncSurface. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_internal_decl.h | 3 +++ test/i965_test_fixture.cpp | 6 ++ test/i965_test_fixture.h | 6 ++ 3 files changed, 15 insertions(+) diff --git

[Libva] [PATCH intel-driver 7/7] test: add some jpeg encode tests

2016-09-26 Thread U. Artie Eoff
. It is not clear why only this resolution exhibits this problem. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 1 + test/i965_jpeg_encode_test.cpp | 694 + test/i965_jpeg_test_data.h

[Libva] [PATCH intel-driver 2/7] test: wrap jpeg decode tests and data in namespace

2016-09-26 Thread U. Artie Eoff
Put the JPEG decode tests and test data into its own namespace so that it can be distinguished from encode. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_decode_test.cpp | 8 ++--- test/i965_jpeg_test_data.cpp | 4 +++ test/i965_jpeg_test_data.h

[Libva] [PATCH intel-driver 3/7] test: add wrapper for CreateSurfaces2

2016-09-26 Thread U. Artie Eoff
-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_test_fixture.cpp | 20 test/i965_test_fixture.h | 10 +++--- 2 files changed, 23 insertions(+), 7 deletions(-) diff --git a/test/i965_test_fixture.cpp b/test/i965_test_fixture.cpp index d7f30c

[Libva] [PATCH intel-driver v2 1/5] move HAS_* macros to header

2016-09-08 Thread U. Artie Eoff
Move the HAS_* macros (i.e. HAS_JPEG_ENCODING, etc.) to the i965_drv_video.h header so that they can be used by testing. This will allow tests to use the macros to determine if a particular test is supported on the current execution platform. v2: rebased Signed-off-by: U. Artie Eoff

[Libva] [PATCH intel-driver 4/5] test: add getFullTestName to i965 test fixture

2016-09-07 Thread U. Artie Eoff
Add I965TestFixture::getFullTestName to allow tests to log the current test's name. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_test_fixture.cpp | 9 + test/i965_test_fixture.h | 3 +++ 2 files changed, 12 insertions(+) diff --git

[Libva] [PATCH intel-driver 5/5] test: skip jpeg fourcc decode test if unsupported by hw

2016-09-07 Thread U. Artie Eoff
If the hardware does not support jpeg decode, then log this information and return from the test early. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_decode_test.cpp | 9 + 1 file changed, 9 insertions(+) diff --git a/test/i965_jpeg_decode_test.cpp

[Libva] [PATCH intel-driver 3/5] test: add jpeg entrypoint test

2016-09-07 Thread U. Artie Eoff
Add a test to verify the i965_CreateConfig functionality for the VAProfileJPEGBaseline/VAEntrypointVLD entrypoint based on platform supported or not. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_decode_test.cpp | 37 -

[Libva] [PATCH intel-driver] test: fix jpeg decode invalid indexing

2016-09-07 Thread U. Artie Eoff
Use hsample and vsample factor in printComponentDataTo to avoid invalid indexes into the data array. This fixes a segfault in the jpeg fourcc tests that may be triggered during 'make check'. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/i965_jpeg_decode_test.cp

[Libva] [PATCH RFC intel-driver 04/11] i965: compile driver source as convenience library

2016-09-01 Thread U. Artie Eoff
Compile driver source into a noinst convenience library and link it into the driver module. This allows for tests to link to the driver library, too, for testing. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- src/Makefile.am | 17 + 1 file changed, 13 inse

[Libva] [PATCH RFC intel-driver 10/11] test: add some chipset tests

2016-09-01 Thread U. Artie Eoff
Add various tests to verify the integrity of the chipset and pciid definitions and lookups. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 1 + test/i965_chipset_test.cpp | 104 + test/i965_internal_

[Libva] [PATCH RFC intel-driver 05/11] test: link to i965 convenience library

2016-09-01 Thread U. Artie Eoff
Link the i965 convenience library to the i965 test executable for testing. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/test/Makefile.am b/test/Makefile.am index 19ad34746541..f309de202559 100644 ---

[Libva] [PATCH RFC intel-driver 07/11] test: add an i965 initialize test

2016-09-01 Thread U. Artie Eoff
Add basic i965 initialize test to verify certain driver data is properly initialized and to demonstrate the usage of the I965TestFixture class. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 1 + test/i965_initialize_test.cp

[Libva] [PATCH RFC intel-driver 03/11] test: add initial test_i965_drv_video target

2016-09-01 Thread U. Artie Eoff
Add test_i965_drv_video as noinst program with an initial test main() definition. Also provide a test.h header with common test includes and definitions that can be included by all test compilation units. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am

[Libva] [PATCH RFC intel-driver 09/11] test: add some object_heap tests

2016-09-01 Thread U. Artie Eoff
Add some basic object_heap tests to verify logical usage. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am | 1 + test/object_heap_test.cpp | 248 ++ test/test.h | 3 + 3 files change

[Libva] [PATCH RFC intel-driver 08/11] test: add some JPEG decode test cases

2016-09-01 Thread U. Artie Eoff
Test that the driver properly decodes JPEG data that is encoded in various fourcc formats. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/Makefile.am |2 + test/i965_jpeg_decode_test.cpp | 292 +++ test/i965_jpeg_test_data.h

[Libva] [PATCH RFC intel-driver 00/11] Automated (Unit) Test Suite

2016-09-01 Thread U. Artie Eoff
EG_ENCODING(i965)) { RecordProperty("skipped", true); std::cout << “[SKIPPED] “ << testname() << “ unsupported” return; } // do jpeg encode test } Does this seem reasonable enough? Are there other suggestions or recommendation

[Libva] [PATCH intel-driver] shaders/gen9: fix build when no intel-gen4asm available

2016-07-20 Thread U. Artie Eoff
If intel-gen4asm version is < 1.9 or not installed then we shouldn't run the associated make rules. This fixes 'make dist' failure. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- src/shaders/post_processing/gen9/Makefile.am | 2 ++ 1 file changed, 2 insertions(+) diff --

[Libva] [intel-driver][PATCH 2/2] jpeg enc/dec gen9: Allow up to 8K JPEG max resolution on gen9

2016-06-07 Thread U. Artie Eoff
Allow up to 8K * 8K resolution for JPEG encode and decode on gen9 HW (SKL,BXT,KBL). Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- src/gen9_mfd.c | 14 ++ src/i965_device_info.c | 16 ++-- 2 files changed, 24 insertions(+), 6 deletions(-) diff

[Libva] [PATCH 0/2] Per-Codec Max Resolution and 16K JPEG Enc/Dec on Gen9

2016-04-01 Thread U. Artie Eoff
The following patches add per-codec max resolution support to the i965 driver. It also enables up to 16K * 16K max resolution for JPEG HW encode and decode on Gen9 HW (SKL,BXT,KBL). U. Artie Eoff (2): i965_drv: add support for per-codec max resolution jpeg enc/dec gen9: Allow up to 16K JPEG

[Libva] [PATCH 1/2] i965_drv: add support for per-codec max resolution

2016-04-01 Thread U. Artie Eoff
Add a functor to hw_codec_info to allow each hw instance to report maximum resolution on a per-codec basis. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- src/i965_drv_video.c | 32 +++- src/i965_drv_video.h | 9 + 2 files changed, 36 inse

[Libva] [PATCH 2/2] jpeg enc/dec gen9: Allow up to 16K JPEG max resolution on gen9

2016-04-01 Thread U. Artie Eoff
Allow up to 16K * 16K resolution for JPEG encode and decode on gen9 HW (SKL,BXT,KBL). Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- src/gen9_mfd.c | 14 ++ src/i965_device_info.c | 16 ++-- 2 files changed, 24 insertions(+), 6 deletions(-)

[Libva] [PATCH 2/2] test/encode: remove dead code

2015-09-16 Thread U. Artie Eoff
Remove commented code and their functions that are no longer executed. This erradicates some compiler warnings for unused functions. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/encode/h264encode.c | 171 --- 1 file change

[Libva] [PATCH 1/2] test/encode: remove unused variables and functions

2015-09-16 Thread U. Artie Eoff
This gets rid of compiler warnings caused by various unused variables and functions. Signed-off-by: U. Artie Eoff <ullysses.a.e...@intel.com> --- test/encode/avcenc.c | 44 ++-- test/encode/mpeg2vaenc.c | 1 - test/loadsurface.h

[Libva] [PATCH libva-intel-driver] gen8_mfd: free surface used by JPEG decode

2015-05-19 Thread U. Artie Eoff
by default. However, one can enable this code path by defining JPEG_WA during compilation. In that case, we would enable the same memory leak. Thus, apply the same fix in gen8_mfd to fix it. Signed-off-by: U. Artie Eoff ullysses.a.e...@intel.com --- src/gen8_mfd.c | 12 1 file