On Wed, Apr 25, 2018 at 8:42 AM, Thomas Hellstrom wrote:
> Hi,
>
> On 12/07/2017 03:49 PM, Daniel Vetter wrote:
>>
>> We're spotting this very rarely in CI, but have no idea. Let's add
>> more debug info about what's going on here.
>>
>> References: https://bugs.freedesktop.org/show_bug.cgi?id=102
Hi!
On 12/07/2017 03:49 PM, Daniel Vetter wrote:
We're spotting this very rarely in CI, but have no idea. Let's add
more debug info about what's going on here.
References:https://bugs.freedesktop.org/show_bug.cgi?id=102707
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_mode_config.c | 3
Hi,
On 12/07/2017 03:49 PM, Daniel Vetter wrote:
We're spotting this very rarely in CI, but have no idea. Let's add
more debug info about what's going on here.
References: https://bugs.freedesktop.org/show_bug.cgi?id=102707
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_mode_config.c |
== Series Details ==
Series: YCBCR 4:2:0/4:4:4 output support for LSPCON (rev7)
URL : https://patchwork.freedesktop.org/series/36068/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4024_full -> Patchwork_8585_full =
== Summary - WARNING ==
Minor unknown changes coming wit
Em Qui, 2018-04-05 às 17:20 -0700, Rodrigo Vivi escreveu:
> On Thu, Feb 22, 2018 at 12:55:10AM -0300, Paulo Zanoni wrote:
> > From: Manasi Navare
> >
> > This is an important part of the DDI initalization as well as
> > for changing the voltage during DisplayPort link training.
> >
> > The Volta
On Fri, 2018-04-20 at 17:14 +, Souza, Jose wrote:
> On Fri, 2018-04-20 at 15:06 +0530, vathsala nagaraju wrote:
> > From: Vathsala Nagaraju
> >
> > Prints live state of psr1.Extending the existing
> > PSR2 live state function to cover psr1.
> >
> > Tested on KBL with psr2 and psr1 panel.
Em Seg, 2018-04-09 às 16:23 -0700, James Ausmus escreveu:
> On Wed, Mar 28, 2018 at 02:57:58PM -0700, Paulo Zanoni wrote:
> > This commit introduces the definitions for the ICL clocks and adds
> > the
> > basic functions to the shared DPLL framework. It adds code for the
> > Enable and Disable sequ
On Fri, 2018-04-20 at 15:27 -0700, José Roberto de Souza wrote:
> IGT tests could be improved with sink status, knowing for sure that
> hardware have activate or exit PSR.
>
> Reviewed-by: Dhinakaran Pandiyan
> Cc: Rodrigo Vivi
> Signed-off-by: José Roberto de Souza
> ---
>
> No changes sin
On Tue, 2018-04-24 at 14:20 -0700, Rodrigo Vivi wrote:
> On Mon, Apr 23, 2018 at 05:42:40PM -0700, Souza, Jose wrote:
> > On Fri, 2018-04-20 at 15:57 -0700, Rodrigo Vivi wrote:
> > > On Fri, Apr 20, 2018 at 03:27:56PM -0700, José Roberto de Souza
> > > wrote:
> > > > Any write in any display reg
On Tue, 2018-04-24 at 16:47 -0700, Dhinakaran Pandiyan wrote:
>
>
> On Fri, 2018-04-20 at 15:27 -0700, José Roberto de Souza wrote:
> > This will be helpful to debug what hardware is actually tracking
> > and causing PSR to exit.
> >
> > BSpec: 7721
> >
> > Signed-off-by: José Roberto de Sou
On Fri, 2018-04-20 at 15:27 -0700, José Roberto de Souza wrote:
> This will be helpful to debug what hardware is actually tracking
> and causing PSR to exit.
>
> BSpec: 7721
>
> Signed-off-by: José Roberto de Souza
> Cc: Dhinakaran Pandiyan
> Cc: Rodrigo Vivi
> ---
>
> New patch in this se
== Series Details ==
Series: drm/i915/icl: Correctly clear lost ctx-switch interrupts across reset
for Gen11
URL : https://patchwork.freedesktop.org/series/42229/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4089_full -> Patchwork_8793_full =
== Summary - FAILURE ==
Se
On 4/24/2018 2:39 PM, Oscar Mateo wrote:
Interrupt handling in Gen11 is quite different from previous platforms.
v2: Rebased (Michel)
v3: Rebased with wiggle
v4: Rebased, remove TODO warning correctly (Daniele)
v5: Rebased, made gen11_gtiir const while at it (Michel)
v6: Rebased
v7: Adapt to the
On Wed, 2018-04-11 at 18:54 -0400, Lyude Paul wrote:
> While having the modeset_retry_work in intel_connector makes sense with
> SST, this paradigm doesn't make a whole ton of sense when it comes to
> MST since we have to deal with multiple connectors. In most cases, it's
> more useful to just u
== Series Details ==
Series: drm/i915/icl: Correctly clear lost ctx-switch interrupts across reset
for Gen11
URL : https://patchwork.freedesktop.org/series/42229/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4089 -> Patchwork_8793 =
== Summary - SUCCESS ==
No regressio
Interrupt handling in Gen11 is quite different from previous platforms.
v2: Rebased (Michel)
v3: Rebased with wiggle
v4: Rebased, remove TODO warning correctly (Daniele)
v5: Rebased, made gen11_gtiir const while at it (Michel)
v6: Rebased
v7: Adapt to the style currently in upstream
Suggested-by:
On Tue, 2018-04-24 at 13:55 -0700, Rodrigo Vivi wrote:
> On Mon, Apr 23, 2018 at 07:56:58PM -0700, Dhinakaran Pandiyan wrote:
> > Sink crc is calculated by the sink for static frames irrespective of
> > what the driver sets in TEST_SINK_START dpcd. Since PSR is the only use
> > case for sink crc
On Mon, Apr 23, 2018 at 05:42:40PM -0700, Souza, Jose wrote:
> On Fri, 2018-04-20 at 15:57 -0700, Rodrigo Vivi wrote:
> > On Fri, Apr 20, 2018 at 03:27:56PM -0700, José Roberto de Souza
> > wrote:
> > > Any write in any display register was causing HW to exit PSR,
> > > masking it to allow more pow
On Fri, Apr 13, 2018 at 6:01 AM Daniel Vetter
wrote:
> This tries to align with the X.org communities's long-standing
> tradition of trying to be an inclusive community and handing out
> commit rights fairly freely.
> We also tend to not revoke commit rights for people no longer
> regularly acti
On Mon, Apr 23, 2018 at 07:56:59PM -0700, Dhinakaran Pandiyan wrote:
> With sink-crc now being relevant only for PSR static frames, move the
> code to intel_psr.c and rename the function.
>
> Signed-off-by: Dhinakaran Pandiyan
Acked-by: Rodrigo Vivi
> ---
> drivers/gpu/drm/i915/i915_debugfs.
On Mon, Apr 23, 2018 at 07:56:58PM -0700, Dhinakaran Pandiyan wrote:
> Sink crc is calculated by the sink for static frames irrespective of
> what the driver sets in TEST_SINK_START dpcd. Since PSR is the only use
> case for sink crc, we don't really need the sink_crc_{start, stop} code.
>
> The s
On Mon, Apr 23, 2018 at 07:56:57PM -0700, Dhinakaran Pandiyan wrote:
> We attempt to read 6 bytes, make sure we have.
>
> Signed-off-by: Dhinakaran Pandiyan
Reviewed-by: Rodrigo Vivi
> ---
> drivers/gpu/drm/i915/intel_dp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --gi
== Series Details ==
Series: drm/i915: fix intel_dvo_dev_ops::mode_valid's return type
URL : https://patchwork.freedesktop.org/series/42209/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4087_full -> Patchwork_8792_full =
== Summary - FAILURE ==
Serious unknown changes c
On 04/24/2018 02:42 PM, Chris Wilson wrote:
Quoting Patchwork (2018-04-24 20:02:06)
== Series Details ==
Series: drm/i915/selftests: Fix uninitialized variable
URL : https://patchwork.freedesktop.org/series/42194/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4087_full
On Wed, 2018-04-11 at 19:42 -0400, Lyude Paul wrote:
> Does what it says on the label, it's a little confusing debugging atomic
> check failures otherwise.
>
> Cc: Manasi Navare
> Cc: Ville Syrjälä
> Signed-off-by: Lyude Paul
> ---
> drivers/gpu/drm/drm_atomic.c | 5 -
> 1 file changed,
Quoting Patchwork (2018-04-24 20:02:06)
> == Series Details ==
>
> Series: drm/i915/selftests: Fix uninitialized variable
> URL : https://patchwork.freedesktop.org/series/42194/
> State : failure
>
> == Summary ==
>
> = CI Bug Log - changes from CI_DRM_4087_full -> Patchwork_8790_full =
>
> =
On Tue, Apr 24, 2018 at 05:26:30PM +0300, Ville Syrjälä wrote:
> On Tue, Apr 24, 2018 at 04:18:37PM +0200, Daniel Vetter wrote:
> > On Tue, Apr 24, 2018 at 04:02:50PM +0300, Ville Syrjala wrote:
> > > From: Ville Syrjälä
> > >
> > > We're currently failing to reset everything in display_info.hdmi
== Series Details ==
Series: drm/i915/breadcrumbs: Keep the fake irq armed across reset (rev3)
URL : https://patchwork.freedesktop.org/series/40825/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4087_full -> Patchwork_8791_full =
== Summary - FAILURE ==
Serious unknown c
On Tue, Apr 24, 2018 at 7:30 PM, Emil Velikov wrote:
> On 13 April 2018 at 11:00, Daniel Vetter wrote:
>> This tries to align with the X.org communities's long-standing
>> tradition of trying to be an inclusive community and handing out
>> commit rights fairly freely.
>>
>> We also tend to not re
== Series Details ==
Series: drm/i915/selftests: Fix uninitialized variable
URL : https://patchwork.freedesktop.org/series/42194/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4087_full -> Patchwork_8790_full =
== Summary - FAILURE ==
Serious unknown changes coming with
On Tue, 2018-04-24 at 16:26 +0300, Mika Kahola wrote:
> On Mon, 2018-04-23 at 19:56 -0700, Dhinakaran Pandiyan wrote:
> > Sink crc is calculated by the sink for static frames irrespective of
> > what the driver sets in TEST_SINK_START dpcd. Since PSR is the only
> > use
> > case for sink crc, we do
== Series Details ==
Series: drm/edid: Reset more of the display info
URL : https://patchwork.freedesktop.org/series/42191/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4086_full -> Patchwork_8788_full =
== Summary - WARNING ==
Minor unknown changes coming with Patchwor
== Series Details ==
Series: drm/i915: add support for specifying DMC firmware override by module
param (rev3)
URL : https://patchwork.freedesktop.org/series/34157/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4086_full -> Patchwork_8787_full =
== Summary - WARNING ==
On 13 April 2018 at 11:00, Daniel Vetter wrote:
> This tries to align with the X.org communities's long-standing
> tradition of trying to be an inclusive community and handing out
> commit rights fairly freely.
>
> We also tend to not revoke commit rights for people no longer
> regularly active in
On 04/20/2018 01:53 PM, Rodrigo Vivi wrote:
On Fri, Apr 20, 2018 at 01:49:45PM -0700, Oscar Mateo wrote:
On 04/20/2018 01:46 PM, Rodrigo Vivi wrote:
On Fri, Apr 20, 2018 at 01:33:54PM -0700, Oscar Mateo wrote:
Disable GWL clock gating to prevent two different issues that
might cause hangs.
== Series Details ==
Series: drm/i915: fix intel_dvo_dev_ops::mode_valid's return type
URL : https://patchwork.freedesktop.org/series/42209/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4087 -> Patchwork_8792 =
== Summary - SUCCESS ==
No regressions found.
External U
Quoting Tvrtko Ursulin (2018-04-24 15:48:10)
>
> On 24/04/2018 15:04, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2018-04-24 14:55:51)
> >>
> >> On 24/04/2018 12:28, Chris Wilson wrote:
> >>> Quoting Tvrtko Ursulin (2018-04-24 12:17:15)
>
> On 24/04/2018 11:40, Chris Wilson wrote:
>
On Tue, Apr 24, 2018 at 05:11:37PM +0200, Maarten Lankhorst wrote:
> Op 24-04-18 om 17:21 schreef Ville Syrjälä:
> > On Tue, Apr 24, 2018 at 01:36:32PM +0200, Maarten Lankhorst wrote:
> >> When calculating limits we want to be as pessimistic as possible,
> >> so we have to explicitly say whether we
== Series Details ==
Series: drm/i915: fix intel_dvo_dev_ops::mode_valid's return type
URL : https://patchwork.freedesktop.org/series/42209/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
e19139a5a800 drm/i915: fix intel_dvo_dev_ops::mode_valid's return type
-:25: CHECK:PARENTHE
== Series Details ==
Series: drm/i915/breadcrumbs: Keep the fake irq armed across reset (rev3)
URL : https://patchwork.freedesktop.org/series/40825/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4087 -> Patchwork_8791 =
== Summary - SUCCESS ==
No regressions found.
Ex
On Tue, Apr 24, 2018 at 01:36:33PM +0200, Maarten Lankhorst wrote:
> With the previous patch drm_atomic_helper_check_plane_state correctly
> calculates clipping and the xf86-video-intel ddx is fixed to fall back
> to GPU correctly when SetPlane fails, we can remove the hack where
> we try to pan/zo
Op 24-04-18 om 17:21 schreef Ville Syrjälä:
> On Tue, Apr 24, 2018 at 01:36:32PM +0200, Maarten Lankhorst wrote:
>> When calculating limits we want to be as pessimistic as possible,
>> so we have to explicitly say whether we want to round up or down
>> to accurately calculate whether we are below m
== Series Details ==
Series: drm/i915/selftests: Fix uninitialized variable
URL : https://patchwork.freedesktop.org/series/42194/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4087 -> Patchwork_8790 =
== Summary - SUCCESS ==
No regressions found.
External URL:
https:
All implementations of the method intel_dvo_dev_ops::mode_valid(), as
well as the underlying struct drm_connector_helper_funcs::mode_valid()
use 'enum drm_mode_status' for the method's return type but the
declaration of intel_dvo_dev_ops::mode_valid() uses an 'int' for it.
Fix this by using 'enum
On Tue, Apr 24, 2018 at 01:36:32PM +0200, Maarten Lankhorst wrote:
> When calculating limits we want to be as pessimistic as possible,
> so we have to explicitly say whether we want to round up or down
> to accurately calculate whether we are below min_scale or above
> max_scale.
>
> Signed-off-by
On Tue, 24 Apr 2018, Luc Van Oostenryck wrote:
> All implementations of the method intel_dvo_dev_ops::mode_valid(), as
> well as the underlying struct drm_connector_helper_funcs::mode_valid()
> use 'enum drm_mode_status' for the method's return type but the
> declaration of intel_dvo_dev_ops::mode
== Series Details ==
Series: drm/i915: Skip printing global offsets for per-engine scratch pages
URL : https://patchwork.freedesktop.org/series/42187/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4084_full -> Patchwork_8786_full =
== Summary - FAILURE ==
Serious unknown
== Series Details ==
Series: series starting with [1/2] drm/rect: Add midpoint to scale calculation
functions
URL : https://patchwork.freedesktop.org/series/42186/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4084_full -> Patchwork_8785_full =
== Summary - FAILURE ==
S
== Series Details ==
Series: series starting with [1/6] drm/i915: Stop tracking
timeline->inflight_seqnos
URL : https://patchwork.freedesktop.org/series/42192/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4087 -> Patchwork_8789 =
== Summary - FAILURE ==
Serious unknown
Quoting Tvrtko Ursulin (2018-04-24 15:46:49)
>
> On 24/04/2018 14:14, Chris Wilson wrote:
> > In the next patch, rings are the central timeline as requests may jump
> > between engines. Therefore in the future as we retire in order along the
> > engine timeline, we may retire out-of-order within a
On 24/04/2018 15:46, Tvrtko Ursulin wrote:
On 24/04/2018 14:14, Chris Wilson wrote:
In the next patch, rings are the central timeline as requests may jump
between engines. Therefore in the future as we retire in order along the
engine timeline, we may retire out-of-order within a ring (as the
== Series Details ==
Series: series starting with [1/6] drm/i915: Stop tracking
timeline->inflight_seqnos
URL : https://patchwork.freedesktop.org/series/42192/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915: Stop tracking timeline->inflight_seqnos
-O:drivers/gpu/dr
On Tue, 24 Apr 2018, Chris Wilson wrote:
> Quoting José Roberto de Souza (2018-04-19 00:41:58)
>> If the initial fbdev configuration(intel_fbdev_initial_config()) runs and
>> there still no sink connected it will cause
>> drm_fb_helper_initial_config() to return 0 as no error happened(but
>> inter
== Series Details ==
Series: series starting with [1/6] drm/i915: Stop tracking
timeline->inflight_seqnos
URL : https://patchwork.freedesktop.org/series/42192/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
9a8374d0cd41 drm/i915: Stop tracking timeline->inflight_seqnos
-:17: ER
On 24/04/2018 15:04, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2018-04-24 14:55:51)
On 24/04/2018 12:28, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2018-04-24 12:17:15)
On 24/04/2018 11:40, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2018-04-24 11:14:21)
On 23/04/2018 19:08, Chris Wil
On 24/04/2018 14:14, Chris Wilson wrote:
In the next patch, rings are the central timeline as requests may jump
between engines. Therefore in the future as we retire in order along the
engine timeline, we may retire out-of-order within a ring (as the ring now
occurs along multiple engines), lead
Instead of synchronously cancelling the timer and re-enabling it inside
the reset callbacks, keep the timer enabled and let it die on its next
wakeup if no longer required. This allows
intel_engine_reset_breadcrumbs() to be used from an atomic
(timer/softirq) context such as required for resetting
On Tue, Apr 24, 2018 at 04:18:37PM +0200, Daniel Vetter wrote:
> On Tue, Apr 24, 2018 at 04:02:50PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > We're currently failing to reset everything in display_info.hdmi
> > which will potentially cause us to use stale information when
> > sw
== Series Details ==
Series: drm/edid: Reset more of the display info
URL : https://patchwork.freedesktop.org/series/42191/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4086 -> Patchwork_8788 =
== Summary - SUCCESS ==
No regressions found.
External URL:
https://patc
On Tue, Apr 24, 2018 at 04:02:50PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> We're currently failing to reset everything in display_info.hdmi
> which will potentially cause us to use stale information when
> swapping monitors. Eg. if the user replaces a HDMI 2.0 monitor
> with a HDMI
Imre Deak writes:
> On Fri, Apr 20, 2018 at 11:27:55AM +0100, Chris Wilson wrote:
>> Quoting Mika Kuoppala (2018-04-20 10:54:26)
>> > We use jiffies to determine when wait expires. However
>> > Imre did find out that jiffies can and will do a >1
>> > increments on certain situations [1]. When thi
Quoting Tvrtko Ursulin (2018-04-24 14:55:51)
>
> On 24/04/2018 12:28, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2018-04-24 12:17:15)
> >>
> >> On 24/04/2018 11:40, Chris Wilson wrote:
> >>> Quoting Tvrtko Ursulin (2018-04-24 11:14:21)
>
> On 23/04/2018 19:08, Chris Wilson wrote:
>
== Series Details ==
Series: drm/i915: add support for specifying DMC firmware override by module
param (rev3)
URL : https://patchwork.freedesktop.org/series/34157/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4086 -> Patchwork_8787 =
== Summary - SUCCESS ==
No regress
On 24/04/2018 12:28, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2018-04-24 12:17:15)
On 24/04/2018 11:40, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2018-04-24 11:14:21)
On 23/04/2018 19:08, Chris Wilson wrote:
-static int reserve_engine(struct intel_engine_cs *engine)
+static int reserv
On 04/24/2018 08:22 AM, Chris Wilson wrote:
Quoting Gustavo A. R. Silva (2018-04-24 14:15:45)
There is a potential execution path in which variable err is
returned without being properly initialized previously.
Fix this by initializing variable err to 0.
err is only returned along an error
== Series Details ==
Series: drm/i915: add support for specifying DMC firmware override by module
param (rev3)
URL : https://patchwork.freedesktop.org/series/34157/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
c62c2ab8334b drm/i915: add support for specifying DMC firmware ove
Quoting Gustavo A. R. Silva (2018-04-24 14:30:58)
>
>
> On 04/24/2018 08:22 AM, Chris Wilson wrote:
> > Quoting Gustavo A. R. Silva (2018-04-24 14:15:45)
> >> There is a potential execution path in which variable err is
> >> returned without being properly initialized previously.
> >>
> >> Fix th
There is a potential execution path in which variable err is
returned without being properly initialized previously.
Fix this by initializing variable err to 0.
Addresses-Coverity-ID: 1468362 ("Uninitialized scalar variable")
Fixes: f4ecfbfc32ed ("drm/i915: Check whitelist registers across resets
On Mon, 2018-04-23 at 19:56 -0700, Dhinakaran Pandiyan wrote:
> Sink crc is calculated by the sink for static frames irrespective of
> what the driver sets in TEST_SINK_START dpcd. Since PSR is the only
> use
> case for sink crc, we don't really need the sink_crc_{start, stop}
> code.
>
> The seco
On 04/24/2018 03:01 PM, Daniel Vetter wrote:
On Fri, Apr 20, 2018 at 07:56:34PM +0200, Thomas Hellstrom wrote:
On 04/20/2018 08:51 AM, Daniel Vetter wrote:
Control nodes are no more!
Signed-off-by: Daniel Vetter
Cc: VMware Graphics
Cc: Sinclair Yeh
Cc: Thomas Hellstrom
---
drivers/gpu/d
Quoting Gustavo A. R. Silva (2018-04-24 14:15:45)
> There is a potential execution path in which variable err is
> returned without being properly initialized previously.
>
> Fix this by initializing variable err to 0.
err is only returned along an error path, returning 0 would not be
useful. Whi
Quoting Chris Wilson (2018-04-24 14:14:11)
> In the next patch, rings are the central timeline as requests may jump
> between engines. Therefore in the future as we retire in order along the
> engine timeline, we may retire out-of-order within a ring (as the ring now
> occurs along multiple engines
In the next patch, rings are the central timeline as requests may jump
between engines. Therefore in the future as we retire in order along the
engine timeline, we may retire out-of-order within a ring (as the ring now
occurs along multiple engines), leading to much hilarity in miscomputing
the pos
When userspace is passing around swapbuffers using DRI, we frequently
have to open and close the same object in the foreign address space.
This shows itself as the same object being rebound at roughly 30fps
(with a second object also being rebound at 30fps), which involves us
having to rewrite the
In the future, we want to move a request between engines. To achieve
this, we first realise that we have two timelines in effect here. The
first runs through the GTT is required for ordering vma access, which is
tracked currently by engine. The second is implied by sequential
execution of commands
We need to move to a more flexible timeline that doesn't assume one
fence context per engine, and so allow for a single timeline to be used
across a combination of engines. This means that preallocating a fence
context per engine is now a hindrance, and so we want to introduce the
singular timeline
We don't need to track every ring for its lifetime as they are managed
by the contexts/engines. What we do want to track are the live rings so
that we can sporadically clean up requests if userspace falls behind. We
can simply restrict the gt->rings list to being only gt->live_rings.
v2: s/live/ac
In commit 9b6586ae9f6b ("drm/i915: Keep a global seqno per-engine"), we
moved from a global inflight counter to per-engine counters in the
hope that will be easy to run concurrently in future. However, with the
advent of the desire to move requests between engines, we do need a
global counter to pr
Quoting Tvrtko Ursulin (2018-04-24 10:40:57)
>
> On 24/04/2018 10:33, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2018-04-24 10:27:23)
> >>
> >> On 24/04/2018 09:16, Chris Wilson wrote:
> >>> If we have more than a few, possibly several thousand request in the
> >>> queue, don't show the centr
From: Ville Syrjälä
We're currently failing to reset everything in display_info.hdmi
which will potentially cause us to use stale information when
swapping monitors. Eg. if the user replaces a HDMI 2.0 monitor
with a HDMI 1.x monitor we will continue to think that the monitor
supports scrambling.
On Fri, Apr 20, 2018 at 07:56:34PM +0200, Thomas Hellstrom wrote:
> On 04/20/2018 08:51 AM, Daniel Vetter wrote:
> > Control nodes are no more!
> >
> > Signed-off-by: Daniel Vetter
> > Cc: VMware Graphics
> > Cc: Sinclair Yeh
> > Cc: Thomas Hellstrom
> > ---
> > drivers/gpu/drm/vmwgfx/vmwgfx
== Series Details ==
Series: drm/i915: Skip printing global offsets for per-engine scratch pages
URL : https://patchwork.freedesktop.org/series/42187/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4084 -> Patchwork_8786 =
== Summary - SUCCESS ==
No regressions found.
Martin Peres writes:
> On 20/04/18 19:55, Patchwork wrote:
>> == Series Details ==
>>
>> Series: series starting with [1/2] drm/i915: Use ktime on wait_for
>> URL : https://patchwork.freedesktop.org/series/42035/
>> State : failure
>>
>> == Summary ==
>>
>> = CI Bug Log - changes from CI_DRM
== Series Details ==
Series: series starting with [1/2] drm/rect: Add midpoint to scale calculation
functions
URL : https://patchwork.freedesktop.org/series/42186/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4084 -> Patchwork_8785 =
== Summary - SUCCESS ==
No regressi
Quoting Mika Kuoppala (2018-04-24 13:26:11)
> Chris Wilson writes:
>
> > Ideally, we want to atomically flush and disable the tasklet before
> > resetting the GPU. At present, we rely on being the only part to touch
> > our tasklet and serialisation of the reset process to ensure that we can
> >
Chris Wilson writes:
> Ideally, we want to atomically flush and disable the tasklet before
> resetting the GPU. At present, we rely on being the only part to touch
> our tasklet and serialisation of the reset process to ensure that we can
> suspend the tasklet from the mix of reset/wedge pathways
Quoting Chris Wilson (2018-04-24 02:08:39)
> printk unhelpfully inserts a '\n' between consecutive calls, and since
> our drm_printf wrapper may be emitting info a seq_file instead,
> KERN_CONT is not an option. To work with any drm_printf destination, we
> need to build up the output into a tempor
On 20/04/18 19:55, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/2] drm/i915: Use ktime on wait_for
> URL : https://patchwork.freedesktop.org/series/42035/
> State : failure
>
> == Summary ==
>
> = CI Bug Log - changes from CI_DRM_4072_full -> Patchwork_8764_full
== Series Details ==
Series: series starting with [1/2] drm/rect: Add midpoint to scale calculation
functions
URL : https://patchwork.freedesktop.org/series/42186/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
aca1294901eb drm/rect: Add midpoint to scale calculation functions
On Mon, Apr 23, 2018 at 06:44:53AM +, Lisovskiy, Stanislav wrote:
>
> > This table is seriously deprecated, because it's unreadable and
> > unmaintainable. Quoting from the docs:
>
> > "Because this table is very unwieldy, do not add any new properties here.
> > Instead document them in a sec
Use i915.dmc_firmware_path to override default firmware for the platform
and bypassing version checks.
v2: add missing param struct member declaration (David)
Tested-by: David Weinehall
Reviewed-by: David Weinehall
Cc: Anusha Srivatsa
Cc: David Weinehall
Signed-off-by: Jani Nikula
---
drive
On 24/04/2018 12:52, Chris Wilson wrote:
Knowing the offset of the per-engine scratch/HWS page during boot is not
very informative, so remove the DRM_DEBUG.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_engine_cs.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers
Quoting Tvrtko Ursulin (2018-04-24 12:57:41)
>
> On 24/04/2018 02:08, Chris Wilson wrote:
> > printk unhelpfully inserts a '\n' between consecutive calls, and since
> > our drm_printf wrapper may be emitting info a seq_file instead,
> > KERN_CONT is not an option. To work with any drm_printf desti
On 24/04/2018 02:08, Chris Wilson wrote:
printk unhelpfully inserts a '\n' between consecutive calls, and since
our drm_printf wrapper may be emitting info a seq_file instead,
KERN_CONT is not an option. To work with any drm_printf destination, we
need to build up the output into a temporary buf
Knowing the offset of the per-engine scratch/HWS page during boot is not
very informative, so remove the DRM_DEBUG.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_engine_cs.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_engine_cs.c
b/drivers/gpu
With the previous patch drm_atomic_helper_check_plane_state correctly
calculates clipping and the xf86-video-intel ddx is fixed to fall back
to GPU correctly when SetPlane fails, we can remove the hack where
we try to pan/zoom when out of min/max scaling range. This was already
poor behavior where
When calculating limits we want to be as pessimistic as possible,
so we have to explicitly say whether we want to round up or down
to accurately calculate whether we are below min_scale or above
max_scale.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic_helper.c | 4 +--
drivers
Quoting Tvrtko Ursulin (2018-04-24 12:17:15)
>
> On 24/04/2018 11:40, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2018-04-24 11:14:21)
> >>
> >> On 23/04/2018 19:08, Chris Wilson wrote:
> >>> -static int reserve_engine(struct intel_engine_cs *engine)
> >>> +static int reserve_gt(struct drm_i91
On 24/04/2018 11:40, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2018-04-24 11:14:21)
On 23/04/2018 19:08, Chris Wilson wrote:
-static int reserve_engine(struct intel_engine_cs *engine)
+static int reserve_gt(struct drm_i915_private *i915)
{
- struct drm_i915_private *i915 = engine->i9
Chris Wilson writes:
> Quoting Mika Kuoppala (2018-04-23 14:03:02)
>> Chris Wilson writes:
>> > @@ -148,6 +149,12 @@ static void intel_breadcrumbs_fake_irq(struct
>> > timer_list *t)
>> > if (!b->irq_armed)
>> > return;
>> >
>> > + /* If the user has disabled the fake-
1 - 100 of 119 matches
Mail list logo
