Hi Ben, haven't got around to testing the patches, but they basically look good to me. Some comments:
On Fri, 2006-01-27 at 12:15 +1100, Benjamin Herrenschmidt wrote: > > > http://gate.crashing.org/~benh/radeon-memmap-7.0-2.diff There should be no need to check for info->cursor_offset == 0 in the cursor functions. Longer term, I think we should just reserve a static FB region for the cursor upfront instead of going through all these hoops with EXA. Also, unless I'm missing something, you're removing the code that forces the display priority to high for Radeon 7200. > http://gate.crashing.org/~benh/radeon-memmap-drm-3.diff The way you handle backwards compatibility here is brilliant, thanks. The only minor issue I see is that the setparam ioctl can be called by unprivileged clients, but that applies to the existing colour tiling part as well, and it may not be a problem thanks to the offset fixups. -- Earthling Michel Dänzer | Debian (powerpc), X and DRI developer Libre software enthusiast | http://svcs.affero.net/rm.php?r=daenzer ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://sel.as-us.falkag.net/sel?cmd=lnk&kid3432&bid#0486&dat1642 -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel