Re: [PATCH libdrm 0/5] improve reuse implementation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 15, 2018 at 06:20:09PM -0700, James Xiong wrote:
> From: "Xiong, James" <james.xiong@xxxxxxxxx>
> 
> With gem_reuse enabled, when a buffer size is different than
> the sizes of buckets, it is aligned to the next bucket's size,
> which means about 25% more memory than the requested is allocated
> in the worst senario. For example:
> 
> Orignal size    Actual
> 32KB+1Byte      40KB
> .
> .
> .
> 8MB+1Byte       10MB
> .
> .
> .
> 96MB+1Byte      112MB
> 
> This is very memory expensive and make the reuse feature less
> favorable than it deserves to be.
> 
> This series aligns the reuse buffer size on page size instead to
> save memory. Performed gfxbench tests on Gen9 without LLC, the
> performances and reuse ratioes (reuse count/allocation count) were
> same as before, saved memory usage by 1% ~ 7%(gl_manhattan: peak
> allocated memory size was reduced from 448401408 to 419078144).
> 
> v2: split the patch to a series of small functional changes (Chris)

Mesa gen driver stopped using the libdrm buffer allocator. The gen2/3
driver still uses it, but I think that's not the one you benchmarked. The
17.1 release was the first one with that change.

I think you want to port your changes over to mesa to future proof it,
merging this to upstream makes little sense.
-Daniel

> 
> Xiong, James (5):
>   drm: add a macro DRMLISTFOREACHENTRYREVERSE
>   intel: reorganize internal function
>   intel: get a cached bucket by size
>   intel: get a cached buffer by size for reuse
>   intel: purge cached bucket when its cached buffer is evicted
> 
>  intel/intel_bufmgr_gem.c | 180 +++++++++++++++++++++++++----------------------
>  libdrm_lists.h           |   6 ++
>  2 files changed, 100 insertions(+), 86 deletions(-)
> 
> -- 
> 2.7.4
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux