Re: [patch v2] i915: take struct_mutex lock in intel_setup_overlay()

This patch never made it into 2.6.35 but it's still valid. Can you pick it up for 2.6.36? It still applies cleanly to 2.6.35-rc3 with fuzz (8 lines offset). regards, dan carpenter On Mon, May 03, 2010 at 09:58:32AM +0200, Dan Carpenter wrote: > I changed the drm_gem_object_unreference() to > dr

[patch v2] i915: take struct_mutex lock in intel_setup_overlay()

This patch never made it into 2.6.35 but it's still valid. Can you pick it up for 2.6.36? It still applies cleanly to 2.6.35-rc3 with fuzz (8 lines offset). regards, dan carpenter On Mon, May 03, 2010 at 09:58:32AM +0200, Dan Carpenter wrote: > I changed the drm_gem_object_unreference() to > dr

[patch v2] i915: take struct_mutex lock in intel_setup_overlay()

I changed the drm_gem_object_unreference() to drm_gem_object_unreference_unlocked(). "reg_bo" is a local variable so there are no synchronization issues, but the problem is that if struct_mutex isn't held, it triggers a BUG_ON() in drm_gem_object_free(). I also took the "struct_mutex" when callin

[patch v2] i915: take struct_mutex lock in intel_setup_overlay()

I changed the drm_gem_object_unreference() to drm_gem_object_unreference_unlocked(). "reg_bo" is a local variable so there are no synchronization issues, but the problem is that if struct_mutex isn't held, it triggers a BUG_ON() in drm_gem_object_free(). I also took the "struct_mutex" when callin