Summary: | Nouveau driver from git fails to build | ||
---|---|---|---|
Product: | xorg | Reporter: | Steven Ward <STEVENWARD666> |
Component: | Driver/nouveau | Assignee: | Nouveau Project <nouveau> |
Status: | RESOLVED FIXED | QA Contact: | Xorg Project Team <xorg-team> |
Severity: | major | ||
Priority: | high | ||
Version: | 6.99.99.904 (7.0 RC4) | ||
Hardware: | x86 (IA32) | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Steven Ward
2009-08-10 01:28:14 UTC
Excerpt from #nouveau: 10:14:41 < okias> Hello, have anyone idea why i can't compile ddx? http://pastebin.ca/1523661 10:15:25 < darktama> okias: your xserver isn't new enough where the referred paste contains the same bug as you: nouveau_exa.c:576: error: ‘EXA_MIXED_PIXMAPS’ undeclared (first use in this function) It was introduced in the DDX commit commit 30ddd08bb2739514edf6d6e137f533b34a4369d2, "exa: turn on new exa migration mode for driver pixmaps". The "fix" is in xorg-server commit http://cgit.freedesktop.org/xorg/xserver/commit/?id=03ecb164f2592c954aa408bf121e0c67b604d854 "exa: A simple 3rd backend implementation.", so the solution is to update your xorg-server from git. An alternative solution is to downgrade xorg-server to a version prior EXA 2.5. Today that means any xorg-server version officially released. Closing as not a bug. The driver needs to check that EXA_MIXED_PIXMAPS is defined before using it. The radeon driver just guards that code by #ifdef EXA_MIXED_PIXMAPS. Michel: While it would be easy enough to do that, sure, I have no intentions of attempting to preserve compatibility between pre-release servers. We currently check for >= EXA 2.5 before assuming mixed pixmaps are available, that seems sufficient no? If it was sufficient, would this report exist? If Maarten intended the minor version to be used for this, he should have bumped it. Even if it was sufficient, from my POV it's unnecessarily both more complicated and limiting than just #ifdef EXA_MIXED_PIXMAPS. But of course it's ultimately up to you. The reporter is using non-released software, it shouldn't be unexpected that things break. Why bother with compatibility for something that's never going to make it into a release? Are we going to do the same when CreatePixmap2() gets modified again before the release? In which case we have #ifdefs for 3 different APIs... It's no real trouble to allow use of both, I just didn't see the point. Not to mention that before mixed pixmaps, driver pixmaps were essentially unusably slow on some chipsets. (In reply to comment #5) > Are we going to do the same when CreatePixmap2() gets modified again before the > release? How would you handle that with #ifdefs? It's gonna need an EXA version change (minor bump, probably). > It's no real trouble to allow use of both, I just didn't see the point. Bisectability, e.g.? Anyway, I think I've presented my arguments and am going to stop now. Do as you see fit. Hi guys, As of the latest updates tonight,the nouveau driver from git builds successfully now.The latest xorg package version is now 1.6.99-33.20090807. I would like to thank you very much for trying to help me. I have a build problem with Mesa,but I'm going to file a different bug report on that one. With your permission,I'm going to close this bug as fixed. Regards, STEVE555 |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.