[PATCH 13/15] drm/i915: Guard debugfs against invalid access when display is disabled
José Roberto de Souza
jose.souza at intel.com
Sat Nov 10 02:19:03 UTC 2018
Without this checks in this debugfs, it would try access memory and
resorces from display causing the driver to crash.
Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
---
drivers/gpu/drm/i915/i915_debugfs.c | 56 ++++++++++++++++++++++++-----
1 file changed, 47 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index 670db5073d70..f9b68bac82dd 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -1636,6 +1636,9 @@ static int i915_frontbuffer_tracking(struct seq_file *m, void *unused)
{
struct drm_i915_private *dev_priv = node_to_i915(m->private);
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
seq_printf(m, "FB tracking busy bits: 0x%08x\n",
dev_priv->fb_tracking.busy_bits);
@@ -1650,7 +1653,7 @@ static int i915_fbc_status(struct seq_file *m, void *unused)
struct drm_i915_private *dev_priv = node_to_i915(m->private);
struct intel_fbc *fbc = &dev_priv->fbc;
- if (!HAS_FBC(dev_priv))
+ if (!HAS_FBC(dev_priv) || !HAS_DISPLAY(dev_priv))
return -ENODEV;
intel_runtime_pm_get(dev_priv);
@@ -1726,7 +1729,7 @@ static int i915_ips_status(struct seq_file *m, void *unused)
{
struct drm_i915_private *dev_priv = node_to_i915(m->private);
- if (!HAS_IPS(dev_priv))
+ if (!HAS_IPS(dev_priv) || !HAS_DISPLAY(dev_priv))
return -ENODEV;
intel_runtime_pm_get(dev_priv);
@@ -1753,6 +1756,9 @@ static int i915_sr_status(struct seq_file *m, void *unused)
struct drm_i915_private *dev_priv = node_to_i915(m->private);
bool sr_enabled = false;
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
intel_runtime_pm_get(dev_priv);
intel_display_power_get(dev_priv, POWER_DOMAIN_INIT);
@@ -1891,6 +1897,9 @@ static int i915_gem_framebuffer_info(struct seq_file *m, void *data)
struct drm_framebuffer *drm_fb;
int ret;
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
ret = mutex_lock_interruptible(&dev->struct_mutex);
if (ret)
return ret;
@@ -2774,7 +2783,7 @@ i915_edp_psr_debug_set(void *data, u64 val)
struct drm_modeset_acquire_ctx ctx;
int ret;
- if (!CAN_PSR(dev_priv))
+ if (!CAN_PSR(dev_priv) || !HAS_DISPLAY(dev_priv))
return -ENODEV;
DRM_DEBUG_KMS("Setting PSR debug to %llx\n", val);
@@ -3222,6 +3231,9 @@ static int i915_display_info(struct seq_file *m, void *unused)
struct drm_connector *connector;
struct drm_connector_list_iter conn_iter;
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
intel_runtime_pm_get(dev_priv);
seq_printf(m, "CRTC info\n");
seq_printf(m, "---------\n");
@@ -3326,6 +3338,9 @@ static int i915_shared_dplls_info(struct seq_file *m, void *unused)
struct drm_device *dev = &dev_priv->drm;
int i;
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
drm_modeset_lock_all(dev);
for (i = 0; i < dev_priv->num_shared_dpll; i++) {
struct intel_shared_dpll *pll = &dev_priv->shared_dplls[i];
@@ -3395,7 +3410,7 @@ static int i915_ipc_status_open(struct inode *inode, struct file *file)
{
struct drm_i915_private *dev_priv = inode->i_private;
- if (!HAS_IPC(dev_priv))
+ if (!HAS_IPC(dev_priv) || !HAS_DISPLAY(dev_priv))
return -ENODEV;
return single_open(file, i915_ipc_status_show, dev_priv);
@@ -3442,7 +3457,7 @@ static int i915_ddb_info(struct seq_file *m, void *unused)
enum pipe pipe;
int plane;
- if (INTEL_GEN(dev_priv) < 9)
+ if (INTEL_GEN(dev_priv) < 9 || !HAS_DISPLAY(dev_priv))
return -ENODEV;
drm_modeset_lock_all(dev);
@@ -3553,6 +3568,9 @@ static int i915_drrs_status(struct seq_file *m, void *unused)
struct intel_crtc *intel_crtc;
int active_crtc_cnt = 0;
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
drm_modeset_lock_all(dev);
for_each_intel_crtc(dev, intel_crtc) {
if (intel_crtc->base.state->active) {
@@ -3579,6 +3597,9 @@ static int i915_dp_mst_info(struct seq_file *m, void *unused)
struct drm_connector *connector;
struct drm_connector_list_iter conn_iter;
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
drm_connector_list_iter_begin(dev, &conn_iter);
drm_for_each_connector_iter(connector, &conn_iter) {
if (connector->connector_type != DRM_MODE_CONNECTOR_DisplayPort)
@@ -3697,6 +3718,11 @@ static int i915_displayport_test_active_show(struct seq_file *m, void *data)
static int i915_displayport_test_active_open(struct inode *inode,
struct file *file)
{
+ struct drm_i915_private *dev_priv = inode->i_private;
+
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
return single_open(file, i915_displayport_test_active_show,
inode->i_private);
}
@@ -3718,6 +3744,9 @@ static int i915_displayport_test_data_show(struct seq_file *m, void *data)
struct drm_connector_list_iter conn_iter;
struct intel_dp *intel_dp;
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
drm_connector_list_iter_begin(dev, &conn_iter);
drm_for_each_connector_iter(connector, &conn_iter) {
struct intel_encoder *encoder;
@@ -3762,6 +3791,9 @@ static int i915_displayport_test_type_show(struct seq_file *m, void *data)
struct drm_connector_list_iter conn_iter;
struct intel_dp *intel_dp;
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
drm_connector_list_iter_begin(dev, &conn_iter);
drm_for_each_connector_iter(connector, &conn_iter) {
struct intel_encoder *encoder;
@@ -3875,7 +3907,8 @@ static int pri_wm_latency_open(struct inode *inode, struct file *file)
{
struct drm_i915_private *dev_priv = inode->i_private;
- if (INTEL_GEN(dev_priv) < 5 && !IS_G4X(dev_priv))
+ if ((INTEL_GEN(dev_priv) < 5 && !IS_G4X(dev_priv)) ||
+ !HAS_DISPLAY(dev_priv))
return -ENODEV;
return single_open(file, pri_wm_latency_show, dev_priv);
@@ -3885,7 +3918,7 @@ static int spr_wm_latency_open(struct inode *inode, struct file *file)
{
struct drm_i915_private *dev_priv = inode->i_private;
- if (HAS_GMCH_DISPLAY(dev_priv))
+ if (HAS_GMCH_DISPLAY(dev_priv) || !HAS_DISPLAY(dev_priv))
return -ENODEV;
return single_open(file, spr_wm_latency_show, dev_priv);
@@ -3895,7 +3928,7 @@ static int cur_wm_latency_open(struct inode *inode, struct file *file)
{
struct drm_i915_private *dev_priv = inode->i_private;
- if (HAS_GMCH_DISPLAY(dev_priv))
+ if (HAS_GMCH_DISPLAY(dev_priv) || !HAS_DISPLAY(dev_priv))
return -ENODEV;
return single_open(file, cur_wm_latency_show, dev_priv);
@@ -4646,6 +4679,11 @@ static ssize_t i915_hpd_storm_ctl_write(struct file *file,
static int i915_hpd_storm_ctl_open(struct inode *inode, struct file *file)
{
+ struct drm_i915_private *dev_priv = inode->i_private;
+
+ if (!HAS_DISPLAY(dev_priv))
+ return -ENODEV;
+
return single_open(file, i915_hpd_storm_ctl_show, inode->i_private);
}
@@ -4737,7 +4775,7 @@ static int i915_drrs_ctl_set(void *data, u64 val)
struct drm_device *dev = &dev_priv->drm;
struct intel_crtc *crtc;
- if (INTEL_GEN(dev_priv) < 7)
+ if (INTEL_GEN(dev_priv) < 7 || !HAS_DISPLAY(dev_priv))
return -ENODEV;
for_each_intel_crtc(dev, crtc) {
--
2.19.1
More information about the Intel-gfx-trybot
mailing list