On 19 January 2018 at 11:36, Tapani Pälli <[email protected]> wrote:
> +#define ANV_CONTEXT_LOW_PRIORITY ((I915_CONTEXT_MIN_USER_PRIORITY-1)/2) > +#define ANV_CONTEXT_MEDIUM_PRIORITY (I915_CONTEXT_DEFAULT_PRIORITY) > +#define ANV_CONTEXT_HIGH_PRIORITY ((I915_CONTEXT_MAX_USER_PRIORITY+1)/2) You want to document why the current values are picked and not, say PRIO*3/4. Ideally the i965 code will be updated as well, but that can happen as follow-up. Don't we need anv_gem_has_context_priority and anv_gem_set_context_priority stubs in anv_gem_stubs.c? Quick looks suggests so, admittedly I haven't run make check to verify. -Emil _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
