On Sat, Feb 23, 2013 at 05:30:09PM -0800, Ben Widawsky wrote:
> Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
> ---
> drivers/gpu/drm/i915/i915_irq.c | 18 +++++++++++-------
> 1 file changed, 11 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> index 29037e0..ebaf558 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -952,24 +952,25 @@ static void i915_get_extra_instdone(struct drm_device *dev,
>
> #ifdef CONFIG_DEBUG_FS
> static struct drm_i915_error_object *
> -i915_error_object_create(struct drm_i915_private *dev_priv,
> - struct drm_i915_gem_object *src)
> +i915_error_object_create_sized(struct drm_i915_private *dev_priv,
> + struct drm_i915_gem_object *src,
> + int num_pages)
> {
> struct drm_i915_error_object *dst;
> - int i, count;
> + int i;
> u32 reloc_offset;
>
> if (src == NULL || src->pages == NULL)
> return NULL;
>
> - count = src->base.size / PAGE_SIZE;
> + num_pages = src->base.size / PAGE_SIZE;
Probably didn't mean to overwrite your new parameter with the old value.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
