Hi Lepton, On Sat, Mar 17, 2018 at 3:09 AM, Lepton Wu <lep...@chromium.org> wrote: > If user calls map twice for kms_sw_displaytarget, the first mapped > buffer could get leaked. Instead of calling mmap every time, just > reuse previous mapping. Since user could map same displaytarget with > different flags, we have to keep two different pointers, one for rw > mapping and one for ro mapping. Also introduce reference count for > mapped buffer so we can unmap them at right time. > > Change-Id: I65308f0ff2640bd57b2577c6a3469540c9722859
Change-Id shouldn't be included in upstream patches. > Signed-off-by: Lepton Wu <lep...@chromium.org> > --- > .../winsys/sw/kms-dri/kms_dri_sw_winsys.c | 44 +++++++++++++++---- > 1 file changed, 36 insertions(+), 8 deletions(-) Other than that: Reviewed-by: Tomasz Figa <tf...@chromium.org> Thanks for addressing the comments. Best regards, Tomasz _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
