Re: [PATCH] drm/i915: Fix SRC_COPY width on 830/845g

[Thomas Richter <thor@math.tu-berlin.de>]

Am 12.09.2014 08:37, schrieb Chris Wilson:
> One small change I forgot to make in
>
> commit c4d69da167fa967749aeb70bc0e94a457e5d00c1
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Mon Sep 8 14:25:41 2014 +0100
>
>      drm/i915: Evict CS TLBs between batches
>
> was to update the copy width for the compact BLT copy instruction.
>
> Reported-by: Thomas Richter <thor@math.tu-berlin.de>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Thomas Richter <thor@math.tu-berlin.de>
> Cc: Jani Nikula <jani.nikula@intel.com>
> Cc: stable@vger.kernel.org
> ---
>   drivers/gpu/drm/i915/intel_ringbuffer.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c
> index d0156d6..45d70fd 100644
> --- a/drivers/gpu/drm/i915/intel_ringbuffer.c
> +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c
> @@ -1386,7 +1386,7 @@ i830_emit_batchbuffer(struct i915_gem_request *rq,
>   		 */
>   		intel_ring_emit(ring, SRC_COPY_BLT_CMD | BLT_WRITE_RGBA);
>   		intel_ring_emit(ring, BLT_DEPTH_32 | BLT_ROP_SRC_COPY | 4096);
> -		intel_ring_emit(ring, DIV_ROUND_UP(len, 4096) << 16 | 1024);
> +		intel_ring_emit(ring, DIV_ROUND_UP(len, 4096) << 16 | 4096);
>   		intel_ring_emit(ring, cs_offset);
>   		intel_ring_emit(ring, 4096);
>   		intel_ring_emit(ring, offset);
Thanks Chris, this has done the trick! I get again a working display and 
a working GPU on my 830 machine.

Tested-by: Thomas Richter <thor@math.tu-berlin.de>