✗ Fi.CI.CHECKPATCH: warning for CCS static load balance

[Patchwork]

== Series Details ==

Series: CCS static load balance
URL   : https://patchwork.freedesktop.org/series/137715/
State : warning

== Summary ==

Error: dim checkpatch failed
f36edbebc969 drm/i915/gt: Avoid using masked workaround for CCS_MODE setting
9744b72c26af drm/i915/gt: Move the CCS mode variable to a global position
a9ed0d3824af drm/i915/gt: Allow the creation of multi-mode CCS masks
083c104fa607 drm/i915/gt: Refactor uabi engine class/instance list creation
-:54: WARNING:AVOID_BUG: Do not crash the kernel unless it is absolutely unavoidable--use WARN_ON_ONCE() plus recovery code (if feasible) instead of BUG() or variants
#54: FILE: drivers/gpu/drm/i915/gt/intel_engine_user.c:233:
+		GEM_BUG_ON(uabi_class >= ARRAY_SIZE(class_instance));

-:70: WARNING:AVOID_BUG: Do not crash the kernel unless it is absolutely unavoidable--use WARN_ON_ONCE() plus recovery code (if feasible) instead of BUG() or variants
#70: FILE: drivers/gpu/drm/i915/gt/intel_engine_user.c:247:
+		GEM_BUG_ON(uabi_class >=

total: 0 errors, 2 warnings, 0 checks, 56 lines checked
e5a08b266450 drm/i915/gem: Mark and verify UABI engine validity
2b9b405eede7 drm/i915/gt: Introduce for_each_enabled_engine() and apply it in selftests
-:30: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#30: FILE: drivers/gpu/drm/i915/gt/intel_gt.h:196:
+#define for_each_enabled_engine(engine__, gt__, id__) \
+	for ((id__) = 0; \
+	     (id__) < I915_NUM_ENGINES; \
+	     (id__)++) \
+		for_each_if (((engine__) = (gt__)->engine[(id__)]) && \
+			     (!RB_EMPTY_NODE(&(engine__)->uabi_node)))

-:30: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'engine__' - possible side-effects?
#30: FILE: drivers/gpu/drm/i915/gt/intel_gt.h:196:
+#define for_each_enabled_engine(engine__, gt__, id__) \
+	for ((id__) = 0; \
+	     (id__) < I915_NUM_ENGINES; \
+	     (id__)++) \
+		for_each_if (((engine__) = (gt__)->engine[(id__)]) && \
+			     (!RB_EMPTY_NODE(&(engine__)->uabi_node)))

-:30: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'id__' - possible side-effects?
#30: FILE: drivers/gpu/drm/i915/gt/intel_gt.h:196:
+#define for_each_enabled_engine(engine__, gt__, id__) \
+	for ((id__) = 0; \
+	     (id__) < I915_NUM_ENGINES; \
+	     (id__)++) \
+		for_each_if (((engine__) = (gt__)->engine[(id__)]) && \
+			     (!RB_EMPTY_NODE(&(engine__)->uabi_node)))

-:34: WARNING:SPACING: space prohibited between function name and open parenthesis '('
#34: FILE: drivers/gpu/drm/i915/gt/intel_gt.h:200:
+		for_each_if (((engine__) = (gt__)->engine[(id__)]) && \

total: 1 errors, 1 warnings, 2 checks, 738 lines checked
225f2e16a369 drm/i915/gt: Manage CCS engine creation within UABI exposure
641b8d9ba13c drm/i915/gt: Remove cslices mask value from the CCS structure
df72b5db7987 drm/i915/gt: Expose the number of total CCS slices
50847d494e8d drm/i915/gt: Store engine-related sysfs kobjects
575c97dc58c8 drm/i915/gt: Store active CCS mask
-:70: WARNING:AVOID_BUG: Do not crash the kernel unless it is absolutely unavoidable--use WARN_ON_ONCE() plus recovery code (if feasible) instead of BUG() or variants
#70: FILE: drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.c:120:
+	GEM_BUG_ON(ccs_mode);

total: 0 errors, 1 warnings, 0 checks, 83 lines checked
a5f4f7b8ec59 drm/i915: Protect access to the UABI engines list with a mutex
d832e381b9a3 drm/i915/gt: Isolate single sysfs engine file creation
ac990eb960d8 drm/i915/gt: Implement creation and removal routines for CCS engines
-:119: WARNING:AVOID_BUG: Do not crash the kernel unless it is absolutely unavoidable--use WARN_ON_ONCE() plus recovery code (if feasible) instead of BUG() or variants
#119: FILE: drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.c:206:
+		GEM_BUG_ON(rb_find_add(&e->uabi_node,

-:123: WARNING:AVOID_BUG: Do not crash the kernel unless it is absolutely unavoidable--use WARN_ON_ONCE() plus recovery code (if feasible) instead of BUG() or variants
#123: FILE: drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.c:210:
+		GEM_BUG_ON(intel_engine_lookup_user(i915, e->uabi_class,

total: 0 errors, 2 warnings, 0 checks, 146 lines checked
ba1301876c47 drm/i915/gt: Allow the user to change the CCS mode through sysfs