== Series Details ==
Series: drm/i915: Move hsw GT w/a to engine initialisation
URL : https://patchwork.freedesktop.org/series/33095/
State : failure
== Summary ==
Series 33095v1 drm/i915: Move hsw GT w/a to engine initialisation
https://patchwork.freedesktop.org/api/1.0/series/33095/revisions
>
> On 10/30/2017 5:21 PM, Pandiyan, Dhinakaran wrote:
> > On Sun, 2017-10-29 at 03:04 +, Kumar, Abhay wrote:
> >> + Subhransu
> >>
> >> -Original Message-
> >> From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
> >> Of
> Kumar, Abhay
> >> Sent: Thursday, Octob
> >> b/drivers/gpu/drm/i915/intel_cdclk.c index
> >> e8884c2ade98..185a70f0921c
> >> 100644
> >> --- a/drivers/gpu/drm/i915/intel_cdclk.c
> >> +++ b/drivers/gpu/drm/i915/intel_cdclk.c
> >> @@ -1920,7 +1920,7 @@ int intel_crtc_compute_min_cdclk(const struct
> >> intel_crt
In commit b7048ea12fbb ("drm/i915: Do .init_clock_gating() earlier to
avoid it clobbering watermarks") init_clock_gating was called earlier in
the module load sequence, moving it before we acquired the forcewake
used to initialise the engines. This revealed that on Haswell, at least,
some of those
Quoting Joonas Lahtinen (2017-10-31 14:14:52)
> On Mon, 2017-10-30 at 13:17 -0700, Oscar Mateo wrote:
> > By doing this, we can dump these workarounds in debugfs for validation
> > (which,
> > at the moment, we are only able to do for the contexts WAs).
> >
> > v2:
> > - Wrong macro used for MM
Quoting Rodrigo Vivi (2017-11-02 23:52:45)
> On Wed, Oct 4, 2017 at 6:07 AM, Joonas Lahtinen
> wrote:
> > On Tue, 2017-10-03 at 15:56 -0700, Sujaritha Sundaresan wrote:
> >> We currently have two module parameters that control GuC:
> >> "enable_guc_loading" and "enable_guc_submission".
> >> Whene
== Series Details ==
Series: drm/i915/glk: Refactor handling of PLANE_COLOR_CTL for GLK+
URL : https://patchwork.freedesktop.org/series/33087/
State : warning
== Summary ==
Series 33087v1 drm/i915/glk: Refactor handling of PLANE_COLOR_CTL for GLK+
https://patchwork.freedesktop.org/api/1.0/seri
On Wed, Oct 4, 2017 at 6:07 AM, Joonas Lahtinen
wrote:
> On Tue, 2017-10-03 at 15:56 -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
Quoting Rodrigo Vivi (2017-11-02 23:34:52)
> On Thu, Nov 02, 2017 at 10:36:26PM +, Oscar Mateo wrote:
> >
> >
> > On 11/02/2017 07:56 AM, Joonas Lahtinen wrote:
> > > On Thu, 2017-11-02 at 07:54 -0700, Rodrigo Vivi wrote:
> > > > On Thu, Nov 02, 2017 at 02:43:16PM +, Joonas Lahtinen wrote
On Thu, Nov 02, 2017 at 10:36:26PM +, Oscar Mateo wrote:
>
>
> On 11/02/2017 07:56 AM, Joonas Lahtinen wrote:
> > On Thu, 2017-11-02 at 07:54 -0700, Rodrigo Vivi wrote:
> > > On Thu, Nov 02, 2017 at 02:43:16PM +, Joonas Lahtinen wrote:
> > > > On Thu, 2017-11-02 at 12:42 +, Chris Wils
Since GLK, some plane configuration settings have moved to the
PLANE_COLOR_CTL register. Refactor handling of the register to work like
PLANE_CTL. This also allows us to fix the set/read of the plane Alpha
Mode for GLK+.
Signed-off-by: James Ausmus
Cc: Paulo Zanoni
---
drivers/gpu/drm/i915/i915
On 11/02/2017 07:56 AM, Joonas Lahtinen wrote:
On Thu, 2017-11-02 at 07:54 -0700, Rodrigo Vivi wrote:
On Thu, Nov 02, 2017 at 02:43:16PM +, Joonas Lahtinen wrote:
On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
As we now record the default HW state and so only emit the "golden"
re
The patch https://patchwork.freedesktop.org/patch/166512/ which relies on HW
tracking for frontbuffer flushes/invalidations leads to an indefinite block
when running any of the *psr* subtests of the kms_frontbuffer_tracking test.
===Failure analysis===
Subtest: kms_frontbuffer_tracking --run-su
Quoting Anusha Srivatsa (2017-11-02 20:28:10)
> On Wed, Nov 01, 2017 at 01:24:15PM +, Chris Wilson wrote:
> > Quoting Michal Wajdeczko (2017-11-01 13:14:33)
> > > On Wed, 01 Nov 2017 01:11:20 +0100, Anusha Srivatsa
> > > > @@ -172,13 +174,18 @@ static int guc_ucode_xfer_dma(struct
> > > > d
On Wed, Nov 01, 2017 at 01:24:15PM +, Chris Wilson wrote:
> Quoting Michal Wajdeczko (2017-11-01 13:14:33)
> > On Wed, 01 Nov 2017 01:11:20 +0100, Anusha Srivatsa
> > > @@ -172,13 +174,18 @@ static int guc_ucode_xfer_dma(struct
> > > drm_i915_private *dev_priv,
> > >*/
> > > r
== Series Details ==
Series: tests: add device information tests (rev2)
URL : https://patchwork.freedesktop.org/series/32764/
State : warning
== Summary ==
Test kms_busy:
Subgroup extended-modeset-hang-oldfb-with-reset-render-B:
pass -> DMESG-WARN (shard-hsw)
sha
On 10/25/2017 08:56 AM, Michal Wajdeczko wrote:
On Tue, 24 Oct 2017 19:21:23 +0200, Sujaritha Sundaresan
wrote:
Updating GuC and HuC firmware select function to support removing
i915_modparams.enable_guc_loading module parameter.
Hmm, is it correct order of patches ? this modparam was alr
== Series Details ==
Series: drm/i915: Plane assert/readout cleanups etc. (rev8)
URL : https://patchwork.freedesktop.org/series/31758/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
pass -> FAIL (shard-hsw) fdo#99912
Test kms_flip:
S
On Wed, Nov 01, 2017 at 02:14:33PM +0100, Michal Wajdeczko wrote:
> On Wed, 01 Nov 2017 01:11:20 +0100, Anusha Srivatsa
> wrote:
>
> >Calculate the time that GuC takes to load using
> >jiffies. This information could be very useful in
> ^^^
> This is no longer true.
True. Will sending an a
On Thu, 2017-11-02 at 07:27 -0700, Rodrigo Vivi wrote:
> On Thu, Nov 02, 2017 at 10:34:37AM +, Jani Nikula wrote:
> > On Wed, 01 Nov 2017, Anusha Srivatsa wrote:
> > > There is a new version of DMC available for KBL.
> >
> > Nobody's going to pull this to linux-firmware if you don't send it
== Series Details ==
Series: i915: Cannonlake perf support (rev2)
URL : https://patchwork.freedesktop.org/series/32762/
State : failure
== Summary ==
Test kms_flip:
Subgroup blocking-wf_vblank:
pass -> FAIL (shard-hsw)
Test kms_busy:
Subgroup extende
On 02/11/17 16:35, Chris Wilson wrote:
Quoting Lionel Landwerlin (2017-11-02 16:29:48)
+/* Query various aspects of the topology of the GPU. Below is a list of the
+ * currently supported queries. The motivation of this more detailed query
+ * mechanism is to expose asynmetric properties of the
== Series Details ==
Series: tests: add device information tests (rev2)
URL : https://patchwork.freedesktop.org/series/32764/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
6d16875736b9fb1ebf4bf3dc5a941f9e431d58e0 lib/igt_debugfs: Remove support for
legacy
Quoting Imre Deak (2017-11-02 13:38:09)
> Doing modeset on internal panels may have a considerable overhead due to
> the panel specific power sequencing delays. To avoid long test runtimes
> in the CI fast-feedback test split out the testing of internal panels
> from the plane modeset subtests. Cre
== Series Details ==
Series: kms_atomic_transition: Split out modeset tests on internal panels
URL : https://patchwork.freedesktop.org/series/33052/
State : warning
== Summary ==
Test kms_setmode:
Subgroup basic:
pass -> FAIL (shard-hsw) fdo#99912
Test kms_a
On Thu, Nov 02, 2017 at 04:34:26PM -, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/3] drm/i915: Check if the stolen memory
> "reserved" area is enabled or not
> URL : https://patchwork.freedesktop.org/series/33060/
> State : warning
>
> == Summary ==
>
> Tes
== Series Details ==
Series: drm/i915: Plane assert/readout cleanups etc. (rev8)
URL : https://patchwork.freedesktop.org/series/31758/
State : success
== Summary ==
Series 31758v8 drm/i915: Plane assert/readout cleanups etc.
https://patchwork.freedesktop.org/api/1.0/series/31758/revisions/8/mb
>-Original Message-
>From: Jani Nikula [mailto:jani.nik...@linux.intel.com]
>Sent: Thursday, November 2, 2017 3:35 AM
>To: Srivatsa, Anusha ; intel-
>g...@lists.freedesktop.org
>Cc: Vivi, Rodrigo
>Subject: Re: [Intel-gfx] [PATCH] drm/i915/dmc: DMC 1.04 for Kabylake
>
>On Wed, 01 Nov 2017
== Series Details ==
Series: i915: Cannonlake perf support (rev2)
URL : https://patchwork.freedesktop.org/series/32762/
State : success
== Summary ==
Series 32762v2 i915: Cannonlake perf support
https://patchwork.freedesktop.org/api/1.0/series/32762/revisions/2/mbox/
Test chamelium:
S
From: Ville Syrjälä
Check that the planes are in the state we expect them to be. For
now we can only check whether each plane is correctly enabled or
disabled. In the future we may want to expand the plane state
readout to support a more through verification.
v2: Verify all planes part of the st
On 10/25/2017 08:26 AM, Michal Wajdeczko wrote:
On Tue, 24 Oct 2017 19:21:21 +0200, Sujaritha Sundaresan
wrote:
We currently have two module parameters that control GuC:
"enable_guc_loading" and "enable_guc_submission". Whenever
we need submission=1, we also need loading=1.We also need
load
On 02/11/17 16:35, Chris Wilson wrote:
Quoting Lionel Landwerlin (2017-11-02 16:29:48)
+/* Query various aspects of the topology of the GPU. Below is a list of the
+ * currently supported queries. The motivation of this more detailed query
+ * mechanism is to expose asynmetric properties of the
Quoting Lionel Landwerlin (2017-11-02 16:29:48)
> +/* Query various aspects of the topology of the GPU. Below is a list of the
> + * currently supported queries. The motivation of this more detailed query
> + * mechanism is to expose asynmetric properties of the GPU. Starting with
> CNL,
> + * sli
== Series Details ==
Series: series starting with [1/3] drm/i915: Check if the stolen memory
"reserved" area is enabled or not
URL : https://patchwork.freedesktop.org/series/33060/
State : warning
== Summary ==
Test kms_busy:
Subgroup extended-modeset-hang-oldfb-with-reset-render-A:
We can verify that topology is consistent with previous getparam like
EU_TOTAL.
We also verify that CS timestamp frequency is always filled.
v2: Use v2 of kernel uapi (Lionel)
Fix Gen < 8 issue
Signed-off-by: Lionel Landwerlin
---
tests/Makefile.sources| 1 +
tests/intel_device_info.
We were missing some registers and also can name one for which we only had
the offset.
Signed-off-by: Lionel Landwerlin
---
drivers/gpu/drm/i915/i915_perf.c | 3 ++-
drivers/gpu/drm/i915/i915_reg.h | 14 ++
2 files changed, 16 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/
With the introduction of asymetric slices in CNL, we cannot rely on the
previous SUBSLICE_MASK getparam. Here we introduce a more detailed way of
querying the Gen's GPU topology that doesn't aggregate numbers.
This is essential for monitoring parts of the GPU with the OA unit, because
counters nee
Gen8/9 aren't very different and we can merge some of this code.
Signed-off-by: Lionel Landwerlin
---
drivers/gpu/drm/i915/i915_perf.c | 48 +---
1 file changed, 25 insertions(+), 23 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm
Now that we have that information in topology fields, let's just reused it.
Signed-off-by: Lionel Landwerlin
---
drivers/gpu/drm/i915/i915_debugfs.c | 26 ++
1 file changed, 10 insertions(+), 16 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c
b/drivers/gpu
This name was added with the whitelisting of registers for building up OA
configs. It is contained in a range gen8 whitelist :
addr >= RPM_CONFIG0.reg && addr <= NOA_CONFIG(8).reg
Hence why the name isn't used anywhere.
Signed-off-by: Lionel Landwerlin
---
drivers/gpu/drm/i915/i915_reg.h |
Hi,
This is a change only on patch 8 (topology) where it makes the uapi a
bit simpler to use and also leaves some room for future queries (one
of the case could be query on media capabilities, where different
Gens/GT have different numbers of media rings).
Cheers,
Lionel Landwerlin (9):
drm/i9
We use to have this fixed per generation, but starting with CNL userspace
cannot tell just off the PCI ID. Let's make this information available. This
is particularly useful for performance monitoring where much of the
normalization work is done using those timestamps (this include pipeline
statist
Now that we have this stored in the device info, we can drop it from perf
part of the driver.
Note that this requires to init perf after we've computed the frequency,
hence why we move i915_perf_init() from i915_driver_init_early() to after
intel_device_info_runtime_init().
Signed-off-by: Lionel
This adds new registers to the whitelist to configs emitted from userspace.
Signed-off-by: Lionel Landwerlin
---
drivers/gpu/drm/i915/Makefile | 3 +-
drivers/gpu/drm/i915/i915_oa_cnl.c | 121 +
drivers/gpu/drm/i915/i915_oa_cnl.h | 34 +++
driv
We can enable GT3 as well as GT2.
Signed-off-by: Lionel Landwerlin
---
drivers/gpu/drm/i915/Makefile | 3 +-
drivers/gpu/drm/i915/i915_drv.h | 2 +
drivers/gpu/drm/i915/i915_oa_cflgt3.c | 109 ++
drivers/gpu/drm/i915/i915_oa_cflgt3.h | 34 ++
On Thu, Nov 02, 2017 at 05:19:07PM +0200, Jani Nikula wrote:
> On Thu, 02 Nov 2017, Ville Syrjälä wrote:
> > On Thu, Nov 02, 2017 at 02:56:51PM +0100, Maarten Lankhorst wrote:
> >> This interface is deprecated, and has been replaced by the upstream
> >> drm crc interface.
> >
> > Before we nuke th
== Series Details ==
Series: tests/debugfs_test: Pretty print subdirectories
URL : https://patchwork.freedesktop.org/series/33045/
State : warning
== Summary ==
Test drv_module_reload:
Subgroup basic-no-display:
pass -> DMESG-WARN (shard-hsw) fdo#102707 +1
Test km
On 11/02/2017 02:23 AM, Sagar Arun Kamble wrote:
On 10/25/2017 9:49 PM, Michal Wajdeczko wrote:
On Tue, 24 Oct 2017 19:21:25 +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
== Series Details ==
Series: kms_atomic_transition: Split out modeset tests on internal panels
URL : https://patchwork.freedesktop.org/series/33052/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
6d16875736b9fb1ebf4bf3dc5a941f9e431d58e0 lib/igt_debugfs: Rem
== Series Details ==
Series: drm/i915: Set up mocs tables before restarting the engines
URL : https://patchwork.freedesktop.org/series/33051/
State : warning
== Summary ==
Test kms_busy:
Subgroup extended-modeset-hang-newfb-with-reset-render-B:
pass -> DMESG-WARN
Quoting Mika Kuoppala (2017-11-02 15:40:50)
> Chris Wilson writes:
>
> > In the next few patches, we will have a hard requirement that we emit a
> > context-switch to the perma-pinned i915->kernel_context (so that we can
> > save the HW state using that context-switch). As the first context
> > i
Chris Wilson writes:
> In the next few patches, we will have a hard requirement that we emit a
> context-switch to the perma-pinned i915->kernel_context (so that we can
> save the HW state using that context-switch). As the first context
> itself may be classed as a kernel context, we want to be
== Series Details ==
Series: series starting with [1/3] drm/i915: Check if the stolen memory
"reserved" area is enabled or not
URL : https://patchwork.freedesktop.org/series/33060/
State : success
== Summary ==
Series 33060v1 series starting with [1/3] drm/i915: Check if the stolen memory
"r
On 11/2/2017 8:25 PM, Chris Wilson wrote:
Quoting Sagar Arun Kamble (2017-11-02 14:35:17)
On 11/2/2017 6:12 PM, Chris Wilson wrote:
GT powersaving is tightly coupled to the request infrastructure. To
avoid complications with the order of initialisation in the next patch
(where we want to sen
== Series Details ==
Series: Lib: Move __gem_context_create to common ioctl wrapper library. (rev3)
URL : https://patchwork.freedesktop.org/series/31775/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
6d16875736b9fb1ebf4bf3dc5a941f9e431d58e0 lib/igt_debugfs
From: Ville Syrjälä
Apparently there are some machines that put semi-sensible looking values
into the stolen "reserved" base and size, except those values are actually
outside the stolen memory. There is a bit in the register which
supposedly could tell us whether the reserved area is even enable
From: Ville Syrjälä
Now that we should be properly filtering out the cases when the stolen
reserved area is disabled, let's convert the debug message about a
misplaced reserved area into an error.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/i915_gem_stolen.c | 6 +++---
1 file change
From: Ville Syrjälä
While I have no solid proof that ILK follows the ELK path when it
comes to the stolen memory reserved area, there are some hints that
it might be the case. Unfortunately my ILK doesn't have this enabled,
and no way to enable it via the BIOS it seems.
So let's have ILK use the
On Thu, 02 Nov 2017, Ville Syrjälä wrote:
> On Thu, Nov 02, 2017 at 02:56:51PM +0100, Maarten Lankhorst wrote:
>> This interface is deprecated, and has been replaced by the upstream
>> drm crc interface.
>
> Before we nuke this I would like to see an option in the new interface
> to not filter out
== Series Details ==
Series: drm/i915/dmc: DMC 1.04 for Kabylake
URL : https://patchwork.freedesktop.org/series/33022/
State : warning
== Summary ==
Test kms_busy:
Subgroup extended-modeset-hang-newfb-with-reset-render-B:
pass -> DMESG-WARN (shard-hsw)
Sub
On Thu, 02 Nov 2017, Joonas Lahtinen wrote:
> On Thu, 2017-11-02 at 07:47 -0700, Rodrigo Vivi wrote:
>> On Thu, Nov 02, 2017 at 08:06:29AM +, Jani Nikula wrote:
>> > On Wed, 01 Nov 2017, Rodrigo Vivi wrote:
>> > > On Wed, Nov 01, 2017 at 04:21:08PM +, Ben Widawsky wrote:
>> > > > On 17-11
Quoting Mika Kuoppala (2017-10-31 15:27:34)
> From: Mika Kuoppala
>
> As all our access to execlist ports are through head and tail
> helpers, we can now move the head instead of memmoving the array.
>
> v2: use memset (Chris)
>
> Cc: Michał Winiarski
> Cc: Joonas Lahtinen
> Cc: Chris Wilson
== Series Details ==
Series: tests/debugfs_test: Pretty print subdirectories
URL : https://patchwork.freedesktop.org/series/33045/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
6d16875736b9fb1ebf4bf3dc5a941f9e431d58e0 lib/igt_debugfs: Remove support for
l
Quoting Mika Kuoppala (2017-11-02 14:32:38)
> From: Mika Kuoppala
>
> Instead of trusting that first available port is at index 0,
> use accessor to hide this. This is a preparation for a
> following patches where head can be at arbitrary location
> in the port array.
>
> v2: improved commit mes
On Thu, 2017-11-02 at 07:47 -0700, Rodrigo Vivi wrote:
> On Thu, Nov 02, 2017 at 08:06:29AM +, Jani Nikula wrote:
> > On Wed, 01 Nov 2017, Rodrigo Vivi wrote:
> > > On Wed, Nov 01, 2017 at 04:21:08PM +, Ben Widawsky wrote:
> > > > On 17-11-01 18:09:47, Joonas Lahtinen wrote:
> > > > > + Ki
On Thu, 2017-11-02 at 07:54 -0700, Rodrigo Vivi wrote:
> On Thu, Nov 02, 2017 at 02:43:16PM +, Joonas Lahtinen wrote:
> > On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
> > > As we now record the default HW state and so only emit the "golden"
> > > renderstate once to prepare the HW, th
Quoting Sagar Arun Kamble (2017-11-02 14:35:17)
>
>
> On 11/2/2017 6:12 PM, Chris Wilson wrote:
> > GT powersaving is tightly coupled to the request infrastructure. To
> > avoid complications with the order of initialisation in the next patch
> > (where we want to send requests to hw during GEM i
On Thu, Nov 02, 2017 at 02:43:16PM +, Joonas Lahtinen wrote:
> On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
> > As we now record the default HW state and so only emit the "golden"
> > renderstate once to prepare the HW, there is no advantage in keeping the
> > renderstate batch around
On Thu, Nov 02, 2017 at 08:06:29AM +, Jani Nikula wrote:
> On Wed, 01 Nov 2017, Rodrigo Vivi wrote:
> > On Wed, Nov 01, 2017 at 04:21:08PM +, Ben Widawsky wrote:
> >> On 17-11-01 18:09:47, Joonas Lahtinen wrote:
> >> > + Kimmo and Paul
> >> >
> >> > On Wed, 2017-11-01 at 07:43 -0700, Ben
== Series Details ==
Series: drm/i915: Remove support for legacy debugfs crc interface
URL : https://patchwork.freedesktop.org/series/33053/
State : warning
== Summary ==
Series 33053v1 drm/i915: Remove support for legacy debugfs crc interface
https://patchwork.freedesktop.org/api/1.0/series/3
On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
> As we now record the default HW state and so only emit the "golden"
> renderstate once to prepare the HW, there is no advantage in keeping the
> renderstate batch around as it will never be used again.
>
> Signed-off-by: Chris Wilson
Revie
Quoting Ville Syrjälä (2017-11-02 14:23:00)
> On Thu, Nov 02, 2017 at 12:42:24PM +, Chris Wilson wrote:
> > Take a copy of the HW state after a reset upon module loading by
> > executing a context switch from a blank context to the kernel context,
> > thus saving the default hw state over the b
On 11/2/2017 6:12 PM, Chris Wilson wrote:
GT powersaving is tightly coupled to the request infrastructure. To
avoid complications with the order of initialisation in the next patch
(where we want to send requests to hw during GEM init) move the
powersaving initialisation into the purview of i91
== Series Details ==
Series: series starting with [1/2] drm/i915: Reject unknown syncobj flags (rev2)
URL : https://patchwork.freedesktop.org/series/32893/
State : success
== Summary ==
Test kms_flip:
Subgroup plain-flip-ts-check-interruptible:
pass -> FAIL
From: Mika Kuoppala
Instead of trusting that first available port is at index 0,
use accessor to hide this. This is a preparation for a
following patches where head can be at arbitrary location
in the port array.
v2: improved commit message, elsp_ready readability (Chris)
v3: s/execlist_port_ind
== Series Details ==
Series: drm/i915: Set up mocs tables before restarting the engines
URL : https://patchwork.freedesktop.org/series/33051/
State : success
== Summary ==
Series 33051v1 drm/i915: Set up mocs tables before restarting the engines
https://patchwork.freedesktop.org/api/1.0/series
On Thu, Nov 02, 2017 at 10:34:37AM +, Jani Nikula wrote:
> On Wed, 01 Nov 2017, Anusha Srivatsa wrote:
> > There is a new version of DMC available for KBL.
>
> Nobody's going to pull this to linux-firmware if you don't send it to
> the linux-firmware folks...
That's intentional. The idea is
On Thu, Nov 02, 2017 at 02:56:51PM +0100, Maarten Lankhorst wrote:
> This interface is deprecated, and has been replaced by the upstream
> drm crc interface.
Before we nuke this I would like to see an option in the new interface
to not filter out the "bad" CRCs. When analyzing how the hardware
beh
On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
> Take a copy of the HW state after a reset upon module loading by
> executing a context switch from a blank context to the kernel context,
> thus saving the default hw state over the blank context image.
> We can then use the default hw state
On Thu, Nov 02, 2017 at 12:42:24PM +, Chris Wilson wrote:
> Take a copy of the HW state after a reset upon module loading by
> executing a context switch from a blank context to the kernel context,
> thus saving the default hw state over the blank context image.
> We can then use the default hw
Mika Kuoppala writes:
> Chris Wilson writes:
>
>> Quoting Mika Kuoppala (2017-11-02 10:38:30)
>>> Chris Wilson writes:
>>>
>>> > Quoting Mika Kuoppala (2017-10-31 15:27:33)
>>> >> +static inline struct execlist_port *
>>> >> +execlists_port_next(struct intel_engine_execlists * const execlists,
Chris Wilson writes:
> Quoting Mika Kuoppala (2017-11-02 10:38:30)
>> Chris Wilson writes:
>>
>> > Quoting Mika Kuoppala (2017-10-31 15:27:33)
>> >> +static inline struct execlist_port *
>> >> +execlists_port_next(struct intel_engine_execlists * const execlists,
>> >> + const
== Series Details ==
Series: drm/i915/dmc: DMC 1.04 for Kabylake
URL : https://patchwork.freedesktop.org/series/33022/
State : success
== Summary ==
Series 33022v1 drm/i915/dmc: DMC 1.04 for Kabylake
https://patchwork.freedesktop.org/api/1.0/series/33022/revisions/1/mbox/
Test kms_cursor_lega
Quoting Mika Kuoppala (2017-11-02 13:56:50)
> Chris Wilson writes:
> > -
> > - execlists_init_reg_state(vaddr + LRC_STATE_PN * PAGE_SIZE,
> > - ctx, engine, ring);
> > + regs = vaddr + LRC_STATE_PN * PAGE_SIZE;
> > + execlists_init_reg_state(regs, ctx, engi
This interface is deprecated, and has been replaced by the upstream
drm crc interface.
Signed-off-by: Maarten Lankhorst
Cc: Tomi Sarvela
Cc: Petri Latvala
Cc: Jani Nikula
---
drivers/gpu/drm/i915/i915_debugfs.c | 7 +-
drivers/gpu/drm/i915/i915_drv.h | 10 -
drivers/gpu/drm/i915/i9
Chris Wilson writes:
> Take a copy of the HW state after a reset upon module loading by
> executing a context switch from a blank context to the kernel context,
> thus saving the default hw state over the blank context image.
> We can then use the default hw state to initialise any future context
On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
> intel_modeset_gem_init() now only sets up the legacy overlay, so let's
> remove the function and call the setup directly during driver load. This
> should help us find a better point in the initialisation sequence for it
> later.
>
> Signed-
== Series Details ==
Series: drm/i915: silence coverity warning (rev2)
URL : https://patchwork.freedesktop.org/series/33041/
State : warning
== Summary ==
Test kms_busy:
Subgroup extended-modeset-hang-oldfb-with-reset-render-A:
pass -> DMESG-WARN (shard-hsw)
Test
On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
> GT powersaving is tightly coupled to the request infrastructure. To
> avoid complications with the order of initialisation in the next patch
> (where we want to send requests to hw during GEM init) move the
> powersaving initialisation into t
On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
> In the next few patches, we will have a hard requirement that we emit a
> context-switch to the perma-pinned i915->kernel_context (so that we can
> save the HW state using that context-switch). As the first context
> itself may be classed as
Quoting Ville Syrjälä (2017-11-02 13:22:51)
> On Thu, Nov 02, 2017 at 12:06:27PM +, Chris Wilson wrote:
> > Quoting Lionel Landwerlin (2017-11-02 11:41:26)
> > > Because dev_priv is 0-ed it's not currently an issue, but since we
> > > have dev_priv->perf.oa.test_config.uuid size at uuid + 1, we
Doing modeset on internal panels may have a considerable overhead due to
the panel specific power sequencing delays. To avoid long test runtimes
in the CI fast-feedback test split out the testing of internal panels
from the plane modeset subtests. Create fast and slow versions of these
new subtests
On Thu, 2017-11-02 at 12:42 +, Chris Wilson wrote:
> Now that we always execute a context switch upon module load, there is
> no need to queue a delayed task for doing so. The purpose of the delayed
> task is to enable GT powersaving, for which we need the HW state to be
> valid (i.e. having lo
On Wed, Nov 01, 2017 at 01:08:47PM +0200, Imre Deak wrote:
> On Wed, Nov 01, 2017 at 10:48:50AM +, Chris Wilson wrote:
> > Quoting Imre Deak (2017-11-01 09:56:22)
> > > On Tue, Oct 31, 2017 at 10:23:25PM +, Chris Wilson wrote:
> > > > Quoting Imre Deak (2017-10-31 13:44:47)
> > > > > Doing
On Tue, Oct 31, 2017 at 11:02:46AM +, Lionel Landwerlin wrote:
> On 31/10/17 09:21, Petri Latvala wrote:
> > On Mon, Oct 30, 2017 at 02:11:10PM +, Patchwork wrote:
> > > == Series Details ==
> > >
> > > Series: tests: add device information tests
> > > URL : https://patchwork.freedesktop
On Thu, Nov 02, 2017 at 12:06:27PM +, Chris Wilson wrote:
> Quoting Lionel Landwerlin (2017-11-02 11:41:26)
> > Because dev_priv is 0-ed it's not currently an issue, but since we
> > have dev_priv->perf.oa.test_config.uuid size at uuid + 1, we could
> > just copy the null character and silence
Op 27-10-17 om 14:25 schreef Petri Latvala:
> On Fri, Oct 27, 2017 at 10:05:39AM +0200, Maarten Lankhorst wrote:
>> Op 27-10-17 om 09:39 schreef Jani Nikula:
>>> On Thu, 26 Oct 2017, James Ausmus wrote:
On Thu, Oct 26, 2017 at 01:38:51PM +0200, Maarten Lankhorst wrote:
> In kernel v4.10 t
On Thu, Nov 02, 2017 at 11:12:09AM +0100, Daniel Vetter wrote:
> On Wed, Nov 01, 2017 at 08:29:18PM +0200, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Try to fix the code to actually clip the plane to the crtc bounds
> > instead of the user provided crtc coordinates (which would be a no-
After a reset, we may immediately begin executing requests on restarting
the engines. Ergo this has to be last step with all re-initialisation
completed beforehand. The mocs setup was after we started executing the
requests; do it earlier!
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i91
== Series Details ==
Series: series starting with [1/6] drm/i915: Force the switch to the
i915->kernel_context
URL : https://patchwork.freedesktop.org/series/33046/
State : failure
== Summary ==
Series 33046v1 series starting with [1/6] drm/i915: Force the switch to the
i915->kernel_context
1 - 100 of 151 matches
Mail list logo
