i965/drm: Drop libpciaccess dependencies.

i965 doesn't use drm_intel_get_aperture_sizes(), so we can delete
support for it.  This avoids a build dependency on libpciaccess.

Chris also notes:

"There's a really old bug that hopefully has been closed already
 (although as far as I can tell, it has never been fixed) about
 how using libpciaccess from libdrm_intel breaks the world (since
 libpciaccess uses a singleton that is torn down at the first request
 rather than upon the last user)."

This bug should go away in two commits when we switch over to our
internal copy of libdrm_intel.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=84325
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Acked-by: Jason Ekstrand <jason@jlekstrand.net>

diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.h b/src/mesa/drivers/dri/i965/brw_bufmgr.h
index 0a6f70c665a35670cbc4c2e355855a657b7c6862..29620ca5a992bb25d41804f8a74304fb4f6dfd13 100644 (file)
--- a/src/mesa/drivers/dri/i965/brw_bufmgr.h
+++ b/src/mesa/drivers/dri/i965/brw_bufmgr.h
@@ -186,7 +186,6 @@ void drm_intel_gem_bo_start_gtt_access(drm_intel_bo *bo, int write_enable);
 
 int drm_intel_get_pipe_from_crtc_id(drm_intel_bufmgr *bufmgr, int crtc_id);
 
-int drm_intel_get_aperture_sizes(int fd, size_t *mappable, size_t *total);
 int drm_intel_bufmgr_gem_get_devid(drm_intel_bufmgr *bufmgr);
 int drm_intel_gem_bo_wait(drm_intel_bo *bo, int64_t timeout_ns);
 

diff --git a/src/mesa/drivers/dri/i965/intel_bufmgr.c b/src/mesa/drivers/dri/i965/intel_bufmgr.c
index a285340039fe2370e05333f11a860f88e29dee39..cdb4359739bc473e5fec243ca9f6c685f217fd4a 100644 (file)
--- a/src/mesa/drivers/dri/i965/intel_bufmgr.c
+++ b/src/mesa/drivers/dri/i965/intel_bufmgr.c
@@ -36,7 +36,6 @@
 #include <errno.h>
 #include <drm.h>
 #include <i915_drm.h>
-#include <pciaccess.h>
 #include "libdrm_macros.h"
 #include "intel_bufmgr.h"
 #include "intel_bufmgr_priv.h"
@@ -325,50 +324,3 @@ drm_intel_get_pipe_from_crtc_id(drm_intel_bufmgr *bufmgr, int crtc_id)
                return bufmgr->get_pipe_from_crtc_id(bufmgr, crtc_id);
        return -1;
 }
-
-static size_t
-drm_intel_probe_agp_aperture_size(int fd)
-{
-       struct pci_device *pci_dev;
-       size_t size = 0;
-       int ret;
-
-       ret = pci_system_init();
-       if (ret)
-               goto err;
-
-       /* XXX handle multiple adaptors? */
-       pci_dev = pci_device_find_by_slot(0, 0, 2, 0);
-       if (pci_dev == NULL)
-               goto err;
-
-       ret = pci_device_probe(pci_dev);
-       if (ret)
-               goto err;
-
-       size = pci_dev->regions[2].size;
-err:
-       pci_system_cleanup ();
-       return size;
-}
-
-int
-drm_intel_get_aperture_sizes(int fd, size_t *mappable, size_t *total)
-{
-
-       struct drm_i915_gem_get_aperture aperture;
-       int ret;
-
-       ret = drmIoctl(fd, DRM_IOCTL_I915_GEM_GET_APERTURE, &aperture);
-       if (ret)
-               return ret;
-
-       *mappable = 0;
-       /* XXX add a query for the kernel value? */
-       if (*mappable == 0)
-               *mappable = drm_intel_probe_agp_aperture_size(fd);
-       if (*mappable == 0)
-               *mappable = 64 * 1024 * 1024; /* minimum possible value */
-       *total = aperture.aper_size;
-       return 0;
-}