[PATCH] drm/radeon: async event synchronization for drmWaitVblank

[Michel Dänzer]

On Don, 2010-05-06 at 18:52 +0200, Jerome Glisse wrote: 
> Bring radeon up to speed with the async event synchronization for
> drmWaitVblank. See c9a9c5e02aedc1a2815877b0268f886d2640b771 for
> more information. Without this patch event never get delivered
> to userspace client.
> 
> Signed-off-by: Jerome Glisse <jglisse at redhat.com>
> ---
>  drivers/gpu/drm/radeon/radeon_drv.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c
> index 4b05563..b3749d4 100644
> --- a/drivers/gpu/drm/radeon/radeon_drv.c
> +++ b/drivers/gpu/drm/radeon/radeon_drv.c
> @@ -216,6 +216,7 @@ static struct drm_driver driver_old = {
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
>  		 .fasync = drm_fasync,
> +		 .read = drm_read,
>  #ifdef CONFIG_COMPAT
>  		 .compat_ioctl = radeon_compat_ioctl,
>  #endif
> @@ -304,6 +305,7 @@ static struct drm_driver kms_driver = {
>  		 .mmap = radeon_mmap,
>  		 .poll = drm_poll,
>  		 .fasync = drm_fasync,
> +		 .read = drm_read,
>  #ifdef CONFIG_COMPAT
>  		 .compat_ioctl = radeon_kms_compat_ioctl,
>  #endif

At least the KMS DRM minor should be bumped for this and checked by the
X driver before enabling the DRI2 functionality that needs it, otherwise
clients trying to use that functionality will hang.


-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer