On Sat, 20 Apr 2019 at 05:25, John Stultz <john.stu...@linaro.org> wrote: > > From: Adrian Salido <sali...@google.com> > > When calling drmModeAtomicAddProperty allocation of memory happens as > needed in increments of 16 elements. This can be very slow if there are > multiple properties to be updated in an Atomic Commit call. > > Increase this to as many as can fit in a memory PAGE to avoid having to > reallocate memory too often. > > Cc: Emil Velikov <emil.veli...@collabora.com> > Cc: Sean Paul <seanp...@chromium.org> > Cc: Alistair Strachan <astrac...@google.com> > Cc: Marissa Wall <maris...@google.com> > Signed-off-by: John Stultz <john.stu...@linaro.org> > --- > xf86drmMode.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/xf86drmMode.c b/xf86drmMode.c > index 8f8633e..c878d9e 100644 > --- a/xf86drmMode.c > +++ b/xf86drmMode.c > @@ -1259,7 +1259,7 @@ drm_public drmModeAtomicReqPtr > drmModeAtomicDuplicate(drmModeAtomicReqPtr old) > return NULL; > } > memcpy(new->items, old->items, > - old->size_items * sizeof(*new->items)); > + old->cursor * sizeof(*new->items)); This seems like another (unrelated) perf. tweak. Splitting to separate patch may be an overkill, so a trivial mention in the commit message will do.
With that: Reviewed-by: Emil Velikov <emil.veli...@collabora.com> -Emil _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
