kmstest_get_crtc was skipping because at that point the crtc was not
active yet, instead we should only use igt_assert_plane_visible
directly. Unexport kmstest_get_crtc, since nothing here should need it.
While at it fix a small leak in igt_assert_plane_visible, the only
remaining user.
Additional
> -Original Message-
> From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Daniel Vetter
> Sent: Wednesday, October 18, 2017 5:36 PM
> To: Latvala, Petri
> Cc: intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH i-g-t] kms_atomic_transition: Outp
== Series Details ==
Series: series starting with [CI,1/4] lib: Add DROP_IDLE
URL : https://patchwork.freedesktop.org/series/32268/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
d4a4fbec0c079c7bec7fa6a2c756eea02314799e igt/prime_mmap_coherency: Call
prime
If the device is in runtime suspend, resuming takes time and reduces our
powersaving. If this was for a small write into an object, that resume
will take longer than any savings in using the indirect GGTT access to
avoid the cpu cache.
Signed-off-by: Chris Wilson
Reviewed-by: Daniel Vetter
---
On 10/18/2017 8:50 PM, Radoslaw Szwichtenberg wrote:
Moving code out of the boost function will allow its usage
in other/new test scenarios.
Signed-off-by: Radoslaw Szwichtenberg
---
tests/pm_rps.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/tests
Emphasize that we want to test synchronisation using the dmabuf API
(prime_sync_start, prime_sync_end) and so drop the manual
synchronisation using the GEM API (gem_sync).
Signed-off-by: Chris Wilson
---
tests/prime_mmap_coherency.c | 4
1 file changed, 4 deletions(-)
diff --git a/tests/pr
As the driver itself keeps a cache of objects, these too need to be
flushed prior to producing a stable count of objects.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102655
Signed-off-by: Chris Wilson
Reviewed-by: Joonas Lahtinen
---
lib/igt_debugfs.c | 3 ++-
1 file changed, 2 inser
As part of the general procedure for ensuring the GPU is idle, we also
want to ask the driver to flush its idle_worker. The idle_worker is
responsible for releasing both the driver's internal cache of buffers
and cache of state (such as the prolonged GT wakeref). By flushing the
idle_worker we ensu
A new flag for an old API; now we can request that the driver flush its
idle_worker to release internal caches.
Signed-off-by: Chris Wilson
Reviewed-by: Joonas Lahtinen
---
lib/igt_debugfs.h | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/lib/igt_debugfs.h b/lib/igt_
Release the internal caches (by flushing the idle_worker) to maximise the
available memory for use by tests (and to reduce sporadic skipping when
on the cusp).
Signed-off-by: Chris Wilson
Reviewed-by: Joonas Lahtinen
---
lib/intel_os.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
Quoting Patchwork (2017-10-19 01:18:19)
> == Series Details ==
>
> Series: igt/prime_mmap_coherency: Call prime_sync_start before read after
> write
> URL : https://patchwork.freedesktop.org/series/32219/
> State : success
>
> == Logs ==
>
> For more details see:
> https://intel-gfx-ci.01.or
== Series Details ==
Series: series starting with [1/2] drm/i915: Let's use more enum intel_dpll_id
pll_id.
URL : https://patchwork.freedesktop.org/series/32245/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
pass -> FAIL (shard-hsw) fdo#99
Hi Dave,
Here goes another round of drm/i915 fixes for 4.14.
drm-intel-fixes-2017-10-18-1:
Fix for stable:
- Fix DDI translation tables for BDW (Chris).
Critical fix:
- Fix GPU Hang on GVT (Changbin).
Other fixes:
- Fix eviction when GGTT is idle (Chris).
- CNL PLL fixes (Rodrigo).
- Fix pw
== Series Details ==
Series: DDI buf trans cleanup (rev3)
URL : https://patchwork.freedesktop.org/series/32049/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
pass -> FAIL (shard-hsw) fdo#99912
Test kms_chv_cursor_fail:
Subgroup pipe
On 2017.10.18 13:25:43 +0300, Joonas Lahtinen wrote:
> When running under virtualization (vGPU active), we must disable
> the azy PPGTT page table initialization optimization introduced by:
>
> 14826673247e ("drm/i915: Only initialize partially filled pagetables")
>
> We must do this because GVT-
== Series Details ==
Series: tests/kms_fbcon_fbt: Report fbc_status on error
URL : https://patchwork.freedesktop.org/series/32256/
State : warning
== Summary ==
IGT patchset tested on top of latest successful build
9a3f6f59d4c208219f7d3178106bdef628ff68d0 tests/intel-ci: Remove
fast-feedback-
knowing the assertion triggered on wait_until_enabled() is not that
useful without knowing what exactly caused the failure. It might be an
user error, like too little stolen memory by the bios, or an actual
issue in the kernel. So, let's make life easier, particularly for the
CI, by printing the
== Series Details ==
Series: tests/pm_rps: Move some test logic out of boost function (rev2)
URL : https://patchwork.freedesktop.org/series/32131/
State : success
== Summary ==
Test prime_self_import:
Subgroup export-vs-gem_close-race:
fail -> PASS (shard-hs
== Series Details ==
Series: igt/prime_mmap_coherency: Call prime_sync_start before read after write
URL : https://patchwork.freedesktop.org/series/32219/
State : success
== Summary ==
Test prime_self_import:
Subgroup export-vs-gem_close-race:
fail -> PASS (
== Series Details ==
Series: series starting with [RFC,1/2] drm/i915: Add connector property to
force bpc
URL : https://patchwork.freedesktop.org/series/32251/
State : warning
== Summary ==
Series 32251v1 series starting with [RFC,1/2] drm/i915: Add connector property
to force bpc
https://pa
On Wed, Oct 18, 2017 at 09:50:59PM +, Rodrigo Vivi wrote:
> On Wed, Oct 18, 2017 at 08:48:24PM +, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Track the system agent voltage we request from pcode in the cdclk state
> > on CNL. Annoyingly we can't actually read out the current valu
Currently the user space does not have a way to force the bpc. The
default bpc to be programmed is decided by the driver and is run
against connector limitations. Creating a new property for the userspace
to recommend a certain color depth while scanning out the pixels.
Cc: Ville Syrjälä
Cc: Paul
Use the newly added force_bpc connector property to limit pipe bpp.
Cc: Ville Syrjälä
Cc: Paulo Zanoni
Cc: Manasi Navare
Signed-off-by: Radhakrishna Sripada
---
drivers/gpu/drm/i915/intel_display.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/drivers/gpu/drm/
== Series Details ==
Series: series starting with [1/4] lib: Add DROP_IDLE
URL : https://patchwork.freedesktop.org/series/32211/
State : success
== Summary ==
Test perf:
Subgroup polling:
fail -> PASS (shard-hsw) fdo#102252
Test kms_flip:
Subgroup dp
Release the internal caches (by flushing the idle_worker) to maximise the
available memory for use by tests (and to reduce sporadic skipping when
on the cusp).
Signed-off-by: Chris Wilson
Reviewed-by: Joonas Lahtinen
---
lib/intel_os.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
As part of the general procedure for ensuring the GPU is idle, we also
want to ask the driver to flush its idle_worker. The idle_worker is
responsible for releasing both the driver's internal cache of buffers
and cache of state (such as the prolonged GT wakeref). By flushing the
idle_worker we ensu
As the driver itself keeps a cache of objects, these too need to be
flushed prior to producing a stable count of objects.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102655
Signed-off-by: Chris Wilson
Reviewed-by: Joonas Lahtinen
---
lib/igt_debugfs.c | 3 ++-
1 file changed, 2 inser
A new flag for an old API; now we can request that the driver flush its
idle_worker to release internal caches.
Signed-off-by: Chris Wilson
Reviewed-by: Joonas Lahtinen
---
lib/igt_debugfs.h | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/lib/igt_debugfs.h b/lib/igt_
Quoting Patchwork (2017-10-18 22:36:41)
> == Series Details ==
>
> Series: drm/i915: Flush the idle-worker for debugfs/i915_drop_caches (rev2)
> URL : https://patchwork.freedesktop.org/series/32102/
> State : warning
>
> == Summary ==
>
> Test perf:
> Subgroup polling:
>
On Wed, Oct 18, 2017 at 08:48:24PM +, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Track the system agent voltage we request from pcode in the cdclk state
> on CNL. Annoyingly we can't actually read out the current value since
> there's no pcode command to do that, so we'll have to just ass
On Wed, 18 Oct 2017 00:50:49 +0200, Sujaritha Sundaresan
wrote:
The Additional Data Struct (ADS) contains objects that are required by
guc post FW load and are not necessarily submission-only (although that's
our current only use-case). If in the future we load GuC with submission
disabled to
== Series Details ==
Series: drm/i915: Flush the idle-worker for debugfs/i915_drop_caches (rev2)
URL : https://patchwork.freedesktop.org/series/32102/
State : warning
== Summary ==
Test perf:
Subgroup polling:
fail -> PASS (shard-hsw) fdo#102252
Test kms_cur
On Wed, 18 Oct 2017 12:29:13 +0200, Sagar Arun Kamble
wrote:
On 10/18/2017 4:20 AM, Sujaritha Sundaresan wrote:
Updating GuC and HuC firmware select function to support removing
i915_modparams.enable_guc_loading module parameter.
v2: Clarifying the commit message (Anusha)
v3: Unify seq_p
On Mon, Oct 16, 2017 at 05:57:01PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> encoder->type is unreliable for DP/HDMI, so pass it in explicity into
> cnl_ddi_vswing_sequence(). This matches what we do for BXT.
>
I still dont get why we cant use encoder->type reliably?
Since if I trace
== Series Details ==
Series: drm/i915: CNL DVFS thing
URL : https://patchwork.freedesktop.org/series/32247/
State : warning
== Summary ==
Series 32247v1 drm/i915: CNL DVFS thing
https://patchwork.freedesktop.org/api/1.0/series/32247/revisions/1/mbox/
Test debugfs_test:
Subgroup read_a
From: Ville Syrjälä
Track the system agent voltage we request from pcode in the cdclk state
on CNL. Annoyingly we can't actually read out the current value since
there's no pcode command to do that, so we'll have to just assume that
it worked.
Cc: Mika Kahola
Cc: Manasi Navare
Cc: Rodrigo Vivi
From: Ville Syrjälä
On CNL we may need to bump up the system agent voltage not only due
to CDCLK but also when driving DDI port with a sufficiently high clock.
To that end start tracking the minimum acceptable voltage for each crtc.
We do the tracking via crtcs because we don't have any kind of e
From: Ville Syrjälä
Track the system agent voltage we request from pcode in the cdclk state
on SKL/KBL/CFL. Annoyingly we can't actually read out the current value since
there's no pcode command to do that, so we'll have to just assume that
it worked.
Cc: Mika Kahola
Cc: Manasi Navare
Cc: Rodr
From: Ville Syrjälä
Track the system agent voltage we request from pcode in the cdclk state
on BXT/GLK. Annoyingly we can't actually read out the current value since
there's no pcode command to do that, so we'll have to just assume that
it worked.
Cc: Mika Kahola
Cc: Manasi Navare
Cc: Rodrigo
From: Ville Syrjälä
Store the punit DSPFREQUAR value into cdclk_state->voltage on
VLV/CHV. Since we can actually read that out from the hardware
this can give us a bit more cross checking between the hardware
and software state.
Cc: Mika Kahola
Cc: Manasi Navare
Cc: Rodrigo Vivi
Signed-off-by
From: Ville Syrjälä
For CNL we'll need to start considering the port clocks when we select
the voltage level for the system agent. To that end start tracking the
voltage in the cdclk state (since that already has to adjust it).
Cc: Mika Kahola
Cc: Manasi Navare
Cc: Rodrigo Vivi
Signed-off-by:
From: Ville Syrjälä
Track the system agent voltage we request from pcode in the cdclk state
on BDW. Annoyingly we can't actually read out the current value since
there's no pcode command to do that, so we'll have to just assume that
it worked.
Cc: Mika Kahola
Cc: Manasi Navare
Cc: Rodrigo Vivi
From: Ville Syrjälä
Redo some switch statements in the cdclk code to use a common
fall through for the default case. Makes everything look a bit
more uniform
Cc: Mika Kahola
Cc: Manasi Navare
Cc: Rodrigo Vivi
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/intel_cdclk.c | 68 +
From: Ville Syrjälä
Here's my idea on how the CNL DVFS thing ought to be implemented.
Not sure I gleaned the details of the port clock limits correctly, but
other than that I think this should work.
Entire series available here:
git://github.com/vsyrjala/linux.git dvfs_voltage_4
Cc: Mika Kahola
== Series Details ==
Series: tests/kms_plane_lowres: Rework tests to work without fbcon. (rev2)
URL : https://patchwork.freedesktop.org/series/32016/
State : failure
== Summary ==
Test kms_plane_lowres:
Subgroup pipe-A-tiling-none:
pass -> SKIP (shard-hsw)
On Wed, Oct 18, 2017 at 08:09:24PM +0300, Ville Syrjälä wrote:
> On Mon, Oct 16, 2017 at 04:41:49PM -0700, Ausmus, James wrote:
> > On Mon, Oct 16, 2017 at 7:56 AM, Ville Syrjala
> > wrote:
> > > From: Ville Syrjälä
> > >
> > > Introduce intel_ddi_get_buf_trans_hdmi() and start using it where we
== Series Details ==
Series: series starting with [1/2] drm/i915: Let's use more enum intel_dpll_id
pll_id.
URL : https://patchwork.freedesktop.org/series/32245/
State : success
== Summary ==
Series 32245v1 series starting with [1/2] drm/i915: Let's use more enum
intel_dpll_id pll_id.
https:
On Wed, Oct 18, 2017 at 09:19:34PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Handle missing buf trans tables, or out of bounds buf trans levels
> the same way everywhere. These should never be hit under normal
> conditions, but let's play it safe for now.
>
> v2: Avoid the hdmi_level
On Wed, Oct 18, 2017 at 09:19:58PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> A bunch of functions are now exclusively used for HDMI, so naming the
> variables with hdmi prefix/suffix is redundant. Also use int rather
> than u32 for the translation level consistently.
>
> v2: Rebase d
== Series Details ==
Series: kms_atomic_transition: Output more finegrained progress info to avoid
CI watchdog timeout
URL : https://patchwork.freedesktop.org/series/32206/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
pass -> FAIL (shard-
On Wed, Oct 18, 2017 at 08:54:12PM +0300, Ville Syrjälä wrote:
> On Wed, Oct 18, 2017 at 10:43:54AM -0700, James Ausmus wrote:
> > On Mon, Oct 16, 2017 at 05:57:05PM +0300, Ville Syrjala wrote:
> > > From: Ville Syrjälä
> > >
> > > A bunch of functions are now exclusively used for HDMI, so naming
From: Paulo Zanoni
These functions even have their own page in our spec,
so extract them from cnl_set_cdclk().
v2: (By Rodrigo) Fixed inverted logic on error return of
cnl_dvfs_pre_change.
v3: Rebase on top of rps rework with pcu_lock.
Cc: Ville Syrjälä
Signed-off-by: Paulo Zanoni
Signed
No functional change expected. Just let's use this enum
when possible and also same standard pll_id name
so we can rework gen9+ port clock later.
Cc: Mika Kahola
Cc: Paulo Zanoni
Signed-off-by: Rodrigo Vivi
Reviewed-by: Mika Kahola
---
drivers/gpu/drm/i915/intel_ddi.c | 34 ++-
== Series Details ==
Series: tests/kms_cursor_legacy: Remove strict vblank checking on
2x-*-vs-cursor tests.
URL : https://patchwork.freedesktop.org/series/32202/
State : failure
== Summary ==
Test kms_atomic_transition:
Subgroup plane-all-transition-nonblocking-fencing:
== Series Details ==
Series: DDI buf trans cleanup (rev3)
URL : https://patchwork.freedesktop.org/series/32049/
State : success
== Summary ==
Series 32049v3 DDI buf trans cleanup
https://patchwork.freedesktop.org/api/1.0/series/32049/revisions/3/mbox/
fi-bdw-5557u total:289 pass:268 dwa
On Wed, Oct 18, 2017 at 09:15:09PM +0300, Ville Syrjälä wrote:
> On Wed, Oct 18, 2017 at 08:52:40PM +0300, Ville Syrjälä wrote:
> > On Wed, Oct 18, 2017 at 10:41:44AM -0700, James Ausmus wrote:
> > > On Mon, Oct 16, 2017 at 05:57:04PM +0300, Ville Syrjala wrote:
> > > > From: Ville Syrjälä
> > > >
== Series Details ==
Series: drm/i915: Disable lazy PPGTT page table optimization for vGPU
URL : https://patchwork.freedesktop.org/series/32201/
State : success
== Summary ==
Test perf:
Subgroup polling:
fail -> PASS (shard-hsw) fdo#102252
fdo#102252 https:
Em Qui, 2017-10-05 às 07:59 -0700, Rodrigo Vivi escreveu:
> On Wed, Oct 04, 2017 at 01:20:57PM +, Mika Kahola wrote:
> > On Tue, 2017-10-03 at 00:06 -0700, Rodrigo Vivi wrote:
> > > When port clock is zero or undefined we base our
> > > calculation on cdclk. So, same function can be used
> > >
From: Ville Syrjälä
A bunch of functions are now exclusively used for HDMI, so naming the
variables with hdmi prefix/suffix is redundant. Also use int rather
than u32 for the translation level consistently.
v2: Rebase due to hdmi_level=-1 avoidance
Cc: James Ausmus
Signed-off-by: Ville Syrjälä
From: Ville Syrjälä
Handle missing buf trans tables, or out of bounds buf trans levels
the same way everywhere. These should never be hit under normal
conditions, but let's play it safe for now.
v2: Avoid the hdmi_level=-1 case (James)
Cc: James Ausmus
Signed-off-by: Ville Syrjälä
---
driver
On Wed, Oct 18, 2017 at 08:52:40PM +0300, Ville Syrjälä wrote:
> On Wed, Oct 18, 2017 at 10:41:44AM -0700, James Ausmus wrote:
> > On Mon, Oct 16, 2017 at 05:57:04PM +0300, Ville Syrjala wrote:
> > > From: Ville Syrjälä
> > >
> > > Handle missing buf trans tables, or out of bounds buf trans level
On Wed, Oct 18, 2017 at 10:43:54AM -0700, James Ausmus wrote:
> On Mon, Oct 16, 2017 at 05:57:05PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > A bunch of functions are now exclusively used for HDMI, so naming the
> > variables with hdmi prefix/suffix is redundant. Also use int rat
On Wed, Oct 18, 2017 at 10:41:44AM -0700, James Ausmus wrote:
> On Mon, Oct 16, 2017 at 05:57:04PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Handle missing buf trans tables, or out of bounds buf trans levels
> > the same way everywhere. These should never be hit under normal
> >
On Wed, Oct 18, 2017 at 08:09:24PM +0300, Ville Syrjälä wrote:
> On Mon, Oct 16, 2017 at 04:41:49PM -0700, Ausmus, James wrote:
> > On Mon, Oct 16, 2017 at 7:56 AM, Ville Syrjala
> > wrote:
> > > From: Ville Syrjälä
> > >
> > > Introduce intel_ddi_get_buf_trans_hdmi() and start using it where we
On Mon, Oct 16, 2017 at 05:57:05PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> A bunch of functions are now exclusively used for HDMI, so naming the
> variables with hdmi prefix/suffix is redundant. Also use int rather
> than u32 for the translation level consistently.
Is there a case
On Mon, Oct 16, 2017 at 05:57:04PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Handle missing buf trans tables, or out of bounds buf trans levels
> the same way everywhere. These should never be hit under normal
> conditions, but let's play it safe for now.
>
> Signed-off-by: Ville Syr
== Series Details ==
Series: drm/i915: Do not rely on wm preservation for ILK watermarks
URL : https://patchwork.freedesktop.org/series/32197/
State : failure
== Summary ==
Test kms_setmode:
Subgroup basic:
pass -> FAIL (shard-hsw) fdo#99912
Test perf:
On Mon, Oct 16, 2017 at 05:57:03PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> SKL DDI B/C/D only have 9 usable buf trans registers for DP/eDP. That
> matches the normal DP buf trans tables, but the low vswing eDP tables
> have 10 entries. Thus the eDP tables can only be used safely wit
On Mon, Oct 16, 2017 at 04:41:49PM -0700, Ausmus, James wrote:
> On Mon, Oct 16, 2017 at 7:56 AM, Ville Syrjala
> wrote:
> > From: Ville Syrjälä
> >
> > Introduce intel_ddi_get_buf_trans_hdmi() and start using it where we
> > can.
> >
> > Signed-off-by: Ville Syrjälä
> > ---
> > drivers/gpu/drm
== Series Details ==
Series: lib/i915: Move submission related helpers to lib/i915/gem_submission
URL : https://patchwork.freedesktop.org/series/32184/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
pass -> FAIL (shard-hsw) fdo#99912
Test pe
On Mon, Oct 16, 2017 at 05:57:02PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> default_index contained in the BXT buf_trans tables is actually useless.
> For DP we should always have a valid level selected (otherwise the link
> training logic would be buggy), and for HDMI we can just do
On Mon, Oct 16, 2017 at 05:57:01PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> encoder->type is unreliable for DP/HDMI, so pass it in explicity into
> cnl_ddi_vswing_sequence(). This matches what we do for BXT.
>
> v2: Pass intel_encoder down to cnl_ddi_vswing_program(), and
> clea
On 10/18/2017 09:50 AM, Joonas Lahtinen wrote:
On Wed, 2017-10-18 at 09:25 -0700, Sujaritha wrote:
On 10/18/2017 03:58 AM, Joonas Lahtinen wrote:
On Tue, 2017-10-17 at 15:50 -0700, Sujaritha Sundaresan wrote:
We currently have two module parameters that control GuC:
"enable_guc_loading" and
On Wed, 2017-10-18 at 09:25 -0700, Sujaritha wrote:
>
> On 10/18/2017 03:58 AM, Joonas Lahtinen wrote:
> > On Tue, 2017-10-17 at 15:50 -0700, Sujaritha Sundaresan wrote:
> > > We currently have two module parameters that control GuC:
> > > "enable_guc_loading" and "enable_guc_submission". Whenever
On 18/10/17 09:04, Chris Wilson wrote:
Quoting Daniele Ceraolo Spurio (2017-10-18 16:49:24)
On 13/10/17 09:37, Daniele Ceraolo Spurio wrote:
On 13/10/17 01:31, Chris Wilson wrote:
Quoting Chris Wilson (2017-10-12 23:57:38)
Quoting Daniele Ceraolo Spurio (2017-10-12 23:27:27)
+igt_cork_
On Mon, Oct 16, 2017 at 05:57:00PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Make BXT less special by following the CNL approach and handling
> it in intel_ddi_dp_voltage_max() alognside every other DDI platform.
>
> v2: Clean up the argument types to bxt_ddi_vswing_sequence() while
On 10/18/2017 03:58 AM, Joonas Lahtinen wrote:
On Tue, 2017-10-17 at 15:50 -0700, Sujaritha Sundaresan wrote:
We currently have two module parameters that control GuC:
"enable_guc_loading" and "enable_guc_submission". Whenever
we need i915_modparams.enable_guc_submission=1, we also need
enable
On Wed, Oct 18, 2017 at 02:33:37PM +, Deucher, Alexander wrote:
Please fix your mail client to word wrap within paragraphs at something
substantially less than 80 columns. Doing this makes your messages much
easier to read and reply to.
> Your conflict change affectively reverted 1e444864833
On 10/18/2017 6:42 PM, Tvrtko Ursulin wrote:
On 18/10/2017 07:46, Sagar Arun Kamble wrote:
GuC log runtime/relay channel data is released during i915 unregister,
So only GuC log vma needs to be released during submission_fini.
Signed-off-by: Sagar Arun Kamble
Cc: Michal Wajdeczko
Cc: Danie
Quoting Daniele Ceraolo Spurio (2017-10-18 16:49:24)
>
>
> On 13/10/17 09:37, Daniele Ceraolo Spurio wrote:
> >
> >
> > On 13/10/17 01:31, Chris Wilson wrote:
> >> Quoting Chris Wilson (2017-10-12 23:57:38)
> >>> Quoting Daniele Ceraolo Spurio (2017-10-12 23:27:27)
> +igt_cork_t *igt_cork_
On 10/18/2017 6:37 PM, Tvrtko Ursulin wrote:
On 18/10/2017 07:46, Sagar Arun Kamble wrote:
With guc_log_level parameter sanitized we can remove the GuC submission
checks from flush_guc_logs and i915_guc_log_register/unregister and
intel_uc_fini_hw. It is important to note that GuC log runtime
== Series Details ==
Series: tests/pm_rps: Move some test logic out of boost function (rev2)
URL : https://patchwork.freedesktop.org/series/32131/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
9a3f6f59d4c208219f7d3178106bdef628ff68d0 tests/intel-ci: Remove
On 13/10/17 09:37, Daniele Ceraolo Spurio wrote:
On 13/10/17 01:31, Chris Wilson wrote:
Quoting Chris Wilson (2017-10-12 23:57:38)
Quoting Daniele Ceraolo Spurio (2017-10-12 23:27:27)
+igt_cork_t *igt_cork_new(int fd);
_new does not imply plugged.
+void igt_cork_signal(igt_cork_t *cork
On 10/18/2017 6:29 PM, Tvrtko Ursulin wrote:
On 18/10/2017 07:46, Sagar Arun Kamble wrote:
Parameter guc_log_level needs to be sanitized based on GuC support and
enable_guc_loading parameter since it depends on them like
enable_guc_submission. This will make GuC logging paths independent of
e
Moving code out of the boost function will allow its usage
in other/new test scenarios.
Signed-off-by: Radoslaw Szwichtenberg
---
tests/pm_rps.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/tests/pm_rps.c b/tests/pm_rps.c
index 89f3e31c..fa548260 100644
== Series Details ==
Series: drm/i915/crt: split compute_config hook by platforms
URL : https://patchwork.freedesktop.org/series/32135/
State : success
== Summary ==
Test drv_suspend:
Subgroup sysfs-reader-hibernate:
dmesg-fail -> FAIL (shard-hsw) k.org#196691
Tes
Quoting Daniel Vetter (2017-10-18 15:47:27)
> On Wed, Oct 18, 2017 at 03:19:44PM +0100, Chris Wilson wrote:
> > We never declared that we were about to read from the mmap after copying
> > into using the BLT (a missed call to prime_sync_start); leaving its
> > coherency ill-defined. For completenes
On Wed, Oct 18, 2017 at 02:30:50PM +, Deucher, Alexander wrote:
> The patch below effectively reverts 1e4448648333a. If you drop the patch
> below, you should be fine.
As I said in my followup mail I've dropped all these patches due to
build failures, please resend both the pull request and
On Wed, Oct 18, 2017 at 03:19:44PM +0100, Chris Wilson wrote:
> We never declared that we were about to read from the mmap after copying
> into using the BLT (a missed call to prime_sync_start); leaving its
> coherency ill-defined. For completeness, add the missing
> prime_sync_end() as well.
>
>
> -Original Message-
> From: Mark Brown [mailto:broo...@kernel.org]
> Sent: Wednesday, October 18, 2017 5:58 AM
> To: Deucher, Alexander; Mukunda, Vijendar; Zhu, Rex; Daniel Vetter; Intel
> Graphics; DRI; Liam Girdwood
> Cc: Linux-Next Mailing List; Linux Kernel Mailing List
> Subject: linu
On Tue, Oct 17, 2017 at 11:08:07PM +0300, Juha-Pekka Heikkila wrote:
This one lacked a commit message. I just slapped in something
rudimentary this time. In the future make sure the commit messages
are there, and that they're useful ;)
Entire series pushed to dinq. Thanks for the patches.
> Sign
== Series Details ==
Series: igt/prime_mmap_coherency: Call prime_sync_start before read after write
URL : https://patchwork.freedesktop.org/series/32219/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
9a3f6f59d4c208219f7d3178106bdef628ff68d0 tests/intel-ci
On Wed, Oct 18, 2017 at 02:43:38PM +0300, Petri Latvala wrote:
> On Wed, Oct 18, 2017 at 02:29:33PM +0300, Imre Deak wrote:
> > The CI software watchdog (owatch) will timeout if the test doesn't
> > output anything for a long time on standard out or error. At least the
> > plane-all-modeset-transit
> -Original Message-
> From: Mark Brown [mailto:broo...@kernel.org]
> Sent: Wednesday, October 18, 2017 6:08 AM
> To: Deucher, Alexander; Mukunda, Vijendar; Zhu, Rex; Daniel Vetter; Intel
> Graphics; DRI; Liam Girdwood
> Cc: Linux-Next Mailing List; Linux Kernel Mailing List; alsa-devel@als
== Series Details ==
Series: drm/i915: remove g4x lowfreq_avail and has_pipe_cxsr
URL : https://patchwork.freedesktop.org/series/32134/
State : success
== Summary ==
Test drv_suspend:
Subgroup sysfs-reader-hibernate:
dmesg-fail -> FAIL (shard-hsw) k.org#196691
Tes
On Wed, Oct 18, 2017 at 11:01:13AM +0200, Daniel Vetter wrote:
> On Tue, Oct 17, 2017 at 09:23:43AM -0700, Jeff McGee wrote:
> > On Tue, Oct 17, 2017 at 05:36:54PM +0200, Daniel Vetter wrote:
> > > On Tue, Oct 17, 2017 at 08:25:33AM -0700, jeff.mc...@intel.com wrote:
> > > > From: Jeff McGee
> > >
We never declared that we were about to read from the mmap after copying
into using the BLT (a missed call to prime_sync_start); leaving its
coherency ill-defined. For completeness, add the missing
prime_sync_end() as well.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103168
Signed-off-b
On 10/18/2017 6:16 PM, Tvrtko Ursulin wrote:
On 18/10/2017 07:46, Sagar Arun Kamble wrote:
GuC interrupts handling is core GuC functionality. Better to keep it
with other core functions in intel_guc.c. Since they are used from
uC functions, GuC logging, i915 irq handling keeping them grouped
On 10/18/2017 6:12 PM, Tvrtko Ursulin wrote:
On 18/10/2017 07:46, Sagar Arun Kamble wrote:
In order to separate GuC IRQ handling functions from i915_irq.c we need
to export the low level pm irq handlers. Export pm_iir, reset_pm_iir and
enable/disable_pm_irq functions.
Suggested-by: Michal Wa
On 10/18/2017 4:33 PM, Szwichtenberg, Radoslaw wrote:
On Wed, 2017-10-18 at 11:10 +0530, Sagar Arun Kamble wrote:
On 10/17/2017 6:13 PM, Radoslaw Szwichtenberg wrote:
Moving code out of the boost function will allow its usage
in other/new test scenarios.
Signed-off-by: Radoslaw Szwichtenberg
1 - 100 of 162 matches
Mail list logo
