[Intel-gfx] [PATCH i-g-t v4] tests/pm_rpm tests for set_caching and set_tiling

v4: re-bind the gem objects each time before calling disable_all_screens_and_wait(). v3: Use smaller sizes when allocating gem objects for caching tests. v2: use mmap to gtt instead off cpu and various style-changes. Reviewed-by: Imre Deak

[Intel-gfx] [PATCH i-g-t] tests/pm_rpm tests for set_caching and set_tiling ioctl(s)

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/pm_rpm.c | 114 + 1 file changed, 114 insertions(+) diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c index c4fb19c..e9ba9ea 100644 --- a/tests/pm_rpm.c +++ b/tests/pm

Re: [Intel-gfx] [PATCH] drm/i915: Add RPM references in the *_get_hw_state functions

Hi, I'm still seeing these warning(s) with both this patch and the one for i915_driver_unload() (http://patchwork.freedesktop.org/patch/69227/), in more than two cases: $ tests/kms_flip --run-subtest basic-flip-vs-wf_vblank [ 226.580012] [

Re: [Intel-gfx] [PATCH v2] drm/i915: Add RPM references in the *_get_hw_state functions

On Tue, Jan 05, 2016 at 02:54:31PM +0100, Daniel Vetter wrote: > On Mon, Jan 4, 2016 at 4:05 PM, Ville Syrjälä > wrote: > > I think what we really need here is some kind of > > intel_display_power_get_unless_zero() thing. We need to make sure not > > only that the

Re: [Intel-gfx] [PATCH i-g-t 1/3] lib/igt_kms migrate existing documentation to header file.

Hi, On Mon, Dec 21, 2015 at 05:09:51PM +0100, Daniel Vetter wrote: > On Mon, Dec 21, 2015 at 01:07:23PM +0200, Marius Vlad wrote: > > Signed-off-by: Marius Vlad <marius.c.v...@intel.com> > > Hm, why that? Given that this is against the usual style in igt this needs

Re: [Intel-gfx] [PATCH i-g-t 1/3] lib/igt_kms migrate existing documentation to header file.

On Tue, Dec 22, 2015 at 11:20:19AM +0100, Daniel Vetter wrote: > On Tue, Dec 22, 2015 at 12:05:10PM +0200, Marius Vlad wrote: > > Hi, > > > > On Mon, Dec 21, 2015 at 05:09:51PM +0100, Daniel Vetter wrote: > > > On Mon, Dec 21, 2015 at 01:07:23PM +0200, Marius

Re: [Intel-gfx] [PATCH i-g-t] tests/pm_rpm tests for set_caching and set_tiling ioctl(s)

On Wed, Nov 25, 2015 at 10:08:21PM +0200, Imre Deak wrote: > On ke, 2015-11-25 at 19:16 +0200, marius.c.v...@intel.com wrote: > > From: Marius Vlad <marius.c.v...@intel.com> > > > > Signed-off-by: Marius Vlad <marius.c.v...@intel.com>

[Intel-gfx] [PATCH i-g-t] tests/pm_rpm tests for set_caching and set_tiling ioctl(s)

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/pm_rpm.c | 120 + 1 file changed, 120 insertions(+) diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c index c4fb19c..d34b2b2 100644 --- a/tests/pm_rpm.c +++ b/tests/pm

[Intel-gfx] [PATCH i-g-t] overlay/intel-gpu-overlay Fix power consumption not being displayed

The power metter was not showing up due to a check over I915_PERF_ENERGY. ENOENT is returned when I915_PERF_ENERGY is not available, and we use that for relaying on debugfs i915_energy_uJ. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- overlay/power.c | 4 ++-- 1 file chan

[Intel-gfx] [PATCH i-g-t] lib/igt_kms: Fix passing invalid fd to openat(2).

Introduced by 0e11befe442. openat(2) uses a relative path. Fix by passing the correct fd. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> CC: Chris Wilson <ch...@chris-wilson.co.uk> --- lib/igt_kms.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/igt

[Intel-gfx] iommu/vt-d: Use per-cpu IOVA caching, suspend-to-ram causes machine hang

Hi Omer the following commit introduced a regression when doing suspend-to-ram. The machine never wakes up. We have more than a few machines that hang: SNB-i7-2620m, IVB-3370, HSW-4470r, BDW-5600u, BDW-525u. You can easily replicate by doing a rtcwake -s 10 -m mem.

Re: [Intel-gfx] iommu/vt-d: Use per-cpu IOVA caching, suspend-to-ram causes machine hang

Yes, that does it. Thanks. On Mon, Jun 06, 2016 at 12:02:30AM +0300, Adam Morrison wrote: > > Hi Omer the following commit introduced a regression when doing > > suspend-to-ram. The machine never wakes up. We have more than > > a few machines that hang: SNB-i7-2620m, IVB-3370, HSW-4470r,

