Existing profile/entrypoint methods require too
much state overhead in the fixture and force tests
to make multiple calls to different methods to get
the information they need. Thus, add new methods
for profile/entrypoint queries so tests can regain
more control and simplicity. These new methods
also do not require member variables to operate.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
vaPutSurface only used in libva-utils. The actual application/omx
implementation never use it.
It will introduce many header files in android framework.Android change
rapidly, the directory or file name changed in versionto version. It's
hard to catch up and maintain support for many versionof android.
Define valid bitmasks for remaining read/write config
attributes...
VAConfigAttribEncPackedHeaders
VAConfigAttribEncInterlaced
VAConfigAttribFEIFunctionType
This will ensure VAAPIGetCreateConfig/CreateConfigWithAttributes
test cases only use one bitmask for each supported value when
calling vaCreateConfig for these bitfield config attributes.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
These fixture methods have been replaced with consolidated
and more flexible methods and are no longer used.
Also, remove associated legacy member variables.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
Take into account bitfield attributes when creating
config and surfaces.
Use new fixture methods for greater control in test
cases.
Fixes#91Fixes#92
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
Consolidate surface methods with refactored logic to allow
tests to take more control of parameters. Also, the
existing doCreateSurfaces suffers from a surface format
logic bug. The new consolidated createSurfaces does not
suffer from this bug.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
Use the new config methods from fixture.
Also, properly handle config attributes that use bitfields.
That is, some attributes require only one bitmask/value
for vaCreateConfig, therefore we cannot directly use
the attribute returned from vaGetConfigAttributes since
those bitfield attributes will return a bitfield value with
"all" OR'd supported bitmask choices.
Fixes#86
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
Add definitions for valid VA rate controls and decode
slice mode values that can be used by test cases.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
Define 4 new config methods that provide the functionality
of the existing 11 similar methods. The usage of the existing
methods will be replaced by the new methods in successive
commits.
These new methods will also give control back to the test
cases to handle config attributes depending on need/use-case
and reduce the number fixture member variables needed.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
The m_ prefix for a variable indicates it's a member
variable. The g_ prefix indicates it's a global variable.
Use the correct prefix in test_data.h for global variables
and update usage accordingly.
Also, cleanup test_data.h formatting.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
Use const and references appropriately.
Streamline logic, variable names and code formatting
Signed-off-by: Jayesh Kumar Tank <jayesh.kumarx.tank@intel.com>
Use const and references appropriately.
Streamline logic, variable names and code formatting
Also remove useless test.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
The locally generated va_version.h was erroneous and
not even being used in the tests. This is because
va/va.h already includes the "real" va/va_version.h.
Thus, remove local va_version.h.in.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
m_querySurfaceAttribList was redefined in
doQuerySurfacesWithConfigAttribs which shadows the member
variable of the same name. Removing the redefinition does
not break tests, so remove it.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
The minimum surface resolution needs to be considered
when creating a context in CreateContext test.
Fixes#83
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
Add method to get minimum allowed surface resolution.
If driver does not specify, then assume 1x1 is min.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
If driver does not specify the max surface resolution,
then assume "any" max is allowed (up to numeric_limits).
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
The header is only needed in one compilation unit.
Thus, move it's contents to the compilation unit to
reduce unnecessary complexity and maintenance.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
The header is only needed in one compilation unit.
Thus, move it's contents to the compilation unit to
reduce unnecessary complexity and maintenance.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
The header is only needed in one compilation unit.
Thus, move it's contents to the compilation unit to
reduce unnecessary complexity and maintenance.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
The header is only needed in one compilation unit.
Thus, move its contents to the compilation unit to
reduce unnecessary complexity and maintenance.
Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>