[PATCH v3] drm: do not sleep on vblank while holding a mutex

2011-10-31 Thread Daniel Vetter
On Mon, Oct 31, 2011 at 01:10:21PM -0400, Ilija Hadzic wrote: > drm_wait_vblank must be DRM_UNLOCKED because otherwise it > will grab the drm_global_mutex and then go to sleep until the vblank > event it is waiting for. That can wreck havoc in the windowing system > because if one process issues th

[PATCH v3] drm: do not sleep on vblank while holding a mutex

2011-10-31 Thread Ilija Hadzic
OK, v4 coming up and your suggestion will be copied verbatim. Hopefully that does it. This patch is probably going to become a record-holder in comment/code lines ratio ;-) -- Ilija On Mon, 31 Oct 2011, Daniel Vetter wrote: > On Mon, Oct 31, 2011 at 01:10:21PM -0400, Ilija Hadzic wrote: >>

Re: [PATCH v3] drm: do not sleep on vblank while holding a mutex

2011-10-31 Thread Ilija Hadzic
OK, v4 coming up and your suggestion will be copied verbatim. Hopefully that does it. This patch is probably going to become a record-holder in comment/code lines ratio ;-) -- Ilija On Mon, 31 Oct 2011, Daniel Vetter wrote: On Mon, Oct 31, 2011 at 01:10:21PM -0400, Ilija Hadzic wrote:

Re: [PATCH v3] drm: do not sleep on vblank while holding a mutex

2011-10-31 Thread Daniel Vetter
On Mon, Oct 31, 2011 at 01:10:21PM -0400, Ilija Hadzic wrote: > drm_wait_vblank must be DRM_UNLOCKED because otherwise it > will grab the drm_global_mutex and then go to sleep until the vblank > event it is waiting for. That can wreck havoc in the windowing system > because if one process issues th

[PATCH v3] drm: do not sleep on vblank while holding a mutex

2011-10-31 Thread Ilija Hadzic
drm_wait_vblank must be DRM_UNLOCKED because otherwise it will grab the drm_global_mutex and then go to sleep until the vblank event it is waiting for. That can wreck havoc in the windowing system because if one process issues this ioctl, it will block all other processes for the duration of all vb

[PATCH v3] drm: do not sleep on vblank while holding a mutex

2011-10-31 Thread Ilija Hadzic
drm_wait_vblank must be DRM_UNLOCKED because otherwise it will grab the drm_global_mutex and then go to sleep until the vblank event it is waiting for. That can wreck havoc in the windowing system because if one process issues this ioctl, it will block all other processes for the duration of all vb