Re: [Intel-gfx] ✗ Ro.CI.BAT: warning for drm/i915: Check VBT for port presence in addition to the strap on VLV/CHV

On Fri, Jun 03, 2016 at 03:28:52PM +0300, Ville Syrjälä wrote: > On Fri, Jun 03, 2016 at 10:06:54AM -, Patchwork wrote: > > == Series Details == > > > > Series: drm/i915: Check VBT for port presence in addition to the strap on > > VLV/CHV > > URL :

[Intel-gfx] DMAR error -> GPU hang, [WAS Re: i915 bug]

Hi, Thanks for the feedback. Have you tried with drm-intel-nightly [1] to see if this still happens? [1] https://anongit.freedesktop.org/git/drm-intel.git CC'ing the list. Maybe someone is aware of this. Seems there are bunch of duplicates for it. On Thu, Jun 02, 2016 at 09:06:05PM +0300,

Re: [Intel-gfx] [PATCH i-g-t] tests/drv_missed_irq_hang: Fix gem_blt path

On Mon, Jun 13, 2016 at 04:26:22PM +0300, Mika Kuoppala wrote: > Don't add SOURCE_DIR to the path for gem_blt as if this > script is invocated on some other directory, the path to > gem_blt will be concatenated two times. > > References: https://bugs.freedesktop.org/show_bug.cgi?id=88437 > Cc:

Re: [Intel-gfx] [PATCH i-g-t] demos/intel_sprite_on: Fix connector iteration bug

On Mon, Jun 13, 2016 at 07:51:06AM -0700, Jim Bride wrote: > Instead of looping until the first disconnected port is found, > now go through all possible connectors, drawing the sprite on > any connected display. > > Signed-off-by: Jim Bride > --- >

Re: [Intel-gfx] [PATCH i-g-t v2 1/8] tests/gem_exec_flush: Remove tests from BAT.

+0100, Chris Wilson wrote: > On Fri, May 27, 2016 at 07:27:46PM +0300, Marius Vlad wrote: > > You are lacking an explanation. Please tell me what this test is about > and why it is not suitable criteria for a basic acceptance test. > -Chris > > -- > Chris Wilson, Inte

Re: [Intel-gfx] [PATCH i-g-t v2 1/8] tests/gem_exec_flush: Remove tests from BAT.

On Mon, May 30, 2016 at 11:50:00AM +0100, Chris Wilson wrote: > On Mon, May 30, 2016 at 01:44:52PM +0300, Marius Vlad wrote: > > The explanation is the same as in the previous series: the GEM tests are > > taking too long. Either I sack them under nightly runs or decrease the >

[Intel-gfx] [ANNOUNCE] intel-gpu-tools 1.15

and other various fixes (Jani Nikula) - build: Add optional --enable-werror (hard-fail in case of compiler warnings) (Marius Vlad) - New KMS test: tests/prime_mmap_kms (show case dma-buf new API and processes restrictions) (Tiago Vignatti) - Several tests have gained to ability to be run

[Intel-gfx] [PATCH i-g-t 1/8] tests/gem_exec_flush: Tune down BAT timeout to ~1sec.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_exec_flush.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_exec_flush.c b/tests/gem_exec_flush.c index d08b843..b608060 100644 --- a/tests/gem_exec_flush.c +++ b/tests/gem_exec_flush.c @@

[Intel-gfx] [PATCH i-g-t 4/8] tests/gem_close_race: Tune down for BAT ~1s.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_close_race.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_close_race.c b/tests/gem_close_race.c index 94fb905..45aa2cc 100644 --- a/tests/gem_close_race.c +++ b/tests/gem_close_race.c @@

Re: [Intel-gfx] [PATCH i-g-t 4/8] tests/gem_close_race: Tune down for BAT ~1s.

e: > >>> > >>> On 27/05/16 12:58, Chris Wilson wrote: > >>>> On Fri, May 27, 2016 at 02:50:34PM +0300, Marius Vlad wrote: > >>>>> Signed-off-by: Marius Vlad <marius.c.v...@intel.com> > >>>> > >>>> Nak. It's

Re: [Intel-gfx] [PATCH i-g-t 4/8] tests/gem_close_race: Tune down for BAT ~1s.

On Fri, May 27, 2016 at 01:16:06PM +0100, Chris Wilson wrote: > On Fri, May 27, 2016 at 01:10:07PM +0100, Tvrtko Ursulin wrote: > > > > On 27/05/16 12:58, Chris Wilson wrote: > > >On Fri, May 27, 2016 at 02:50:34PM +0300, Marius Vlad wrote: > > >>Si

Re: [Intel-gfx] [PATCH i-g-t 4/8] tests/gem_close_race: Tune down for BAT ~1s.

