On Mon, Dec 07, 2009 at 06:51:34PM +0100, Thomas Hellstrom wrote: > Jerome Glisse wrote: > > Thomas attached is updated patch, they apply on top of my ttm changes. > > I modified radeon to return -ERESTARTSYS and updated libdrm_radeon to > > deal with this change. I didn't changed the patch that much but i added > > my signoff maybe only ack is need, not sure about that. > > > > Cheers, > > Jerome > > > > > Thanks, Jerome. > > I haven't yet looked at the libdrm_radeon change, and my hope was that no > user-space change was needed, since the -ERESTARTSYS is converted to an > -EINTR in the kernel signal layer before returned to user-space, > and the -EINTR is handled in drmCommand[Write|Read|None]. > > /Thomas >
No, we were using ioctl directly, i converted libdrm-radeon to use drmIoctl which handle errno=EINTR or EAGAIN, this to run fine but haven't checked how much restart i get. Cheers, Jerome ------------------------------------------------------------------------------ Join us December 9, 2009 for the Red Hat Virtual Experience, a free event focused on virtualization and cloud computing. Attend in-depth sessions from your desk. Your couch. Anywhere. http://p.sf.net/sfu/redhat-sfdev2dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
