Re: [PATCH v2] drm/i915/gt: Return bool values from a boolean helper

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The patch fails to fully implement the explicit boolean returns claimed in the commit message.
--

commit 0d74cf5750ffd2182532ac4105376be52c64bca2
Author: Andi Shyti <andi.shyti@linux.intel.com>

drm/i915/gt: Return bool values from a boolean helper

This commit updates intel_has_gpu_reset() to return false instead of NULL
when GPU reset is disabled via parameters. The stated goal is to have the
boolean helper return explicit true or false values.

Link: https://lore.kernel.org/r/20260625114442.2912230-1-andi.shyti@linux.intel.com

> diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c b/drivers/gpu/drm/i915/gt/intel_reset.c
> index b2cf672564dd9..6053f3e8aa27c 100644
> --- a/drivers/gpu/drm/i915/gt/intel_reset.c
> +++ b/drivers/gpu/drm/i915/gt/intel_reset.c
> @@ -797,7 +797,7 @@ static int __intel_gt_reset(struct intel_gt *gt, intel_engine_mask_t engine_mask
>  bool intel_has_gpu_reset(const struct intel_gt *gt)
>  {
>  	if (!gt->i915->params.reset)
> -		return NULL;
> +		return false;
>  
>  	return intel_get_gpu_reset(gt);

[Severity: Medium]
Does this fully achieve the goal stated in the commit message?

Looking at intel_has_gpu_reset() in drivers/gpu/drm/i915/gt/intel_reset.c,
intel_get_gpu_reset() returns a function pointer (reset_func).

By returning intel_get_gpu_reset(gt) directly, the code still relies on the C
compiler's implicit pointer-to-bool conversion.

Would it be better to return an explicit boolean value here as well, such as
!!intel_get_gpu_reset(gt) or intel_get_gpu_reset(gt) != NULL, to fully align
with the commit message's intent?

>  }

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260626215232.861416-1-andi.shyti@linux.intel.com?part=1