[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Reserve powerctx for chv from the stolen allocator

2017-11-04 Thread Patchwork
== Series Details == Series: drm/i915: Reserve powerctx for chv from the stolen allocator URL : https://patchwork.freedesktop.org/series/33192/ State : failure == Summary == Series 33192v1 drm/i915: Reserve powerctx for chv from the stolen allocator https://patchwork.freedesktop.org/api/1.0/se

[Intel-gfx] [PATCH] drm/i915: Reserve powerctx for chv from the stolen allocator

2017-11-04 Thread Chris Wilson
Ensure that we do not overwrite the cherryview power context by reserving its range in the stolen allocator; exactly like how we handle the same reservation for valleyview. Signed-off-by: Chris Wilson Cc: Mika Kuoppala --- drivers/gpu/drm/i915/i915_drv.h | 3 +- drivers/gpu/drm/i915/intel_pm.c

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [CI,1/7] drm/i915: Define an engine class enum for the uABI

2017-11-04 Thread Patchwork
== Series Details == Series: series starting with [CI,1/7] drm/i915: Define an engine class enum for the uABI URL : https://patchwork.freedesktop.org/series/33183/ State : success == Summary == Test kms_setmode: Subgroup basic: pass -> FAIL (shard-hsw) fdo#

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,1/7] drm/i915: Define an engine class enum for the uABI

2017-11-04 Thread Patchwork
== Series Details == Series: series starting with [CI,1/7] drm/i915: Define an engine class enum for the uABI URL : https://patchwork.freedesktop.org/series/33183/ State : success == Summary == Series 33183v1 series starting with [CI,1/7] drm/i915: Define an engine class enum for the uABI ht

[Intel-gfx] [CI 6/7] drm/i915: Record the default hw state after reset upon load

2017-11-04 Thread Chris Wilson
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, ensuring that each starts wit

[Intel-gfx] [CI 7/7] drm/i915: Report whether we have true context isolation

2017-11-04 Thread Chris Wilson
Let userspace know if they can trust that new contexts are created using HW default values; and avoid inheriting state from existing contexts. Note: I intend to squash this into the bugfix once we agree on the uabi. Signed-off-by: Chris Wilson Cc: Mika Kuoppala Cc: Joonas Lahtinen Cc: Tvrtko U

[Intel-gfx] [CI 1/7] drm/i915: Define an engine class enum for the uABI

2017-11-04 Thread Chris Wilson
From: Tvrtko Ursulin We want to be able to report back to userspace details about an engine's class, and in return for userspace to be able to request actions regarding certain classes of engines. To isolate the uABI from any variations between hw generations, we define an abstract class for the

[Intel-gfx] [CI 5/7] drm/i915: Mark the context state as dirty/written

2017-11-04 Thread Chris Wilson
In the next few patches, we will want to both copy out of the context image and write a valid image into a new context. To be completely safe, we should then couple in our domain tracking to ensure that we don't have any issues with stale data remaining in unwanted cachelines. Historically, we omi

[Intel-gfx] [CI 2/7] drm/i915: Force the switch to the i915->kernel_context

2017-11-04 Thread Chris Wilson
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 explicit in our comparison. For

[Intel-gfx] [CI 4/7] drm/i915: Inline intel_modeset_gem_init()

2017-11-04 Thread Chris Wilson
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-off-by: Chris Wilson Reviewed-by: Joonas Lahtinen Reviewed-by: Mi

[Intel-gfx] [CI 3/7] drm/i915: Move GT powersaving init to i915_gem_init()

2017-11-04 Thread Chris Wilson
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 i915_gem_init(). Signed-off-by: Chris Wilson Cc

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [CI,01/10] drm/i915: Define an engine class enum for the uABI

2017-11-04 Thread Patchwork
== Series Details == Series: series starting with [CI,01/10] drm/i915: Define an engine class enum for the uABI URL : https://patchwork.freedesktop.org/series/33181/ State : failure == Summary == Series 33181v1 series starting with [CI,01/10] drm/i915: Define an engine class enum for the uAB

[Intel-gfx] [CI 07/10] drm/i915: Report whether we have true context isolation

2017-11-04 Thread Chris Wilson
Let userspace know if they can trust that new contexts are created using HW default values; and avoid inheriting state from existing contexts. Note: I intend to squash this into the bugfix once we agree on the uabi. Signed-off-by: Chris Wilson Cc: Mika Kuoppala Cc: Joonas Lahtinen Cc: Tvrtko U

[Intel-gfx] [CI 04/10] drm/i915: Inline intel_modeset_gem_init()

2017-11-04 Thread Chris Wilson
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-off-by: Chris Wilson Reviewed-by: Joonas Lahtinen Reviewed-by: Mi

[Intel-gfx] [CI 05/10] drm/i915: Mark the context state as dirty/written

2017-11-04 Thread Chris Wilson
In the next few patches, we will want to both copy out of the context image and write a valid image into a new context. To be completely safe, we should then couple in our domain tracking to ensure that we don't have any issues with stale data remaining in unwanted cachelines. Historically, we omi

