diff options
author | George Spelvin <[email protected]> | 2020-03-25 19:24:29 +0000 |
---|---|---|
committer | Rodrigo Vivi <[email protected]> | 2020-08-17 16:16:54 -0400 |
commit | 21118e8e56479ef33460fbd63a5ad0535843b666 (patch) | |
tree | 520b12eb13c3a62ee9043628826cd617b2141b49 /tools/perf/scripts/python/export-to-sqlite.py | |
parent | 13149e8bafc4657254831ba6c16ed8780aa64a06 (diff) |
drm/i915/selftests: Avoid passing a random 0 into ilog2
igt_mm_config() calls ilog2() on the (pseudo)random 21-bit number
s>>12. Once in 2 million seeds, this is zero and ilog2 summons
the nasal demons.
There was an attempt to handle this case with a max(), but that's
too late; ms could already be something bizarre.
Given that the low 12 bits of s and ms are always zero, it's a lot
simpler just to divide them by 4096, then everything fits into 32
bits, and we can easily generate a random number 1 <= s <= 0x1fffff.
Fixes: 14d1b9a6247c ("drm/i915: buddy allocator")
Signed-off-by: George Spelvin <[email protected]>
Cc: Matthew Auld <[email protected]>
Cc: Jani Nikula <[email protected]>
Cc: Joonas Lahtinen <[email protected]>
Cc: Rodrigo Vivi <[email protected]>
Cc: [email protected]
Reviewed-by: Matthew Auld <[email protected]>
Signed-off-by: Chris Wilson <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Signed-off-by: Rodrigo Vivi <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/export-to-sqlite.py')
0 files changed, 0 insertions, 0 deletions