drm/gem: Check locking in drm_gem_object_unreference

2015-10-17 Thread Dan Carpenter
Hello Daniel Vetter, This is a semi-automatic email about new static checker warnings. The patch 5771551c4877: "drm/gem: Check locking in drm_gem_object_unreference" from Oct 15, 2015, leads to the following Smatch complaint: include/drm/drm_gem.h:147 drm_gem_object_u

[PATCH 09/25] drm/gem: Check locking in drm_gem_object_unreference

2015-10-15 Thread David Herrmann
Hi On Thu, Oct 15, 2015 at 10:55 AM, Daniel Vetter wrote: > On Thu, Oct 15, 2015 at 10:36:08AM +0200, David Herrmann wrote: >> Hi >> >> On Thu, Oct 15, 2015 at 9:36 AM, Daniel Vetter >> wrote: >> > Pretty soon only some drivers will need dev->struct_mutex in their >> > gem_free_object callbacks

[PATCH 09/25] drm/gem: Check locking in drm_gem_object_unreference

2015-10-15 Thread Daniel Vetter
On Thu, Oct 15, 2015 at 10:36:08AM +0200, David Herrmann wrote: > Hi > > On Thu, Oct 15, 2015 at 9:36 AM, Daniel Vetter > wrote: > > Pretty soon only some drivers will need dev->struct_mutex in their > > gem_free_object callbacks. Hence it's really important to make sure > > everything still kee

[PATCH 09/25] drm/gem: Check locking in drm_gem_object_unreference

2015-10-15 Thread David Herrmann
Hi On Thu, Oct 15, 2015 at 9:36 AM, Daniel Vetter wrote: > Pretty soon only some drivers will need dev->struct_mutex in their > gem_free_object callbacks. Hence it's really important to make sure > everything still keeps getting this right. > > Signed-off-by: Daniel Vetter > --- > include/drm/

[PATCH 09/25] drm/gem: Check locking in drm_gem_object_unreference

2015-10-15 Thread Daniel Vetter
Pretty soon only some drivers will need dev->struct_mutex in their gem_free_object callbacks. Hence it's really important to make sure everything still keeps getting this right. Signed-off-by: Daniel Vetter --- include/drm/drm_gem.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/d