[Intel-gfx] [CI 08/10] drm/i915: Remove redundant intel_autoenable_gt_powersave()

2017-11-04 Thread Chris Wilson
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 loaded a context and initialised basic state). We used to defer t

[Intel-gfx] [CI 01/10] drm/i915: Define an engine class enum for the uABI

2017-11-04 Thread Chris Wilson
From: Tvrtko Ursulin We want to be able to report back to userspace details about an engine's class, and in return for userspace to be able to request actions regarding certain classes of engines. To isolate the uABI from any variations between hw generations, we define an abstract class for the

[Intel-gfx] [CI 09/10] drm/i915: Always sanitize GT wakeref before restarting engines

2017-11-04 Thread Chris Wilson
The first request submitted to an engine will take the prolonged GT wakeref. If we discard that wakeref to issue a reset/suspend/etc, then before restarting the engines, reacquire the GT's wakeref. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_drv.c | 3 +- drivers/gpu/drm/

[Intel-gfx] [CI 02/10] drm/i915: Force the switch to the i915->kernel_context

2017-11-04 Thread Chris Wilson
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 explicit in our comparison. For

[Intel-gfx] [CI 10/10] drm/i915: Stop caching the "golden" renderstate

2017-11-04 Thread Chris Wilson
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 Reviewed-by: Joonas Lahtinen --- drivers/gpu/drm/i915/i915_drv.h

[Intel-gfx] [CI 03/10] drm/i915: Move GT powersaving init to i915_gem_init()

2017-11-04 Thread Chris Wilson
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 i915_gem_init(). Signed-off-by: Chris Wilson Cc

[Intel-gfx] [CI 06/10] drm/i915: Record the default hw state after reset upon load

2017-11-04 Thread Chris Wilson
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, ensuring that each starts wit

[Intel-gfx] ✗ Fi.CI.BAT: warning for drm/fbdev: Panel orientation connector property support (rev3)

2017-11-04 Thread Patchwork
== Series Details == Series: drm/fbdev: Panel orientation connector property support (rev3) URL : https://patchwork.freedesktop.org/series/32447/ State : warning == Summary == Series 32447v3 drm/fbdev: Panel orientation connector property support https://patchwork.freedesktop.org/api/1.0/serie

[Intel-gfx] [PATCH v5 0/7] drm/fbdev: Panel orientation connector property support

2017-11-04 Thread Hans de Goede
Hi All, Here is v5 of my series to add a "panel orientation" property to the drm-connector for the LCD panel to let userspace know about LCD panels which are not mounted upright, as well as detecting upside-down panels without needing quirks (like we do for 90 degree rotated screens). New in v5:

[Intel-gfx] [PATCH v5 6/7] efifb: Set info->fbcon_rotate_hint based on drm_get_panel_orientation_quirk

2017-11-04 Thread Hans de Goede
On some hardware the LCD panel is not mounted upright in the casing, but rotated by 90 degrees. In this case we want the console to automatically be rotated to compensate. The drm subsys has a quirk table for this, use the drm_get_panel_orientation_quirk function to get the panel orientation and s

[Intel-gfx] [PATCH v5 3/7] drm: Add support for a panel-orientation connector property

2017-11-04 Thread Hans de Goede
On some devices the LCD panel is mounted in the casing in such a way that the up/top side of the panel does not match with the top side of the device (e.g. it is mounted upside-down). This commit adds the necessary infra for lcd-panel drm_connector-s to have a "panel orientation" property to commu

[Intel-gfx] [PATCH v5 4/7] drm/fb-helper: Apply panel orientation connector prop to the primary plane

2017-11-04 Thread Hans de Goede
Apply the "panel orientation" drm connector prop to the primary plane so that fbcon and fbdev using userspace programs display the right way up. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=94894 Signed-off-by: Hans de Goede --- Changes in v2: -New patch in v2 of this patch-set Changes in

[Intel-gfx] [PATCH v5 2/7] drm: Add panel orientation quirks

2017-11-04 Thread Hans de Goede
Some x86 clamshell design devices use portrait tablet screens and a display engine which cannot rotate in hardware, so the firmware just leaves things as is and we cannot figure out that the display is oriented non upright from the hardware. So at least on x86, we need a quirk table for this. This

[Intel-gfx] [PATCH v5 1/7] fbcon: Add fbcon_rotate_hint to struct fb_info

2017-11-04 Thread Hans de Goede
On some hardware the LCD panel is not mounted upright in the casing, but upside-down or rotated 90 degrees. In this case we want the console to automatically be rotated to compensate. The fbdev-driver may know about the need to rotate. Add a new fbcon_rotate_hint field to struct fb_info, which get

[Intel-gfx] [PATCH v5 5/7] drm/i915: Add "panel orientation" property to the panel connector

