Commit 6d4f49b7 authored by Jani Nikula's avatar Jani Nikula Committed by Rodrigo Vivi
Browse files

drm/xe: make compound literal initialization const 



Be careful about having const in the compound literal initialization to
keep the initializers in rodata. Here, the impact is 1.8k of mutable
data moved to rodata.

add/remove: 0/1 grow/shrink: 0/0 up/down: 0/-1804 (-1804)
Data                                         old     new   delta
__compound_literal                          1804       -   -1804
Total: Before=42425, After=40621, chg -4.25%
add/remove: 0/0 grow/shrink: 1/0 up/down: 1804/0 (1804)
RO Data                                      old     new   delta
__compound_literal                          7696    9500   +1804
Total: Before=138535, After=140339, chg +1.30%

Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: default avatarLucas De Marchi <lucas.demarchi@intel.com>
Link: https://lore.kernel.org/r/20230309121746.479146-1-jani.nikula@intel.com


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

drivers/gpu/drm/xe/xe_rtp.h

+2 −2
Original line number Diff line number Diff line
@@ -363,7 +363,7 @@ struct xe_reg_sr; 
 */

#define XE_RTP_RULES(r1, ...)							\

	.n_rules = COUNT_ARGS(r1, ##__VA_ARGS__),				\

	.rules = (struct xe_rtp_rule[]) {					\

	.rules = (const struct xe_rtp_rule[]) {					\

		CALL_FOR_EACH(__ADD_XE_RTP_RULE_PREFIX, r1, ##__VA_ARGS__)	\

	}
@@ -390,7 +390,7 @@ struct xe_reg_sr; 
 */

#define XE_RTP_ACTIONS(a1, ...)							\

	.n_actions = COUNT_ARGS(a1, ##__VA_ARGS__),				\

	.actions = (struct xe_rtp_action[]) {					\

	.actions = (const struct xe_rtp_action[]) {				\

		CALL_FOR_EACH(__ADD_XE_RTP_ACTION_PREFIX, a1, ##__VA_ARGS__)	\

	}