[Intel-gfx] [PATCH 5/9] drm/i915: Use a define for the default priority [0]
Chris Wilson
chris at chris-wilson.co.uk
Thu May 4 13:58:17 UTC 2017
On Thu, May 04, 2017 at 04:32:34PM +0300, Joonas Lahtinen wrote:
> On ke, 2017-05-03 at 12:37 +0100, Chris Wilson wrote:
> > Explicitly assign the default priority, and give it a name (macro).
> >
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
>
> <SNIP>
>
> > kref_init(&ctx->ref);
> > list_add_tail(&ctx->link, &dev_priv->context_list);
> > ctx->i915 = dev_priv;
> > + ctx->priority = I915_PRIORITY_DFL;
>
> I915_PRIORITY_DEFAULT would work better.
On the one hand I have the symmetry with MIN, DFL, MAX, on the other
hand DFL is plain bizarre.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list