2017-11-04 Thread Hans de Goede
Ideally we could use the VBT for this, that would be simple, in intel_dsi_init() check dev_priv->vbt.dsi.config->rotation, set connector->display_info.panel_orientation accordingly and call drm_connector_init_panel_orientation_property(), done. Unfortunately vbt.dsi.config->rotation is always 0 ev

[Intel-gfx] [PATCH v5 7/7] fbcon: Remove dmi quirk table

2017-11-04 Thread Hans de Goede
This is now all handled in the drivers and communicated through fb_info.fbcon_rotate_hint. Signed-off-by: Hans de Goede --- drivers/video/fbdev/core/Makefile | 3 - drivers/video/fbdev/core/fbcon.c| 4 +- drivers/video/fbdev/core/fbcon.h| 6 -- drivers/vid

Re: [Intel-gfx] [PATCH v3 6/7] efifb: Set info->fbcon_rotate_hint based on drm_get_panel_orientation_quirk

2017-11-04 Thread Hans de Goede
Hi, On 30-10-17 10:53, Daniel Vetter wrote: On Mon, Oct 23, 2017 at 09:14:24AM +0200, Hans de Goede wrote: On some hardware the LCD panel is not mounted upright in the casing, but rotated by 90 degrees. In this case we want the console to automatically be rotated to compensate. The drm subsys

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [v4,01/10] drm/i915: Define an engine class enum for the uABI

2017-11-04 Thread Patchwork
== Series Details == Series: series starting with [v4,01/10] drm/i915: Define an engine class enum for the uABI URL : https://patchwork.freedesktop.org/series/33173/ State : failure == Summary == Test kms_pipe_crc_basic: Subgroup hang-read-crc-pipe-A: pass -> DME

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v4,01/10] drm/i915: Define an engine class enum for the uABI

2017-11-04 Thread Patchwork
== Series Details == Series: series starting with [v4,01/10] drm/i915: Define an engine class enum for the uABI URL : https://patchwork.freedesktop.org/series/33173/ State : success == Summary == Series 33173v1 series starting with [v4,01/10] drm/i915: Define an engine class enum for the uAB

[Intel-gfx] [PATCH v4 02/10] drm/i915: Force the switch to the i915->kernel_context

2017-11-04 Thread Chris Wilson
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 explicit in our comparison. For

[Intel-gfx] [PATCH v4 05/10] drm/i915: Inline intel_modeset_gem_init()

2017-11-04 Thread Chris Wilson
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-off-by: Chris Wilson Reviewed-by: Joonas Lahtinen Reviewed-by: Mi

[Intel-gfx] [PATCH v4 07/10] drm/i915: Record the default hw state after reset upon load

2017-11-04 Thread Chris Wilson
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, ensuring that each starts wit

[Intel-gfx] [PATCH v4 09/10] drm/i915: Remove redundant intel_autoenable_gt_powersave()

2017-11-04 Thread Chris Wilson
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 loaded a context and initialised basic state). We used to defer t

[Intel-gfx] [PATCH v4 08/10] drm/i915: Report whether we have true context isolation

2017-11-04 Thread Chris Wilson
Let userspace know if they can trust that new contexts are created using HW default values; and avoid inheriting state from existing contexts. Note: I intend to squash this into the bugfix once we agree on the uabi. Signed-off-by: Chris Wilson Cc: Mika Kuoppala Cc: Joonas Lahtinen Cc: Tvrtko U

[Intel-gfx] [PATCH v4 04/10] drm/i915: Always sanitize GT wakeref before restarting engines

2017-11-04 Thread Chris Wilson
The first request submitted to an engine will take the prolonged GT wakeref. If we discard that wakeref to issue a reset/suspend/etc, then before restarting the engines, reacquire the GT's wakeref. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_drv.c | 1 - drivers/gpu/drm/i915/i915_

[Intel-gfx] [PATCH v4 10/10] drm/i915: Stop caching the "golden" renderstate

2017-11-04 Thread Chris Wilson
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 Reviewed-by: Joonas Lahtinen --- drivers/gpu/drm/i915/i915_drv.h

[Intel-gfx] [PATCH v4 06/10] drm/i915: Mark the context state as dirty/written

2017-11-04 Thread Chris Wilson
In the next few patches, we will want to both copy out of the context image and write a valid image into a new context. To be completely safe, we should then couple in our domain tracking to ensure that we don't have any issues with stale data remaining in unwanted cachelines. Historically, we omi

[Intel-gfx] [PATCH v4 03/10] drm/i915: Move GT powersaving init to i915_gem_init()

2017-11-04 Thread Chris Wilson
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 i915_gem_init(). Signed-off-by: Chris Wilson Cc

[Intel-gfx] [PATCH v4 01/10] drm/i915: Define an engine class enum for the uABI

2017-11-04 Thread Chris Wilson
From: Tvrtko Ursulin We want to be able to report back to userspace details about an engine's class, and in return for userspace to be able to request actions regarding certain classes of engines. To isolate the uABI from any variations between hw generations, we define an abstract class for the