On Fri, May 27, 2016 at 01:10:07PM +0100, Tvrtko Ursulin wrote: > > On 27/05/16 12:58, Chris Wilson wrote: > >On Fri, May 27, 2016 at 02:50:34PM +0300, Marius Vlad wrote: > >>Signed-off-by: Marius Vlad <marius.c.v...@intel.com> > > > >Nak. It's a race d

[Intel-gfx] [PATCH i-g-t 3/8] tests/gem_exec_nop: Tune down timeout for BAT ~1s.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_exec_nop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c index e90d5eb..1b2d144 100644 --- a/tests/gem_exec_nop.c +++ b/tests/gem_exec_nop.c @@ -213,7

[Intel-gfx] [PATCH i-g-t 8/8] tests/gem_exec_suspend: Remove hibernate test from BAT.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_exec_suspend.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_exec_suspend.c b/tests/gem_exec_suspend.c index cd133cc..3d529bd 100644 --- a/tests/gem_exec_suspend.c +++ b/tests/gem_exec_sus

[Intel-gfx] [PATCH i-g-t 5/8] tests/gem_sync: Tune down for BAT to ~1s for basic-each and ~2s for basic-all.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_sync.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/gem_sync.c b/tests/gem_sync.c index 320bce3..f824d1e 100644 --- a/tests/gem_sync.c +++ b/tests/gem_sync.c @@ -126,7 +126,7 @@ sync_ring(

[Intel-gfx] [PATCH i-g-t 7/8] tests/gem_tiled_pread_basic: Rename test and BATify.

Add a extended subtest and reduce the number of iteration to qualify for BAT. Renamed to test to gem_tiled_pread. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/Makefile.sources| 2 +- tests/gem_tiled_pread.c | 231 ++

[Intel-gfx] [PATCH i-g-t 6/8] tests/gem_exec_fault: Decrease the counts to lower the time for BAT.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_storedw_loop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_storedw_loop.c b/tests/gem_storedw_loop.c index 317b8c6..89f0182 100644 --- a/tests/gem_storedw_loop.c +++ b/tests/gem_storedw_

Re: [Intel-gfx] [PATCH i-g-t] tests/gem_{ctx, exec}_create: Adjust timeout values for BAT

On Fri, May 20, 2016 at 08:53:12PM +0100, Chris Wilson wrote: > On Fri, May 20, 2016 at 08:02:46PM +0300, Marius Vlad wrote: > > Tune down from 20s to 2s. Add the old timeout values under extended tests. > > Does it fail with the new timeout? If not, increase it. > > This

[Intel-gfx] [PATCH i-g-t 2/8] tests/gem_ctx_switch: Tune down for BAT ~1s.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_ctx_switch.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/gem_ctx_switch.c b/tests/gem_ctx_switch.c index 7b27336..066a5fc 100644 --- a/tests/gem_ctx_switch.c +++ b/tests/gem_ctx_switch.c @@

Re: [Intel-gfx] [PATCH i-g-t] igt_core: Search "." as final fallback for igt_fopen_data()

Pushed. On Thu, May 26, 2016 at 04:02:05PM -0700, Matt Roper wrote: > Some validation teams seem to run tests out of source directories that > have been nfs mounted or rsync'd to different locations on the target > machine. This causes the igt_srcdir that the tests were built with to > be

[Intel-gfx] [PATCH i-g-t v2 6/8] tests/gem_storedw_loop: Remove from BAT.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_storedw_loop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_storedw_loop.c b/tests/gem_storedw_loop.c index 317b8c6..a0c5af1 100644 --- a/tests/gem_storedw_loop.c +++ b/tests/gem_storedw_

[Intel-gfx] [PATCH i-g-t v2 2/8] tests/gem_ctx_switch: Remove test from BAT.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_ctx_switch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_ctx_switch.c b/tests/gem_ctx_switch.c index 7b27336..ce52365 100644 --- a/tests/gem_ctx_switch.c +++ b/tests/gem_ctx_switch.c @@

[Intel-gfx] [PATCH i-g-t v2 1/8] tests/gem_exec_flush: Remove tests from BAT.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_exec_flush.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/tests/gem_exec_flush.c b/tests/gem_exec_flush.c index d08b843..2f271bb 100644 --- a/tests/gem_exec_flush.c +++ b

[Intel-gfx] [PATCH i-g-t v2 4/8] tests/gem_close_race: Remove basic-threads from BAT.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_close_race.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_close_race.c b/tests/gem_close_race.c index 94fb905..80da731 100644 --- a/tests/gem_close_race.c +++ b/tests/gem_close_race.c @@

[Intel-gfx] [PATCH i-g-t v2 3/8] tests/gem_exec_nop: Remove test from BAT.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_exec_nop.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c index e90d5eb..8b31b27 100644 --- a/tests/gem_exec_nop.c +++ b/tests/gem_exec_nop.c @@ -212,9 +212,6 @@ ig

[Intel-gfx] [PATCH i-g-t v2 5/8] tests/gem_sync: Remove basic tests.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_sync.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/gem_sync.c b/tests/gem_sync.c index 320bce3..a46f120 100644 --- a/tests/gem_sync.c +++ b/tests/gem_sync.c @@ -221,12 +221,12 @@ ig

[Intel-gfx] [PATCH i-g-t v2 7/8] tests/gem_tile_pread_basic: Remove gem_tiled_pread_basic from BAT.

Test is renamed to gem_tiled_pread. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/Makefile.sources| 2 +- tests/gem_tiled_pread.c | 228 ++ tests/gem_tiled_pread_basic.c | 215

[Intel-gfx] [PATCH i-g-t 8/8] tests/gem_exec_suspend: Remove hibernate from BAT.

We are already skipping this test in CI on more than one platform, so migrate them to nightly. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_exec_suspend.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_exec_suspend.c b/tests/gem_exec_sus

[Intel-gfx] [PATCH i-g-t] tests/vgem_reload_basic: Include drm_lib.sh to please check target.

Scripts are required to have it included. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> CC: Chris Wilson <ch...@chris-wilson.co.uk> --- tests/vgem_reload_basic | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/vgem_reload_basic b/tests/vgem_reload_basic index b150

[Intel-gfx] [PATCH i-g-t v2] tests/drm_lib.sh: Extract fake getopt options so it can sourced into tests that require it.

This should allow tests/vgem_reload_basic to pass check target and it doesn't depend of i915 being present. v2: Do not source drm_lib.sh, only fake getopt parsing option to again access to --list-subtests option (Chris) Signed-off-by: Marius Vlad <marius.c.v...@intel.com> CC: Chris Wils

Re: [Intel-gfx] [RFC i-g-t v3 08/13] lib/stubs: Add stubs for intel_bufmgr.

On Mon, Jun 20, 2016 at 03:52:35PM +0100, Emil Velikov wrote: > Hi Rob, > > A couple of nitpicks and a case of missing git add :-) > > On 15 June 2016 at 10:51, wrote: > > > +if HAVE_LIBDRM_INTEL > > +else > > +libintel_tools_la_SOURCES += \ > > +

