On Tue, Jan 24, 2017 at 01:02:23PM +0200, Petri Latvala wrote:
> On Tue, Jan 24, 2017 at 09:31:39AM +, Chris Wilson wrote:
> > On Wed, Jan 18, 2017 at 08:54:10PM +, Chris Wilson wrote:
> > > Execlists introduces a new wrinkle to filling rings, in that each
> > >
etting their
$favoritefeatureoftheday into BAT, I'm being a little more lenient
than I should with allowing more tests into fast-feedback at this
time. As soon as we can deploy Ezbench-driven extended testing (not
far now btw), fast-feedback will go on a diet.
--
Petri Latvala
___
k>
Indeed.
Acked-by: Petri Latvala <petri.latv...@intel.com>
> ---
> tests/intel-ci/fast-feedback.testlist | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/intel-ci/fast-feedback.testlist
> b/tests/intel-ci/fast-feedback.testlist
> ind
CC: Robert Foss <robert.f...@collabora.com>
CC: Lyude <ly...@redhat.com>
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tests/chamelium.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/chamelium.c b/tests/chamelium.c
index d914d
Reading the CRC files in debugfs will never
terminate. drv_suspend/debugfs-reader will leave a cat process around
which blocks all other access to the device until manually killed if
CRC files are not omitted.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tests/drv_suspend
gem_exec_reloc --list-subtests breaks otherwise.
v2: use igt_subtest_group (Chris)
CC: Chris Wilson <ch...@chris-wilson.co.uk>
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tests/gem_exec_reloc.c | 30 ++
1 file changed, 18 insertions(+),
gem_exec_reloc --list-subtests breaks otherwise.
CC: Chris Wilson <ch...@chris-wilson.co.uk>
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tests/gem_exec_reloc.c | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/tests/gem_exec_rel
On Mon, Jan 30, 2017 at 01:07:12PM +0200, Joonas Lahtinen wrote:
> delete mode 100755 tests/test_rte_check
Remove test_rte_check also from tests/Makefile.sources. With that,
Reviewed-by: Petri Latvala <petri.latv...@intel.com>
___
Intel-gf
On Fri, Jan 27, 2017 at 09:36:23PM +, Chris Wilson wrote:
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Petri Latvala <petri.latv...@intel.com>
> ---
> tests/intel-ci/fast-feedback.testlist | 5 +
> 1 file changed, 5 insertions(+)
>
>
this by moving the removal of the enum until after all of the
> tests have had their dependency on the enum fixed.
This is how it's done. No deliberately broken commits please.
--
Petri Latvala
>
> Another solution is just to collapse all of the test fixes into this patch.
> Whic
Just one more nitpick (in two parts) below. With that, you can
consider this now to be
Reviewed-by: Petri Latvala <petri.latv...@intel.com>
This patch can be pushed when the kernel changes that contain "must be
kept in sync" numbers in intel_dp_compliance.c have landed (and
> filling multiple execlist rings (for the same engine) to BAT.
> >
> > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> > Cc: Petri Latvala <petri.latv...@intel.com>
>
> Ping? This provides coverage of some hairy paths on execlists/guc that
> curre
if (device_override)
> return 0;
libdrm version check in configure.ac needs to be bumped to a version
containing that ioctl token.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
NAK.
DRIVER_VGEM is omitted from DRIVER_ANY intentionally. Vgem is unable
to modeset, unable to render, practically it only supports the
vgem-specific tests. See also: lib/drmtest.c, __drm_open_driver().
--
Petri Latvala
___
Intel-gfx mailing list
cessing garbage pointers on
> non-Intel devices.
>
> Signed-off-by: Brian Starkey <brian.star...@arm.com>
Reviewed-by: Petri Latvala <petri.latv...@intel.com>
Pushed, thanks.
> ---
> tests/kms_atomic_transition.c |2 +-
> 1 file changed, 1 insert
On Thu, Feb 09, 2017 at 08:20:58AM +, Chris Wilson wrote:
> Another bug found first in the wild...
>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Petri Latvala <petri.latv...@intel.com>
Behaviour on farm2 looks alright.
Acked-by: Petri Latvala
uot;h";
> +
> +static void print_help(void)
> +{
> +igt_info("\n [options]\n"
> + "-h, --help display usage\n"
> + "-q, --qemu the qemu path\n"
> + "-a, --hda the hda raw image / qco
lly for the test list that has
been used in the proof-of-concept setup (attached). The list runs in a
hair under three hours on SNB at this time, and has been collected
with the sophisticated method of "take everything, then throw
bathwater out, ignore babies". Naturally the list is not s
On 02/15/2017 08:24 PM, Michel Thierry wrote:
I cant be the only one that have added .tags by mistake.
v2: Do not ignore .gitignore
Cc: Petri Latvala <petri.latv...@intel.com>
Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Signed-off-by: Michel Thierry <michel.th
On 02/16/2017 09:34 PM, Michel Thierry wrote:
I cant be the only one that have added .tags by mistake.
v2: Do not ignore .gitignore
v3: Move !.gitignore at the end, otherwise it'll ignore new .gitignore
files in another directory (Petri)
Cc: Petri Latvala <petri.latv...@intel.com>
Cc:
NAK on these with the current drv_selftest code.
The subtest enumeration on IGT's side needs to contain these subtests
even if the running kernel doesn't have selftests, or these particular
subtests.
--
Petri Latvala
On Thu, Feb 02, 2017 at 09:18:00AM +, Chris Wilson wrote
On Wed, Jan 18, 2017 at 10:21:47AM +, Chris Wilson wrote:
> On Wed, Jan 18, 2017 at 12:14:00PM +0200, Petri Latvala wrote:
> > Thanks for the reminder about this ordering change.
> >
> > The crash from disable_display use will cause $stuff in CI, its
> > ab
Thanks for the reminder about this ordering change.
The crash from disable_display use will cause $stuff in CI, its
ability to recover and resume is not quite there yet. But as it's at
the end, we can live with it until a fix lands.
Series is
Acked-by: Petri Latvala <petri.latv...@intel.
Patches 1-4 and 6 are
Reviewed-by: Petri Latvala <petri.latv...@intel.com>
On Thu, Jan 19, 2017 at 11:35:00AM -0500, Robert Foss wrote:
> This series contains a few separate fixes mostly for memory allocations
> but also some small unrelated small fixes.
>
> This series
I was a bit confused there and there are other cases where
root rights are required.
Assuming root privileges is fine, especially if you're nice and warn the
user if proper rights are not active. Can the tool be used as a normal
user at all?
--
Petri Latvala
___
In
The series is
Tested-by: Petri Latvala <petri.latv...@intel.com>
On 09/30/2016 02:44 PM, Chris Wilson wrote:
dma_buf_export() adds a reference to the owning module to the dmabuf (to
prevent the driver from being unloaded whilst a third party still refers
to the dmabuf). H
See tests/intel-ci/README for rationale and explanation.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
The plans for CI are to use explicit test lists in the future, as laid
out in this commit. Comments, acks, nacks, flames, etc would be very
welcome.
The only list
On Wed, Oct 05, 2016 at 11:48:26PM +0100, Lionel Landwerlin wrote:
> - -o*)
> - file=${1##-o}
> - shift
> - ;;
This breaks using -ofilename without spaces.
--
Petri Latvala
___
Intel-gfx mailing li
s a nonrepresentable character present.
>
> while true; do
>case "$1" in
> @@ -38,6 +54,14 @@ while true; do
> file=${1##--output=}
> shift
> ;;
> + -c)
> + build_command &
See tests/intel-ci/README for rationale and explanation.
v2: Use the current BAT set for fast-feedback.testlist first
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
Thanks for the feedback. This commit only contains the current test
set, extended test suites are left for
For the series:
Reviewed-by: Petri Latvala <petri.latv...@intel.com>
On 10/05/2016 03:21 PM, Chris Wilson wrote:
dma_buf_export() adds a reference to the owning module to the dmabuf (to
prevent the driver from being unloaded whilst a third party still refers
to the dmabuf). H
The drop_caches sysctl has a max value of 4, so writing 7 to it just
fails. Avoid the earlier two-writes problem by opening the fd twice.
v2: Don't lseek(), open() twice. (Chris)
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
lib/intel_os.c | 13 +++--
1 file chang
Add myself, remove Daniel by request.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
For the record.
Daniel requested to be removed from the list (Daniel, please confirm,
that's how I understood), and I'm going to be maintaining IGT with
Marius until Marius steps down.
MAINT
The drop_caches sysctl has a max value of 4, so writing 7 to it just
fails. Avoid the earlier two-writes problem by seeking to the
beginning between writes.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
lib/intel_os.c | 9 +++--
1 file changed, 7 insertions(+), 2 del
This patch is now pushed, thanks.
--
Petri Latvala
On Wed, Oct 12, 2016 at 02:32:17PM +0100, Tvrtko Ursulin wrote:
>
> On 10/10/2016 15:31, Goel, Akash wrote:
> >
> >
> > On 10/10/2016 7:22 PM, Tvrtko Ursulin wrote:
> > >
> > > On 10/10/2016 11:59
On Tue, Oct 25, 2016 at 01:07:25PM +0300, Petri Latvala wrote:
> Am I understanding correctly that gem folks don't object to
> gem_storedw_loop being removed from BAT?
Interpreting silence as a yes.
Acked-by: Petri Latvala <petri.latv...@intel.com>
Please push this.
--
P
pov. Oh well.
Don't do that.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
;params->fb.fb->fb_id,
>DRM_MODE_PAGE_FLIP_EVENT, NULL);
> igt_assert_eq(rc, 0);
This makes FLIP_PAGEFLIP pass DRM_MODE_PAGE_FLIP_EVENT. Is that intentional?
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Thu, Nov 10, 2016 at 10:43:27AM +, Chris Wilson wrote:
> On Thu, Nov 10, 2016 at 12:20:51PM +0200, Petri Latvala wrote:
> > On Thu, Nov 10, 2016 at 08:05:19AM +, Chris Wilson wrote:
> > > A raw pageflip is nonblocking and asynchronous, but
> > > kms_fr
73 igt@kms_flip@basic-plain-flip
10.33 igt@gem_sync@basic-store-each
10.29 igt@gem_sync@basic-store-all
10.11 igt@gem_sync@basic-each
10.08 igt@gem_sync@basic-all
Does any of these strike as a low-hanging fruit?
--
Petri Latvala
___
Intel-gfx mailing
If executed too soon after prime_vgem tests, the vgem unload test
fails due to its execbuffers still being handled in the kernel. Retry
the unload three times with sleeps before reporting a skip.
When tested on HSW, one 1s sleep wasn't enough, 3s was.
Signed-off-by: Petri Latvala <petri.l
dundant, should it be
removed altogether?
--
Petri Latvala
>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> ---
> tests/gem_storedw_loop.c | 6 +++---
> tests/intel-ci/fast-feedback.testlist | 7 ---
> 2 files changed, 3 insertions(+), 10 de
e
> + skip "debugfs not found"
> fi
Would parsing the output of `mount -t debugfs` be an option?
--
Petri Latvala
>
> -if [ -d /sys/kernel/debug/dri ] ; then
> - debugfs_path=/sys/kernel/debug/dri
> +if [ ! -d $debugfs_path/dri ]; then
> + skip &qu
and sleeping.
We can also order vgem unload test to be first (along with
drv_module_reload) depending on whether we want to catch those stray
module leaks. Comments on that tradeoff?
Comments in the test list: Not yet. Piglit patch for them just sent to
piglit mailing list.
Petri Latvala
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
CC: Chris Wilson <ch...@chris-wilson.co.uk>
---
(Possible duplicate, I didn't see this arrive on the mailing list myself)
To match Chris's intention.
The test itself will receive fixes later, and another module loading
test w
l references to the vgem.ko's object (via dma-buf) will only
> exist if they were constructed by the test and if they were, e.g. i915,
> they too should be will be flushed by igt in its exithandlers.
>
> Other drivers may have similar bridges to cross ofc.
> -Chris
Ok, ex
On 10/18/2016 02:36 PM, Chris Wilson wrote:
On Tue, Oct 18, 2016 at 02:25:21PM +0300, Petri Latvala wrote:
If executed too soon after prime_vgem tests, the vgem unload test
fails due to its execbuffers still being handled in the kernel. Retry
the unload three times with sleeps before
nchmarks, tools, tests, CI test sets, lib)
The goal with this discussion is to reach a suitable tradeoff between
stability from CI point of view and fruitful use of programmer time.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists
54:33AM +0100, Chris Wilson wrote:
> > > > > On Thu, Oct 20, 2016 at 11:45:47AM +0300, Petri Latvala wrote:
> > > > > > On Wed, Oct 19, 2016 at 08:26:17PM +0100, Chris Wilson wrote:
> > > > > > > The inter-engine synchronisation (with and without se
Chris, happy with this revision?
On Fri, Oct 21, 2016 at 09:22:13AM +0530, Nautiyal Ankit wrote:
> From: Ramalingam C
>
> Idleness DRRS:
> By default the DRRS state will be at DRRS_HIGH_RR. When a Display
> content is Idle for more than 1Sec Idleness will be
igt@gem_flink_basic@basic
I pushed this commit, but without adding basic-s3-devices. The needed
kernel config changes will be deployed after CI is using IGT builds
with this commit in, next week.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Thu, Nov 17, 2016 at 01:48:36PM +, Chris Wilson wrote:
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Petri Latvala <petri.latv...@intel.com>
> ---
> tests/intel-ci/fast-feedback.testlist | 1 +
> 1 file changed, 1 insertion(+)
>
>
On Fri, Nov 25, 2016 at 12:00:43PM +0200, Petri Latvala wrote:
> On Thu, Nov 17, 2016 at 01:48:36PM +, Chris Wilson wrote:
> > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> > Cc: Petri Latvala <petri.latv...@intel.com>
> > ---
> > tests/intel
ded gem_alive and gem_exec_store into test (Chris Wilson)
> > - int main() to igt_main (Chris Wilson)
> > - moved tests/gem_alive -> tools/gem_info (Chris Wilson)
> > - added to intel-ci/fast-feedback.testlist (Petri Latvala)
> > - added hda_dynamic_debug() (Petri Latvala)
>
Commit 721d8747e3a2 added sync() calls to igt_main and
igt_simple_main, making self-tests fail to build. #including unistd.h
in igt_core.h fixes that.
Fixes: 721d8747e3a2 ("igt: Add a test for reordering execbufs")
CC: Chris Wilson <ch...@chris-wilson.co.uk>
Signed-off-b
On Tue, Nov 15, 2016 at 01:00:14PM +, Chris Wilson wrote:
> Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
> -Chris
>
Thanks, pushed.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.
) {
+ if ((err = reload(NULL)))
+ igt_fail(err);
+ }
+
+ igt_subtest("basic-exec") {
+ dir = dirname(prg);
+ finish_load(dir);
+ }
+
+ igt_subtest("basic-reload-inject") {
+ for (i = 0; i <
unds scary but LGTM.
Reviewed-by: Petri Latvala <petri.latv...@intel.com>
> ---
> tests/drv_module_reload_basic | 12 ++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/tests/drv_module_reload_basic b/tests/drv_module_reload_basic
>
Also re-sort the file.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tools/.gitignore | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/.gitignore b/tools/.gitignore
index 6ca6fbe..b19c908 100644
--- a/tools/.gitignore
+++ b/tools/.gitignore
@@
Also re-sort the file.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
benchmarks/.gitignore | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/benchmarks/.gitignore b/benchmarks/.gitignore
index 5332406..fa7f8ed 100644
--- a/benchmarks/.gitignore
+++ b/benc
ceed the time
budget.
- Tests must pass on current kernels, on some platform.
This will be documented when we can get CI do the testing for us on
multiple platforms and properly know what kind of requirements can
actually be tested.
For these tests, manual testing was done on IVB, HSW, and BYT. Stable,
fast and pa
tor_destroy(iter);
> }
> - if (pci_dev == NULL)
> - errx(1, "Couldn't find graphics card");
> + igt_require_f(pci_dev, "Couldn't find Intel graphics card\n");
>
> error = pci_device_probe(pci_dev);
> igt_fail_on_f(error !=
The test is producing a lot of CI noise.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
Will be pushed shortly. Visible in CI results tomorrow-ish.
tests/intel-ci/fast-feedback.testlist | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/intel-ci/fast-feedback.testl
On Fri, Oct 14, 2016 at 02:50:49PM +0300, Jani Nikula wrote:
> On Fri, 14 Oct 2016, Petri Latvala <petri.latv...@intel.com> wrote:
> > On Thu, Oct 13, 2016 at 03:59:55PM +0300, Jani Nikula wrote:
> >> While at it, make debugfs_path point at the debugfs root, not
&g
tests run in a usual configuration,
and as a nice side-effect, with the admin-specified global module
parameters (kernel cmdline, module config in /etc...). Those are
currently ignored because of the reload (oopsie).
Chris, Jani, comments on moving all reloads to the bottom?
--
Petri Latvala
___
> v2:
> * Add mode unset on hotplug uevent on disconnect (Manasi Navare)
>
> Cc: Petri Latvala <petri.latv...@intel.com>
> Cc: Marius Vlad <marius.c.v...@intel.com>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> Signed-off-by: Manasi Navare <manasi.d.nav...
("/sbin/modprobe -s r sw_sync"));
> +
> + err = false;
> + if (access(DEVFS_SW_SYNC, R_OK | W_OK) < 0) {
> + char buf[128];
> +
> + snprintf(buf, sizeof(buf), "%s/sw_sync", igt_debugfs_mount());
Hi
The copyright statements still need the year
corrected. intel_dp_compliance needs to be added to tools/.gitignore
Some new comments also:
- Why do some of the prints have \r\n?
- Building intel_dp_compliance should actually be made conditional upon
HAVE_UDEV
--
Petri Latvala
On Fri
for measuring fence wakeup latencies. (Chris
Wilson)
- prime_lookup: New microbenchmark for stressing prime_fd_to_handle
and prime_handle_to_fd. (Chris Wilson)
Test changes:
- Multiple new tests.
- Added an explicit list of tests used for Intel CI. (Petri Latvala)
- Converted multiple shell
on farm2.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Mon, Jan 02, 2017 at 04:33:03PM +0200, Ander Conselvan De Oliveira wrote:
> Hi Petri,
>
> I'm not sure what are the i-g-t rules regarding R-b tags nowadays. Does this
> require one or can I push it?
The series is
Reviewed-by: Petri Latvala <petri.latv...@intel.com>
Some o
ot eDP\n");
+ sink_crc.supported = false;
return;
}
Paulo?
--
Petri Latvala
On Thu, Dec 22, 2016 at 06:42:06PM -0200, Paulo Zanoni wrote:
> What I'm currently seeing is that sometimes the first check during
> setup_sink_crc() returns valid sink CRC, but then the
ibpciaccess-dev
> + libprocps3-dev
> libunwind-dev
> python-docutils
> x11proto-dri2-dev
> --
> 2.1.4
s/libprocps3-dev/libprocps-dev/
procps package name on Debian is just libprocps-dev. Ubuntu has
libprocps3-dev but only on 14.04, and even there the package
s, run by 'make [dist]check'.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
check" checks everything "make check" does, as does
executing the script by hand.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tests/igt_command_line.sh | 45 +
1 file changed, 29 insertions(+), 16 deletions(-)
diff --git a
for the same reason.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
CC: Chris Wilson <ch...@chris-wilson.co.uk>
---
lib/igt_kmod.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c
index 4b8ea81..b366ade 100644
--- a/li
Kernel selftest launchers use dynamic subtest enumeration. When
running on a kernel without selftests, they output nothing from
--list-subtests and exit with 0. Handle this specialty in the checker.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tests/igt_command_line.
Test at check and distcheck that testlist files only contain tests
that are still present.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
configure.ac | 1 +
tests/Makefile.am | 2 ++
tests/intel-ci/.gitignore | 1 +
tests/in
When generating the .gitignore file, use /.gitignore instead of
.gitignore to not have it apply to subdirectories.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tests/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/Makefile.am b
Document the test ordering choices in fast-feedback.testlist.
For comments in testlists, piglit commit
commit 0c535186d624071098c10003fdafe8f475ed9ae7
Author: Petri Latvala <petri.latv...@intel.com>
Date: Wed Feb 1 12:57:45 2017 +0200
framework/programs/run.py: Allow comments i
This series introduces a build-time checker for testlists, the test
list for extended testing, and as an added bonus, introduces the
ability to comment test lists.
Petri Latvala (4):
intel-ci: Add comments on test order to fast-feedback.testlist
tests/Makefile.am: Only ignore generated
Test list with wider coverage and longer full runtime.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
CC: Tomi Sarvela <tomi.p.sarv...@intel.com>
---
This is a preliminary testlist for extended testing. Tomi has been
running it semi-manually in CI already.
I think I have
ECBUFFER2_WR is not there, neither is
DRM_I915_GEM_EXECBUFFER2_WR.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
rsion checks.
Thanks, pushed.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
assembler/gen8_disasm.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/assembler/gen8_disasm.c b/assembler/gen8_disasm.c
index 7fc7a65..68bc033 100644
--- a/assembler/gen8_disasm.c
+++ b/assembler/gen8_disasm.c
@@ -138,11
efp_conn() and related have been unused since 86a546f6f798
("tools/intel_bios_reader: Dump out more information from the child
device structure")
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tools/intel_vbt_decode.c | 22 --
1 file changed, 22 de
os path\n\n"
> + "[example]\n"
> + " ./intel_gvtg_test -q /usr/bin/qemu-system-x86_64 -a "
> + "/home/img/ubuntu-16.04.img -b /usr/bin/bios.bin\n"
> + );
> +}
> +
> +static void arg_mismatch(char *arg)
> +{
> +igt_info("argument mismatch: %s\n", arg);
> +}
> +
> +int main(int argc, char *argv[])
> +{
> +int opt = -1;
> +const char *optstring = "hq:a:b:";
> +static struct option long_options[] = {
> +{"help", 0, NULL, 'h'},
> +{"qemu", 1, NULL, 'q'},
> +{"hda", 1, NULL, 'a'},
> +{"bios", 1, NULL, 'b'},
> +{0, 0, 0, 0}
> +};
> +
> +int ret = 0;
> +int flag_cnt = 0;
> +int h_flag = 0;
> +int q_flag = 0;
> +int a_flag = 0;
> +int b_flag = 0;
> +
> +if (check_gvtg_support() == 1) {
> +igt_skip("GVT-g technology is not supported in your system.\n");
> +}
> +
> +if (check_tools() == 1) {
> +igt_skip("Please install the \"uuid-runtime\" tool.\n");
> +} else if (check_tools() == 2) {
> +igt_skip("Please install the \"arp-scan\" tool.\n");
> +} else if (check_tools() == 3) {
> +igt_skip("Please prepare the \"qemu-ifup\" script.\n");
> +}
> +
The pattern
if (something) {
igt_skip("text");
}
can be written as
igt_skip_on_f(something, "text")
but that's not a huge deal.
Calling check_tools() multiple times is a bit wasteful. Also not a
huge deal.
Otherwise looks good to me.
--
Petri Latvala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
These four removals are ones the compiler was yelling at me for the
loudest.
Petri Latvala (4):
lib/intel_device_info: Remove unused whitney definition
tools/intel_vbt_decode: Remove unused code
tests/testdisplay: Remove unused SPRITE_COLOR_KEY
assembler/gen8_disasm.c: Remove unused
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
lib/intel_device_info.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
index db4f383..41d94b5 100644
--- a/lib/intel_device_info.c
+++ b/lib/intel_device_info.c
@@ -9,1
Unused since 99bdf4f0ff3e ("tests/testdisplay: Use cairo helpers to draw to
plane")
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
tests/testdisplay.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/testdisplay.c b/tests/testdisplay.c
index a974f42
headers copied from the kernel
sources, but we can live with that. Generating static testlists for CI
use is now possible, and running new selftests without IGT changes
stays possible.
Petri Latvala (2):
lib: Add i915 and drm-mm selftest headers from the kernel
Always expose IGT subtests
unning
kernel does not support selftests.
v2: Also add them to Makefile.sources
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
lib/Makefile.sources | 3 +++
lib/drm_mm_selftests.h| 24
lib/i915_live_selftests.h | 19
and rebuilding IGT.
v2: Use correct names for the testcases
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
lib/igt_kmod.c | 52 ++--
lib/igt_kmod.h | 8 +++-
tests/drm_mm.c | 17 -
tests/drv_selftest.
ich wasn't
possible before. It doesn't change behaviour from before when run with
selftests enabled. It makes other machinery work again, the ones that
expect subtest enumeration to work regardless of the runtime
environment. Changing _that_ rule with the kselftest runners is the
one change that is without
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
Marius has stepped down from being a maintainer a while ago, it's time
to make the file match reality.
I'd like to take this opportunity to thank Marius for his work and
wish him the best in his new endeavours.
MAINTAINERS |
On Mon, Mar 13, 2017 at 02:15:34PM +, Chris Wilson wrote:
> On Mon, Mar 13, 2017 at 01:02:04PM +0200, Petri Latvala wrote:
> > On Mon, Mar 13, 2017 at 10:50:04AM +, Chris Wilson wrote:
> > > Still completely lacking justification. The above is a non sequitur;
>
code to get_sink_crc()
kms_frontbuffer_tracking: refactor sink CRC reliability handling
kms_frontbuffer_tracking: destroy all FBs from all formats
kms_frontbuffer_tracking: fix compression checking
Petri Latvala (12):
kms_frontbuffer_tracking: fix sink CRC detection regression
intel-ci: Rem
For the record, I pushed this series so I could get the IGT release
out. I'm sure there are various ways this functionality can be
improved, so constructive suggestions are still definitely welcome.
--
Petri Latvala
___
Intel-gfx mailing list
Intel
and rebuilding IGT.
Signed-off-by: Petri Latvala <petri.latv...@intel.com>
---
lib/igt_kmod.c | 52 ++--
lib/igt_kmod.h | 8 +++-
tests/drm_mm.c | 17 -
tests/drv_selftest.c | 25 +++--
4 files c
, but we can live with that. Generating static testlists for CI
use is now possible, and running new selftests without IGT changes
stays possible.
Petri Latvala (2):
lib: Add i915 and drm-mm selftest headers from the kernel
Always expose IGT subtests for known kernel selftests
lib
1 - 100 of 508 matches
Mail list logo