drm/i915/guc: Fix flag query helper function to not modify state
authorJohn Harrison <John.C.Harrison@Intel.com>
Thu, 17 Feb 2022 21:29:42 +0000 (13:29 -0800)
committerJohn Harrison <John.C.Harrison@Intel.com>
Tue, 1 Mar 2022 16:48:25 +0000 (08:48 -0800)
commiteee5215bd784eee5f8520f2deebf437c3fa95abb
treeb9f8e579ce3b70ee19e5c76b45734e53a7ea607f
parentfb87550d2517878ebcd112f080865a2dc38e9bae
drm/i915/guc: Fix flag query helper function to not modify state

A flag query helper was actually writing to the flags word rather than
just reading. Fix that. Also update the function's comment as it was
out of date.

NB: No need for a 'Fixes' tag. The test was only ever used inside a
BUG_ON during context registration. Rather than asserting that the
condition was true, it was making the condition true. So, in theory,
there was no consequence because we should never have hit a BUG_ON
anyway. Which means the write should always have been a no-op.

Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220217212942.629922-1-John.C.Harrison@Intel.com
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c