From: Kenneth Graunke Date: Tue, 10 Apr 2018 08:27:56 +0000 (-0700) Subject: i965: Require softpin support for Cannonlake and later. X-Git-Tag: upstream/19.0.0~4738 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3ea2d791f3e72859a755e6e50a5330fc41a4bd2f;p=platform%2Fupstream%2Fmesa.git i965: Require softpin support for Cannonlake and later. This isn't strictly necessary, but anyone running Cannonlake will already have Kernel 4.5 or later, so there's no reason to support the relocation model on Gen10+. This will let us avoid dealing with them for new features. Reviewed-by: Scott D Phillips Reviewed-by: Jordan Justen --- diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.c b/src/mesa/drivers/dri/i965/brw_bufmgr.c index b5e3eb6..7ac3bca 100644 --- a/src/mesa/drivers/dri/i965/brw_bufmgr.c +++ b/src/mesa/drivers/dri/i965/brw_bufmgr.c @@ -1732,6 +1732,16 @@ brw_bufmgr_init(struct gen_device_info *devinfo, int fd) 4096, _4GB); util_vma_heap_init(&bufmgr->vma_allocator[BRW_MEMZONE_OTHER], 1 * _4GB, gtt_size - 1 * _4GB); + } else if (devinfo->gen >= 10) { + /* Softpin landed in 4.5, but GVT used an aliasing PPGTT until + * kernel commit 6b3816d69628becb7ff35978aa0751798b4a940a in + * 4.14. Gen10+ GVT hasn't landed yet, so it's not actually a + * problem - but extending this requirement back to earlier gens + * might actually mean requiring 4.14. + */ + fprintf(stderr, "i965 requires softpin (Kernel 4.5) on Gen10+."); + free(bufmgr); + return NULL; } }