On Fri, 16 Jul 2021 at 20:21, Matthew Auld <matthew.william.a...@gmail.com> wrote: > > On Fri, 16 Jul 2021 at 15:14, Jason Ekstrand <ja...@jlekstrand.net> 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 <ja...@jlekstrand.net> > > Cc: Matthew Auld <matthew.a...@intel.com> > > If CI is happy, > Reviewed-by: Matthew Auld <matthew.a...@intel.com>
Might be good to also update the mman selftests to use this new helper.
