Re: [PATCH] drm: Micro-optimise drm_mm_for_each_node_in_range()

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

 



On la, 2017-02-04 at 11:19 +0000, Chris Wilson wrote:
> As we require valid start/end parameters, we can replace the initial
> potential NULL with a pointer to the drm_mm.head_node and so reduce the
> test on every iteration from a NULL + address comparison to just an
> address comparison.
> 
> add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-26 (-26)
> function                                     old     new   delta
> i915_gem_evict_for_node                      719     693     -26
> 
> (No other users outside of the test harness.)
> 
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>

Slightly confused by the mixing of [start, end] and [start, end).

Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux