On ma, 2016-12-12 at 11:53 +0000, Chris Wilson wrote: > The scan state occupies a large proportion of the struct drm_mm and is > rarely used and only contains temporary state. That makes it suitable to > moving to its struct and onto the stack of the callers. > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> <SNIP> > +void drm_mm_scan_init_with_range(struct drm_mm_scan *scan, > + struct drm_mm *mm, > u64 size, > u64 alignment, > unsigned long color, > u64 start, > u64 end) Just call drm_mm_scan_init() here and override check_range and a few others. Or add __drm_mm_scan_init(), but get rid of the duplicate the code. With that changed; Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx