Re: [PATCH] drm/syncobj: reset file ptr to NULL when released.

2017-12-19 Thread Daniel Vetter
On Tue, Dec 19, 2017 at 09:30:12PM +1000, Dave Airlie wrote: > From: Dave Airlie > > The vk cts test: > dEQP-VK.api.external.semaphore.opaque_fd.export_multiple_times_temporary > > triggers a lot of > VFS: Close: file count is 0 > > This patch fixes it, but I'm guessing it's racy and someone wi

Re: [PATCH] drm/syncobj: reset file ptr to NULL when released.

2017-12-19 Thread Chris Wilson
Quoting Dave Airlie (2017-12-19 11:30:12) > From: Dave Airlie > > The vk cts test: > dEQP-VK.api.external.semaphore.opaque_fd.export_multiple_times_temporary > > triggers a lot of > VFS: Close: file count is 0 > > This patch fixes it, but I'm guessing it's racy and someone will > smell rcu, but

[PATCH] drm/syncobj: reset file ptr to NULL when released.

2017-12-19 Thread Dave Airlie
From: Dave Airlie The vk cts test: dEQP-VK.api.external.semaphore.opaque_fd.export_multiple_times_temporary triggers a lot of VFS: Close: file count is 0 This patch fixes it, but I'm guessing it's racy and someone will smell rcu, but I just wanted to send out the proof of fixing it so I remembe