| From 25e27302c30bff1c1142f5947ca3514fb1efb447 Mon Sep 17 00:00:00 2001 |
| From: Mika Kuoppala <mika.kuoppala@linux.intel.com> |
| Date: Thu, 6 Jun 2013 15:18:39 +0300 |
| Subject: drm/i915: export error state to string conversion |
| |
| In preparation for accessing error state from sysfs, export |
| error state to string conversion function. Also tuck buffer |
| error handling inside the function. |
| |
| Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> |
| Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> |
| Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> |
| (cherry picked from commit fc16b48be665d94337a861486dd25499971742a2) |
| (cherry picked from drm-intel-next-queued) |
| Signed-off-by: Darren Hart <dvhart@linux.intel.com> |
| --- |
| drivers/gpu/drm/i915/i915_debugfs.c | 24 ++++++++---------------- |
| drivers/gpu/drm/i915/i915_drv.h | 7 +++++++ |
| 2 files changed, 15 insertions(+), 16 deletions(-) |
| |
| diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c |
| index f82134f8e9fb..b64af15eb388 100644 |
| --- a/drivers/gpu/drm/i915/i915_debugfs.c |
| +++ b/drivers/gpu/drm/i915/i915_debugfs.c |
| @@ -830,15 +830,8 @@ static void i915_ring_error_state(struct drm_i915_error_state_buf *m, |
| err_printf(m, " ring->tail: 0x%08x\n", error->cpu_ring_tail[ring]); |
| } |
| |
| -struct i915_error_state_file_priv { |
| - struct drm_device *dev; |
| - struct drm_i915_error_state *error; |
| -}; |
| - |
| - |
| -static int i915_error_state(struct i915_error_state_file_priv *error_priv, |
| - struct drm_i915_error_state_buf *m) |
| - |
| +int i915_error_state_to_str(struct drm_i915_error_state_buf *m, |
| + const struct i915_error_state_file_priv *error_priv) |
| { |
| struct drm_device *dev = error_priv->dev; |
| drm_i915_private_t *dev_priv = dev->dev_private; |
| @@ -848,7 +841,7 @@ static int i915_error_state(struct i915_error_state_file_priv *error_priv, |
| |
| if (!error) { |
| err_printf(m, "no error state collected\n"); |
| - return 0; |
| + goto out; |
| } |
| |
| err_printf(m, "Time: %ld s %ld us\n", error->time.tv_sec, |
| @@ -958,6 +951,10 @@ static int i915_error_state(struct i915_error_state_file_priv *error_priv, |
| if (error->display) |
| intel_display_print_error_state(m, dev, error->display); |
| |
| +out: |
| + if (m->bytes == 0 && m->err) |
| + return m->err; |
| + |
| return 0; |
| } |
| |
| @@ -1051,15 +1048,10 @@ static ssize_t i915_error_state_read(struct file *file, char __user *userbuf, |
| |
| error_str.start = *pos; |
| |
| - ret = i915_error_state(error_priv, &error_str); |
| + ret = i915_error_state_to_str(&error_str, error_priv); |
| if (ret) |
| goto out; |
| |
| - if (error_str.bytes == 0 && error_str.err) { |
| - ret = error_str.err; |
| - goto out; |
| - } |
| - |
| ret_count = simple_read_from_buffer(userbuf, count, &tmp_pos, |
| error_str.buf, |
| error_str.bytes); |
| diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h |
| index e90b919c0d32..e4ac32298055 100644 |
| --- a/drivers/gpu/drm/i915/i915_drv.h |
| +++ b/drivers/gpu/drm/i915/i915_drv.h |
| @@ -917,6 +917,11 @@ struct drm_i915_error_state_buf { |
| loff_t pos; |
| }; |
| |
| +struct i915_error_state_file_priv { |
| + struct drm_device *dev; |
| + struct drm_i915_error_state *error; |
| +}; |
| + |
| struct i915_gpu_error { |
| /* For hangcheck timer */ |
| #define DRM_I915_HANGCHECK_PERIOD 1500 /* in ms */ |
| @@ -1925,6 +1930,8 @@ int i915_debugfs_init(struct drm_minor *minor); |
| void i915_debugfs_cleanup(struct drm_minor *minor); |
| __printf(2, 3) |
| void i915_error_printf(struct drm_i915_error_state_buf *e, const char *f, ...); |
| +int i915_error_state_to_str(struct drm_i915_error_state_buf *estr, |
| + const struct i915_error_state_file_priv *error); |
| |
| /* i915_suspend.c */ |
| extern int i915_save_state(struct drm_device *dev); |
| -- |
| 1.8.5.rc3 |
| |