Re: [Intel-gfx] [PATCH i-g-t] assembler/: Fix lex warnings for %empty and %nonassoc.

iau wrote: > > > On Mon, May 16, 2016 at 01:39:10PM +0300, Marius Vlad wrote: > > > > Signed-off-by: Marius Vlad <marius.c.v...@intel.com> > > > > --- > > > > assembler/gram.y | 74 > > > >

[Intel-gfx] [PATCH i-g-t 1/2] tests/{kms_cursor_legacy, drv_missed_irq}, lib/drmtest: Fix warnings about unused function return values.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> CC: Chris Wilson <ch...@chris-wilson.co.uk> --- lib/drmtest.c | 8 +++- tests/drv_missed_irq.c| 2 +- tests/kms_cursor_legacy.c | 8 ++-- 3 files changed, 10 insertions(+), 8 deletions(-) diff --git a/lib

[Intel-gfx] [PATCH i-g-t 2/2] lib/drmtest: Report proper err message when opening the driver.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> CC: Chris Wilson <ch...@chris-wilson.co.uk> --- lib/drmtest.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/lib/drmtest.c b/lib/drmtest.c index 1e28f60..d2ee2d1 100644 --- a/lib/drmtest.c +++ b/l

[Intel-gfx] [PATCH i-g-t 1/2 v2] lib/igt_kms: Various fixes and add remaining documentation

keeps the initial format of the documentation. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 160 +++--- lib/igt_kms.h | 123 ++-- 2 files changed, 236 insertions(+), 47 del

[Intel-gfx] [PATCH i-g-t 2/2] lib/igt_kms: Short description between Intel/DRM terminology.

lib/igt_kms: Briefly describe Intel-to-DRM mapping between pipes, encoders and connectors. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 82 +++ 1 file changed, 82 insertions(+) diff --git a/lib/igt_kms.c

[Intel-gfx] [PATCH i-g-t] lib/igt_kms: Add COMIT_ATOMIC to igt_display_commit2()

So far, we had only COMMIT_UNIVERSAL and COMMIT_LEGACY, using drmModeSetPlane()/drmSetCrtc(). This patch adds COMMIT_ATOMIC to igt_display_commit2() that makes use of drmModeAtomicCommit(). Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c

