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

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

 



On Fri, May 07, 2010 at 09:52:12AM +0200, Michel Dänzer wrote:
> 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.
> 

I need another interface change before using this functionality
i will bump minor with the new interface i will send patch today.

Cheers,
Jerome
_______________________________________________
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