Bug 23231 - Nouveau driver from git fails to build
Summary: Nouveau driver from git fails to build
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/nouveau (show other bugs)
Version: 6.99.99.904 (7.0 RC4)
Hardware: x86 (IA32) Linux (All)
: high major
Assignee: Nouveau Project
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-10 01:28 UTC by Steven Ward
Modified: 2009-08-13 12:56 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Steven Ward 2009-08-10 01:28:14 UTC
Hi to all,
         I'm currently using Fedora Rawhide 12 Constantine.
I also complile D.R.M the unamed git repository for the nouveau kernel moudules,Mesa and the Nouveau driver form git.My graphics card is an nvidia geforce 6,800 G.T with 256Mb of memory,my monitor is a A.D.I Microscan G1000.

I've been building the former source code from git ok,but I'm having trouble building the Nouveau driver.I update the commits from the nouveau git repository regulary.

The problem is after ./configure and then run make.I'll try and type the error I'm getting as accurately as possible:

Make all-recursive
make[1]: Entering directory '/opt/xf86-video-nouvuea'
Making all in src
make[2]: entering directory '/opt/xf86-video-nouveau/src'
CC nouveau_exa.lo
nouveau_exa.c: In function 'nouveau_exa_init'
nouveau_exa.c:576:error:'EXA_MIXED_PIXMAPS'undelclared(first use in this function
nouveau_exa.c:576:error:for each function declared in)
make[2]:*** [nouveau_exa.lo] Error1

I hope anybody can solve this problem.
Regards,
       STEVE555
Comment 1 Pekka Paalanen 2009-08-10 03:29:56 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.
Comment 2 Michel Dänzer 2009-08-10 03:51:31 UTC
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.
Comment 3 Ben Skeggs 2009-08-10 04:01:31 UTC
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?
Comment 4 Michel Dänzer 2009-08-10 04:09:20 UTC
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.
Comment 5 Ben Skeggs 2009-08-10 04:21:40 UTC
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.
Comment 6 Michel Dänzer 2009-08-10 04:28:12 UTC
(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.
Comment 7 Steven Ward 2009-08-13 12:56:20 UTC
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.