Re: Kernel 3.7.0-rc1 crash on i915 unlock_page

Re: Kernel 3.7.0-rc1 crash on i915 unlock_page

[Daniel Vetter]

[adding relevant mailing lists.]

On Fri, Oct 19, 2012 at 2:03 PM, Krzysztof Kolasa <kkolasa@winsoft.pl> wrote:
>
> crash : unlock page, shmem_read_mapping_page_gfp,
> i915_gem_object_get_pages_gtt
>
> laptop AMILO Pro V3405 with 2GB, kernel 32bit/PAE
>
> error screenshot :
>
> https://docs.google.com/open?id=0B1LAMAFWTdewbjFGSFNFcTNGSFE

Unfortunately the oops has scrolled off a bit. Can you try to grab a
full backtrace, either with netconsole or by using a higher screen
resolution (so that the entire thing fits)?

Thanks, Daniel
>
> Krzysztof
>



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

[PATCH] drm/i915: Initialize obj->pages before use by i915_gem_object_do_bit17_swizzle()

[Chris Wilson]

If we leave obj->pages set to NULL before attempting to deswizzle them,
then an OOPS is well deserved.

Fixes regression introduced in commit 9da3da660d8c19a54f6e93361d147509be3fff84
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Fri Jun 1 15:20:22 2012 +0100

    drm/i915: Replace the array of pages with a scatterlist

Reported-by: Krzysztof Kolasa
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: stable@vger.kernel.org
---
 drivers/gpu/drm/i915/i915_gem.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index ad2ed75..6ab7c68 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -1953,10 +1953,11 @@ i915_gem_object_get_pages_gtt(struct drm_i915_gem_object *obj)
 		sg_set_page(sg, page, PAGE_SIZE, 0);
 	}
 
+	obj->pages = st;
+
 	if (i915_gem_object_needs_bit17_swizzle(obj))
 		i915_gem_object_do_bit_17_swizzle(obj);
 
-	obj->pages = st;
 	return 0;
 
 err_pages:
-- 
1.7.10.4

Re: [PATCH] drm/i915: Initialize obj->pages before use by i915_gem_object_do_bit17_swizzle()

[Krzysztof Kolasa]

[-- Attachment #1.1: Type: text/plain, Size: 1219 bytes --]

On 19.10.2012 16:51, Chris Wilson wrote:
> If we leave obj->pages set to NULL before attempting to deswizzle them,
> then an OOPS is well deserved.
>
> Fixes regression introduced in commit 9da3da660d8c19a54f6e93361d147509be3fff84
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Fri Jun 1 15:20:22 2012 +0100
>
>      drm/i915: Replace the array of pages with a scatterlist
>
> Reported-by: Krzysztof Kolasa
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: stable@vger.kernel.org
> ---
>   drivers/gpu/drm/i915/i915_gem.c |    3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
> index ad2ed75..6ab7c68 100644
> --- a/drivers/gpu/drm/i915/i915_gem.c
> +++ b/drivers/gpu/drm/i915/i915_gem.c
> @@ -1953,10 +1953,11 @@ i915_gem_object_get_pages_gtt(struct drm_i915_gem_object *obj)
>   		sg_set_page(sg, page, PAGE_SIZE, 0);
>   	}
>   
> +	obj->pages = st;
> +
>   	if (i915_gem_object_needs_bit17_swizzle(obj))
>   		i915_gem_object_do_bit_17_swizzle(obj);
>   
> -	obj->pages = st;
>   	return 0;
>   
>   err_pages:
Patch solves the problem.

Thanks.

Krzysztof


[-- Attachment #1.2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 3657 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915: Initialize obj->pages before use by i915_gem_object_do_bit17_swizzle()

[Jesse Barnes]

On Fri, 19 Oct 2012 15:51:06 +0100
Chris Wilson <chris@chris-wilson.co.uk> wrote:

> If we leave obj->pages set to NULL before attempting to deswizzle them,
> then an OOPS is well deserved.
> 
> Fixes regression introduced in commit 9da3da660d8c19a54f6e93361d147509be3fff84
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Fri Jun 1 15:20:22 2012 +0100
> 
>     drm/i915: Replace the array of pages with a scatterlist
> 
> Reported-by: Krzysztof Kolasa
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: stable@vger.kernel.org
> ---
>  drivers/gpu/drm/i915/i915_gem.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
> index ad2ed75..6ab7c68 100644
> --- a/drivers/gpu/drm/i915/i915_gem.c
> +++ b/drivers/gpu/drm/i915/i915_gem.c
> @@ -1953,10 +1953,11 @@ i915_gem_object_get_pages_gtt(struct drm_i915_gem_object *obj)
>  		sg_set_page(sg, page, PAGE_SIZE, 0);
>  	}
>  
> +	obj->pages = st;
> +
>  	if (i915_gem_object_needs_bit17_swizzle(obj))
>  		i915_gem_object_do_bit_17_swizzle(obj);
>  
> -	obj->pages = st;
>  	return 0;
>  
>  err_pages:

Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>

-- 
Jesse Barnes, Intel Open Source Technology Center