Bug 8873 - server should use libdrm not link its own...
Summary: server should use libdrm not link its own...
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: unspecified
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Dave Airlie
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: xorg-7.2
  Show dependency treegraph
 
Reported: 2006-11-03 01:14 UTC by Dave Airlie
Modified: 2006-12-04 14:21 UTC (History)
6 users (show)

See Also:
i915 platform:
i915 features:


Attachments
patch for libdrm which should cover problems... (7.87 KB, patch)
2006-11-03 01:17 UTC, Dave Airlie
no flags Details | Splinter Review
patch for libdrm to add server hooks.. (9.34 KB, patch)
2006-11-03 20:26 UTC, Dave Airlie
no flags Details | Splinter Review
patch to server to use system libdrm.. (148.86 KB, patch)
2006-11-03 20:28 UTC, Dave Airlie
no flags Details | Splinter Review
move drmSetServerInfo() call, remove libdrm remnants (17.70 KB, patch)
2006-11-12 15:37 UTC, George -
no flags Details | Splinter Review

Description Dave Airlie 2006-11-03 01:14:59 UTC
The server has its own copy of libdrm it should link to the real one this
requires a few changes...
Comment 1 Dave Airlie 2006-11-03 01:17:35 UTC
Created attachment 7641 [details] [review]
patch for libdrm which should cover problems...

this is an initial patch to libdrm to add the necessary functionality...

I'll try and add the server side patch later..
Comment 2 Dave Airlie 2006-11-03 20:26:40 UTC
Created attachment 7644 [details] [review]
patch for libdrm to add server hooks..

new libdrm patch... note make the drm lib version number returned by
drmGetLibVersion has to be 1.x not 2.x as radeon driver checks it and falls
over
Comment 3 Dave Airlie 2006-11-03 20:28:06 UTC
Created attachment 7645 [details] [review]
patch to server to use system libdrm..

Remove libdrm from server and use the system one...
Comment 4 Dave Airlie 2006-11-08 15:46:13 UTC
This is all checked into the trees now, libdrm 2.3.0 is released.

I think this should go in 7.2 as we will have AIGLX problems with newer DRI
drivers otherwise...
Comment 5 George - 2006-11-12 15:37:48 UTC
Created attachment 7762 [details] [review]
move drmSetServerInfo() call, remove libdrm remnants

I think the drmSetServerInfo() call should be moved in DRIScreenInit() before
the drmOpen() call. Also, there are a few libdrm remnants to be removed.
Comment 6 George - 2006-11-12 15:59:20 UTC
the above patch is now committed to master
(5bfe7f4dfc6ab51b6790d74faf11574385234423)
Comment 7 Dave Airlie 2006-12-04 14:21:12 UTC
fixed in 7.2 also.


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.