diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2016-01-15 16:51:46 +0000 |
---|---|---|
committer | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2016-01-21 11:00:35 +0000 |
commit | 426960bed3217f72a1b7bb94f084d79cc616ec0f (patch) | |
tree | f26fb2afc54529609c28397c43fdc1207fb33907 /mm | |
parent | de1add360522c876c25ef2bbbbab1c94bdb509ab (diff) | |
download | linux-stable-426960bed3217f72a1b7bb94f084d79cc616ec0f.tar.gz linux-stable-426960bed3217f72a1b7bb94f084d79cc616ec0f.tar.bz2 linux-stable-426960bed3217f72a1b7bb94f084d79cc616ec0f.zip |
drm/i915: Seal busy-ioctl uABI and prevent leaking of internal ids
Tvrtko was looking through the execbuffer-ioctl and noticed that the
uABI was tightly coupled to our internal engine identifiers. Close
inspection also revealed that we leak those internal engine identifiers
through the busy-ioctl, and those internal identifiers already do not
match the user identifiers. Fortuitiously, there is only one user of the
set of busy rings from the busy-ioctl, and they only wish to choose
between the RENDER and the BLT engines.
Let's fix the userspace ABI while we still can.
v2: Update the uAPI documentation to explain the identifiers.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Testcase: igt/gem_busy
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/1452876706-21620-1-git-send-email-chris@chris-wilson.co.uk
Diffstat (limited to 'mm')
0 files changed, 0 insertions, 0 deletions