diff options
| author | Chris Wilson <[email protected]> | 2020-09-28 22:59:42 +0100 |
|---|---|---|
| committer | Rodrigo Vivi <[email protected]> | 2020-09-30 14:24:54 -0400 |
| commit | c60b93cd4862d108214a14e655358ea714d7a12a (patch) | |
| tree | a6e315d87f045634119cd9c67dfa5f62e0a6beb7 /tools/perf/scripts/python/bin/export-to-postgresql-report | |
| parent | 651dabe27f9638f569f6a794f9d3cc1889cd315e (diff) | |
drm/i915: Avoid mixing integer types during batch copies
Be consistent and use unsigned long throughout the chunk copies to
avoid the inherent clumsiness of mixing integer types of different
widths and signs. Failing to take acount of a wider unsigned type when
using min_t can lead to treating it as a negative, only for it flip back
to a large unsigned value after passing a boundary check.
Fixes: ed13033f0287 ("drm/i915/cmdparser: Only cache the dst vmap")
Testcase: igt/gen9_exec_parse/bb-large
Reported-by: "Candelaria, Jared" <[email protected]>
Signed-off-by: Chris Wilson <[email protected]>
Cc: Mika Kuoppala <[email protected]>
Cc: Joonas Lahtinen <[email protected]>
Cc: "Candelaria, Jared" <[email protected]>
Cc: "Bloomfield, Jon" <[email protected]>
Cc: <[email protected]> # v4.9+
Reviewed-by: Mika Kuoppala <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
(cherry picked from commit b7eeb2b4132ccf1a7d38f434cde7043913d1ed3c)
Signed-off-by: Rodrigo Vivi <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/bin/export-to-postgresql-report')
0 files changed, 0 insertions, 0 deletions