diff options
author | Scott D Phillips <[email protected]> | 2018-04-19 07:54:28 -0700 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2018-04-30 11:34:19 -0700 |
commit | 8ffc6ee251bdcf4a91916b73c1be71d58e680d34 (patch) | |
tree | 091d1ce053cd9ef6c35e4677b123187effd62816 /src/intel/vulkan/anv_gem.c | |
parent | 1c5f4f4e17f74d823d9e38c678e40e9f49e2c053 (diff) |
intel: fix check for 48b ppgtt support
The previous logic of the supports_48b_addresses wasn't actually
checking if i915.ko was running with full_48bit_ppgtt. The ENOENT
it was checking for was actually coming from the invalid context
id provided in the test execbuffer. There is no path in the
kernel driver where the presence of
EXEC_OBJECT_SUPPORTS_48B_ADDRESS leads to an error.
Instead, check the default context's GTT_SIZE param for a value
greater than 4 GiB
v2 (Ken): Fix in i965 as well.
v3 Check GTT_SIZE instead of HAS_ALIASING_PPGTT (Chris Wilson)
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/intel/vulkan/anv_gem.c')
-rw-r--r-- | src/intel/vulkan/anv_gem.c | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/src/intel/vulkan/anv_gem.c b/src/intel/vulkan/anv_gem.c index 2a8f8b14b7e..3ba6d198a8a 100644 --- a/src/intel/vulkan/anv_gem.c +++ b/src/intel/vulkan/anv_gem.c @@ -377,24 +377,6 @@ anv_gem_get_aperture(int fd, uint64_t *size) return 0; } -bool -anv_gem_supports_48b_addresses(int fd) -{ - struct drm_i915_gem_exec_object2 obj = { - .flags = EXEC_OBJECT_SUPPORTS_48B_ADDRESS, - }; - - struct drm_i915_gem_execbuffer2 execbuf = { - .buffers_ptr = (uintptr_t)&obj, - .buffer_count = 1, - .rsvd1 = 0xffffffu, - }; - - int ret = anv_ioctl(fd, DRM_IOCTL_I915_GEM_EXECBUFFER2, &execbuf); - - return ret == -1 && errno == ENOENT; -} - int anv_gem_gpu_get_reset_stats(struct anv_device *device, uint32_t *active, uint32_t *pending) |