[Intel-gfx] [PATCH i-g-t 1/3] lib/igt_kms: Add support for 4K and audio HDMI EDID injection.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 181 ++ lib/igt_kms.h | 2 + 2 files changed, 183 insertions(+) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 90c8da7..ed40c48 100644 --- a/lib/igt_kms.c

[Intel-gfx] [PATCH i-g-t 3/3] lib/igt_kms: Remove check against HDMI connectors on HSW and BDW.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index ed40c48..5c911d5 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -384,16 +384,8 @

[Intel-gfx] [PATCH i-g-t v2 0/3] 4K and audio HDMI EDID injection

. Marius Vlad (3): lib/igt_kms: Add support for 4K and audio HDMI EDID injection. tests/kms_hdmi_inject: Test to make use of HDMI injection capabilities. lib/igt_kms: Remove check against HDMI connectors on HSW and BDW. lib/igt_kms.c | 191

[Intel-gfx] [PATCH i-g-t 2/3] tests/kms_hdmi_inject: Test to make use of HDMI injection capabilities.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/Makefile.sources | 1 + tests/kms_hdmi_inject.c | 236 2 files changed, 237 insertions(+) create mode 100644 tests/kms_hdmi_inject.c diff --git a/tests/Makefile.sources b

[Intel-gfx] [PATCH i-g-t v2] tests/kms_pipe_crc_basic: Don't suspend the machine if the pipe is not present.

suspend-read-crc-pipe will perform a suspend and then skip the test in case the pipe is not present or is fused w/ another pipe. Skip the test before doing the suspend. v2: Somehow the subject was trimmed. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/kms_pipe_crc_b

Re: [Intel-gfx] [PATCH] lib/igt_kms: Add COMMIT_ATOMIC to igt_display_commit2()

these seem fit. On Thu, Feb 04, 2016 at 05:23:01PM +0530, Mayuresh Gharpure wrote: > Co-Author : Marius Vlad <marius.c.v...@intel.com> > > So far we have had only two commit styles, COMMIT_LEGACY > and COMMIT_UNIVERSAL. This patch adds another commit style > COMMIT_A

[Intel-gfx] [PATCH i-g-t] tests/kms_pipe_crc_basic: Don't suspend the machine if

suspend-read-crc-pipe will perform a suspend and then skip the test in case the pipe is not present on the platform. Skip the test before doing the suspend. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/kms_pipe_crc_basic.c | 1 + 1 file changed, 1 insertion(+) diff

[Intel-gfx] [PATCH i-g-t] tests/prime_mmap: Encapsulate check_for_dma_buf_mmap() in igt_fixture.

This unbreaks distcheck target that in turn runs each test with --list-subtests. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/prime_mmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/prime_mmap.c b/tests/prime_mmap.c index 29a0cfd..1ea61c2

Re: [Intel-gfx] [PATCH] lib/igt_kms: Add COMIT_ATOMIC to igt_display_commit2()

Hi, I still do not see FB_ID set to 0 when disabling the plane in igt_atomic_prepare_plane_commit(). See http://lists.freedesktop.org/archives/intel-gfx/2016-January/085790.html On Fri, Jan 29, 2016 at 02:17:11PM +0530, Mayuresh Gharpure wrote: > Co-Author : Marius V

Re: [Intel-gfx] [PATCH] drm/i915: Support HDMI EDID injection

On Tue, Feb 02, 2016 at 05:32:21PM +0200, Jani Nikula wrote: > On Tue, 02 Feb 2016, Marius Vlad <marius.c.v...@intel.com> wrote: > > Use the drm_property_blob data for user-supplied EDID blobs. > > > > Signed-off-by: Marius Vlad <marius.c.v...@intel.com>

[Intel-gfx] [PATCH v2] drm/i915: Support HDMI EDID injection

Make a copy of drm_property_blob data for user-supplied EDID blobs. v2: Allocate and memcopy drm_property_blob->data instead of pointing to it. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- drivers/gpu/drm/i915/intel_hdmi.c | 10 ++ 1 file changed, 10 insertions

Re: [Intel-gfx] [PATCH i-g-t v6] lib/igt_kms: Add COMMIT_ATOMIC to igt_display_commit2()

; > > Co-Author : Marius Vlad <marius.c.v...@intel.com> > Co-Author : Pratik Vishwakarma <pratik.vishwaka...@intel.com> > > So far we have had only two commit styles, COMMIT_LEGACY > and COMMIT_UNIVERSAL. This patch adds another commit style > COMMIT_ATOMIC which

Re: [Intel-gfx] [PATCH i-g-t] lib/igt_kms: Add COMIT_ATOMIC to igt_display_commit2()

eddy > <avinash.reddy.pall...@intel.com> > Subject: Re: [Intel-gfx] [PATCH i-g-t] lib/igt_kms: Add COMIT_ATOMIC to > igt_display_commit2() > > On Fri, Jan 15, 2016 at 11:06:24AM +0200, Marius Vlad wrote: > > So far, we had only COMMIT_UNIVERSAL and COMMIT_LEGACY, u

