Quoting Venkata Sandeep Dhanalakota (2019-12-16 18:53:32)
> typecheck() macro creates an huge stack size causing
> issues with static analysis with coverity, addressing
> this with creating a local pointer.
>
> Fixes: 639f2f24895f ("drm/i915: Introduce new macros for tracing")
> Cc: Joonas Lahtinen <[email protected]>
> Cc: Rodrigo Vivi <[email protected]>
> Cc: Daniel Vetter <[email protected]>
> Cc: Chris Wilson <[email protected]>
> Cc: Jani Nikula <[email protected]>
> Signed-off-by: Venkata Sandeep Dhanalakota <[email protected]>
> ---
> drivers/gpu/drm/i915/gt/intel_gt.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/intel_gt.h
> b/drivers/gpu/drm/i915/gt/intel_gt.h
> index 9d9e8831daeb..2355cf129e9c 100644
> --- a/drivers/gpu/drm/i915/gt/intel_gt.h
> +++ b/drivers/gpu/drm/i915/gt/intel_gt.h
> @@ -12,9 +12,9 @@
>
> struct drm_i915_private;
>
> -#define GT_TRACE(gt__, fmt, ...) do { \
> - typecheck(struct intel_gt, *(gt__)); \
> - GEM_TRACE("%s " fmt, dev_name(gt->i915->drm.dev), \
> +#define GT_TRACE(gt, fmt, ...) do { \
> + const struct intel_gt *gt__ __maybe_unused = (gt); \
> + GEM_TRACE("%s " fmt, dev_name(gt__->i915->drm.dev), \
> ##__VA_ARGS__); \
> } while (0)
Reviewed-by: Chris Wilson <[email protected]>
-Chris
---------------------------------------------------------------------
Intel Corporation (UK) Limited
Registered No. 1134945 (England)
Registered Office: Pipers Way, Swindon SN3 1RJ
VAT No: 860 2173 47
This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
