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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
> ---
>  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
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux