On Friday, September 29, 2017 3:25:09 AM PDT Chris Wilson wrote: > Hook up the RendererQuery for __DRI2_RENDERER_HAS_CONTEXT_PRIORITY to > report the available DRM_I915_GEM_CONTEXT_SETPARAM options based on the > the default context. The kernel will validate the request to change the > property, so we get an accurate reflection of available support (based on > kernel version and privilege) and we should only have to do it once > during screen setup -- although the SETPARAM should be fast, they are > still an ioctl each. > > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> > Cc: Kenneth Graunke <kenn...@whitecape.org> > Reviewed-by: Emil Velikov <emil.veli...@collabora.com> > --- > src/mesa/drivers/dri/i965/intel_screen.c | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/intel_screen.c > b/src/mesa/drivers/dri/i965/intel_screen.c > index 22d9f19298..112935a580 100644 > --- a/src/mesa/drivers/dri/i965/intel_screen.c > +++ b/src/mesa/drivers/dri/i965/intel_screen.c > @@ -1373,6 +1373,19 @@ brw_query_renderer_integer(__DRIscreen *dri_screen, > case __DRI2_RENDERER_HAS_TEXTURE_3D: > value[0] = 1; > return 0; > + case __DRI2_RENDERER_HAS_CONTEXT_PRIORITY: > + value[0] = 0; > + if (brw_hw_context_set_priority(screen->bufmgr, > + 0, BRW_CONTEXT_HIGH_PRIORITY) == 0) > + value[0] |= __DRI2_RENDERER_HAS_CONTEXT_PRIORITY_HIGH; > + if (brw_hw_context_set_priority(screen->bufmgr, > + 0, BRW_CONTEXT_LOW_PRIORITY) == 0) > + value[0] |= __DRI2_RENDERER_HAS_CONTEXT_PRIORITY_LOW; > + /* reset to default last, just in case */ > + if (brw_hw_context_set_priority(screen->bufmgr, > + 0, BRW_CONTEXT_MEDIUM_PRIORITY) == 0) > + value[0] |= __DRI2_RENDERER_HAS_CONTEXT_PRIORITY_MEDIUM; > + return 0; > default: > return driQueryRendererIntegerCommon(dri_screen, param, value); > } >
Interesting, you're setting the priority of the default context (which we never use), in order to test whether it works? Does the kernel actually set anything in that case, or does it just ignore it? If it ignores it, that seems fine. Series is: Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> Typically we wait to pull in new headers until the kernel patches land in airlied's drm-next tree.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
