Jerome Glisse wrote: > 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 > Ah Ok.
Usually you only see restarts in the X server process when moving windows around. /Thomas ------------------------------------------------------------------------------ Return on Information: Google Enterprise Search pays you back Get the facts. http://p.sf.net/sfu/google-dev2dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
