diff options
| author | Kenneth Graunke <kenneth@whitecape.org> | 2014-02-10 01:54:23 -0800 |
|---|---|---|
| committer | Kenneth Graunke <kenneth@whitecape.org> | 2014-02-11 02:23:22 -0800 |
| commit | eaf3358e0a1323ed417b6875e70fdcdc30ed97e0 (patch) | |
| tree | 2f4e0b7e61e4f664976f7719b67be38cb96f3ddc /src/mesa/drivers/dri/i965/intel_screen.c | |
| parent | b47d231526821f5cff99546a984103a7222bc66c (diff) | |
| download | external_mesa3d-eaf3358e0a1323ed417b6875e70fdcdc30ed97e0.zip external_mesa3d-eaf3358e0a1323ed417b6875e70fdcdc30ed97e0.tar.gz external_mesa3d-eaf3358e0a1323ed417b6875e70fdcdc30ed97e0.tar.bz2 | |
i965: Don't call abort() on an unknown device.
If we don't recognize the PCI ID, we can't reasonably load the driver.
However, calling abort() is quite rude - it means the application that
tried to initialize us (possibly the X server) can't continue via
fallback paths. We already have a more polite mechanism - failing to
create the context. So, just use that.
While we're at it, improve the error message.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73024
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Tested-by: Lu Hua <huax.lu@intel.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/intel_screen.c')
| -rw-r--r-- | src/mesa/drivers/dri/i965/intel_screen.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/intel_screen.c b/src/mesa/drivers/dri/i965/intel_screen.c index 6ab3609..acdb5f3 100644 --- a/src/mesa/drivers/dri/i965/intel_screen.c +++ b/src/mesa/drivers/dri/i965/intel_screen.c @@ -1328,6 +1328,8 @@ __DRIconfig **intelInitScreen2(__DRIscreen *psp) intelScreen->deviceID = drm_intel_bufmgr_gem_get_devid(intelScreen->bufmgr); intelScreen->devinfo = brw_get_device_info(intelScreen->deviceID); + if (!intelScreen->devinfo) + return false; intelScreen->hw_must_use_separate_stencil = intelScreen->devinfo->gen >= 7; |