[Intel-gfx] ✗ Fi.CI.BAT: failure for Add retries for dp dual mode reads (rev3)

== Series Details == Series: Add retries for dp dual mode reads (rev3) URL : https://patchwork.freedesktop.org/series/29155/ State : failure == Summary == Series 29155v3 Add retries for dp dual mode reads https://patchwork.freedesktop.org/api/1.0/series/29155/revisions/3/mbox/ Test gem_ringfi

Re: [Intel-gfx] [PATCH 02/12] drm/i915: Skip fence alignemnt check for the CCS plane

On 17-08-24 22:10:50, Ville Syrjälä wrote: From: Ville Syrjälä The CCS won't have the same stride as the main surface anyway so trying to guard against the fence stride not matching the CCS stride is not sensible. Just skip the fence vs. fb alignment check for the aux plane. Cc: Ben Widawsky

Re: [Intel-gfx] [PATCH 03/12] drm/i915: Switch over to the LLC/eLLC hotspot avoidance hash mode for CCS

On 17-08-24 22:10:51, Ville Syrjälä wrote: From: Ville Syrjälä Use the LLC/eLLC hotspot avoidance mode for CCS on LLC machines. This is reported to give better performance. Not seeing in the diff how this only hits eLLC machines. Am I misreading when this is needed. Testing has indicated t

Re: [Intel-gfx] [PATCH 05/12] drm/i915: Nuke a pointless unreachable()

This was specifically requested by Emil. On 17-08-24 22:10:53, Ville Syrjälä wrote: From: Ville Syrjälä The unreachable() is very much unreachable and the compiler knows that, so there's no point in having it. Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville Syrjäl

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc: Add GuC Load time to debugfs

== Series Details == Series: drm/i915/guc: Add GuC Load time to debugfs URL : https://patchwork.freedesktop.org/series/29324/ State : failure == Summary == CHK include/config/kernel.release CHK include/generated/uapi/linux/version.h CHK include/generated/utsrelease.h CHK

[Intel-gfx] [PATCH] drm/i915/guc: Add GuC Load time to debugfs

Calculate the time that GuC takes to load. This information could be very useful in determining if GuC is taking unreasonably long time to load in a certain platforms. Cc: Oscar Mateo Cc: Michal Wajdeczko Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/i915_debugfs.c | 4 driv

[Intel-gfx] ✗ Fi.CI.IGT: warning for tests/perf_pmu: test i915 RFC PMU

== Series Details == Series: tests/perf_pmu: test i915 RFC PMU URL : https://patchwork.freedesktop.org/series/29313/ State : warning == Summary == Test kms_setmode: Subgroup basic: pass -> FAIL (shard-hsw) fdo#99912 Test vgem_basic: Subgroup unload:

[Intel-gfx] ✓ Fi.CI.BAT: success for tests/perf_pmu: test i915 RFC PMU

== Series Details == Series: tests/perf_pmu: test i915 RFC PMU URL : https://patchwork.freedesktop.org/series/29313/ State : success == Summary == IGT patchset tested on top of latest successful build 29d488034a50cd6fbad792cae61321995f0ab51c aubdump: Log some information about the execbuf cal

Re: [Intel-gfx] [PATCH v3] drm/i915: Deal with upside-down mounted LCD panels

Hi Hans, [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on next-20170824] [cannot apply to v4.13-rc6] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Hans-de-Goede

[Intel-gfx] [RFC i-g-t] tests/perf_pmu: test i915 RFC PMU

i915 RFC PMU: * https://patchwork.freedesktop.org/series/27488/ * https://patchwork.freedesktop.org/series/28842/ Tests: * init: try to initialize all possible metrics exposed in i915 PMU (limit to 0-instance of engines) * invalid_init: verify that i915 PMU correctly error out on invalid initi

Re: [Intel-gfx] [PATCH] drm/i915: Allow null render state batchbuffers bigger than one page

On Thu, Aug 24, 2017 at 3:39 PM, Oscar Mateo wrote: > > > On 08/23/2017 05:01 PM, Rodrigo Vivi wrote: >> >> On Tue, Jul 18, 2017 at 8:15 AM, Oscar Mateo >> wrote: >>> >>> >>> >>> On 07/14/2017 08:08 AM, Chris Wilson wrote: Quoting Oscar Mateo (2017-07-14 15:52:59) > > >

