Commit 7c8f8a70 authored by Rodrigo Vivi's avatar Rodrigo Vivi Committed by Daniel Vetter

drm/i915: Force PSR exit by inactivating it.

The perfect solution for psr_exit is the hardware tracking the changes and
doing the psr exit by itself. This scenario works for HSW and BDW with some
environments like Gnome and Wayland.

However there are many other scenarios that this isn't true. Mainly one right
now is KDE users on HSW and BDW with PSR on. User would miss many screen
updates. For instances any key typed could be seen only when mouse cursor is
moved. So this patch introduces the ability of trigger PSR exit on kernel side
on some common cases that.

Most of the cases are coverred by psr_exit at set_domain. The remaining cases
are coverred by triggering it at set_domain, busy_ioctl, sw_finish and
mark_busy.

The downside here might be reducing the residency time on the cases this
already work very wall like Gnome environment. But so far let's get focused
on fixinge issues sio PSR couild be used for everybody and we could even
get it enabled by default. Later we can add some alternatives to choose the
level of PSR efficiency over boot flag of even over crtc property.

v2: remove exit from connector_dpms. Daniel pointed this is the wrong way and
also this isn't needed for BDW and HSW anyway.

Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: default avatarVijay Purushothaman <vijay.a.purushothaman@intel.com>
Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 62942ed7
...@@ -637,6 +637,9 @@ struct i915_psr { ...@@ -637,6 +637,9 @@ struct i915_psr {
bool sink_support; bool sink_support;
bool source_ok; bool source_ok;
bool setup_done; bool setup_done;
bool enabled;
bool active;
struct delayed_work work;
}; };
enum intel_pch { enum intel_pch {
......
...@@ -1395,6 +1395,8 @@ i915_gem_set_domain_ioctl(struct drm_device *dev, void *data, ...@@ -1395,6 +1395,8 @@ i915_gem_set_domain_ioctl(struct drm_device *dev, void *data,
goto unlock; goto unlock;
} }
intel_edp_psr_exit(dev, true);
/* Try to flush the object off the GPU without holding the lock. /* Try to flush the object off the GPU without holding the lock.
* We will repeat the flush holding the lock in the normal manner * We will repeat the flush holding the lock in the normal manner
* to catch cases where we are gazumped. * to catch cases where we are gazumped.
...@@ -1440,6 +1442,8 @@ i915_gem_sw_finish_ioctl(struct drm_device *dev, void *data, ...@@ -1440,6 +1442,8 @@ i915_gem_sw_finish_ioctl(struct drm_device *dev, void *data,
if (ret) if (ret)
return ret; return ret;
intel_edp_psr_exit(dev, true);
obj = to_intel_bo(drm_gem_object_lookup(dev, file, args->handle)); obj = to_intel_bo(drm_gem_object_lookup(dev, file, args->handle));
if (&obj->base == NULL) { if (&obj->base == NULL) {
ret = -ENOENT; ret = -ENOENT;
...@@ -4236,6 +4240,8 @@ i915_gem_busy_ioctl(struct drm_device *dev, void *data, ...@@ -4236,6 +4240,8 @@ i915_gem_busy_ioctl(struct drm_device *dev, void *data,
if (ret) if (ret)
return ret; return ret;
intel_edp_psr_exit(dev, true);
obj = to_intel_bo(drm_gem_object_lookup(dev, file, args->handle)); obj = to_intel_bo(drm_gem_object_lookup(dev, file, args->handle));
if (&obj->base == NULL) { if (&obj->base == NULL) {
ret = -ENOENT; ret = -ENOENT;
......
...@@ -8818,12 +8818,15 @@ void intel_mark_idle(struct drm_device *dev) ...@@ -8818,12 +8818,15 @@ void intel_mark_idle(struct drm_device *dev)
intel_runtime_pm_put(dev_priv); intel_runtime_pm_put(dev_priv);
} }
void intel_mark_fb_busy(struct drm_i915_gem_object *obj, void intel_mark_fb_busy(struct drm_i915_gem_object *obj,
struct intel_engine_cs *ring) struct intel_engine_cs *ring)
{ {
struct drm_device *dev = obj->base.dev; struct drm_device *dev = obj->base.dev;
struct drm_crtc *crtc; struct drm_crtc *crtc;
intel_edp_psr_exit(dev, true);
if (!i915.powersave) if (!i915.powersave)
return; return;
...@@ -9290,6 +9293,9 @@ static int intel_crtc_page_flip(struct drm_crtc *crtc, ...@@ -9290,6 +9293,9 @@ static int intel_crtc_page_flip(struct drm_crtc *crtc,
if (work == NULL) if (work == NULL)
return -ENOMEM; return -ENOMEM;
/* Exit PSR early in page flip */
intel_edp_psr_exit(dev, true);
work->event = event; work->event = event;
work->crtc = crtc; work->crtc = crtc;
work->old_fb_obj = to_intel_framebuffer(old_fb)->obj; work->old_fb_obj = to_intel_framebuffer(old_fb)->obj;
...@@ -11581,6 +11587,8 @@ static void intel_setup_outputs(struct drm_device *dev) ...@@ -11581,6 +11587,8 @@ static void intel_setup_outputs(struct drm_device *dev)
if (SUPPORTS_TV(dev)) if (SUPPORTS_TV(dev))
intel_tv_init(dev); intel_tv_init(dev);
intel_edp_psr_init(dev);
list_for_each_entry(encoder, &dev->mode_config.encoder_list, base.head) { list_for_each_entry(encoder, &dev->mode_config.encoder_list, base.head) {
encoder->base.possible_crtcs = encoder->crtc_mask; encoder->base.possible_crtcs = encoder->crtc_mask;
encoder->base.possible_clones = encoder->base.possible_clones =
......
...@@ -1818,10 +1818,11 @@ static bool intel_edp_psr_match_conditions(struct intel_dp *intel_dp) ...@@ -1818,10 +1818,11 @@ static bool intel_edp_psr_match_conditions(struct intel_dp *intel_dp)
static void intel_edp_psr_do_enable(struct intel_dp *intel_dp) static void intel_edp_psr_do_enable(struct intel_dp *intel_dp)
{ {
struct drm_device *dev = intel_dp_to_dev(intel_dp); struct intel_digital_port *intel_dig_port = dp_to_dig_port(intel_dp);
struct drm_device *dev = intel_dig_port->base.base.dev;
struct drm_i915_private *dev_priv = dev->dev_private;
if (!intel_edp_psr_match_conditions(intel_dp) || if (intel_edp_is_psr_enabled(dev))
intel_edp_is_psr_enabled(dev))
return; return;
/* Enable PSR on the panel */ /* Enable PSR on the panel */
...@@ -1829,6 +1830,9 @@ static void intel_edp_psr_do_enable(struct intel_dp *intel_dp) ...@@ -1829,6 +1830,9 @@ static void intel_edp_psr_do_enable(struct intel_dp *intel_dp)
/* Enable PSR on the host */ /* Enable PSR on the host */
intel_edp_psr_enable_source(intel_dp); intel_edp_psr_enable_source(intel_dp);
dev_priv->psr.enabled = true;
dev_priv->psr.active = true;
} }
void intel_edp_psr_enable(struct intel_dp *intel_dp) void intel_edp_psr_enable(struct intel_dp *intel_dp)
...@@ -1848,8 +1852,7 @@ void intel_edp_psr_enable(struct intel_dp *intel_dp) ...@@ -1848,8 +1852,7 @@ void intel_edp_psr_enable(struct intel_dp *intel_dp)
/* Setup PSR once */ /* Setup PSR once */
intel_edp_psr_setup(intel_dp); intel_edp_psr_setup(intel_dp);
if (intel_edp_psr_match_conditions(intel_dp) && if (intel_edp_psr_match_conditions(intel_dp))
!intel_edp_is_psr_enabled(dev))
intel_edp_psr_do_enable(intel_dp); intel_edp_psr_do_enable(intel_dp);
} }
...@@ -1858,7 +1861,7 @@ void intel_edp_psr_disable(struct intel_dp *intel_dp) ...@@ -1858,7 +1861,7 @@ void intel_edp_psr_disable(struct intel_dp *intel_dp)
struct drm_device *dev = intel_dp_to_dev(intel_dp); struct drm_device *dev = intel_dp_to_dev(intel_dp);
struct drm_i915_private *dev_priv = dev->dev_private; struct drm_i915_private *dev_priv = dev->dev_private;
if (!intel_edp_is_psr_enabled(dev)) if (!dev_priv->psr.enabled)
return; return;
I915_WRITE(EDP_PSR_CTL(dev), I915_WRITE(EDP_PSR_CTL(dev),
...@@ -1868,13 +1871,13 @@ void intel_edp_psr_disable(struct intel_dp *intel_dp) ...@@ -1868,13 +1871,13 @@ void intel_edp_psr_disable(struct intel_dp *intel_dp)
if (_wait_for((I915_READ(EDP_PSR_STATUS_CTL(dev)) & if (_wait_for((I915_READ(EDP_PSR_STATUS_CTL(dev)) &
EDP_PSR_STATUS_STATE_MASK) == 0, 2000, 10)) EDP_PSR_STATUS_STATE_MASK) == 0, 2000, 10))
DRM_ERROR("Timed out waiting for PSR Idle State\n"); DRM_ERROR("Timed out waiting for PSR Idle State\n");
dev_priv->psr.enabled = false;
} }
void intel_edp_psr_update(struct drm_device *dev) void intel_edp_psr_update(struct drm_device *dev)
{ {
struct drm_i915_private *dev_priv = dev->dev_private; struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_encoder *encoder;
struct intel_dp *intel_dp = NULL;
if (!HAS_PSR(dev)) if (!HAS_PSR(dev))
return; return;
...@@ -1882,6 +1885,17 @@ void intel_edp_psr_update(struct drm_device *dev) ...@@ -1882,6 +1885,17 @@ void intel_edp_psr_update(struct drm_device *dev)
if (!dev_priv->psr.setup_done) if (!dev_priv->psr.setup_done)
return; return;
intel_edp_psr_exit(dev, true);
}
void intel_edp_psr_work(struct work_struct *work)
{
struct drm_i915_private *dev_priv =
container_of(work, typeof(*dev_priv), psr.work.work);
struct drm_device *dev = dev_priv->dev;
struct intel_encoder *encoder;
struct intel_dp *intel_dp = NULL;
list_for_each_entry(encoder, &dev->mode_config.encoder_list, base.head) list_for_each_entry(encoder, &dev->mode_config.encoder_list, base.head)
if (encoder->type == INTEL_OUTPUT_EDP) { if (encoder->type == INTEL_OUTPUT_EDP) {
intel_dp = enc_to_intel_dp(&encoder->base); intel_dp = enc_to_intel_dp(&encoder->base);
...@@ -1889,9 +1903,66 @@ void intel_edp_psr_update(struct drm_device *dev) ...@@ -1889,9 +1903,66 @@ void intel_edp_psr_update(struct drm_device *dev)
if (!intel_edp_psr_match_conditions(intel_dp)) if (!intel_edp_psr_match_conditions(intel_dp))
intel_edp_psr_disable(intel_dp); intel_edp_psr_disable(intel_dp);
else else
if (!intel_edp_is_psr_enabled(dev)) intel_edp_psr_do_enable(intel_dp);
intel_edp_psr_do_enable(intel_dp); }
}
void intel_edp_psr_inactivate(struct drm_device *dev)
{
struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_connector *connector;
struct intel_encoder *encoder;
struct intel_crtc *intel_crtc;
struct intel_dp *intel_dp = NULL;
list_for_each_entry(connector, &dev->mode_config.connector_list,
base.head) {
if (connector->base.dpms != DRM_MODE_DPMS_ON)
continue;
encoder = to_intel_encoder(connector->base.encoder);
if (encoder->type == INTEL_OUTPUT_EDP) {
intel_dp = enc_to_intel_dp(&encoder->base);
intel_crtc = to_intel_crtc(encoder->base.crtc);
dev_priv->psr.active = false;
I915_WRITE(EDP_PSR_CTL(dev), I915_READ(EDP_PSR_CTL(dev))
& ~EDP_PSR_ENABLE);
} }
}
}
void intel_edp_psr_exit(struct drm_device *dev, bool schedule_back)
{
struct drm_i915_private *dev_priv = dev->dev_private;
if (!HAS_PSR(dev))
return;
if (!dev_priv->psr.setup_done)
return;
cancel_delayed_work_sync(&dev_priv->psr.work);
if (dev_priv->psr.active)
intel_edp_psr_inactivate(dev);
if (schedule_back)
schedule_delayed_work(&dev_priv->psr.work,
msecs_to_jiffies(100));
}
void intel_edp_psr_init(struct drm_device *dev)
{
struct drm_i915_private *dev_priv = dev->dev_private;
if (!HAS_PSR(dev))
return;
INIT_DELAYED_WORK(&dev_priv->psr.work, intel_edp_psr_work);
} }
static void intel_disable_dp(struct intel_encoder *encoder) static void intel_disable_dp(struct intel_encoder *encoder)
......
...@@ -830,6 +830,9 @@ void intel_edp_psr_enable(struct intel_dp *intel_dp); ...@@ -830,6 +830,9 @@ void intel_edp_psr_enable(struct intel_dp *intel_dp);
void intel_edp_psr_disable(struct intel_dp *intel_dp); void intel_edp_psr_disable(struct intel_dp *intel_dp);
void intel_edp_psr_update(struct drm_device *dev); void intel_edp_psr_update(struct drm_device *dev);
void intel_dp_set_drrs_state(struct drm_device *dev, int refresh_rate); void intel_dp_set_drrs_state(struct drm_device *dev, int refresh_rate);
void intel_edp_psr_exit(struct drm_device *dev, bool schedule_back);
void intel_edp_psr_init(struct drm_device *dev);
/* intel_dsi.c */ /* intel_dsi.c */
void intel_dsi_init(struct drm_device *dev); void intel_dsi_init(struct drm_device *dev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment