Quoting Joonas Lahtinen (2018-02-08 11:25:38)
> Quoting Chris Wilson (2018-02-08 10:51:51)
> > Userspace provides a 64b value for the priority, we need to be careful
> > to preserve the full range before validation to prevent truncation (and
> > letting an illegal value pass).
> >
> > Reported-by: Antonio Argenziano <antonio.argenziano@xxxxxxxxx>
> > Fixes: ac14fbd460d0 ("drm/i915/scheduler: Support user-defined priorities")
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > Cc: Antonio Argenziano <antonio.argenziano@xxxxxxxxx>
> > Cc: Michal Winiarski <michal.winiarski@xxxxxxxxx>
> > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxx>
> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
>
> Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
Pushed with a s/int64_t/s64/ to appease checkpatch.
Thanks, I was hoping to get Antonio's new gem_ctx_param checks in first,
oh well.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
