drm/i915: Fix cmd parser desc matching with masks
authorMika Kuoppala <mika.kuoppala@linux.intel.com>
Mon, 17 Aug 2020 19:59:26 +0000 (22:59 +0300)
committerMika Kuoppala <mika.kuoppala@linux.intel.com>
Fri, 21 Aug 2020 10:07:04 +0000 (13:07 +0300)
commit3b4efa148da36f158cce3f662e831af2834b8e0f
treed8560a517eaaa439d97a54e5719e579ce13aa599
parentd24f1341a63cebdff22454a6464810e6f34030c6
drm/i915: Fix cmd parser desc matching with masks

Our variety of defined gpu commands have the actual
command id field and possibly length and flags applied.

We did start to apply the mask during initialization of
the cmd descriptors but forgot to also apply it on comparisons.

Fix comparisons in order to properly deny access with
associated commands.

v2: fix lri with correct mask (Chris)

References: 926abff21a8f ("drm/i915/cmdparser: Ignore Length operands during command matching")
Reported-by: Nicolai Stange <nstange@suse.de>
Cc: stable@vger.kernel.org # v5.4+
Cc: Miroslav Benes <mbenes@suse.cz>
Cc: Takashi Iwai <tiwai@suse.de>
Cc: Tyler Hicks <tyhicks@canonical.com>
Cc: Jon Bloomfield <jon.bloomfield@intel.com>
Cc: Chris Wilson <chris.p.wilson@intel.com>
Signed-off-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20200817195926.12671-1-mika.kuoppala@linux.intel.com
drivers/gpu/drm/i915/i915_cmd_parser.c