Bugzilla – Bug 146687
Radeon DRM driver patch
Last modified: 2006-03-26 08:54:50 UTC
As you can see in Bug #127757, we (SuSE/Novell/X.Org) currently have big problems with the radeon driver in general. Benjamin Herrenschmidt has been working hard to adress theses issues and generated patches for X and Kernel accordingly. We already include the X driver patch for Beta3. =============================================================================== From: Benjamin Herrenschmidt <benh@kernel.crashing.org> To: Discuss issues related to the xorg tree <xorg@lists.freedesktop.org> Date: Thu, 26 Jan 2006 10:43:39 +1100 Cc: dri-devel@lists.sourceforge.net Subject: [PATCH] new radeon memory map fixes Ok, so finally here is a new version of the patch. This time, it's against modular and it comes with a DRM patch. The X driver and the DRM patch should both work with the unpatched counterpart though you'll only get the full benefit of the fixes with both patches applied. As I had to shuffle a lot of code around in the X driver, there may still be bugs lurking around. Especially look for regressions around Xinerama and MergedFB as I haven't yet had a chance to test with those (especially Xinerama is doing a lot of very dodgy stuffs in the radeon driver). Please, try to test all sort of combinations of color tiling on/off, dri enabled/disabled, bit depth, hw/sw cursor etc... Patches are available at: Xorg driver patch: http://gate.crashing.org/~benh/radeon-memmap-7.0-2.diff DRM patch: http://gate.crashing.org/~benh/radeon-memmap-drm-1.diff Please, report any problem, thanks, Ben. =============================================================================== Could you apply the DRM patch to the SUSE kernel? Ben has got several positive feedback mails. Therefore I don't think that this patch will break any things. The DRM patch has been updated meanwhile (radeon-memmap-drm-3.diff). Unfortunately the patch is against DRM CVS and would need to be ported back to the DRM sources in the current kernel (probably a trivial task). :-(
Created attachment 65719 [details] Current DRM patch by Benjamin
Created attachment 66151 [details] Patch against kernel-source of 10.1 Beta3
Are these patches accepted by upstream yet?
I don't know. At least the patch is no yet in DRM CVS. Proably this needs to be discussed with Benjamin and Dave Airlie <airlied@gmail.com>. AFAIK he synchronizes the DRM CVS and upstream kernel sources.
Yes, he does. Please discuss it with him, and find out the acceptance of it. We don't want to carry around such a patch for the next 7 years unless we know it's accepted into mainline...
done. I've included you in Cc ...
Greg, as you can see there's nothing to do on your side for now. I'll keep you up-to-date. Bugreport remains therefore on status "NEEDINFO".
Due to the immature state of this patch set, I really do not think we can accept it at this late stage, sorry. We are _way_ past the "add new features" deadline...
The patch has not been committed yet to DRM CVS. I'll keep you up-to-date.
Small update. Dave Airlie finally committed benh's memory management patch to DRM CVS (2006-02-18/2006-02-19). --> http://gate.crashing.org/~benh/radeon-memmap-drm-5.diff
Greg, care for a little bit of DRM fun? :-)
Sure, I'll take it, but it should still be NEEDINFO, as I don't want to add such a big change so late in the cycle...
Of course. Only apply it when it has been committed upstream. Unfortunately I probably won't notice when it gets committed upstream. :-(
Still not upstream (checked 2.6.16). Anyway, it's way to late for 10.1. Closing as WONTFIX.