Re: [Intel-gfx] [PATCH v3] drm/i915: Deal with upside-down mounted LCD panels

Hi Hans, [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on next-20170824] [cannot apply to v4.13-rc6] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Hans-de

Re: [Intel-gfx] [PATCH] drm/i915/cnl: WaForceEnableNonCoherent

On 08/23/2017 03:02 PM, Rodrigo Vivi wrote: Must Force Non-Coherent whenever executing a 3D context. This is a workaround for a possible hang in the unlikely event a TLB invalidation occurs during a PSD flush. Set masked bit 4 in 0x7300 during boot. This bug should not be present in HW anym

Re: [Intel-gfx] [PATCH] drm/i915: Allow null render state batchbuffers bigger than one page

On 08/23/2017 05:01 PM, Rodrigo Vivi wrote: On Tue, Jul 18, 2017 at 8:15 AM, Oscar Mateo wrote: On 07/14/2017 08:08 AM, Chris Wilson wrote: Quoting Oscar Mateo (2017-07-14 15:52:59) On 07/13/2017 03:28 PM, Rodrigo Vivi wrote: On Wed, May 3, 2017 at 9:31 AM, Chris Wilson wrote: On Wed

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Fix up the CCS code

== Series Details == Series: drm/i915: Fix up the CCS code URL : https://patchwork.freedesktop.org/series/29308/ State : success == Summary == Test kms_setmode: Subgroup basic: fail -> PASS (shard-hsw) fdo#99912 fdo#99912 https://bugs.freedesktop.org/show_b

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915/bios: child device config refactoring

== Series Details == Series: drm/i915/bios: child device config refactoring URL : https://patchwork.freedesktop.org/series/29307/ State : warning == Summary == Test kms_cursor_legacy: Subgroup nonblocking-modeset-vs-cursor-atomic: pass -> SKIP (shard-hsw) Te

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Fix up the CCS code

== Series Details == Series: drm/i915: Fix up the CCS code URL : https://patchwork.freedesktop.org/series/29308/ State : success == Summary == Series 29308v1 drm/i915: Fix up the CCS code https://patchwork.freedesktop.org/api/1.0/series/29308/revisions/1/mbox/ Test kms_flip: Subgroup

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Make some RPS functions static

Since this on has no functional change and passed ci and is rv-b I merged patch 1 to dinq already Thanks for patch and review. For next time it would be good to send separated patches since patches 1 and 2 have no directly relation... So CI test them individually. On Thu, Aug 24, 2017 at 3:28 A

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/bios: child device config refactoring

== Series Details == Series: drm/i915/bios: child device config refactoring URL : https://patchwork.freedesktop.org/series/29307/ State : success == Summary == Series 29307v1 drm/i915/bios: child device config refactoring https://patchwork.freedesktop.org/api/1.0/series/29307/revisions/1/mbox/

[Intel-gfx] [PATCH 11/12] drm: Check that the plane supports the request format+modifier combo

From: Ville Syrjälä Currently we only check that the plane supports the pixel format of the fb we're about to feed to it. Extend it to check also the modifier, and more specifically that the combination of the format and modifier is supported. Cc: dri-de...@lists.freedesktop.org Cc: Ben Widawsky

[Intel-gfx] [PATCH 12/12] drm/i915: Remove the pipe/plane ID checks from skl_check_ccs_aux_surface()

From: Ville Syrjälä The core now checks that the plane supports the fb's format+modifier combination, so we can drop the related checks from skl_check_ccs_aux_surface(). These checks were specific to SKL/KBL/BXT anyway. Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville

[Intel-gfx] [PATCH 10/12] drm: Fix modifiers_property kernel doc

From: Ville Syrjälä The member is called 'modifiers_property' instead of 'modifiers'. Adjust the kernel docs to match. Cc: dri-de...@lists.freedesktop.org Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville Syrjälä --- include/drm/drm_mode_config.h | 2 +- 1 file chang

[Intel-gfx] [PATCH 07/12] drm/i915: Clean up the sprite modifier checks

From: Ville Syrjälä Split the g4x and snb cases into separate functions to match how we deal with all other platforms. Also sort the switch cases to match the format lists we've declared earlier, to ease comparisons. Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville Sy

[Intel-gfx] [PATCH 08/12] drm/i915: Add CCS capability for sprites

From: Ville Syrjälä Allow sprites to scan out compressed framebuffers. Since different platforms have a different set of planes that support CCS let's add a small helper to determine whether a specific plane supports CCS or not. Currently that information is spread around in many places, and not

[Intel-gfx] [PATCH 09/12] drm/i915: Allow up to 32KB stride on SKL+ "sprites"

From: Ville Syrjälä SKL+ "sprites" no longer have 16KB max stride limit that earlier platforms had. Bump up the limit to 32KB. Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_sprite.c | 3 ++- 1 file changed, 2 insertions(+),

[Intel-gfx] [PATCH 04/12] drm/i915: Add a comment exlaining CCS hsub/vsub

From: Ville Syrjälä Let's document why we claim hsub==8,vsub==16 for CCS even though the memory layout would suggest that we use 16x8 instead. Cc: Daniel Vetter Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 7 ++

[Intel-gfx] [PATCH 02/12] drm/i915: Skip fence alignemnt check for the CCS plane

From: Ville Syrjälä The CCS won't have the same stride as the main surface anyway so trying to guard against the fence stride not matching the CCS stride is not sensible. Just skip the fence vs. fb alignment check for the aux plane. Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-

[Intel-gfx] [PATCH 06/12] drm/i915: Add the missing Y/Yf modifiers for SKL+ sprites

From: Ville Syrjälä Y/Yf somehow dropped out from the SKL+ sprite modifier list. Add them in. Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_sprite.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/i915/

[Intel-gfx] [PATCH 01/12] drm/i915: Treat fb->offsets[] as a raw byte offset instead of a linear offset

From: Ville Syrjälä Userspace wants to treat fb->offsets[] as raw byte offsets into the gem bo. Adjust the kernel code to match. Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 108 +

[Intel-gfx] [PATCH 03/12] drm/i915: Switch over to the LLC/eLLC hotspot avoidance hash mode for CCS

From: Ville Syrjälä Use the LLC/eLLC hotspot avoidance mode for CCS on LLC machines. This is reported to give better performance. Testing has indicated that we don't need to enforce any massive 2 or 4 MiB alignment for all compressed resources even though there are still plenty of stale comments

[Intel-gfx] [PATCH 05/12] drm/i915: Nuke a pointless unreachable()

From: Ville Syrjälä The unreachable() is very much unreachable and the compiler knows that, so there's no point in having it. Cc: Ben Widawsky Cc: Jason Ekstrand Cc: Daniel Stone Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 2 -- 1 file changed, 2 deletions(-) di

[Intel-gfx] [PATCH 00/12] drm/i915: Fix up the CCS code

From: Ville Syrjälä Looks like we ended up with a stale version of my CCS code in dinq. This series contains the remainder in smaller chunks. I also ended up adding a bunch of extra cleanup etc. on top. The most important thing we need to get in is the change to the fb->offsets[] interpretation

[Intel-gfx] [PATCH 09/10] drm/i915/bios: throw away unused DVO_* macros

