igt_aux.h already provides the optimal igt_fls(), so use that in preference to open coding the brute force version. Reported-by: Stuart Summers <stuart.summers@xxxxxxxxx> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Cc: Stuart Summers <stuart.summers@xxxxxxxxx> --- tests/i915/gem_blits.c | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/tests/i915/gem_blits.c b/tests/i915/gem_blits.c index e0346a7c7..f9cb12bb8 100644 --- a/tests/i915/gem_blits.c +++ b/tests/i915/gem_blits.c @@ -57,16 +57,6 @@ enum mode { WC, }; -static int fls(uint64_t x) -{ - int t; - - for (t = 0; x >> t; t++) - ; - - return t; -} - static unsigned int get_tiling_stride(const struct device *device, unsigned int width, unsigned int tiling) @@ -81,7 +71,7 @@ get_tiling_stride(const struct device *device, else stride = ALIGN(stride, 128); if (device->gen < 4) - stride = 1 << fls(stride - 1); + stride = 1 << igt_fls(stride - 1); } else { if (device->gen >= 8) stride = ALIGN(stride, 64); -- 2.24.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx