On Fri, 16 Jul 2021 at 20:21, Matthew Auld
<[email protected]> wrote:
>
> On Fri, 16 Jul 2021 at 15:14, Jason Ekstrand <[email protected]> wrote:
> >
> > Instead of hand-rolling the same three calls in each function, pull them
> > into an i915_gem_object_create_user helper.  Apart from re-ordering of
> > the placements array ENOMEM check, there should be no functional change.
> >
> > v2 (Matthew Auld):
> >  - Add the call to i915_gem_flush_free_objects() from
> >    i915_gem_dumb_create() in a separate patch
> >  - Move i915_gem_object_alloc() below the simple error checks
> >
> > Signed-off-by: Jason Ekstrand <[email protected]>
> > Cc: Matthew Auld <[email protected]>
>
> If CI is happy,
> Reviewed-by: Matthew Auld <[email protected]>

Might be good to also update the mman selftests to use this new helper.
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to