Apparently meant to be additional DEVICE_PORT_* or DVO_PORT_* macros, but left unused at some point. Remove. No functional changes. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_vbt_defs.h | 5 - 1 file changed, 5 deletions(

[Intel-gfx] [PATCH 10/10] drm/i915/bios: drop the rest of the p_ prefixes from pointers

Not really kernel style, and sticks out like a sore thumb. No functional changes. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_bios.c | 84 +++ 1 file changed, 41 insertions(+), 43 deletions(

[Intel-gfx] [PATCH 08/10] drm/i915/bios: group device type definitions together

No idea why some definitions were defined at a different place from the rest. Move them together. No functional changes. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_vbt_defs.h | 103 +- 1 file c

[Intel-gfx] [PATCH 06/10] drm/i915/bios: throw away struct old_child_dev_config

The old_child_dev_config struct is no longer needed except for its size; replace with a macro. No functional change. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_bios.c | 7 +++ drivers/gpu/drm/i915/intel_vbt_defs.h |

[Intel-gfx] [PATCH 04/10] drm/i915/bios: add legacy contents to common child device config

Add legacy contents to common child device config, in preparation for using a single child device config. Use unions where BDB versions of the config differ. Use the naming from old_child_dev_config for legacy fields. No functional changes. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä

[Intel-gfx] [PATCH 05/10] drm/i915/bios: throw away high level child device union

All the child device config fields, including legacy, are now available in the same struct, so use it for everything. As this change touches plenty of code with "p_child", rename them to "child" while at it. Also do some simple unification and constification where not intrusive. This in the name o

[Intel-gfx] [PATCH 07/10] drm/i915/bios: document child device config dvo_port values a bit better

Move closer to child device config struct while at it. No functional changes. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_vbt_defs.h | 41 ++- 1 file changed, 21 insertions(+), 20 deletions(-)

[Intel-gfx] [PATCH 03/10] drm/i915/bios: remove the raw version of child device config

Convert the only user of the raw field, switching to the recently added struct fields. No functional changes. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_bios.c | 2 +- drivers/gpu/drm/i915/intel_vbt_defs.h | 3 --- 2 file

[Intel-gfx] [PATCH 02/10] drm/i915/bios: document BDB versions of child device config fields

Document everything that was introduced after version 155, which seems to be the baseline for some of the later documentation. No functional changes. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_vbt_defs.h | 16

[Intel-gfx] [PATCH 01/10] drm/i915/bios: amend child device config parameters

Add both some new and some old fields to child device config parameters. Prepare for switching to just one child device config. Use naming from struct old_child_dev_config for common fields. No functional changes. Cc: Animesh Manna Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula

[Intel-gfx] [PATCH 00/10] drm/i915/bios: child device config refactoring

Refactor and clarify the child device config definitions and parsing. Turn the top level union to handle old and new BDB versions inside out; have a common struct with unions for the conflicting parts inside instead. This should make it easier to add new child device config fields in the future.

Re: [Intel-gfx] [PATCH 18/23] drm/i915/selftests: huge page tests

Hi Matthew, [auto build test ERROR on mmotm/master] [cannot apply to drm-intel/for-linux-next linus/master v4.13-rc6 next-20170824] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Matthew-Auld

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [2/2] drm/i915: Create vfuncs for the reset/enable/disable GuC functions (rev3)

== Series Details == Series: series starting with [2/2] drm/i915: Create vfuncs for the reset/enable/disable GuC functions (rev3) URL : https://patchwork.freedesktop.org/series/29251/ State : failure == Summary == CHK include/config/kernel.release CHK include/generated/uapi/linux/

[Intel-gfx] [PATCH] drm/i915: Create vfuncs for the reset/enable/disable GuC functions

Facilitates creating Gen-specific versions of these functions later on. v2: Move everything into guc.interrupts (Michal) Cc: Rodrigo Vivi Cc: Tvrtko Ursulin Cc: Michal Wajdeczko Cc: Daniele Ceraolo Spurio Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_guc_submission.c | 4 ++-- d

Re: [Intel-gfx] [PATCH] drm/i915/dp: Fix the channel equalization failure condition during Link Training

On Thu, Aug 24, 2017 at 03:33:27PM +0300, Ville Syrjälä wrote: > On Thu, Aug 17, 2017 at 08:03:04PM -0700, Manasi Navare wrote: > > In the channel EQ retry loop, we break from the loop in case > > of failure to get link status or failure in clock recovery or > > failure to update link training. In

Re: [Intel-gfx] [PATCH v3 1/2] drm: Add retries for dp dual mode read

On Thu, Aug 24, 2017 at 06:53:43PM +0530, Shashank Sharma wrote: > From the CI builds, its been observed that during a driver > reload/insert, dp dual mode read function sometimes fails to > read from dual mode devices (like LSPCON) over i2c-over-aux > channel. > > This patch: > - adds some delay

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm: rename u32 in __u32 in uapi

== Series Details == Series: drm: rename u32 in __u32 in uapi URL : https://patchwork.freedesktop.org/series/29274/ State : failure == Summary == Series 29274v1 drm: rename u32 in __u32 in uapi https://patchwork.freedesktop.org/api/1.0/series/29274/revisions/1/mbox/ Test gem_exec_flush:

Re: [Intel-gfx] [PATCH] drm: rename u32 in __u32 in uapi

Quoting Lionel Landwerlin (2017-08-24 14:21:38) > All other fields use __ Aye, it is the uapi way. > Cc: Ben Widawsky > Fixes: db1689aa61b ("drm: Create a format/modifier blob") > Signed-off-by: Lionel Landwerlin Reviewed-by: Chris Wilson I checked that these are the only naked u32 in our ua

Re: [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/3] drm/i915: Assert the context is not closed on object-close

Quoting Patchwork (2017-08-22 13:06:37) > == Series Details == > > Series: series starting with [1/3] drm/i915: Assert the context is not closed > on object-close > URL : https://patchwork.freedesktop.org/series/29137/ > State : success > > == Summary == > > Series 29137v1 series starting wit

[Intel-gfx] ✗ Fi.CI.BAT: failure for Add retries for dp dual mode reads (rev3)

== Series Details == Series: Add retries for dp dual mode reads (rev3) URL : https://patchwork.freedesktop.org/series/29155/ State : failure == Summary == Series 29155v3 Add retries for dp dual mode reads https://patchwork.freedesktop.org/api/1.0/series/29155/revisions/3/mbox/ Test gem_exec_f

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Assert the context is not closed on object-close

Quoting Lofstedt, Marta (2017-08-22 12:59:59) > Thanks Chris, > With this series the test pin-pointed in the bug now pass. > > Tested-by: Marta Lofstedt > > > -Original Message- > > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf > > Of Chris Wilson > > Sent: T

[Intel-gfx] [PULL] drm-misc-fixes

Hi Dave, One fix which is cc:stable from Chris this week. The patch fixes a race where the driver could still be tracking a gem object while its handle has already been recycled. drm-misc-fixes-2017-08-24: Core Changes: - Release driver tracking before making the object available again (Chris) Cc

Re: [Intel-gfx] [PATCH v3 1/2] drm: Add retries for dp dual mode read

On Thu, Aug 24, 2017 at 06:53:43PM +0530, Shashank Sharma wrote: > >From the CI builds, its been observed that during a driver > reload/insert, dp dual mode read function sometimes fails to > read from dual mode devices (like LSPCON) over i2c-over-aux > channel. > > This patch: > - adds some delay

[Intel-gfx] ✓ Fi.CI.IGT: success for aubdump: A few improvements

== Series Details == Series: aubdump: A few improvements URL : https://patchwork.freedesktop.org/series/29225/ State : success == Summary == shard-hswtotal:2230 pass:1229 dwarn:0 dfail:0 fail:19 skip:982 time:9603s == Logs == For more details see: https://intel-gfx-ci.01.org/t

[Intel-gfx] ✗ Fi.CI.BAT: failure for lib: Avoid actually throttling from igt_require_gem() (rev2)

== Series Details == Series: lib: Avoid actually throttling from igt_require_gem() (rev2) URL : https://patchwork.freedesktop.org/series/28983/ State : failure == Summary == IGT patchset tested on top of latest successful build 80cc54023e198165eca34450e9cc75c9cffcb072 lib/core: Use igt_info in

[Intel-gfx] [PATCH] drm: rename u32 in __u32 in uapi

All other fields use __ Cc: Ben Widawsky Fixes: db1689aa61b ("drm: Create a format/modifier blob") Signed-off-by: Lionel Landwerlin --- include/uapi/drm/drm_mode.h | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/d

[Intel-gfx] [PATCH v3 1/2] drm: Add retries for dp dual mode read

From the CI builds, its been observed that during a driver reload/insert, dp dual mode read function sometimes fails to read from dual mode devices (like LSPCON) over i2c-over-aux channel. This patch: - adds some delay and few retries, allowing a scope for these devices to settle down and respon

Re: [Intel-gfx] [PATCH igt v2] lib: Avoid actually throttling from igt_require_gem()

On Thu, 2017-08-24 at 13:42 +0100, Chris Wilson wrote: > igt_require_gem() checks whether we can use the i915 fd for > submitting > requests by detecting a wedged driver. It was intended to be used > just > after opening DRIVER_INTEL for a gem test to provide an early skip if > the device was unusa

Re: [Intel-gfx] [PATCH igt v2] lib: Avoid actually throttling from igt_require_gem()

On Thu, Aug 24, 2017 at 01:42:37PM +0100, Chris Wilson wrote: > igt_require_gem() checks whether we can use the i915 fd for submitting > requests by detecting a wedged driver. It was intended to be used just > after opening DRIVER_INTEL for a gem test to provide an early skip if > the device was un

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Create vfuncs for the reset/enable/disable GuC functions

On Wed, Aug 23, 2017 at 04:58:25PM -0700, Oscar Mateo wrote: > Facilitates creating Gen-specific versions of these functions later on. > > Cc: Rodrigo Vivi > Cc: Tvrtko Ursulin > Cc: Michal Wajdeczko > Cc: Daniele Ceraolo Spurio > Signed-off-by: Oscar Mateo > --- > drivers/gpu/drm/i915/i915_

[Intel-gfx] ✓ Fi.CI.BAT: success for aubdump: A few improvements

== Series Details == Series: aubdump: A few improvements URL : https://patchwork.freedesktop.org/series/29225/ State : success == Summary == IGT patchset tested on top of latest successful build 80cc54023e198165eca34450e9cc75c9cffcb072 lib/core: Use igt_info instead of printf with latest DRM

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc : Enable GuC logs even when submission is not enabled

On Wed, Aug 23, 2017 at 03:13:07PM -0700, Sujaritha Sundaresan wrote: > Currently, we only enable GuC logs when enable_guc_submission is set. > But we could be interested in getting GuC logs in other cases as well. > > Cc: Daniele Ceraolo Spurio > Cc: Joonas Lahtinen > Cc: Anusha Srivatsa > Sig

Re: [Intel-gfx] [PATCH 1/2] drm/i915/guc : Removing enable_guc_loading module

On Wed, Aug 23, 2017 at 03:11:31PM -0700, Sujaritha Sundaresan wrote: > Whenever we need i915.enable_guc_submission=1, we also need > enable_guc_loading=1. We also need enable_guc_loading=1 when we want to > verify the HuC, which is every time we have a HuC (but all platforms with HuC > have a G

[Intel-gfx] [PATCH igt v2] lib: Avoid actually throttling from igt_require_gem()

igt_require_gem() checks whether we can use the i915 fd for submitting requests by detecting a wedged driver. It was intended to be used just after opening DRIVER_INTEL for a gem test to provide an early skip if the device was unusable. However, it is also used at the start of library functions lik

Re: [Intel-gfx] [PATCH] drm/i915/dp: Fix the channel equalization failure condition during Link Training

On Thu, Aug 17, 2017 at 08:03:04PM -0700, Manasi Navare wrote: > In the channel EQ retry loop, we break from the loop in case > of failure to get link status or failure in clock recovery or > failure to update link training. In these cases channel_eq_status > is still false even though the retry lo

Re: [Intel-gfx] [PATCH i-g-t] igt/gem_ringfill: Adds ringbuffer full preemption test

Quoting Michał Winiarski (2017-08-24 13:13:30) > > +static void alarm_handler(int sig) > > +{ > > +} > > + > > +static int __execbuf(int fd, struct drm_i915_gem_execbuffer2 *execbuf) > > +{ > > + return ioctl(fd, DRM_IOCTL_I915_GEM_EXECBUFFER2, execbuf); > > +} > > + > > So... __gem_execbuf?

Re: [Intel-gfx] [PATCH v2 1/2] drm: Add retries for dp dual mode read

Regards Shashank On 8/24/2017 5:49 PM, Imre Deak wrote: On Thu, Aug 24, 2017 at 05:40:32PM +0530, Sharma, Shashank wrote: Regards Shashank On 8/24/2017 5:19 PM, Imre Deak wrote: On Wed, Aug 23, 2017 at 06:12:51PM +0530, Shashank Sharma wrote: From the CI builds, its been observed that

Re: [Intel-gfx] [PATCH v2 1/2] drm: Add retries for dp dual mode read

On Thu, Aug 24, 2017 at 05:40:32PM +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 8/24/2017 5:19 PM, Imre Deak wrote: > > On Wed, Aug 23, 2017 at 06:12:51PM +0530, Shashank Sharma wrote: > > > From the CI builds, its been observed that during a driver > > > reload/insert, dp dua

Re: [Intel-gfx] [PATCH i-g-t] igt/gem_ringfill: Adds ringbuffer full preemption test

On Wed, Aug 23, 2017 at 05:11:23PM -0700, Antonio Argenziano wrote: > The testcase added here, stimulates this scenario where a high priority > request is sent while another process keeps submitting requests and > filling its ringbuffer. s/stimulates/simulates You're no longer changing igt/gem_ri

Re: [Intel-gfx] [PATCH v2 1/2] drm: Add retries for dp dual mode read

Regards Shashank On 8/24/2017 5:19 PM, Imre Deak wrote: On Wed, Aug 23, 2017 at 06:12:51PM +0530, Shashank Sharma wrote: From the CI builds, its been observed that during a driver reload/insert, dp dual mode read function sometimes fails to read from dual mode devices (like LSPCON) over i2c-

Re: [Intel-gfx] [PATCH 01/23] mm/shmem: introduce shmem_file_setup_with_mnt

On 23 August 2017 at 23:34, Andrew Morton wrote: > On Wed, 23 Aug 2017 12:31:28 +0300 Joonas Lahtinen > wrote: > >> This patch has been floating around for a while now Acked and without >> further comments. It is blocking us from merging huge page support to >> drm/i915. >> >> Would you mind mer

Re: [Intel-gfx] [PATCH v2 1/2] drm: Add retries for dp dual mode read

On Wed, Aug 23, 2017 at 06:12:51PM +0530, Shashank Sharma wrote: > From the CI builds, its been observed that during a driver > reload/insert, dp dual mode read function sometimes fails to > read from dual mode devices (like LSPCON) over i2c-over-aux > channel. > > This patch: > - adds some delay

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for igt/gem_ringfill: Adds ringbuffer full preemption test

Quoting Patchwork (2017-08-24 01:29:29) > == Series Details == > > Series: igt/gem_ringfill: Adds ringbuffer full preemption test > URL : https://patchwork.freedesktop.org/series/29252/ > State : failure What I had in mind was From f009cc1a2c6564d4a1683dccaf9e9a7d192c1f61 Mon Sep 17 00:00:00

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Make some RPS functions static

Reviewed-by: Sagar Arun Kamble -Original Message- From: Mateo Lozano, Oscar Sent: Thursday, August 24, 2017 5:28 AM To: intel-gfx@lists.freedesktop.org Cc: Mateo Lozano, Oscar ; Kamble, Sagar A ; Vivi, Rodrigo Subject: [PATCH 1/2] drm/i915: Make some RPS functions static They are not

Re: [Intel-gfx] [PATCH i-g-t v3] pm_rps: Changes in waitboost scenario

Quoting Katarzyna Dec (2017-08-22 13:40:53) > CI is observing sporadical failures in pm_rps subtests. > There are a couple of reasons. One of them is the fact that > on gen6, gen7 and gen7.5, max frequency (as in the HW limit) > is not set to RP0, but the value obtaind from PCODE (which > may be di

Re: [Intel-gfx] ✓ Fi.CI.BAT: success for lib/core: Use igt_info instead of printf

On Wed, Aug 23, 2017 at 01:50:12PM +, Patchwork wrote: > == Series Details == > > Series: lib/core: Use igt_info instead of printf > URL : https://patchwork.freedesktop.org/series/29210/ > State : success For anyone else wondering, the difference with this series can be seen comparing e.g.

[Intel-gfx] [PULL] drm-intel-fixes

Hi Dave, drm/i915 fixes for v4.13-rc7. A mixed bag of fixes, with black screen fixes from me and Andy, and a couple of GVT fixes. In retrospect could have gone without the CNL specific fix, but it's trivial. I expect this to be the last round of drm/i915 fixes for v4.13 (fingers crossed). Rodrigo

Re: [Intel-gfx] [PATCH 1/2] mm: Track actual nr_scanned during shrink_slab()

On 08/24/2017 07:11 AM, Minchan Kim wrote: > Hello Chris, > > On Tue, Aug 22, 2017 at 02:53:24PM +0100, Chris Wilson wrote: >> Some shrinkers may only be able to free a bunch of objects at a time, and >> so free more than the requested nr_to_scan in one pass. Can such shrinkers reflect that in th

Re: [Intel-gfx] [maintainer-tools PATCH] dim: Properly handle series on apply_branch

On Wed, 23 Aug 2017, Rodrigo Vivi wrote: > So far we could use *dim* to apply a whole series > in a mbox, but only the very last patch was receiving > all the checks and patchwork link. > > So this patch remove this limitation by using git mailsplit > to split the mbox and than use git am and chec