[PATCH v1 2/4] scripts/config: drop modules support

dmukhin@xen.org posted 4 patches 1 week, 3 days ago
[PATCH v1 2/4] scripts/config: drop modules support
Posted by dmukhin@xen.org 1 week, 3 days ago
From: Denis Mukhin <dmukhin@ford.com> 

Drop the --module (-m) and --module-after (-M) options, as Xen
does not support loadable modules, so options are not applicable.

Signed-off-by: Denis Mukhin <dmukhin@ford.com>
---
 xen/scripts/config | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/xen/scripts/config b/xen/scripts/config
index ea475c07de28..1050812aae8d 100755
--- a/xen/scripts/config
+++ b/xen/scripts/config
@@ -18,7 +18,6 @@ $myname options command ...
 commands:
 	--enable|-e option   Enable option
 	--disable|-d option  Disable option
-	--module|-m option   Turn option into a module
 	--set-str option string
 	                     Set option to "string"
 	--set-val option value
@@ -30,8 +29,6 @@ commands:
                              Enable option directly after other option
 	--disable-after|-D beforeopt option
                              Disable option directly after other option
-	--module-after|-M beforeopt option
-                             Turn option into module directly after other option
 	--refresh            Refresh the config using old settings
 
 	commands can be repeated multiple times
@@ -177,10 +174,6 @@ while [ "$1" != "" ] ; do
 		set_var "${CONFIG_}$ARG" "# ${CONFIG_}$ARG is not set"
 		;;
 
-	--module|-m)
-		set_var "${CONFIG_}$ARG" "${CONFIG_}$ARG=m"
-		;;
-
 	--set-str)
 		# sed swallows one level of escaping, so we need double-escaping
 		set_var "${CONFIG_}$ARG" "${CONFIG_}$ARG=\"${1//\"/\\\\\"}\""
@@ -220,10 +213,6 @@ while [ "$1" != "" ] ; do
 		set_var "${CONFIG_}$B" "# ${CONFIG_}$B is not set" "${CONFIG_}$A"
 		;;
 
-	--module-after|-M)
-		set_var "${CONFIG_}$B" "${CONFIG_}$B=m" "${CONFIG_}$A"
-		;;
-
 	--refresh)
 		yes "" | make oldconfig KCONFIG_CONFIG=$FN
 		;;
-- 
2.52.0
Re: [PATCH v1 2/4] scripts/config: drop modules support
Posted by Stefano Stabellini 2 days, 7 hours ago
On Thu, 15 Jan 2026, dmukhin@xen.org wrote:
> From: Denis Mukhin <dmukhin@ford.com> 
> 
> Drop the --module (-m) and --module-after (-M) options, as Xen
> does not support loadable modules, so options are not applicable.
> 
> Signed-off-by: Denis Mukhin <dmukhin@ford.com>

Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>


> ---
>  xen/scripts/config | 11 -----------
>  1 file changed, 11 deletions(-)
> 
> diff --git a/xen/scripts/config b/xen/scripts/config
> index ea475c07de28..1050812aae8d 100755
> --- a/xen/scripts/config
> +++ b/xen/scripts/config
> @@ -18,7 +18,6 @@ $myname options command ...
>  commands:
>  	--enable|-e option   Enable option
>  	--disable|-d option  Disable option
> -	--module|-m option   Turn option into a module
>  	--set-str option string
>  	                     Set option to "string"
>  	--set-val option value
> @@ -30,8 +29,6 @@ commands:
>                               Enable option directly after other option
>  	--disable-after|-D beforeopt option
>                               Disable option directly after other option
> -	--module-after|-M beforeopt option
> -                             Turn option into module directly after other option
>  	--refresh            Refresh the config using old settings
>  
>  	commands can be repeated multiple times
> @@ -177,10 +174,6 @@ while [ "$1" != "" ] ; do
>  		set_var "${CONFIG_}$ARG" "# ${CONFIG_}$ARG is not set"
>  		;;
>  
> -	--module|-m)
> -		set_var "${CONFIG_}$ARG" "${CONFIG_}$ARG=m"
> -		;;
> -
>  	--set-str)
>  		# sed swallows one level of escaping, so we need double-escaping
>  		set_var "${CONFIG_}$ARG" "${CONFIG_}$ARG=\"${1//\"/\\\\\"}\""
> @@ -220,10 +213,6 @@ while [ "$1" != "" ] ; do
>  		set_var "${CONFIG_}$B" "# ${CONFIG_}$B is not set" "${CONFIG_}$A"
>  		;;
>  
> -	--module-after|-M)
> -		set_var "${CONFIG_}$B" "${CONFIG_}$B=m" "${CONFIG_}$A"
> -		;;
> -
>  	--refresh)
>  		yes "" | make oldconfig KCONFIG_CONFIG=$FN
>  		;;
> -- 
> 2.52.0
>