Re: [Intel-gfx] [PATCH i-g-t 2/2] lib/igt_kms: Short description between Intel/DRM terminology.

On Fri, Jan 15, 2016 at 02:46:45PM +0200, Ville Syrjälä wrote: > On Fri, Jan 15, 2016 at 11:06:42AM +0200, Marius Vlad wrote: > > lib/igt_kms: Briefly describe Intel-to-DRM mapping between pipes, encoders > > and > > connectors. > > > > Signed-off-by: Mar

Re: [Intel-gfx] [PATCH] lib/igt_kms: Add COMIT_ATOMIC to igt_display_commit2()

Hi, On Wed, Jan 27, 2016 at 05:50:52PM +0530, Mayuresh Gharpure wrote: > Co-Author : Marius Vlad <marius.c.v...@intel.com> > > So far we have had only two commit styles, COMMIT_LEGACY > and COMMIT_UNIVERSAL. This patch adds another commit style > COMMIT_A

Re: [Intel-gfx] [PATCH i-g-t] tests/prime_mmap: Encapsulate check_for_dma_buf_mmap() in igt_fixture.

+8)+8, ()) == 0 > (prime_mmap:24023) ioctl-wrappers-CRITICAL: Last errno: 25, Inappropriate > ioctl for device > (prime_mmap:24023) ioctl-wrappers-CRITICAL: error: -1 != 0 > END > prime_mmap: ../../lib/igt_core.c:1014: igt_fail: Assertion > `!test_with_subtests

Re: [Intel-gfx] [PATCH i-g-t v4 1/1] lib/igt_pm: Lib for power management

ves HD audio and SATA link power management. > For SATA link PM there's also code to save the previous settings, > to allow for resetting the values after we've finished testing. > > Signed-off-by: David Weinehall <david.weineh...@intel.com> Reviewed-by: Marius Vlad <marius.c.v.

Re: [Intel-gfx] [PATCH i-g-t 2/3] lib/igt_kms: Add fail exit branch in do_display_commit()

On Fri, Feb 19, 2016 at 02:34:52PM +0200, Gabriel Feceoru wrote: > On Cherryview PIPE_C can only be connected to PORT_D (bspec). > The driver properly reports the crtc_mask for the encoder, however the > mismatch between pipe and port is not reported back to the test. > > Add support for

[Intel-gfx] [PATCH i-g-t] Update MAINTAINERS file.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7825e1e..58f9efb 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1,2 +1,2 @@ Daniel Vetter <daniel.vet...@intel.com

[Intel-gfx] [PATCH 1/3] lib/igt_kms: Add support for 4K and audio HDMI EDID injection.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 176 ++ lib/igt_kms.h | 2 + 2 files changed, 178 insertions(+) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 90c8da7..64aa5d1 100644 --- a/lib/igt_kms.c

[Intel-gfx] [PATCH 2/3] tests/kms_hdmi_inject: Test to make use of HDMI injection capabilities.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/Makefile.sources | 1 + tests/kms_hdmi_inject.c | 165 2 files changed, 166 insertions(+) create mode 100644 tests/kms_hdmi_inject.c diff --git a/tests/Makefile.sources b

[Intel-gfx] [PATCH] drm/i915: Support HDMI EDID injection

Use the drm_property_blob data for user-supplied EDID blobs. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- drivers/gpu/drm/i915/intel_hdmi.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_

[Intel-gfx] [PATCH 3/3] lib/igt_kms: Remove check against HDMI on HSW and BDW when forcing a connector. This should allow running tests/kms_hdmi_inject.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 64aa5d1..9b294dd 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -384,16 +384,8 @

Re: [Intel-gfx] [PATCH] lib/igt_kms: Add COMMIT_ATOMIC to igt_display_commit2()

Hi, See bellow: On Wed, Mar 09, 2016 at 07:43:54PM +0530, Mayuresh Gharpure wrote: > Co-Author : Marius Vlad <marius.c.v...@intel.com> > Co-Author : Pratik Vishwakarma <pratik.vishwaka...@intel.com> > > So far we have had only two commit styles, COMMIT_LEGAC

Re: [Intel-gfx] [CI-ping 1/9] drm/i915: Include engine->last_submitted_seqno in GPU error state

There are some results on PW: https://patchwork.freedesktop.org/series/5400/ Fi CI seems to be a little bit late on the results On Thu, Apr 07, 2016 at 04:06:49PM +0100, Chris Wilson wrote: > On Thu, Apr 07, 2016 at 07:29:10AM +0100, Chris Wilson wrote: > > It's useful to look at the last

Re: [Intel-gfx] System freeze apparently due to GPU memory exhaustion - why?

Hi Adam, If you have cloned piglit into i-g-t directory go in scripts/ directory and run it from there (./run-tests.sh -t basic -s). It might be that a results/ directory already exists so remove the directory before this. There's currently some discussion on how to handle this

Re: [Intel-gfx] [PATCH i-g-t] tests/gem_concurrent_all: Please the compiler regarding fscanfs unused result.

On Tue, Apr 05, 2016 at 06:23:21PM +0100, Chris Wilson wrote: > On Tue, Apr 05, 2016 at 08:13:23PM +0300, Marius Vlad wrote: > > Signed-off-by: Marius Vlad <marius.c.v...@intel.com> > > --- > > tests/gem_concurrent_all.c | 2 +- > > 1 file changed, 1 insertio

[Intel-gfx] [PATCH i-g-t] tests/gem_concurrent_all: Please the compiler regarding fscanfs unused result.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_concurrent_all.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_concurrent_all.c b/tests/gem_concurrent_all.c index 10e5357..e84ded3 100644 --- a/tests/gem_concurrent_all.c +++ b

[Intel-gfx] [PATCH i-g-t 3/3] tests/kms_force_connector_basic: Add HDMI and audio EDID injection.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/kms_force_connector_basic.c | 236 +- 1 file changed, 231 insertions(+), 5 deletions(-) diff --git a/tests/kms_force_connector_basic.c b/tests/kms_force_connector_basic.c index d2c804c..e

[Intel-gfx] [PATCH i-g-t 2/3] lib/igt_kms: Allow forcing HDMI connectors and remove checks against BDW and HSW.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 12d52f5..b90ab3e 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -531,16 +531,9 @

[Intel-gfx] [PATCH i-g-t v3 0/3] 4K and audio HDMI EDID injection

sure we're checking all of them. v2: - Use a proper subject. - Added speaker data block and check that we get indeed a valid ELD for audio injection, otherwise we can not tell if the audio injection succeeded. Marius Vlad (3): lib/igt_kms: Add helpers for injecting 4k HDMI EDID, and HDMI

[Intel-gfx] [PATCH i-g-t 1/3] lib/igt_kms: Add helpers for injecting 4k HDMI EDID, and HDMI-capable audio.

Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 182 ++ lib/igt_kms.h | 2 + 2 files changed, 184 insertions(+) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 82257a6..12d52f5 100644 --- a/lib/igt_kms.c

Re: [Intel-gfx] [PATCH i-g-t 2/7] lib/igt_kms: refactor atomic properties load/commit

On Wed, Mar 23, 2016 at 11:38:05AM +, Lionel Landwerlin wrote: > All properties can be listed regardless of whether we're dealing with > an atomic enabled driver, so load all of the properties ids regardless > at init time. > > Also, we can have only one function loading the property ids and

[Intel-gfx] [PATCH] drm: Add support for EDID injection.

Allow the possibility to return an copy of the injected EDID when the connector has been forced and an EDID has been specified over the debugfs interface. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- drivers/gpu/drm/drm_edid.c | 23 --- 1 file chang

Re: [Intel-gfx] [PATCH i-g-t] scripts/run-test.sh: Piglit overwrite option.

Isn't this as https://patchwork.freedesktop.org/series/4177/? On Fri, Mar 18, 2016 at 07:02:11AM -0700, Rodrigo Vivi wrote: > The following piglit commit adds one option to overwrite files: > > commit ec317ece07afdf9c8a26de04bdec8a94e5d7b2db > Author: Dylan Baker >

Re: [Intel-gfx] [i-g-t PATCH] tests: don't skip kms_panel_fitting when non-eDP connected

Is there a reason why other connectors do not support panel fitting? Removed the check and using a HDMI connector the test passes... On Wed, Mar 23, 2016 at 02:19:25PM +0200, Jani Nikula wrote: > kms_panel_fitting skips if there's a non-eDP display connected. Fix it. > > Bugzilla:

Re: [Intel-gfx] Information about Intel_GPU_TOP tool

Hi, As Sergio stated, tools/intel_gpu_top is rather old and has some limitations, but you are welcome to improve it :-). 1. None that I know of. 2. Most likely true. 3. The tool suggested by Sergio is an alternative. On Fri, Mar 18, 2016 at 03:57:16PM +, Sanchez, AdolfoX

[Intel-gfx] [PATCH i-g-t] tests/kms_setmode: Fix compilation warnings.

[-Wdiscarded-qualifiers] ids, crtc->connector_count, >mode); Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/kms_setmode.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c index 6afe00c..e7a5bee 100644

[Intel-gfx] [ANNOUNCE] intel-gpu-tools 1.14

it timeout to 50 ms. gem_ringfill: fix typo in test name kms_force_connector_basic: Add force-load-detect test tests/kms_force_connector_basic: Disable all crtc's for load-detect. Marius Vlad (4): Update MAINTAINERS file. tests/prime_mmap: Encapsulate check_for_dm

Re: [Intel-gfx] [PATCH i-g-t] Adding tests using COMMIT_ATOMIC path

Library support has landed, please use a proper commit message so we can add this (i.e., tests/kms_rotation_crc: ) On Mon, Mar 07, 2016 at 03:55:03PM +0530, Pratik Vishwakarma wrote: > From: pvishwak > > Depends on: >

Re: [Intel-gfx] [PATCH i-g-t] lib/igt_kms: Fix atomic plane commit when fb_id is 0.

On Tue, Apr 05, 2016 at 10:46:37AM +0200, Maarten Lankhorst wrote: > Op 04-04-16 om 17:47 schreef Marius Vlad: > > igt_atomic_prepare_plane_commit() assumes that the framebuffer is always > > set-up. > > > > Signed-off-by: Marius Vlad <marius.c.v...@intel.com&g

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [v3] mm/vmap: Add a notifier for when we run out of vmap address space (rev3)

We're not catching it, but this gives deadlock trace when running kms_pipe_crc_basic@suspend-read-crc-pipe-A (happens on BSW): [ 132.555497] kms_pipe_crc_basic: starting subtest suspend-read-crc-pipe-A [ 132.734041] PM: Syncing filesystems ... done. [ 132.751624] Freezing user space processes

[Intel-gfx] [PATCH i-g-t] lib/igt_kms: Fix atomic plane commit when fb_id is 0.

igt_atomic_prepare_plane_commit() assumes that the framebuffer is always set-up. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- lib/igt_kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 82257a6..30c5b7e 100644 ---

Re: [Intel-gfx] [PATCH i-g-t 7/7] kms_panel_fitting: Add support for LVDS and DSI connectors.

Think you can squash the last two into one (you're removing what you've added in the previous one). Also see https://lists.freedesktop.org/archives/intel-gfx/2016-March/090617.html. On Fri, Apr 22, 2016 at 05:25:00PM -0400, robert.f...@collabora.com wrote: > From: Robert Foss

Re: [Intel-gfx] [PATCH] kms_atomic: Fix crtc_id comparison failing.

You've already sent this yesterday. Its pushed. On Thu, Apr 21, 2016 at 10:24:28PM -0400, robert.f...@collabora.com wrote: > From: Robert Foss > > Fixed ctrc_id comparison failing due to bad initialization of > crtc variable. > > Signed-off-by: Robert Foss

[Intel-gfx] [PATCH i-g-t] tests/gem_close_race: Remove basic-threads from BAT.

Currently this test causes some machines to hang and segfaults on others. Rename it for now until we figure out the root cause. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95048 Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_close_race.c | 2 +- 1 file chan

[Intel-gfx] [PATCH i-g-t 2/2] autotools, assembler: Fix check target for tests in assembler.

assembler suffers from the same issue as lib/tests, adjust it so it allows us to do a check. Have to make autotools happy by having an empty Makefile.am. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- assembler/Makefile.am

[Intel-gfx] [PATCH i-g-t 1/2] autotools: Check target requires to a have the list of tests built.

We need to have the test list generated before running the check target. Migrated igt_command_line.sh to tests/ from lib/tests/, which allows to building the tests and execute the script. This would allow cleaning followed by a make check. Signed-off-by: Marius Vlad <marius.c.v...@intel.

Re: [Intel-gfx] [PATCH] prime_mmap_kms: show case dma-buf new API and processes restrictions

On Mon, Apr 25, 2016 at 11:07:48AM -0300, Tiago Vignatti wrote: > dma-buf new API consists of: > > - mmap(dma_buf_fd): the ability to map a dma-buf file-descriptor to the > user-space, and most importantly, to actually write on the mapped pointer. > Worth to note that the Direct Rendering Manager

[Intel-gfx] [PATCH i-g-t] tests/gem_close_race: Add igt_fixture block for igt_stop_hang_detector.

igt_fork_hang_detector() was called from a igt_fixture block, while its counterpart (igt_stop_hang_detector) was called normally, causing SIGTERM to be sent when running under check target. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- tests/gem_close_race.c | 3 ++- 1 file chan

Re: [Intel-gfx] i951 ERRORs and WARN_ON()s (was: blank screen on boot with i915/DRM_FBDEV_EMULATION)

Florian, if you're using drm-intel-nigthly submit a bug at https://bugs.freedesktop.org/enter_bug.cgi?product=DRI, with DRM/intel as component. This way we can track some kind of progress/regress. The FIFO underruns should've have been fixed, but maybe there's something related to your platform.

  1   2   3   >