mirror of
				https://github.com/torvalds/linux.git
				synced 2025-10-31 00:28:52 +02:00 
			
		
		
		
	fs/proc: Skip bootloader comment if no embedded kernel parameters
If the "bootconfig" kernel command-line argument was specified or if the kernel was built with CONFIG_BOOT_CONFIG_FORCE, but if there are no embedded kernel parameter, omit the "# Parameters from bootloader:" comment from the /proc/bootconfig file. This will cause automation to fall back to the /proc/cmdline file, which will be identical to the comment in this no-embedded-kernel-parameters case. Link: https://lore.kernel.org/all/20240409044358.1156477-2-paulmck@kernel.org/ Fixes: 8b8ce6c75430 ("fs/proc: remove redundant comments from /proc/bootconfig") Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org> Signed-off-by: Paul E. McKenney <paulmck@kernel.org> Cc: stable@vger.kernel.org Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org> Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
This commit is contained in:
		
							parent
							
								
									fbbdc255fb
								
							
						
					
					
						commit
						c722cea208
					
				
					 3 changed files with 7 additions and 1 deletions
				
			
		|  | @ -63,7 +63,7 @@ static int __init copy_xbc_key_value_list(char *dst, size_t size) | ||||||
| 			dst += ret; | 			dst += ret; | ||||||
| 		} | 		} | ||||||
| 	} | 	} | ||||||
| 	if (ret >= 0 && boot_command_line[0]) { | 	if (cmdline_has_extra_options() && ret >= 0 && boot_command_line[0]) { | ||||||
| 		ret = snprintf(dst, rest(dst, end), "# Parameters from bootloader:\n# %s\n", | 		ret = snprintf(dst, rest(dst, end), "# Parameters from bootloader:\n# %s\n", | ||||||
| 			       boot_command_line); | 			       boot_command_line); | ||||||
| 		if (ret > 0) | 		if (ret > 0) | ||||||
|  |  | ||||||
|  | @ -10,6 +10,7 @@ | ||||||
| #ifdef __KERNEL__ | #ifdef __KERNEL__ | ||||||
| #include <linux/kernel.h> | #include <linux/kernel.h> | ||||||
| #include <linux/types.h> | #include <linux/types.h> | ||||||
|  | bool __init cmdline_has_extra_options(void); | ||||||
| #else /* !__KERNEL__ */ | #else /* !__KERNEL__ */ | ||||||
| /*
 | /*
 | ||||||
|  * NOTE: This is only for tools/bootconfig, because tools/bootconfig will |  * NOTE: This is only for tools/bootconfig, because tools/bootconfig will | ||||||
|  |  | ||||||
|  | @ -487,6 +487,11 @@ static int __init warn_bootconfig(char *str) | ||||||
| 
 | 
 | ||||||
| early_param("bootconfig", warn_bootconfig); | early_param("bootconfig", warn_bootconfig); | ||||||
| 
 | 
 | ||||||
|  | bool __init cmdline_has_extra_options(void) | ||||||
|  | { | ||||||
|  | 	return extra_command_line || extra_init_args; | ||||||
|  | } | ||||||
|  | 
 | ||||||
| /* Change NUL term back to "=", to make "param" the whole string. */ | /* Change NUL term back to "=", to make "param" the whole string. */ | ||||||
| static void __init repair_env_string(char *param, char *val) | static void __init repair_env_string(char *param, char *val) | ||||||
| { | { | ||||||
|  |  | ||||||
		Loading…
	
		Reference in a new issue
	
	 Masami Hiramatsu
						Masami Hiramatsu