Bug 2264

Summary: Don't build Tdfx DRM for platforms where the glide lib doesn't exist
Product: xorg Reporter: Egbert Eich <eich>
Component: Build/MonolithicAssignee: Egbert Eich <eich>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: high CC: daniel, roland.mainz
Version: git   
Hardware: All   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
[FIXED_X11R68x] Fix roland.mainz: 6.8-branch+

Description Egbert Eich 2005-01-11 09:01:22 UTC
To use 3D on Tdfx HW the Glide HW abstraction in libGlide3 is required.
This lib is open source - however it only exists for x86 and AXP.
Therefore it should not be build on any of these platforms.
Patch is going to be attached.
Comment 1 Egbert Eich 2005-01-11 09:03:00 UTC
Created attachment 1665 [details] [review]
[FIXED_X11R68x] Fix

See above.
Comment 2 Egbert Eich 2005-01-11 09:10:57 UTC
2005-01-11  Egbert Eich  <eich-at-freedesktop-dot-org>

        * config/cf/xorg.cf:
        Don't build tdfx DRI driver on any platform but x86 and AXP
        as therequired Glide3 lib is only available there (if at all).
        Readability fixes (Bugzilla #2264).

Comment 3 Roland Mainz 2005-01-23 19:05:02 UTC
Comment on attachment 1665 [details] [review]
[FIXED_X11R68x] Fix

Patch approved for commit into X11R6.8.x stable branch in the 2005-01-14
release-wranglers phone call.
Comment 4 Roland Mainz 2005-01-23 19:12:36 UTC
Comment on attachment 1665 [details] [review]
[FIXED_X11R68x] Fix

patch checked-in into X11R6.8.x stable branch:

/cvs/xorg/xc/ChangeLog,v  <--  ChangeLog
new revision: 1.365.2.127; previous revision: 1.365.2.126
cvs commit: Using deprecated info format strings.  Convert your scripts to use
the new argument format and remove '1's from your info file format strings.
/cvs/xorg/xc/config/cf/xorg.cf,v  <--  xorg.cf
new revision: 1.27.2.6; previous revision: 1.27.2.5
cvs commit: Using deprecated info format strings.  Convert your scripts to use
the new argument format and remove '1's from your info file format strings.
Mailing the commit message to xorg-commit@lists.freedesktop.org...
Comment 5 Daniel Stone 2005-01-23 19:16:55 UTC
As I stated on the list, libglide3 is also available for amd64 and ia64.
Comment 6 Roland Mainz 2005-01-23 20:43:10 UTC
Daniel Stone wrote:
> As I stated on the list, libglide3 is also available for amd64 and ia64.

Can you make a patch for that, please ?
Comment 7 Egbert Eich 2005-01-25 02:13:06 UTC
> As I stated on the list, libglide3 is also available for amd64 and ia64.
There is no pointer to those libs anywhere in the code. 
Please supply a pointer.
Comment 8 Egbert Eich 2005-01-26 02:04:27 UTC
Daniel won't know that he's been asked a question when not in Cc: ;-)

I would like to add the proper defines for tdfx DRI support for all platforms
for which a libglide is available. However we should add a pointer to the site(s)
where the lib may be obtained from.
Comment 9 Daniel Stone 2005-01-30 06:20:03 UTC
Sorry, don't know where they're available from; all I know is that, on
Debian/Ubuntu, libglide3 is built for alpha (AXP), amd64 (x86_64), i386 (ia32,
x86) and ia64.  I assume that this is the largest possible subset, since Debian
builds for about 14 architectures, all things considered, and others do not
build libglide3 (deliberately: the source packages specifies which architectures
it can build on, which defaults to 'any'; the choice of alpha/amd64/i386/ia64
was a deliberate one).
Comment 10 Arkadiusz Miskiewicz 2005-02-06 14:20:08 UTC
Glide also exists on PPC and SPARC architecture.

We (PLD) were successfully building tdfx_dri.so for %{ix86} ia64 alpha ppc 
trough real use of this driver was zero on non x86 arches.

This patch was needed for Glide:
http://cvs.pld-linux.org/cgi-bin/cvsweb/SOURCES/Glide_VG-morearchs.patch

lftp ftp.pld-linux.org:/dists/ac/PLD/ppc/PLD/RPMS> ls Glide*
-rw-r--r--   1 pldac    pld       2108486 Apr 24  2004 Glide2x_SDK-2.1-8.ppc.rpm
-rw-r--r--   1 pldac    pld        111298 Jan 18  2004 Glide_V3-DRI-3.10.0-0.
20010309.10.ppc.rpm
-rw-r--r--   1 pldac    pld        400280 Jan 18  2004 Glide_V3-DRI-devel-3.10.
0-0.20010309.10.ppc.rpm
-rw-r--r--   1 pldac    pld        112916 Jan 18  2004 Glide_V3-DRI-static-3.10.
0-0.20010309.10.ppc.rpm
-rw-r--r--   1 pldac    pld        191313 Jan 18  2004 Glide_V5-DRI-3.10.0-0.
20010309.10.ppc.rpm
-rw-r--r--   1 pldac    pld        472857 Jan 18  2004 Glide_V5-DRI-devel-3.10.
0-0.20010309.10.ppc.rpm
-rw-r--r--   1 pldac    pld        204492 Jan 18  2004 Glide_V5-DRI-static-3.10.
0-0.20010309.10.ppc.rpm
-rw-r--r--   1 pldac    pld        247298 Apr 24  2004 Glide_VG-2.46-8.ppc.rpm
lftp ftp.pld-linux.org:/dists/ac/PLD/sparc/PLD/RPMS> ls Glide*
-rw-r--r--   1 pldac    pld       2109099 Apr 24  2004 Glide2x_SDK-2.1-8.sparc.
rpm
-rw-r--r--   1 pldac    pld        111599 Jan 19  2004 Glide_V3-DRI-3.10.0-0.
20010309.10.sparc.rpm
-rw-r--r--   1 pldac    pld        400561 Jan 19  2004 Glide_V3-DRI-devel-3.10.
0-0.20010309.10.sparc.rpm
-rw-r--r--   1 pldac    pld        117289 Jan 19  2004 Glide_V3-DRI-static-3.10.
0-0.20010309.10.sparc.rpm
-rw-r--r--   1 pldac    pld        195484 Jan 19  2004 Glide_V5-DRI-3.10.0-0.
20010309.10.sparc.rpm
-rw-r--r--   1 pldac    pld        472852 Jan 19  2004 Glide_V5-DRI-devel-3.10.
0-0.20010309.10.sparc.rpm
-rw-r--r--   1 pldac    pld        212293 Jan 19  2004 Glide_V5-DRI-static-3.10.
0-0.20010309.10.sparc.rpm
-rw-r--r--   1 pldac    pld        247366 Apr 24  2004 Glide_VG-2.46-8.sparc.rpm
Comment 11 Daniel Stone 2005-02-06 16:15:04 UTC
Er, I think the people who are using 3Dfx cards on amd64 (they still have PCI
slots, y'know), i386, ia64 and alpha would disagree with you there.  It is still
valuable for non-x86 architectures.
Comment 12 Adam Jackson 2005-04-03 14:35:46 UTC
mass update: the fix for this bug was applied to both HEAD and the stable 6.8 branch, but the bug was 
never closed.  closing now.

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.