On E, 2010-08-16 at 09:51 +0300, Mart Raudsepp wrote: > On E, 2010-08-16 at 09:49 +0300, Martin-Éric Racine wrote: > > On Mon, Aug 16, 2010 at 9:36 AM, Mart Raudsepp <[email protected]> wrote: > > > On L, 2010-08-14 at 11:32 +0300, Martin-Éric Racine wrote: > > >> Greetings, > > >> > > >> As recently announced, we now have a release candidate for the X.org > > >> Geode 2.11.9 video driver. [1] Please test > > >> it extensively and immediately report any significant regression > > >> directly into the X.org Bugzilla. [2] If nothing major gets reported > > >> during this weekend, we'll release what we currently have in GIT by > > >> Monday morning. > > > > > > Looks like the rotation code isn't quite perfect yet. > > > > > > Before doing any rotation: > > > > > > http://dev.gentoo.org/~leio/xorg/geode-before-rotation-20100816.png > > > > > > After rotating to inverted mode, and then (importantly) back to normal > > > mode: > > > > > > http://dev.gentoo.org/~leio/xorg/geode-after-rotation-20100816.png > > > > > > Notice all icon images missing again, various glyphs turned to vertical > > > lines and a broken openbox panel black/grey gradient on the bottom. > > > > > > It seems that right after the rotation patch at commit 8a61ef8f, the > > > problem is already present. > > > > > > I would consider this a release blocker for a while. OTOH, rotation > > > didn't work at all with xserver-1.7+ before, so I guess it depends on > > > how quick we can find a solution for this, and if xserver-1.6 (where > > > rotation worked before) is affected by this. > > > > Thanks for reporting this. > > > > Since OLPC are the main user for this feature, I would have > > appreciated receiving feedback from them earlier than this. > > Bernie's feedback just now was that it works fine. But just asked to > test what happens after going back to normal mode, as that's where the > problem appears - after destroying of the scratch pixmap header that was > created upon entering rotated mode. > > Also on XO-1 things were working fine before, as the reserved memory > approach we had before didn't trigger any issues with xserver-1.6, that > is used on the XO-1 FC-11 images, but didn't work at all since > xserver-1.7+.
Just to be more clear: Rotation works _fine_ with xf86-video-geode-2.11.8 + xserver-1.6(!), so if this change breaks it on xserver-1.6 too (which XO-1 uses), then it's a regression worth fixing. And I'm pretty sure a fix is not hard, after having debugged the issue a bit. I'm rather confident I can get it fixed within a couple hours, but I need to go to earn my salary now. Maybe Hunk can get it fixed today before me too. Mart > > > Personally, given how rotation has already been broken for quite some > > time, I don't consider this breakage as a good enough reason to avoid > > releasing as planned. Instead, I would hope that our AMD developers > > could be sent their own XO-1 to work with, along with a URL to > > instructions on how to build our driver for the OLPC and on how to > > install it onto the laptop's filesystem. I would also hope to see > > continuous feedback from the OLPC developer community towards ensuring > > that our driver keeps on working on the XO-1. > > > > Martin-Éric > > > _______________________________________________ > Xorg-driver-geode mailing list > [email protected] > http://lists.x.org/mailman/listinfo/xorg-driver-geode _______________________________________________ Xorg-driver-geode mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-driver-geode
