drm/xe: make compound literal initialization const
authorJani Nikula <jani.nikula@intel.com>
Thu, 9 Mar 2023 12:17:46 +0000 (14:17 +0200)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Tue, 19 Dec 2023 23:29:46 +0000 (18:29 -0500)
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: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://lore.kernel.org/r/20230309121746.479146-1-jani.nikula@intel.com
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/xe/xe_rtp.h

index 433f0cbff57f736123cc567d3747f8d590bf938f..ac983ce93684d8e8f5a89c41233425462e1b91e4 100644 (file)
@@ -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__)    \
        }