summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2014-02-13Assembler/bdw: Remove the unsupported cache agent for WRITE(...)Zhao Yakui
The Sampler/Constant cache is read-only. And it can't be used as the target cache agent of WRITE message. Reviewed-by: Xiang, Haihao <haihao.xiang@intel.com> Signed-off-by: Zhao Yakui <yakui.zhao@intel.com> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-12tests: fix sub-test enumerationThomas Wood
Ensure sub-tests can be listed correctly by doing any test setup within an igt_fixture block. Signed-off-by: Thomas Wood <thomas.wood@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-12NEWS: Mention Damien's new helper libraryDaniel Vetter
2014-02-12lib: Make igt_skip noreturnDaniel Vetter
This is a remnant from the subtest code before we had proper fixture blocks. With those there's no no reason to have igt_skip calls outside of fixtures or subtests, so we can assert this and hence always jump out. Suggested by Thomas Wood since static analyzers got confused about this. To check for fallout I've run all tests as non-root so that they'll all skip. Only very little fallout resulted. Cc: Thomas Wood <thomas.wood@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-12lib: install exit handler only on success for prefault controlDaniel Vetter
Otherwise we'll hit an igt_skip in the exit handler, which upsets the new in_fixture||in_subtests checks. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-12test/gem_gtt_hog: bail out earlierDaniel Vetter
This way we properly skip instead of failing hard. Just result polish when running on non-intel systems. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11tests/gem_madvise: use correct main blockDaniel Vetter
igt_simple_main doesn't enumerate subtests ... Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11tests/gem_bad_ctx_exec: Properly wrap in igt_fixturesDaniel Vetter
Caught by the new nasty assert in igt_skip. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11tests/gem_close_race: Properly wrap init code in fixturesDaniel Vetter
Blows up otherwise if there's no intel gpu around. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11lib: fix header includeDaniel Vetter
drm/ path is for the headers from kernel sources, without when using the pkgconfig libdrm version. Cc: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11lib/display: Fix typo in igt_plane_get_fd_id() (fb)Damien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11kms_pipe_crc_basic: Remove igt_display_get_n_pipes() usage in mainDamien Lespiau
The fixture won't be run when listing the subtests and thus we can't use igt_display_get_n_pipes() in the list of tests. The alternative here is to list the subtests with the maximum values and skip when not supported on the platform. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib: Fix igt_ouput_get_plane() typo (output)Damien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib: Don't wait in igt_wait_for_keypress() if stdin is not a ttyDamien Lespiau
When running in a non interactive terminal, there's little point in trying to wait for some input on stdin. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11kms_plane: Start a basic display plane testDamien Lespiau
We test the sprite plane positionning in there, for now. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Fix assertion in set_plane()Damien Lespiau
When cycling throuth planes, we still want to reach the cursor plane. We have to special case IGT_PLANE_CURSOR as a shorthand to select the cursor plane (the last plane on the pipe). Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Wait for a vblank after SetPlane()Damien Lespiau
Let's be testing friendly and gently wait for the next vblank before returning from commit() when needed. After igt_display_commit() one can safely look at the CRC. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Print the fb id, not its pointer in the set_fb() log messageDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Add a way to wait at every commit for inspectionDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib: Add a helper to wait for a keypressDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11kms_pipe_crc_basic: Use kmstest_create_color_fb()Damien Lespiau
Reducing again the per-test number of lines. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib: Include drm_fourcc.h from igt_kms.hDamien Lespiau
This include is needed for the DRM_FORMAT* defines used in the fb creation helpers. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11kms_cursor_crc: Use kmstest_create_color_fb()Damien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib: Introduce a new helper kmstest_create_color_fb()Damien Lespiau
We need to create fbs of a single color in a few places. Time to abstract that out to a helper function. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Add support for DRM planesDamien Lespiau
We can now extend our plane support beyond primary and cursor planes. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11tests/kms_cursor_crc: Use igt_pipe_crc_collect_crc()Damien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/crc: Add a helper to read a single CRC valueDamien Lespiau
In this case, we also take care of starting/stopping the CRC collection. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/crc: Factor out reading a single CRC valueDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Check if we're trying to use the same pipe on 2 outputsDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Allow to override the display verbosity with an env varialeDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11kms_cursor_crc: Port the test to the new modeset APIDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Add a way to specify we don't care about the pipe to useDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Add support for the cursor planeDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Fix the SetCrtc disabling log messageDamien Lespiau
We were displaying the value of fb_id (0), when the actual interesting thing about this call it that it'll just down the pipe. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11tests/kms_cursor_crc: Move the array of cursor fb_ids to the stackDamien Lespiau
This array is not used outside ouf create_cursor_fb(). A stack allocated array is enough. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11kms_pipe_crc_basic: Use igt_display_get_n_pipes() instead of hardcoding 3Damien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib/display: Add an accessor to retrieve the number of pipesDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11kms_pipe_crc_basic: Use for_each_connected_output()Damien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib: Introduce a for_each_connected_output() macroDamien Lespiau
So we can easily cycle through them in tests without knowing too many internal details about how igt_display_t organize its data. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib: Introduce symbolic names for display planesDamien Lespiau
It'd be nice to have symbolic names for planes instead of using an index in igt_output_get_plane(). We also namespace the enum to not conflict with anyone. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11kms_pipe_crc_basic: Port to the new modeset APIDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11lib: Introduce a modeset APIDamien Lespiau
The goals here are: - Reduce duplicated code in each KMS test - Provide an API that looks more like what we want for atomic modesets. The hope is then that it'll be easy to switch, at run-time, between the "legacy" path and atomic modesets, keeping the same API for tests. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-02-11tests/gem_seqno_wrap: fix over allocation of arraysThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11assembler: fix condition for printing a warningThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11debugger: fix the -p optionThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11lib/drmtest: Fix up skip_henceforth checksDaniel Vetter
We need to first check whether we'll even run the testcase before deciding to print a premature SKIP/FAIL. Without this and global skipped resulted in superflous output when running only one testcase. References: https://bugs.freedesktop.org/show_bug.cgi?id=74100 Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-11tests/gem_reset_stats: Correctly wrap global test codeDaniel Vetter
Tests _must_ be able to run as non-root and on non-intel platforms, otherwise the enumeration is busted (since QA enumerates testcases on a gpu-less xeon box). Cc: Mika Kuoppala <mika.kuoppala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-07tests/pm_rps: remove setfreqDaniel Vetter
It's unused. Cc: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-07tests/pm_rps: Round requested freq correctlyDaniel Vetter
The kernel will round it, so if we don't we'll have a spurious mismatch. Happens on my machine here with 650-1300MHz range, where the midpoint is 975. Cc: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-02-06lib: fix signed/unsigned comparison issuesThomas Wood
Store the result of set_vt_mode as a signed value so that errors can be caught correctly. Signed-off-by: Thomas Wood <thomas.wood@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>