diff options
| author | Steven Rostedt <[email protected]> | 2025-07-18 20:18:44 +0000 |
|---|---|---|
| committer | Steven Rostedt <[email protected]> | 2025-07-21 20:31:04 +0000 |
| commit | 61f7e318e99d3b398670518dd3f4f8510d1800fc (patch) | |
| tree | 9892602f9cb639ca78bc4c1157eab72574a98425 /drivers/gpu/drm/lima/lima_sched.c | |
| parent | ktest.pl: Have -D option work without a space (diff) | |
| download | kernel-61f7e318e99d3b398670518dd3f4f8510d1800fc.tar.gz kernel-61f7e318e99d3b398670518dd3f4f8510d1800fc.zip | |
ktest.pl: Prevent recursion of default variable options
If a default variable contains itself, do not recurse on it.
For example:
ADD_CONFIG := ${CONFIG_DIR}/temp_config
DEFAULTS
ADD_CONFIG = ${CONFIG_DIR}/default_config ${ADD_CONFIG}
The above works because the temp variable ADD_CONFIG (is a temp because it
is created with ":=") is already defined, it will be substituted in the
variable option. But if it gets commented out:
# ADD_CONFIG := ${CONFIG_DIR}/temp_config
DEFAULTS
ADD_CONFIG = ${CONFIG_DIR}/default_config ${ADD_CONFIG}
Then the above will go into a recursive loop where ${ADD_CONFIG} will
get replaced with the current definition of ADD_CONFIG which contains the
${ADD_CONFIG} and that will also try to get converted. ktest.pl will error
after 100 attempts of recursion and fail.
When replacing a variable with the default variable, if the default
variable contains itself, do not replace it.
Cc: "John Warthog9 Hawley" <[email protected]>
Cc: Dhaval Giani <[email protected]>
Cc: Greg KH <[email protected]>
Link: https://lore.kernel.org/[email protected]
Signed-off-by: Steven Rostedt <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/lima/lima_sched.c')
0 files changed, 0 insertions, 0 deletions
