[PATCH 1/2] drm/tegra: don't take dev->struct_mutex in mmap offset ioctl

2015-08-10 Thread Thierry Reding
On Mon, Aug 10, 2015 at 01:31:42PM +0200, Daniel Vetter wrote: > On Mon, Aug 10, 2015 at 12:30:21PM +0200, Thierry Reding wrote: > > On Wed, Jul 15, 2015 at 03:38:51PM +0200, Daniel Vetter wrote: > > > Since David Herrmann's mmap vma manager rework we don't need to grab > > > dev->struct_mutex any

[PATCH 1/2] drm/tegra: don't take dev->struct_mutex in mmap offset ioctl

2015-08-10 Thread Daniel Vetter
On Mon, Aug 10, 2015 at 12:30:21PM +0200, Thierry Reding wrote: > On Wed, Jul 15, 2015 at 03:38:51PM +0200, Daniel Vetter wrote: > > Since David Herrmann's mmap vma manager rework we don't need to grab > > dev->struct_mutex any more to prevent races when looking up the mmap > > offset. Drop it and

[PATCH 1/2] drm/tegra: don't take dev->struct_mutex in mmap offset ioctl

2015-08-10 Thread Thierry Reding
On Wed, Jul 15, 2015 at 03:38:51PM +0200, Daniel Vetter wrote: > Since David Herrmann's mmap vma manager rework we don't need to grab > dev->struct_mutex any more to prevent races when looking up the mmap > offset. Drop it and instead don't forget to use the unref_unlocked > variant (since the drm

[PATCH 1/2] drm/tegra: don't take dev->struct_mutex in mmap offset ioctl

2015-07-15 Thread Daniel Vetter
Since David Herrmann's mmap vma manager rework we don't need to grab dev->struct_mutex any more to prevent races when looking up the mmap offset. Drop it and instead don't forget to use the unref_unlocked variant (since the drm core still cares). While at it also fix a leak when this ioctl is