Commit e94fbaa8 authored by Mika Kuoppala's avatar Mika Kuoppala Committed by Daniel Vetter

drm/i915: Make next_seqno debugs entry to use i915_gem_set_seqno

This debugs entry can be used to set arbitrary value to next_seqno.
Use i915_gem_set_seqno instead of poking next_seqno.

v2: nasty details of next_seqno and last_seqno handling
moved inside i915_gem_set_seqno as suggested by Chris Wilson.
Signed-off-by: default avatarMika Kuoppala <mika.kuoppala@intel.com>
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent fca26bb4
...@@ -878,7 +878,6 @@ i915_next_seqno_write(struct file *filp, ...@@ -878,7 +878,6 @@ i915_next_seqno_write(struct file *filp,
loff_t *ppos) loff_t *ppos)
{ {
struct drm_device *dev = filp->private_data; struct drm_device *dev = filp->private_data;
drm_i915_private_t *dev_priv = dev->dev_private;
char buf[20]; char buf[20];
u32 val = 1; u32 val = 1;
int ret; int ret;
...@@ -896,19 +895,11 @@ i915_next_seqno_write(struct file *filp, ...@@ -896,19 +895,11 @@ i915_next_seqno_write(struct file *filp,
return ret; return ret;
} }
if (val == 0)
return -EINVAL;
ret = mutex_lock_interruptible(&dev->struct_mutex); ret = mutex_lock_interruptible(&dev->struct_mutex);
if (ret) if (ret)
return ret; return ret;
if (i915_seqno_passed(val, dev_priv->next_seqno)) { ret = i915_gem_set_seqno(dev, val);
dev_priv->next_seqno = val;
DRM_DEBUG_DRIVER("Advancing seqno to %u\n", val);
} else {
ret = -EINVAL;
}
mutex_unlock(&dev->struct_mutex); mutex_unlock(&dev->struct_mutex);
......
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