From 4ad0758f516d9e700c002c6f7614da74dea51005 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Thu, 23 Mar 2017 15:29:43 -0700 Subject: [PATCH] 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 Acked-by: Jason Ekstrand --- src/mesa/drivers/dri/i965/brw_bufmgr.h | 1 - src/mesa/drivers/dri/i965/intel_bufmgr.c | 48 -------------------------------- 2 files changed, 49 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.h b/src/mesa/drivers/dri/i965/brw_bufmgr.h index 0a6f70c..29620ca 100644 --- 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 a285340..cdb4359 100644 --- a/src/mesa/drivers/dri/i965/intel_bufmgr.c +++ b/src/mesa/drivers/dri/i965/intel_bufmgr.c @@ -36,7 +36,6 @@ #include #include #include -#include #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; -} -- 2.7.4