On Wednesday, August 24, 2016 8:42:44 PM PDT Chris Wilson wrote: > From about kernel 4.9, GTT mmaps are virtually unlimited. A new > parameter, I915_PARAM_MMAP_GTT_VERSION, is added to advertise the > feature so query it and use it to avoid limiting tiled allocations to > only fit within the mappable aperture. > > Signed-off-by: Chris Wilson <[email protected]> > Cc: Kenneth Graunke <[email protected]>
Cool! I didn't realize we could do page faulting here, since it's
CPU-related. It's definitely nice to be able to map an unlimited
amount of space, at least as a fallback, even if other methods end
up being more efficient and we prefer those.
[snip]
> diff --git a/src/mesa/drivers/dri/i965/intel_screen.c
> b/src/mesa/drivers/dri/i965/intel_screen.c
> index 98f1c76..62eacba 100644
> --- a/src/mesa/drivers/dri/i965/intel_screen.c
> +++ b/src/mesa/drivers/dri/i965/intel_screen.c
> @@ -1006,6 +1006,17 @@ intel_get_boolean(struct intel_screen *screen, int
> param)
> return (intel_get_param(screen, param, &value) == 0) && value;
> }
>
> +static int
> +intel_get_integer(struct intel_screen *screen, int param)
> +{
> + int value = -1;
> +
> + if (intel_get_param(screen, param, &value) == 0)
> + return value;
Indentation is a bit off here (spaces only, 3 space indent).
> +
> + return -1;
> +}
> +
> static void
> intelDestroyScreen(__DRIscreen * sPriv)
> {
> @@ -1576,6 +1587,27 @@ __DRIconfig **intelInitScreen2(__DRIscreen *psp)
> if (INTEL_DEBUG & DEBUG_AUB)
> drm_intel_bufmgr_gem_set_aub_dump(intelScreen->bufmgr, true);
>
> +#define I915_PARAM_MMAP_GTT_VERSION 40 /* XXX delete me with new libdrm */
Lately some other developers have been getting grumpy about requiring
new libdrm versions not in their distros. Maybe for now, we should do:
#ifndef I915_PARAM_MMAP_GTT_VERSION
#define I915_PARAM_MMAP_GTT_VERSION 40 /* XXX delete me with new libdrm */
#endif
and we can leave it for a little while.
Thanks for doing this, Chris!
Reviewed-by: Kenneth Graunke <[email protected]>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
