Rodrigo Vivi writes:
> "Pandiyan, Dhinakaran" writes:
>
>> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote:
>>> Hi Andy,
>>>
>>> thanks for getting involved with PSR and sorry for not replying sooner.
>>>
>>> I first saw this patch on that bugzilla entry but only now I stop to
>>> really t
== Series Details ==
Series: drm/i915: Rename drm_i915_gem_request to i915_request
URL : https://patchwork.freedesktop.org/series/37958/
State : warning
== Summary ==
Test kms_frontbuffer_tracking:
Subgroup fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
fail -> PASS
On 02/08/2018 03:31 PM, Chris Wilson wrote:
Quoting Jackie Li (2018-02-08 23:03:54)
@@ -95,7 +97,11 @@ struct intel_guc_wopcm {
u32 offset;
u32 size;
u32 top;
- u32 valid;
+
+ /* GuC WOPCM flags below. */
+ u32 valid:1;
+ u32 hw_updated:1;
+
== Series Details ==
Series: lib/igt_debugfs: Add igt_pipe_crc_get_single and igt_pipe_crc_drain, v3.
URL : https://patchwork.freedesktop.org/series/37910/
State : failure
== Summary ==
Test kms_cursor_legacy:
Subgroup cursor-vs-flip-legacy:
fail -> PASS (sh
== Series Details ==
Series: drm/i915/userptr: Probe vma range before gup (rev6)
URL : https://patchwork.freedesktop.org/series/35394/
State : failure
== Summary ==
Test gem_userptr_blits:
Subgroup coherency-sync:
pass -> DMESG-WARN (shard-snb)
Subgroup un
Apologies. This patch was pushed wrongly. Not a part of the 16 patch series of
NV12.
Regards
Vidya
> -Original Message-
> From: Sharma, Shashank
> Sent: Thursday, February 8, 2018 6:17 PM
> To: Srinivas, Vidya ; intel-
> g...@lists.freedesktop.org
> Cc: maarten.lankho...@linux.intel.com;
From: Sharma, Shashank
Sent: Thursday, February 8, 2018 12:17 PM
To: Srinivas, Vidya ; intel-gfx@lists.freedesktop.org
Cc: maarten.lankho...@linux.intel.com; Kamath, Sunil ;
Shankar, Uma ; Kumar, Mahesh1
Subject: Re: [PATCH 03/16] drm/i915/skl+: add NV12 in skl_format_to_fourcc
Regards
Shash
> -Original Message-
> From: Sharma, Shashank
> Sent: Thursday, February 8, 2018 2:35 PM
> To: Srinivas, Vidya ; intel-
> g...@lists.freedesktop.org
> Cc: maarten.lankho...@linux.intel.com; Kamath, Sunil
> ; Shankar, Uma ;
> Konduru, Chandra ; Maiti, Nabendu Bikash
>
> Subject: Re: [PATC
> -Original Message-
> From: Sharma, Shashank
> Sent: Thursday, February 8, 2018 3:16 PM
> To: Srinivas, Vidya ; intel-
> g...@lists.freedesktop.org
> Cc: maarten.lankho...@linux.intel.com; Kamath, Sunil
> ; Shankar, Uma ;
> Konduru, Chandra ; Maiti, Nabendu Bikash
>
> Subject: Re: [PATC
Srinivas, Vidya would like to recall the message, "[PATCH] drm/i915: NV12
changes for Gen10".
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Apologies. This patch was pushed wrongly. Not a part of the 16 patch series of
NV12.
Regards
Vidya
> -Original Message-
> From: Sharma, Shashank
> Sent: Thursday, February 8, 2018 6:17 PM
> To: Srinivas, Vidya ; intel-
> g...@lists.freedesktop.org
> Cc: maarten.lankho...@linux.intel.com;
Apologies. This patch was pushed wrongly. Not a part of the 16 patch series of
NV12.
Regards
Vidya
> -Original Message-
> From: Sharma, Shashank
> Sent: Thursday, February 8, 2018 6:08 PM
> To: Srinivas, Vidya ; intel-
> g...@lists.freedesktop.org
> Cc: maarten.lankho...@linux.intel.com;
> -Original Message-
> From: Sharma, Shashank
> Sent: Thursday, February 8, 2018 4:21 PM
> To: Srinivas, Vidya ; intel-
> g...@lists.freedesktop.org
> Cc: maarten.lankho...@linux.intel.com; Kamath, Sunil
> ; Shankar, Uma ;
> Konduru, Chandra ; Maiti, Nabendu Bikash
>
> Subject: Re: [PATC
== Series Details ==
Series: drm/i915: Don't query PCODE RC6VIDS on platforms not supporting it
URL : https://patchwork.freedesktop.org/series/37936/
State : success
== Summary ==
Test kms_cursor_crc:
Subgroup cursor-64x64-offscreen:
skip -> PASS (shard-apl)
"Pandiyan, Dhinakaran" writes:
> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote:
>> Hi Andy,
>>
>> thanks for getting involved with PSR and sorry for not replying sooner.
>>
>> I first saw this patch on that bugzilla entry but only now I stop to
>> really think why I have written the cod
== Series Details ==
Series: drm/i915/crt: Silence compiler warning for uninitialised status (rev2)
URL : https://patchwork.freedesktop.org/series/33365/
State : success
== Summary ==
Test kms_vblank:
Subgroup pipe-a-ts-continuation-dpms-suspend:
skip -> PASS
On Thu, 2018-02-08 at 17:01 -0800, Andy Lutomirski wrote:
>
>
> > On Feb 8, 2018, at 4:39 PM, Pandiyan, Dhinakaran
> > wrote:
> >
> >
> >> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote:
> >> Hi Andy,
> >>
> >> thanks for getting involved with PSR and sorry for not replying sooner.
> On Feb 8, 2018, at 4:39 PM, Pandiyan, Dhinakaran
> wrote:
>
>
>> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote:
>> Hi Andy,
>>
>> thanks for getting involved with PSR and sorry for not replying sooner.
>>
>> I first saw this patch on that bugzilla entry but only now I stop to
>>
== Series Details ==
Series: drm/i915: Disable unused-but-set compiler warning
URL : https://patchwork.freedesktop.org/series/37925/
State : success
== Summary ==
Test kms_cursor_legacy:
Subgroup cursor-vs-flip-legacy:
fail -> PASS (shard-apl) fdo#103355
Tes
On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote:
> Hi Andy,
>
> thanks for getting involved with PSR and sorry for not replying sooner.
>
> I first saw this patch on that bugzilla entry but only now I stop to
> really think why I have written the code that way.
>
> So some clarity below.
== Series Details ==
Series: drm/i915: Rename drm_i915_gem_request to i915_request
URL : https://patchwork.freedesktop.org/series/37958/
State : success
== Summary ==
Series 37958v1 drm/i915: Rename drm_i915_gem_request to i915_request
https://patchwork.freedesktop.org/api/1.0/series/37958/rev
== Series Details ==
Series: drm/i915: Handle RC6 counter wrap (rev5)
URL : https://patchwork.freedesktop.org/series/37824/
State : success
== Summary ==
Test gem_eio:
Subgroup in-flight-contexts:
pass -> FAIL (shard-hsw) fdo#104676 +1
Test kms_cursor_crc:
Quoting Jackie Li (2018-02-08 23:03:54)
> @@ -95,7 +97,11 @@ struct intel_guc_wopcm {
> u32 offset;
> u32 size;
> u32 top;
> - u32 valid;
> +
> + /* GuC WOPCM flags below. */
> + u32 valid:1;
> + u32 hw_updated:1;
> + u32 need_load_huc_fw:1;
bo
== Series Details ==
Series: series starting with [v9,1/7] drm/i915/guc: Move GuC WOPCM related code
into separate files
URL : https://patchwork.freedesktop.org/series/37957/
State : failure
== Summary ==
Series 37957v1 series starting with [v9,1/7] drm/i915/guc: Move GuC WOPCM
related code
== Series Details ==
Series: drm/i915: Remove redundant check on execlists interrupt
URL : https://patchwork.freedesktop.org/series/37923/
State : failure
== Summary ==
Test kms_cursor_legacy:
Subgroup cursor-vs-flip-legacy:
fail -> PASS (shard-apl) fdo#1033
GuC WOPCM registers are write-once registers. Current driver code accesses
these registers without checking the accessibility to these registers which
will lead to unpredictable driver behaviors if these registers were touch
by other components (such as faulty BIOS code).
This patch moves the GuC
On CNL A0 and Gen9, there's a hardware restriction that requires the
available GuC WOPCM size to be larger than or equal to HuC firmware size.
This patch adds new verfication code to ensure the available GuC WOPCM size
to be larger than or equal to HuC firmware size on both Gen9 and CNL A0.
v6:
Signed-off-by: Jackie Li
---
drivers/gpu/drm/i915/i915_params.c | 2 +-
drivers/gpu/drm/i915/i915_params.h | 2 +-
drivers/gpu/drm/i915/intel_guc_wopcm.c | 6 ++
3 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_params.c
b/drivers/gpu/drm/i915/i
Hardware may have specific restrictions on GuC WOPCM offset and size. On
Gen9, the value of the GuC WOPCM size register needs to be larger than the
value of GuC WOPCM offset register + a Gen9 specific offset (144KB) for
reserved GuC WOPCM. Fail to enforce such a restriction on GuC WOPCM size
will l
CNL has its specific reserved GuC WOPCM size for RC6 and other hardware
contexts.
This patch updates the code to return CNL specific reserved GuC WOPCM size
for RC6 and other hardware contexts so that the GuC WOPCM size can be
calculated correctly for CNL.
v9:
- Created a new patch for these cha
intel_guc_reg.h should only include definition for GuC registers and
related register bits. Non-register related GuC WOPCM macro definitions
should not be defined in intel_guc_reg.h.
This patch cleans up intel_guc_reg.h and moves GuC WOPCM related code into
new created separate files as future pat
GuC related exported functions should start with "intel_guc_" prefix and
pass intel_guc as the first parameter since its guc related. Current
guc_ggtt_offset() failed to follow this code convention and this is a
problem for future patches that needs to access intel_guc data to verify
the GGTT offse
Hi Andy,
thanks for getting involved with PSR and sorry for not replying sooner.
I first saw this patch on that bugzilla entry but only now I stop to
really think why I have written the code that way.
So some clarity below.
On Mon, Feb 05, 2018 at 10:07:09PM +, Andy Lutomirski wrote:
> The
== Series Details ==
Series: drm/i915: Fix kerneldoc warnings for i915_gem_internal
URL : https://patchwork.freedesktop.org/series/37908/
State : warning
== Summary ==
Test kms_rotation_crc:
Subgroup sprite-rotation-90-flip:
pass -> FAIL (shard-apl) fdo#1033
== Series Details ==
Series: drm/i915: Fix kerneldoc warnings in i915_gem_execbuffer
URL : https://patchwork.freedesktop.org/series/37907/
State : success
== Summary ==
Test kms_cursor_crc:
Subgroup cursor-64x64-suspend:
pass -> FAIL (shard-snb) fdo#102365 +
== Series Details ==
Series: drm/i915/snb+: Remove incorrect forcewake check in
debugfs/i915_drpc_info
URL : https://patchwork.freedesktop.org/series/37901/
State : warning
== Summary ==
Test kms_flip:
Subgroup basic-flip-vs-modeset:
pass -> DMESG-WARN (shard-hsw
== Series Details ==
Series: drm/i915: Remove lost comment from i915_gem_context
URL : https://patchwork.freedesktop.org/series/37900/
State : warning
== Summary ==
Test kms_sysfs_edid_timing:
pass -> WARN (shard-apl) fdo#100047
Test kms_flip:
Subgroup modes
== Series Details ==
Series: drm/i915: Fix kerneldoc warnings for i915_gem_request
URL : https://patchwork.freedesktop.org/series/37898/
State : success
== Summary ==
Test gem_eio:
Subgroup in-flight-contexts:
pass -> FAIL (shard-hsw) fdo#104676
Test gem_exe
== Series Details ==
Series: lib/igt_debugfs: Add igt_pipe_crc_get_single and igt_pipe_crc_drain, v3.
URL : https://patchwork.freedesktop.org/series/37910/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
3af87d45da24015b7a6124b59b2c4b854381cab6 tools/intel_a
== Series Details ==
Series: drm/i915: Fix kerneldoc warnings for i915_gem_userptr
URL : https://patchwork.freedesktop.org/series/37897/
State : success
== Summary ==
Test perf_pmu:
Subgroup semaphore-wait-idle-bcs0:
pass -> FAIL (shard-apl) fdo#105011
Test
== Series Details ==
Series: drm/i915/userptr: Probe vma range before gup (rev6)
URL : https://patchwork.freedesktop.org/series/35394/
State : success
== Summary ==
Series 35394v6 drm/i915/userptr: Probe vma range before gup
https://patchwork.freedesktop.org/api/1.0/series/35394/revisions/6/mb
On Thu, Feb 08, 2018 at 06:33:03PM +, Chris Wilson wrote:
> Quoting Imre Deak (2018-02-08 17:41:02)
> > On BXT/GLK GEN6_PCODE_READ_RC6VIDS fails with
> > MAILBOX_P24C_CC_ILLEGAL_CMD, so don't try to do the query on these
> > platforms. Do it only on SNB, IVB and HSW, where we use this command
>
Quoting Imre Deak (2018-02-08 17:41:02)
> On BXT/GLK GEN6_PCODE_READ_RC6VIDS fails with
> MAILBOX_P24C_CC_ILLEGAL_CMD, so don't try to do the query on these
> platforms. Do it only on SNB, IVB and HSW, where we use this command
> anyway for RC6 enabling.
>
> Based on my tests the command also succ
== Series Details ==
Series: drm/i915: Don't query PCODE RC6VIDS on platforms not supporting it
URL : https://patchwork.freedesktop.org/series/37936/
State : success
== Summary ==
Series 37936v1 drm/i915: Don't query PCODE RC6VIDS on platforms not supporting
it
https://patchwork.freedesktop.o
On Thu, Feb 08, 2018 at 09:55:38AM +0100, Maarten Lankhorst wrote:
> References: https://bugs.freedesktop.org/show_bug.cgi?id=104975
> Signed-off-by: Maarten Lankhorst
> ---
> drivers/gpu/drm/i915/i915_drv.h | 2 ++
> drivers/gpu/drm/i915/i915_reg.h | 3 +++
> drivers/gpu/drm/i915/int
== Series Details ==
Series: drm/i915: Fix kerneldoc warnings for intel_ringbuffer
URL : https://patchwork.freedesktop.org/series/37896/
State : warning
== Summary ==
Test gem_exec_suspend:
Subgroup basic-s4:
fail -> SKIP (shard-snb) fdo#103375 +1
Test gem_e
We want to exclude any GGTT objects from being present on our internal
lists to avoid the deadlock we may run into with our requirement for
struct_mutex during invalidate. However, if the gup_fast fails, we put
the userptr onto the workqueue and mark it as active, so that we
remember to serialise t
On Thu, Feb 08, 2018 at 04:39:39PM +, Chris Wilson wrote:
> clang is confused by our if-else-chain that abruptly exits before a
> final else:
>
> drivers/gpu/drm/i915/intel_crt.c:821:11: warning: variable 'status' is used
> uninitialized whenever 'if' condition is false [-Wsometimes-uninitial
Quoting Jordan Crouse (2018-02-08 17:31:50)
> The i915 DRM driver very cleverly used ascii85 encoding for their
> GPU state file. Move the encode functions to a general header file to
> support other drivers that might be interested in the same
> functionality.
>
> [v2 - Update the API to be clean
On BXT/GLK GEN6_PCODE_READ_RC6VIDS fails with
MAILBOX_P24C_CC_ILLEGAL_CMD, so don't try to do the query on these
platforms. Do it only on SNB, IVB and HSW, where we use this command
anyway for RC6 enabling.
Based on my tests the command also succeeds on all LLC platforms, but
it's not clear if it'
The i915 DRM driver very cleverly used ascii85 encoding for their
GPU state file. Move the encode functions to a general header file to
support other drivers that might be interested in the same
functionality.
[v2 - Update the API to be cleaner for the caller suggested by
Chris Wilson]
Signed-off
Quoting Jani Nikula (2018-01-19 14:11:29)
> On Sat, 09 Dec 2017, Chris Wilson wrote:
> > The struct platform_device memdups the provided data pointer requiring
> > us to free the template we construct during lpe_audio_platdev_create():
> >
> > unreferenced object 0x88026eafe400 (size 512):
> >
== Series Details ==
Series: drm/i915/crt: Silence compiler warning for uninitialised status (rev2)
URL : https://patchwork.freedesktop.org/series/33365/
State : success
== Summary ==
Series 33365v2 drm/i915/crt: Silence compiler warning for uninitialised status
https://patchwork.freedesktop.o
Quoting Maarten Lankhorst (2018-02-08 16:50:08)
> Op 08-02-18 om 17:16 schreef Chris Wilson:
> > The unused-but-set warning enabled by W=1 catches out a lot of the
> > atomic helper iterator macros and drown us in their noise (or trip over
> > Werror and die). Path of least resistance is to ignore
== Series Details ==
Series: drm/i915: Disable unused-but-set compiler warning
URL : https://patchwork.freedesktop.org/series/37925/
State : success
== Summary ==
Series 37925v1 drm/i915: Disable unused-but-set compiler warning
https://patchwork.freedesktop.org/api/1.0/series/37925/revisions/1
Op 08-02-18 om 17:16 schreef Chris Wilson:
> The unused-but-set warning enabled by W=1 catches out a lot of the
> atomic helper iterator macros and drown us in their noise (or trip over
> Werror and die). Path of least resistance is to ignore the warning.
>
> Signed-off-by: Chris Wilson
> Cc: Joon
clang is confused by our if-else-chain that abruptly exits before a
final else:
drivers/gpu/drm/i915/intel_crt.c:821:11: warning: variable 'status' is used
uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
else if (ret < 0)
^~~
drivers/gpu/drm
On 11/01/18 18:25, Oscar Mateo wrote:
From: Kelvin Gardiner
ICL 11 has a greater number of maximum subslices. This patch
reflects this.
v2: GEN11 updates to MCR_SELECTOR (Oscar)
Bspec: 21139
BSpec: 21108
Reviewed-by: Daniele Ceraolo Spurio (v1)
Signed-off-by: Kelvin Gardiner
Signed-off-by:
== Series Details ==
Series: series starting with [1/2] drm/i915: Add missing kerneldoc for 'ent' in
i915_driver_init_early
URL : https://patchwork.freedesktop.org/series/37893/
State : failure
== Summary ==
Test kms_flip:
Subgroup 2x-plain-flip-fb-recreate-interruptible:
Quoting Chris Wilson (2018-02-08 13:39:17)
> drivers/gpu/drm/i915/i915_gem_execbuffer.c:1983: warning: No description
> found for parameter 'dev_priv'
> drivers/gpu/drm/i915/i915_gem_execbuffer.c:1983: warning: No description
> found for parameter 'file'
>
> Signed-off-by: Chris Wilson
Reviewe
== Series Details ==
Series: drm/i915: Handle RC6 counter wrap (rev5)
URL : https://patchwork.freedesktop.org/series/37824/
State : success
== Summary ==
Series 37824v5 drm/i915: Handle RC6 counter wrap
https://patchwork.freedesktop.org/api/1.0/series/37824/revisions/5/mbox/
Test gem_exec_sus
Quoting Chris Wilson (2018-02-08 13:42:24)
> drivers/gpu/drm/i915/i915_gem_internal.c:183: warning: No description found
> for parameter 'i915'
> drivers/gpu/drm/i915/i915_gem_internal.c:183: warning: No description found
> for parameter 'size'
>
> Signed-off-by: Chris Wilson
Reviewed-by: Joon
The unused-but-set warning enabled by W=1 catches out a lot of the
atomic helper iterator macros and drown us in their noise (or trip over
Werror and die). Path of least resistance is to ignore the warning.
Signed-off-by: Chris Wilson
Cc: Joonas Lahtinen
Cc: Maarten Lankhorst
---
drivers/gpu/d
Quoting Chris Wilson (2018-02-08 16:06:59)
> Quoting Chris Wilson (2018-02-08 11:42:24)
> > drivers/gpu/drm/i915/i915_gem_internal.c:183: warning: No description found
> > for parameter 'i915'
> > drivers/gpu/drm/i915/i915_gem_internal.c:183: warning: No description found
> > for parameter 'size'
Quoting Chris Wilson (2018-02-08 11:42:24)
> drivers/gpu/drm/i915/i915_gem_internal.c:183: warning: No description found
> for parameter 'i915'
> drivers/gpu/drm/i915/i915_gem_internal.c:183: warning: No description found
> for parameter 'size'
>
> Signed-off-by: Chris Wilson
irc Reviewed-by:
== Series Details ==
Series: drm/i915: Handle RC6 counter wrap (rev4)
URL : https://patchwork.freedesktop.org/series/37824/
State : success
== Summary ==
Series 37824v4 drm/i915: Handle RC6 counter wrap
https://patchwork.freedesktop.org/api/1.0/series/37824/revisions/4/mbox/
Test kms_pipe_crc
Quoting Tvrtko Ursulin (2018-02-08 16:00:36)
> From: Tvrtko Ursulin
>
> We can implement limited RC6 counter wrap-around protection under the
> assumption that clients will be reading this value more frequently than
> the wrap period on a given platform.
>
> With the typical wrap-around period b
On 08/02/18 11:23, Imre Deak wrote:
FORCEWAKE_ACK is depricated by BSpec at least starting from BDW,
referring to the multi-threaded version of it instead. Accessing
FORCEWAKE_ACK triggers an unclaimed register access error - at
least on GLK - see the Reference: below.
The correct registers to u
From: Tvrtko Ursulin
We can implement limited RC6 counter wrap-around protection under the
assumption that clients will be reading this value more frequently than
the wrap period on a given platform.
With the typical wrap-around period being ~90 minutes, even with the
exception of Baytrail which
On Thu, Feb 08, 2018 at 05:13:02PM +0200, Mika Kuoppala wrote:
> Chris Wilson writes:
>
> > Quoting Tvrtko Ursulin (2018-02-08 14:34:38)
> >>
> >> On 08/02/2018 14:22, Ville Syrjälä wrote:
> >> > On Thu, Feb 08, 2018 at 01:06:05PM +, Tvrtko Ursulin wrote:
> >> >> From: Tvrtko Ursulin
> >> >
== Series Details ==
Series: series starting with [v2,1/2] drm/i915/perf: Fix compiler warning for
string truncation
URL : https://patchwork.freedesktop.org/series/37892/
State : success
== Summary ==
Test gem_exec_suspend:
Subgroup basic-s4:
skip -> FAIL (
Quoting Tvrtko Ursulin (2018-02-08 15:46:38)
> From: Tvrtko Ursulin
>
> We can implement limited RC6 counter wrap-around protection under the
> assumption that clients will be reading this value more frequently than
> the wrap period on a given platform.
>
> With the typical wrap-around period b
On 08/02/2018 15:46, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin
We can implement limited RC6 counter wrap-around protection under the
assumption that clients will be reading this value more frequently than
the wrap period on a given platform.
With the typical wrap-around period being ~90 minu
From: Tvrtko Ursulin
We can implement limited RC6 counter wrap-around protection under the
assumption that clients will be reading this value more frequently than
the wrap period on a given platform.
With the typical wrap-around period being ~90 minutes, even with the
exception of Baytrail which
== Series Details ==
Series: drm/i915: Remove redundant check on execlists interrupt
URL : https://patchwork.freedesktop.org/series/37923/
State : success
== Summary ==
Series 37923v1 drm/i915: Remove redundant check on execlists interrupt
https://patchwork.freedesktop.org/api/1.0/series/37923
Quoting Chris Wilson (2018-02-08 15:29:29)
> Quoting Mika Kuoppala (2018-02-08 15:20:30)
> > Chris Wilson writes:
> >
> > > Since commit 4a118ecbe99c ("drm/i915: Filter out spurious execlists
> > > context-switch interrupts") we probe execlists->active, and no longer
> > > have to peek at the exe
Quoting Mika Kuoppala (2018-02-08 15:20:30)
> Chris Wilson writes:
>
> > Since commit 4a118ecbe99c ("drm/i915: Filter out spurious execlists
> > context-switch interrupts") we probe execlists->active, and no longer
> > have to peek at the execlist interrupt to determine if the tasklet still
> > n
== Series Details ==
Series: Selectable platform support
URL : https://patchwork.freedesktop.org/series/37912/
State : failure
== Summary ==
Applying: drm/i915: Make I830 platform support optional
Applying: drm/i915: Make I845G platform support optional
Applying: drm/i915: Make I85X platform s
== Series Details ==
Series: drm/i915: Fix kerneldoc warnings for i915_gem_internal
URL : https://patchwork.freedesktop.org/series/37908/
State : success
== Summary ==
Series 37908v1 drm/i915: Fix kerneldoc warnings for i915_gem_internal
https://patchwork.freedesktop.org/api/1.0/series/37908/r
Chris Wilson writes:
> Since commit 4a118ecbe99c ("drm/i915: Filter out spurious execlists
> context-switch interrupts") we probe execlists->active, and no longer
> have to peek at the execlist interrupt to determine if the tasklet still
> needs to be run to drain the ELSP.
>
> References: 4a118e
On 08/02/2018 11:39, Chris Wilson wrote:
drivers/gpu/drm/i915/i915_gem_execbuffer.c:1983: warning: No description found
for parameter 'dev_priv'
drivers/gpu/drm/i915/i915_gem_execbuffer.c:1983: warning: No description found
for parameter 'file'
Signed-off-by: Chris Wilson
---
drivers/gpu/d
Chris Wilson writes:
> Quoting Tvrtko Ursulin (2018-02-08 14:34:38)
>>
>> On 08/02/2018 14:22, Ville Syrjälä wrote:
>> > On Thu, Feb 08, 2018 at 01:06:05PM +, Tvrtko Ursulin wrote:
>> >> From: Tvrtko Ursulin
>> >>
>> >> Coccinelle transformation:
>> >>
>> >> @@
>> >> expression p, g;
>>
Since commit 4a118ecbe99c ("drm/i915: Filter out spurious execlists
context-switch interrupts") we probe execlists->active, and no longer
have to peek at the execlist interrupt to determine if the tasklet still
needs to be run to drain the ELSP.
References: 4a118ecbe99c ("drm/i915: Filter out spur
Quoting Chris Wilson (2018-02-08 10:54:49)
> drivers/gpu/drm/i915/i915_syncmap.c:92: warning: No description found for
> parameter 'root'
> drivers/gpu/drm/i915/i915_syncmap.c:155: warning: No description found for
> parameter 'root'
> drivers/gpu/drm/i915/i915_syncmap.c:155: warning: No descript
Quoting Tvrtko Ursulin (2018-02-08 16:06:41)
>
> On 08/02/2018 13:26, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2018-02-08 13:05:51)
> >> From: Tvrtko Ursulin
> >>
> >> For Joonas basically. :)
> >>
> >> Rough goal - add Kconfig options to turn off supported platforms and count
> >> on
> >
Quoting Tvrtko Ursulin (2018-02-08 14:34:38)
>
> On 08/02/2018 14:22, Ville Syrjälä wrote:
> > On Thu, Feb 08, 2018 at 01:06:05PM +, Tvrtko Ursulin wrote:
> >> From: Tvrtko Ursulin
> >>
> >> Coccinelle transformation:
> >>
> >> @@
> >> expression p, g;
> >> @@
> >> (
> >> -INTEL_GEN
== Series Details ==
Series: drm/i915: Fix kerneldoc warnings in i915_gem_execbuffer
URL : https://patchwork.freedesktop.org/series/37907/
State : success
== Summary ==
Series 37907v1 drm/i915: Fix kerneldoc warnings in i915_gem_execbuffer
https://patchwork.freedesktop.org/api/1.0/series/37907
Quoting Tvrtko Ursulin (2018-02-08 14:31:05)
>
> On 08/02/2018 14:06, Ville Syrjälä wrote:
> > On Thu, Feb 08, 2018 at 08:18:42AM +, Tvrtko Ursulin wrote:
> >> From: Tvrtko Ursulin
> >>
> >> We can implement limited RC6 counter wrap-around protection under the
> >> assumption that clients wil
On 08/02/2018 14:06, Ville Syrjälä wrote:
On Thu, Feb 08, 2018 at 08:18:42AM +, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin
We can implement limited RC6 counter wrap-around protection under the
assumption that clients will be reading this value more frequently than
the wrap period on a giv
Quoting Joonas Lahtinen (2018-02-08 14:16:56)
> Quoting Chris Wilson (2018-01-04 23:48:45)
> > Quoting Chris Wilson (2018-01-04 21:45:28)
> > > +#define lockdep_rename(lock, name) \
> > > + lockdep_set_class_and_name(lock, (lock)->dep_map.key, name)
> >
>
Quoting Chris Wilson (2018-01-04 23:48:45)
> Quoting Chris Wilson (2018-01-04 21:45:28)
> > +#define lockdep_rename(lock, name) \
> > + lockdep_set_class_and_name(lock, (lock)->dep_map.key, name)
>
> Maybe lockdep_set_name() for similarity to the others.
Quoting Chris Wilson (2017-12-23 00:04:31)
> When we require GEM, reset the global seqno. This gives each test a
> clean slate to work with, and avoids left-over state from previous tests
> impacting on the next. In particular, somes tests may be setting up long
> sequence of stalling batches not e
On 08/02/2018 13:26, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2018-02-08 13:05:51)
From: Tvrtko Ursulin
For Joonas basically. :)
Rough goal - add Kconfig options to turn off supported platforms and count on
compiler DCE to make the driver smaller.
Tested as so much that it boots and rend
On Thu, Feb 08, 2018 at 08:18:42AM +, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> We can implement limited RC6 counter wrap-around protection under the
> assumption that clients will be reading this value more frequently than
> the wrap period on a given platform.
>
> With the typical w
== Series Details ==
Series: drm/i915/snb+: Remove incorrect forcewake check in
debugfs/i915_drpc_info
URL : https://patchwork.freedesktop.org/series/37901/
State : success
== Summary ==
Series 37901v1 drm/i915/snb+: Remove incorrect forcewake check in
debugfs/i915_drpc_info
https://patchwor
== Series Details ==
Series: drm/i915: Use DOUBLE_BUFFER_CTL instead of vblank evasion for GEN9+.
URL : https://patchwork.freedesktop.org/series/37879/
State : success
== Summary ==
Test gem_exec_suspend:
Subgroup basic-s4:
fail -> SKIP (shard-hsw) fdo#10337
Quoting Mika Kuoppala (2018-02-08 11:43:50)
> Chris Wilson writes:
>
> > After we assert the reset request (and wait for 20us), when the device
> > has been fully reset it asserts the reset-status bit. Before we stop
> > requesting the reset and allow the device to return to normal, we should
> >
Quoting Mika Kuoppala (2018-02-08 12:34:08)
> Chris Wilson writes:
>
> > Quoting Mika Kuoppala (2018-02-08 11:43:50)
> >> Any thoughts of starting to log the reset attempts
> >> with timeout, even if the subsequent reset succeeds?
> >
> > If it succeeds, do we care? Capturing why it fails, sure.
== Series Details ==
Series: drm/i915: Remove lost comment from i915_gem_context
URL : https://patchwork.freedesktop.org/series/37900/
State : success
== Summary ==
Series 37900v1 drm/i915: Remove lost comment from i915_gem_context
https://patchwork.freedesktop.org/api/1.0/series/37900/revisio
Em Ter, 2018-02-06 às 19:54 +, Patchwork escreveu:
> == Series Details ==
>
> Series: ICL display initialization, selected patches (rev3)
> URL : https://patchwork.freedesktop.org/series/37668/
> State : warning
>
> == Summary ==
>
> Series 37668v3 ICL display initialization, selected patc
1 - 100 of 192 matches
Mail list logo