Kristian Høgsberg schrieb:
> We need to hold the struct_mutex around pinning and the phys object
> operations.
>
this is missing mutex_lock at one place, leading to oopses like this one:
> BUG: unable to handle kernel paging request at 00100104
> kernel: IP: [<f867dfef>] i915_gem_retire_requests+0xdf/0x150 [i915]
Single line patch attached..
Regards,
Pierre
commit 6ffaf47548472e902f468aad909d9f51fb4f3ead
Author: Pierre Willenbrock <pie...@pirsoft.de>
Date: Sun Feb 22 01:51:17 2009 +0100
Add missing mutex_lock(&dev->struct_mutex)
Signed-off-by: Pierre Willenbrock <pie...@pirsoft.de>
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 4d2baf7..65b635c 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -1008,6 +1008,7 @@ static int intel_crtc_cursor_set(struct drm_crtc *crtc,
temp = CURSOR_MODE_DISABLE;
addr = 0;
bo = NULL;
+ mutex_lock(&dev->struct_mutex);
goto finish;
}
------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA
-OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise
-Strategies to boost innovation and cut costs with open source participation
-Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel
