Unverified Commit b64a1433 authored by Raag Jadav's avatar Raag Jadav Committed by Rodrigo Vivi
Browse files

drm/xe/throttle: Skip reason prefix while emitting array 



The newly introduced "reasons" attribute already signifies possible
reasons for throttling and makes the prefix in individual attribute
names redundant while emitting them as an array. Skip the prefix.

Fixes: 83ccde67 ("drm/xe/gt_throttle: Avoid TOCTOU when monitoring reasons")
Signed-off-by: default avatarRaag Jadav <raag.jadav@intel.com>
Reviewed-by: default avatarSk Anirban <sk.anirban@intel.com>
Link: https://patch.msgid.link/20251203123355.571606-1-raag.jadav@intel.com


Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent da33e457

drivers/gpu/drm/xe/xe_gt_throttle.c

+1 −1
Original line number Diff line number Diff line
@@ -137,7 +137,7 @@ static ssize_t reasons_show(struct kobject *kobj, 
		struct throttle_attribute *other_ta = kobj_attribute_to_throttle(kattr);



		if (other_ta->mask != U32_MAX && reasons & other_ta->mask)

			ret += sysfs_emit_at(buff, ret, "%s ", (*pother)->name);

			ret += sysfs_emit_at(buff, ret, "%s ", (*pother)->name + strlen("reason_"));

	}



	if (drm_WARN_ONCE(&xe->drm, !ret, "Unknown reason: %#x\n", reasons))