summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2018-11-06tools/intel_vbt_decode: Decode HDMI max data rateVille Syrjälä
Decode the HDMI max data rate from the VBT. Cc: Jani Nikula <jani.nikula@intel.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Jani Nikula <jani.nikula@intel.com>
2018-11-06tests/kms_atomic_transition: Do not timeout when number of sprites is huge, v5.Maarten Lankhorst
We mostly care about the plane type, because sometimes the driver may do silly things when only a cursor is enabled for example, so we reduce the number of tests. This puts an upper bound on the number of plane combinations being tested, which is nice for gen11. Changes since v1: - Make 2 groups for overlay planes, and randomly put planes in either group. Changes since v2: - Include igt_rand.h to compiler error. Changes since v3: - Fix logical error resulting in crash. Changes since v4: - Fix logical error harder. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> #v3
2018-11-06tests/kms_properties: Fix compiler warningPetri Latvala
../tests/kms_properties.c:113:11: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers] Fixes: 7983e19ed62e ("tests/kms_properties: Add functional test for "max bpc" property") Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Radhakrishna Sripada <radhakrishna.sripada@intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-11-06runner: Unit tests for json generationPetri Latvala
Test the results.json generation with a top-down approach: With a directory of test run intermediary logs, check that the resulting json would match a reference json file. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-11-06runner/resultgen: Split json generation and file writingPetri Latvala
This allows testing to skip the file writing. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-11-05lib/igt_fb: Generalize the slow read from gtt mmap handlingVille Syrjälä
Make the handling of slow gtt mmap reads generic, and extend it to the pixman converter. Makes the pixman path a bit faster. With testing just XRGB8888 and XBGR8888 on KBL: $ time kms_plane --r pixel-format-pipe-A-planes - real 0m18,757s + real 0m2,635s v2: Use the original src buffer if the malloc fails (Chris) Drop the duplicated comment about things being slow Cc: Chris Wilson <chris@chris-wilson.co.uk> Cc: Maxime Ripard <maxime.ripard@bootlin.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Maxime Ripard <maxime.ripard@bootlin.com> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2018-11-05lib/igt_fb: Use linear.fb in the converterVille Syrjälä
The converter operates between the linear and shadow fbs. When using the blitter path there is no particular reason to assume that the linear fb and actual fb have the same strides for instance. Thus consulting the actual fb for metadata is not really correct. We can also simplify the mmap() path by copying all the original fb metadata into linear.fb as there we map the original fb directly. We just have to keep clearing linear.fb.gem_handle so that the dtor knows which kind of beast it's got. Cc: Maxime Ripard <maxime.ripard@bootlin.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Maxime Ripard <maxime.ripard@bootlin.com>
2018-11-05lib/igt_fb: Assert converted formats harderVille Syrjälä
Add more asserts to make sure the converted formats are correct. Cc: Maxime Ripard <maxime.ripard@bootlin.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Maxime Ripard <maxime.ripard@bootlin.com>
2018-11-05lib/igt_fb: Fix the pixman converterVille Syrjälä
Cairo doesn't do RGB888. The shadow buffer must be in XRGB888 (which is what our YUV converters also assume). We did calculate the shadow stride/size with four bytes per pixel, but we just put the wrong format in the fb metadata. Cc: Maxime Ripard <maxime.ripard@bootlin.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Maxime Ripard <maxime.ripard@bootlin.com>
2018-11-05tests/tools_test: Find the tool binaries also from bindirPetri Latvala
Also print what directory gets used. v2: Use PATH_MAX Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-11-05meson: Enable more warning flagsPetri Latvala
We had quite a bit of warning flags active on autotools builds that were not used for meson builds. Add the same flags autotools builds used to what meson was using (some flags autotools didn't have). For the assembler, disable some of the flags to make it build cleanly again. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-11-05assembler: Compare correct thingsPetri Latvala
This code is supposedly checking of the given register is nr=0 in ARF but was instead checking twice if the register is anything in ARF. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-11-05runner: const-correct the unit testsPetri Latvala
const where possible, and cast away const when passing argv to parse_options, which expects non-const strings, because it passes them to getopt_long, which expects non-const strings... getopt and getopt_long take the argv array as char * const *, or in other words, as pointer-to-const-pointer-to-char. In C, pointer-to-T implicitly converts to pointer-to-const-T and for a char **, the T is char* and "const T" is char * const, ergo char ** converts to char * const *, not const char **. The only const-correctness getopt and getopt_long can really do is char * const * or they lose the ability to directly pass in main()'s arguments, which are an array of non-const pointers to non-const char for legacy reasons. For testing the argument handling, it's very convenient to use an array of string literals, which are of type const char[N], convertible to const char *. To get such an array into getopt, the choices are: 1) Cast away the const in the pointer-to-pointer 2) Cast away the const in the string literal 3) Don't cast anything and eat the compiler warning Option 1 looked cleanest out of all those. tl;dr: Choices made in 1972 force our hand. v2: - Augment commit message Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-11-05runner: Fix declaration of timeofday_doublePetri Latvala
Fixes compiler warning: function declaration isn’t a prototype Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-11-05docs: Remove incorrect reference to i915Petri Latvala
While true that subtest listing doesn't require the i915 driver, same applies to any driver or platform. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-11-02lib/igt_fb: Fix unused variable buf.Stanislav Lisovskiy
Despite the comment temporary buf is not used anywhere in convert_nv12_to_rgb24, so it has to be either removed or used instead of cvt->src.ptr. v2: Put it in use instead of removing as other functions seem to use it. Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2018-11-02tests/kms_properties: Add functional test for "max bpc" propertyRadhakrishna Sripada
Test the values in the range advertised by the "max bpc" property. Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-11-02tests/kms_properties: Add additional prperty test infrastructureRadhakrishna Sripada
We currently test the existimg properties by setting them with default value. Add infrastructure to perform additional test on a desired property. v2: Fix the strcmp logic Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-11-01lib: Help static analyzers figure out the execution flowPetri Latvala
When running any kind of static analysis on IGT, one tends to drown in warnings about using uninitialized variables in subtests, because static analysis is unable to figure out that igt_fixture blocks are always entered if a subtest block is entered. Aid the discovery of correct execution flow by making static analysis always enter all igt_fixture blocks and all subtest blocks. Automatic discovery of static analyzers is done for Clang, Coverity and Klocwork, using macros found by quick googling. For explicit control on possible other analyzers, defining STATIC_ANALYSIS_BUILD=1 manually will activate this hack. v2: - Hack not needed for igt_subtest() - Make sure igt_fixture is entered once instead of an infinite loop v3: - Rebase properly... Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-31gitlab-ci: arm64 cross compilingDaniel Vetter
First I tried to build in a full sysroot, but I didn't figure out why ld never found the libraries (this was on fedora, which might have been part of the problem - if you just need a toolchain for kernel corss-building fedora is perfectly fine). Then I tried using debian's multiarch stuff, which worked a lot better, except there's some minor gotchas: - some libraries conflict and can't be multi-arch installed - some libraries are buggy and cause conflicts when installing - pkgconfig and ld get confused with some other libraries and pick the first one they find (i.e. the x86_64 one, not the arm64 one). That means we need a new docker image with just the cross libaries installed as -dev packages, and none of the native ones. Next up I tried to run the tests. Meson has really nice support for an exe wrapper, and qemu happily runs arm64 on x86_64. With that I managed to run some tests successfully (it's not even that slow!). But most are run through a script, and the script doesn't know about the exe wrapper and hence fails miserably. I discussed this a bit with Dylan and he filed an upstream meson issue: https://github.com/mesonbuild/meson/issues/4427#issuecomment-433553689 Another option would be to use binfmt-misc handling (which works neatly locally), but on the runners we have I can't control that. Maybe I can beg Daniel Stone ... For not let's just mark this stage as optional and that failures are allowed. v2: Use the new gitlab 1.4 feature to rebuild docker images when necessary. Cc: Sean Paul <sean@poorly.run> Cc: Eric Anholt <eric@anholt.net> Cc: Dylan Baker <dylan@pnwbakers.com> Cc: Daniel Stone <daniel@fooishbar.org> Reviewed-by: Eric Anholt <eric@anholt.net> (v1) Reviewed-by: Petri Latvala <petri.latvala@intel.com> (v1) Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-31gitlab-ci: Build docs by defaultDaniel Vetter
I just realized that when you push to a non-HEAD branch, then gitlab doesn't bother to build the deploy stage. Which means if you push to a branch, then that's not properly tested. So build docs right away. We're not going to waste anything, since the deploy:pages job will pick up right from where we left. Reviewed-by: Eric Anholt <eric@anholt.net> Reviewed-by: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-31gitlab-ci: build test libunwind stubsDaniel Vetter
We want to make sure this keeps working nicely Cc: Sean Paul <sean@poorly.run> Cc: Eric Anholt <eric@anholt.net> Reviewed-by: Eric Anholt <eric@anholt.net> Reviewed-by: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-31lib: Make libunwind optional againDaniel Vetter
Apparently it's really not great to require it for a bunch of platforms. Requested by Sean and Eric. v2: Use combo option (Petri). v3: Fix the right option (Petri) v4: try a bit harder ... v5: Even more simplification (Dylan) Cc: Dylan Baker <dylan@pnwbakers.com> Cc: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Eric Anholt <eric@anholt.net> (v1) Reviewed-by: Petri Latvala <petri.latvala@intel.com> Cc: Sean Paul <sean@poorly.run> Cc: Eric Anholt <eric@anholt.net> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-31igt/gem_ppgtt: Unroll 32b memsetChris Wilson
Manually unroll the 32b memset in create_bo() to set a cacheline at a time, for a 2x speed improve of the whole test. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Michał Winiarski <michal.winiarski@intel.com>
2018-10-30igt/gem_tiled_fence_blits: Remember to mark up fence blitsChris Wilson
Older platforms require fence registers to perform blits, and so userspace is expected to mark up the objects to request fences be assigned. Fixes: ff2db94acb53 ("igt/gem_tiled_fence_blits: Remove libdrm_intel dependence") Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108591 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2018-10-30gitlab-ci: Build Docker images if Dockerfiles have changedArkadiusz Hiler
The jobs that build container images and upload them to the registry do not have to be manual any longer. By leveraging the new only-changes feature (GitLab 11.4), we can trigger those jobs conditionally on any alterations to the Dockerfiles. Cc: Daniel Vetter <daniel@ffwll.ch> Cc: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-29tests/kms_content_protection: restore for real before failingDaniel Vetter
Nothing happens if you don't commit ... v2: rebased!? Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108550 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108549 Cc: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Ramalingam C <ramalingam.c@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-26lib/kmod: Unload snd_hdmi_lpe_audioVille Syrjälä
Unload the snd_hdmi_lpe_audio module. Otherwise i915 can't be unloaded. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-26kms_plane_alpha_blend: Start CRC after enabling display.Maarten Lankhorst
This will fix the -EIO from trying to start CRC on a disabled pipe. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108146 Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Reviewed-by: Imre Deak <imre.deak@intel.com>
2018-10-26igt/gem_tiled_fence_blits: Remove libdrm_intel dependenceChris Wilson
Modernise the test to use igt's ioctl library as opposed to the antiquated libdrm_intel. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2018-10-25tests/kms_content_protection: Avoid dmesg falloutDaniel Vetter
Subsequent tests have become really unhappy. Paper over this for now. If this doesn't work, then I think we need to revert. v2: Use Ram's suggestion for comment. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108550 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108549 Cc: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Ramalingam C <ramalingam.c@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-25tools/intel_watermark: Recognize cnl+Ville Syrjälä
Pick the skl+ code path for cnl+. And since this tool has nothing to do with pch let's also replace the has_pch_split check with gen>=5 check. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-10-25README|CONTRIBUTING: Move to .mdDaniel Vetter
Looks so much better in the gitlab UI. Maybe we want to split out some of these README into subdirectories ... v2: Polish layout a bit. Changes are only whitespace, but README is so complicated reworked that git's rename detection doesn't spot the similarities anymore. Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-25CONTRIBUTING: Point towards gitlabPetri Latvala
Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-25kms_content_protection: Add Content Protection testRamalingam C
Pretty simple test: - picks the hdcp capable output with suitable pipe and apply modeset. - checks the connected sink's hdcp capability through debugfs - apply a FB and wait for the flip completion. - clears the content protection property - verifies that it clears - sets the content protection property to desired - verifies that it transitions to enabled - incase of timeout three reattempts are implemented - clear the content protection property and modeset on the crtc Above steps are repeated on all HDCP capable connectors for both legacy and atomic subtests. v2: dynamic subtests are dropped [Daniel] v3: debugfs is used to detect the sink's hdcp capability [Daniel] data structure is made as global variable. v4: debugfs file from connector's debugfs dir is used [Daniel] v5: i915_debugfs_connector_dir() usage is modified [Chris] Signed-off-by: Sean Paul <seanpaul@chromium.org> Signed-off-by: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-25lib/debugfs: function to open connector debugfs dirRamalingam C
Function to open a debugfs directory of a connector associated to a device. v2: instead of string manipulation openat used [Chris] Signed-off-by: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24gitlab-ci: Do meson build with oldest supported mesonArkadiusz Hiler
Documentation building differs for meson < 0.47, so let's make sure that it works. Also remove a comment that has slipped through. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24gitlab-ci: Compare test-list.txt(s) generated by meson and autotoolsArkadiusz Hiler
Let's make sure that we haven't missed any tests with any of the supported build systems. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24gitlab-ci: Compile using autotoolsArkadiusz Hiler
Let's make sure that IGT compiles with autotools. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24gitlab-ci: Parametrize images used with CI_ variablesArkadiusz Hiler
The container stage was already using CI_ variables, pushing to the right registry. Let's do that for the pulling containers too. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24tools/intel_vbt_decode: Dump DSI panel rotationVille Syrjälä
Dump out the rotation field from the MIPI config block. Cc: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Jani Nikula <jani.nikula@intel.com>
2018-10-24tests/kms_rotation_crc: Don't leave crc runningJuha-Pekka Heikkila
Leaving crc running nobody reading it may fail tests randomly with dmesg comment "*ERROR* Overflow of CRC buffer, userspace reads too slow." v2 (Maarten Lankhorst): Don't change clean up code but just add flag for starting crc and stop it at end of test. bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105748 Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2018-10-23Add GitLab's CI/CD configurationArkadiusz Hiler
The CI/CD pipeline is configured the following way: 1. Build docker images and add them to repo's registry (manual step for now, automation needs GitLab to be updated). 2. Build igt with meson on Debian and Fedora. 3. Run `ninja tests` on Fedora. 4. Build and publish docs as an artifact, for GitLab Pages. v2: Typos and fully-featured meson build. v3: Use variables instead of hard-coding registry and project. Cc: Petri Latvala <petri.latvala@intel.com> Cc: Daniel Vetter <daniel@ffwll.ch> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> Acked-by: Petri Latvala <petri.latvala@intel.com>
2018-10-23runner: Add "root" also to the "totals" objectPetri Latvala
Turns out the same information is looked up in different places in different code paths. Piglit's summary module looks up total counts in ['totals']['root'], CI looks up ['totals']['']. The latter is inherited from piglit, so this has probably changed at some point. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108486 Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Andi Shyti <andi.shyti@intel.com> Tested-by: Andi Shyti <andi.shyti@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-10-23tests: Introduce i915 directoryArkadiusz Hiler
We can already move all the tests with distinct prefixes: gem_, gen3_ and i915_. pm_ and drv_ tests will follow in batches, so we can do the adjustments in the reporting/filtering layer of the CI system. v2: Fix test-list.txt generation with meson v3: Fix docs build (Petri) Cc: Chris Wilson <chris@chris-wilson.co.uk> Cc: Petri Latvala <petri.latvala@intel.com> Cc: Martin Peres <martin.peres@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com> Tested-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
2018-10-19runner: Add --overall-timeoutPetri Latvala
With --overall-timeout $foo, the runner will stop executing new tests when $foo seconds have already been used. A resumed run will start over with no time used, using the same timeout. This allows for executing a long list of tests piecemeal, in about $foo length executions. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106127 Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com> Cc: Martin Peres <martin.peres@linux.intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-10-18tools/dpcd_reg: Introduce dump as the default operationRodrigo Vivi
For now this only imports the registers that were used on i915's debugfs dpcd dump. Later this could be extended. With this, we should be able to kill i915_dpcd from the kernel debugfs and rely solely on dpcd_reg for dpcd dumps. Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Tarun Vyas <tarun.vyas@intel.com>
2018-10-17tests/kms_selftest: Integrate kernel selftest test-drm_modesetDeepak Rawat
Call kernel selftest module test-drm_modeset for testing KMS. v2: - Add test alphabetically. - Add test to meson build. v3: Rename to kms_selftest. Signed-off-by: Deepak Rawat <drawat@vmware.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-17lib: Don't call igt_require_fb_modifiers() when no modifierDeepak Rawat
vmwgfx and amdgpu doesn't support fb modifiers, yet kms_addfb() requires modifier support. Since many tests don't need this to run, the requirement can be made less broad. Therefore, tighten the requirement to cases where modifiers are actually needed. v2: * In create_fb() calls to kms_addfb(), remove the modifier flag iff the driver doesn't support modifiers and the modifer is 0 * Don't modify the flag in kms_addfb(). Signed-off-by: Deepak Rawat <drawat@vmware.com> Signed-off-by: Leo Li <sunpeng.li@amd.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-17lib/igt_fb: Check for cairo surface successDeepak Rawat
For vmwgfx cairo surface creation fails due to stride mismatch, add a igt_require_f() for surface. v2: Check for surface creation failure. Signed-off-by: Deepak Rawat <drawat@vmware.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
generated by cgit v1.2.3 (git 2.39.1) at 2025-11-01 13:59:59 +0000