drm/i915/guc: Fix a static analysis warning
authorJohn Harrison <John.C.Harrison@Intel.com>
Wed, 21 Dec 2022 19:30:30 +0000 (11:30 -0800)
committerJohn Harrison <John.C.Harrison@Intel.com>
Thu, 22 Dec 2022 20:42:23 +0000 (12:42 -0800)
commitd830e0dc2e2d4826ebc6a429c2bc098848c9eeda
tree62c495b44f8d087e3c6e89a7f6706d924234bbed
parent7ccf9a5386a48b86c1abc0e7cb42f4b8f961af38
drm/i915/guc: Fix a static analysis warning

A static analyser was complaining about not checking for null
pointers. However, the location of the complaint can only be reached
in the first place if said pointer is non-null. Basically, if we are
using a v69 GuC then the descriptor pool is guaranteed to be alocated
at start of day or submission will be disabled with an ENOMEM error.
And if we are using a later GuC that does not use a descriptor pool
then the v69 submission function would not be called. So, not a
possible null at that point in the code.

Hence adding a GEM_BUG_ON(!ptr) to keep the tool happy.

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/20221221193031.687266-3-John.C.Harrison@Intel.com
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c