All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-23  3:03 ` yingelin
  0 siblings, 0 replies; 18+ messages in thread
From: yingelin @ 2022-02-23  3:03 UTC (permalink / raw)
  To: ebiederm, mcgrof, keescook, yzaikin
  Cc: kexec, linux-kernel, linux-fsdevel, zengweilin, chenjianguo3,
	nixiaoming, qiuguorui1, yingelin, young.liuyang

This move the kernel/kexec_core.c respective sysctls to its own file.

Signed-off-by: yingelin <yingelin@huawei.com>
---
 kernel/kexec_core.c | 20 ++++++++++++++++++++
 kernel/sysctl.c     | 13 -------------
 2 files changed, 20 insertions(+), 13 deletions(-)

diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
index 68480f731192..e57339d49439 100644
--- a/kernel/kexec_core.c
+++ b/kernel/kexec_core.c
@@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
 struct kimage *kexec_image;
 struct kimage *kexec_crash_image;
 int kexec_load_disabled;
+static struct ctl_table kexec_core_sysctls[] = {
+	{
+		.procname	= "kexec_load_disabled",
+		.data		= &kexec_load_disabled,
+		.maxlen		= sizeof(int),
+		.mode		= 0644,
+		/* only handle a transition from default "0" to "1" */
+		.proc_handler	= proc_dointvec_minmax,
+		.extra1		= SYSCTL_ONE,
+		.extra2		= SYSCTL_ONE,
+	},
+	{ }
+};
+
+static int __init kexec_core_sysctl_init(void)
+{
+	register_sysctl_init("kernel", kexec_core_sysctls);
+	return 0;
+}
+late_initcall(kexec_core_sysctl_init);
 
 /*
  * No panic_cpu check version of crash_kexec().  This function is called
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index ae5e59396b5d..00e97c6d6576 100644
--- a/kernel/sysctl.c
+++ b/kernel/sysctl.c
@@ -61,7 +61,6 @@
 #include <linux/capability.h>
 #include <linux/binfmts.h>
 #include <linux/sched/sysctl.h>
-#include <linux/kexec.h>
 #include <linux/bpf.h>
 #include <linux/mount.h>
 #include <linux/userfaultfd_k.h>
@@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
 		.proc_handler	= tracepoint_printk_sysctl,
 	},
 #endif
-#ifdef CONFIG_KEXEC_CORE
-	{
-		.procname	= "kexec_load_disabled",
-		.data		= &kexec_load_disabled,
-		.maxlen		= sizeof(int),
-		.mode		= 0644,
-		/* only handle a transition from default "0" to "1" */
-		.proc_handler	= proc_dointvec_minmax,
-		.extra1		= SYSCTL_ONE,
-		.extra2		= SYSCTL_ONE,
-	},
-#endif
 #ifdef CONFIG_MODULES
 	{
 		.procname	= "modprobe",
-- 
2.26.2


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-23  3:03 ` yingelin
  0 siblings, 0 replies; 18+ messages in thread
From: yingelin @ 2022-02-23  3:03 UTC (permalink / raw)
  To: kexec

This move the kernel/kexec_core.c respective sysctls to its own file.

Signed-off-by: yingelin <yingelin@huawei.com>
---
 kernel/kexec_core.c | 20 ++++++++++++++++++++
 kernel/sysctl.c     | 13 -------------
 2 files changed, 20 insertions(+), 13 deletions(-)

diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
index 68480f731192..e57339d49439 100644
--- a/kernel/kexec_core.c
+++ b/kernel/kexec_core.c
@@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
 struct kimage *kexec_image;
 struct kimage *kexec_crash_image;
 int kexec_load_disabled;
+static struct ctl_table kexec_core_sysctls[] = {
+	{
+		.procname	= "kexec_load_disabled",
+		.data		= &kexec_load_disabled,
+		.maxlen		= sizeof(int),
+		.mode		= 0644,
+		/* only handle a transition from default "0" to "1" */
+		.proc_handler	= proc_dointvec_minmax,
+		.extra1		= SYSCTL_ONE,
+		.extra2		= SYSCTL_ONE,
+	},
+	{ }
+};
+
+static int __init kexec_core_sysctl_init(void)
+{
+	register_sysctl_init("kernel", kexec_core_sysctls);
+	return 0;
+}
+late_initcall(kexec_core_sysctl_init);
 
 /*
  * No panic_cpu check version of crash_kexec().  This function is called
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index ae5e59396b5d..00e97c6d6576 100644
--- a/kernel/sysctl.c
+++ b/kernel/sysctl.c
@@ -61,7 +61,6 @@
 #include <linux/capability.h>
 #include <linux/binfmts.h>
 #include <linux/sched/sysctl.h>
-#include <linux/kexec.h>
 #include <linux/bpf.h>
 #include <linux/mount.h>
 #include <linux/userfaultfd_k.h>
@@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
 		.proc_handler	= tracepoint_printk_sysctl,
 	},
 #endif
-#ifdef CONFIG_KEXEC_CORE
-	{
-		.procname	= "kexec_load_disabled",
-		.data		= &kexec_load_disabled,
-		.maxlen		= sizeof(int),
-		.mode		= 0644,
-		/* only handle a transition from default "0" to "1" */
-		.proc_handler	= proc_dointvec_minmax,
-		.extra1		= SYSCTL_ONE,
-		.extra2		= SYSCTL_ONE,
-	},
-#endif
 #ifdef CONFIG_MODULES
 	{
 		.procname	= "modprobe",
-- 
2.26.2



^ permalink raw reply related	[flat|nested] 18+ messages in thread
* Re: [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
  2022-02-23  3:03 ` yingelin
@ 2022-02-23  8:30   ` Baoquan He
  -1 siblings, 0 replies; 18+ messages in thread
From: Baoquan He @ 2022-02-23  8:30 UTC (permalink / raw)
  To: yingelin
  Cc: ebiederm, mcgrof, keescook, yzaikin, kexec, linux-kernel,
	linux-fsdevel, zengweilin, chenjianguo3, nixiaoming, qiuguorui1,
	young.liuyang

On 02/23/22 at 11:03am, yingelin wrote:
> This move the kernel/kexec_core.c respective sysctls to its own file.

Hmm, why is the move needed? 

With my understanding, sysctls are all put in kernel/sysctl.c,
why is kexec special?

> 
> Signed-off-by: yingelin <yingelin@huawei.com>
> ---
>  kernel/kexec_core.c | 20 ++++++++++++++++++++
>  kernel/sysctl.c     | 13 -------------
>  2 files changed, 20 insertions(+), 13 deletions(-)
> 
> diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
> index 68480f731192..e57339d49439 100644
> --- a/kernel/kexec_core.c
> +++ b/kernel/kexec_core.c
> @@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
>  struct kimage *kexec_image;
>  struct kimage *kexec_crash_image;
>  int kexec_load_disabled;
> +static struct ctl_table kexec_core_sysctls[] = {
> +	{
> +		.procname	= "kexec_load_disabled",
> +		.data		= &kexec_load_disabled,
> +		.maxlen		= sizeof(int),
> +		.mode		= 0644,
> +		/* only handle a transition from default "0" to "1" */
> +		.proc_handler	= proc_dointvec_minmax,
> +		.extra1		= SYSCTL_ONE,
> +		.extra2		= SYSCTL_ONE,
> +	},
> +	{ }
> +};
> +
> +static int __init kexec_core_sysctl_init(void)
> +{
> +	register_sysctl_init("kernel", kexec_core_sysctls);
> +	return 0;
> +}
> +late_initcall(kexec_core_sysctl_init);
>  
>  /*
>   * No panic_cpu check version of crash_kexec().  This function is called
> diff --git a/kernel/sysctl.c b/kernel/sysctl.c
> index ae5e59396b5d..00e97c6d6576 100644
> --- a/kernel/sysctl.c
> +++ b/kernel/sysctl.c
> @@ -61,7 +61,6 @@
>  #include <linux/capability.h>
>  #include <linux/binfmts.h>
>  #include <linux/sched/sysctl.h>
> -#include <linux/kexec.h>
>  #include <linux/bpf.h>
>  #include <linux/mount.h>
>  #include <linux/userfaultfd_k.h>
> @@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
>  		.proc_handler	= tracepoint_printk_sysctl,
>  	},
>  #endif
> -#ifdef CONFIG_KEXEC_CORE
> -	{
> -		.procname	= "kexec_load_disabled",
> -		.data		= &kexec_load_disabled,
> -		.maxlen		= sizeof(int),
> -		.mode		= 0644,
> -		/* only handle a transition from default "0" to "1" */
> -		.proc_handler	= proc_dointvec_minmax,
> -		.extra1		= SYSCTL_ONE,
> -		.extra2		= SYSCTL_ONE,
> -	},
> -#endif
>  #ifdef CONFIG_MODULES
>  	{
>  		.procname	= "modprobe",
> -- 
> 2.26.2
> 


^ permalink raw reply	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-23  8:30   ` Baoquan He
  0 siblings, 0 replies; 18+ messages in thread
From: Baoquan He @ 2022-02-23  8:30 UTC (permalink / raw)
  To: kexec

On 02/23/22 at 11:03am, yingelin wrote:
> This move the kernel/kexec_core.c respective sysctls to its own file.

Hmm, why is the move needed? 

With my understanding, sysctls are all put in kernel/sysctl.c,
why is kexec special?

> 
> Signed-off-by: yingelin <yingelin@huawei.com>
> ---
>  kernel/kexec_core.c | 20 ++++++++++++++++++++
>  kernel/sysctl.c     | 13 -------------
>  2 files changed, 20 insertions(+), 13 deletions(-)
> 
> diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
> index 68480f731192..e57339d49439 100644
> --- a/kernel/kexec_core.c
> +++ b/kernel/kexec_core.c
> @@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
>  struct kimage *kexec_image;
>  struct kimage *kexec_crash_image;
>  int kexec_load_disabled;
> +static struct ctl_table kexec_core_sysctls[] = {
> +	{
> +		.procname	= "kexec_load_disabled",
> +		.data		= &kexec_load_disabled,
> +		.maxlen		= sizeof(int),
> +		.mode		= 0644,
> +		/* only handle a transition from default "0" to "1" */
> +		.proc_handler	= proc_dointvec_minmax,
> +		.extra1		= SYSCTL_ONE,
> +		.extra2		= SYSCTL_ONE,
> +	},
> +	{ }
> +};
> +
> +static int __init kexec_core_sysctl_init(void)
> +{
> +	register_sysctl_init("kernel", kexec_core_sysctls);
> +	return 0;
> +}
> +late_initcall(kexec_core_sysctl_init);
>  
>  /*
>   * No panic_cpu check version of crash_kexec().  This function is called
> diff --git a/kernel/sysctl.c b/kernel/sysctl.c
> index ae5e59396b5d..00e97c6d6576 100644
> --- a/kernel/sysctl.c
> +++ b/kernel/sysctl.c
> @@ -61,7 +61,6 @@
>  #include <linux/capability.h>
>  #include <linux/binfmts.h>
>  #include <linux/sched/sysctl.h>
> -#include <linux/kexec.h>
>  #include <linux/bpf.h>
>  #include <linux/mount.h>
>  #include <linux/userfaultfd_k.h>
> @@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
>  		.proc_handler	= tracepoint_printk_sysctl,
>  	},
>  #endif
> -#ifdef CONFIG_KEXEC_CORE
> -	{
> -		.procname	= "kexec_load_disabled",
> -		.data		= &kexec_load_disabled,
> -		.maxlen		= sizeof(int),
> -		.mode		= 0644,
> -		/* only handle a transition from default "0" to "1" */
> -		.proc_handler	= proc_dointvec_minmax,
> -		.extra1		= SYSCTL_ONE,
> -		.extra2		= SYSCTL_ONE,
> -	},
> -#endif
>  #ifdef CONFIG_MODULES
>  	{
>  		.procname	= "modprobe",
> -- 
> 2.26.2
> 



^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
  2022-02-23  8:30   ` Baoquan He
@ 2022-02-24  1:04     ` yingelin
  -1 siblings, 0 replies; 18+ messages in thread
From: yingelin @ 2022-02-24  1:04 UTC (permalink / raw)
  To: Baoquan He
  Cc: ebiederm, mcgrof, keescook, yzaikin, kexec, linux-kernel,
	linux-fsdevel, zengweilin, chenjianguo3, nixiaoming, qiuguorui1,
	young.liuyang


在 2022/2/23 16:30, Baoquan He 写道:
> On 02/23/22 at 11:03am, yingelin wrote:
>> This move the kernel/kexec_core.c respective sysctls to its own file.
> Hmm, why is the move needed?
>
> With my understanding, sysctls are all put in kernel/sysctl.c,
> why is kexec special?

kernel/sysctl.c is a kitchen sink where everyone leaves their dirty dishes,

this makes it very difficult to maintain. The proc sysctl maintainers do 
not want to

know what sysctl knobs you wish to add for your own piece of code, we

just care about the core logic.

This patch moves the kexec sysctls to the place where they actually 
belong to help

with this maintenance.

>> Signed-off-by: yingelin <yingelin@huawei.com>
>> ---
>>   kernel/kexec_core.c | 20 ++++++++++++++++++++
>>   kernel/sysctl.c     | 13 -------------
>>   2 files changed, 20 insertions(+), 13 deletions(-)
>>
>> diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
>> index 68480f731192..e57339d49439 100644
>> --- a/kernel/kexec_core.c
>> +++ b/kernel/kexec_core.c
>> @@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
>>   struct kimage *kexec_image;
>>   struct kimage *kexec_crash_image;
>>   int kexec_load_disabled;
>> +static struct ctl_table kexec_core_sysctls[] = {
>> +	{
>> +		.procname	= "kexec_load_disabled",
>> +		.data		= &kexec_load_disabled,
>> +		.maxlen		= sizeof(int),
>> +		.mode		= 0644,
>> +		/* only handle a transition from default "0" to "1" */
>> +		.proc_handler	= proc_dointvec_minmax,
>> +		.extra1		= SYSCTL_ONE,
>> +		.extra2		= SYSCTL_ONE,
>> +	},
>> +	{ }
>> +};
>> +
>> +static int __init kexec_core_sysctl_init(void)
>> +{
>> +	register_sysctl_init("kernel", kexec_core_sysctls);
>> +	return 0;
>> +}
>> +late_initcall(kexec_core_sysctl_init);
>>   
>>   /*
>>    * No panic_cpu check version of crash_kexec().  This function is called
>> diff --git a/kernel/sysctl.c b/kernel/sysctl.c
>> index ae5e59396b5d..00e97c6d6576 100644
>> --- a/kernel/sysctl.c
>> +++ b/kernel/sysctl.c
>> @@ -61,7 +61,6 @@
>>   #include <linux/capability.h>
>>   #include <linux/binfmts.h>
>>   #include <linux/sched/sysctl.h>
>> -#include <linux/kexec.h>
>>   #include <linux/bpf.h>
>>   #include <linux/mount.h>
>>   #include <linux/userfaultfd_k.h>
>> @@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
>>   		.proc_handler	= tracepoint_printk_sysctl,
>>   	},
>>   #endif
>> -#ifdef CONFIG_KEXEC_CORE
>> -	{
>> -		.procname	= "kexec_load_disabled",
>> -		.data		= &kexec_load_disabled,
>> -		.maxlen		= sizeof(int),
>> -		.mode		= 0644,
>> -		/* only handle a transition from default "0" to "1" */
>> -		.proc_handler	= proc_dointvec_minmax,
>> -		.extra1		= SYSCTL_ONE,
>> -		.extra2		= SYSCTL_ONE,
>> -	},
>> -#endif
>>   #ifdef CONFIG_MODULES
>>   	{
>>   		.procname	= "modprobe",
>> -- 
>> 2.26.2
>>
> .

^ permalink raw reply	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-24  1:04     ` yingelin
  0 siblings, 0 replies; 18+ messages in thread
From: yingelin @ 2022-02-24  1:04 UTC (permalink / raw)
  To: kexec


? 2022/2/23 16:30, Baoquan He ??:
> On 02/23/22 at 11:03am, yingelin wrote:
>> This move the kernel/kexec_core.c respective sysctls to its own file.
> Hmm, why is the move needed?
>
> With my understanding, sysctls are all put in kernel/sysctl.c,
> why is kexec special?

kernel/sysctl.c is a kitchen sink where everyone leaves their dirty dishes,

this makes it very difficult to maintain. The proc sysctl maintainers do 
not want to

know what sysctl knobs you wish to add for your own piece of code, we

just care about the core logic.

This patch moves the kexec sysctls to the place where they actually 
belong to help

with this maintenance.

>> Signed-off-by: yingelin <yingelin@huawei.com>
>> ---
>>   kernel/kexec_core.c | 20 ++++++++++++++++++++
>>   kernel/sysctl.c     | 13 -------------
>>   2 files changed, 20 insertions(+), 13 deletions(-)
>>
>> diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
>> index 68480f731192..e57339d49439 100644
>> --- a/kernel/kexec_core.c
>> +++ b/kernel/kexec_core.c
>> @@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
>>   struct kimage *kexec_image;
>>   struct kimage *kexec_crash_image;
>>   int kexec_load_disabled;
>> +static struct ctl_table kexec_core_sysctls[] = {
>> +	{
>> +		.procname	= "kexec_load_disabled",
>> +		.data		= &kexec_load_disabled,
>> +		.maxlen		= sizeof(int),
>> +		.mode		= 0644,
>> +		/* only handle a transition from default "0" to "1" */
>> +		.proc_handler	= proc_dointvec_minmax,
>> +		.extra1		= SYSCTL_ONE,
>> +		.extra2		= SYSCTL_ONE,
>> +	},
>> +	{ }
>> +};
>> +
>> +static int __init kexec_core_sysctl_init(void)
>> +{
>> +	register_sysctl_init("kernel", kexec_core_sysctls);
>> +	return 0;
>> +}
>> +late_initcall(kexec_core_sysctl_init);
>>   
>>   /*
>>    * No panic_cpu check version of crash_kexec().  This function is called
>> diff --git a/kernel/sysctl.c b/kernel/sysctl.c
>> index ae5e59396b5d..00e97c6d6576 100644
>> --- a/kernel/sysctl.c
>> +++ b/kernel/sysctl.c
>> @@ -61,7 +61,6 @@
>>   #include <linux/capability.h>
>>   #include <linux/binfmts.h>
>>   #include <linux/sched/sysctl.h>
>> -#include <linux/kexec.h>
>>   #include <linux/bpf.h>
>>   #include <linux/mount.h>
>>   #include <linux/userfaultfd_k.h>
>> @@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
>>   		.proc_handler	= tracepoint_printk_sysctl,
>>   	},
>>   #endif
>> -#ifdef CONFIG_KEXEC_CORE
>> -	{
>> -		.procname	= "kexec_load_disabled",
>> -		.data		= &kexec_load_disabled,
>> -		.maxlen		= sizeof(int),
>> -		.mode		= 0644,
>> -		/* only handle a transition from default "0" to "1" */
>> -		.proc_handler	= proc_dointvec_minmax,
>> -		.extra1		= SYSCTL_ONE,
>> -		.extra2		= SYSCTL_ONE,
>> -	},
>> -#endif
>>   #ifdef CONFIG_MODULES
>>   	{
>>   		.procname	= "modprobe",
>> -- 
>> 2.26.2
>>
> .


^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
  2022-02-24  1:04     ` yingelin
@ 2022-02-24  2:35       ` Baoquan He
  -1 siblings, 0 replies; 18+ messages in thread
From: Baoquan He @ 2022-02-24  2:35 UTC (permalink / raw)
  To: yingelin
  Cc: ebiederm, mcgrof, keescook, yzaikin, kexec, linux-kernel,
	linux-fsdevel, zengweilin, chenjianguo3, nixiaoming, qiuguorui1,
	young.liuyang

On 02/24/22 at 09:04am, yingelin wrote:
> 
> 在 2022/2/23 16:30, Baoquan He 写道:
> > On 02/23/22 at 11:03am, yingelin wrote:
> > > This move the kernel/kexec_core.c respective sysctls to its own file.
> > Hmm, why is the move needed?
> > 
> > With my understanding, sysctls are all put in kernel/sysctl.c,
> > why is kexec special?
> 
> kernel/sysctl.c is a kitchen sink where everyone leaves their dirty dishes,
> 
> this makes it very difficult to maintain. The proc sysctl maintainers do not
> want to
> 
> know what sysctl knobs you wish to add for your own piece of code, we
> 
> just care about the core logic.
> 
> This patch moves the kexec sysctls to the place where they actually belong
> to help

That seems to be an issue everything related to sysctl are all added to
kernel/sysctl.c. Do you have a pointer that someone complained about it
and people agree to scatter them into their own component code?

I understand your concern now, I am personally not confused by that
maybe because I haven't got stuff adding or changing into sysctls. My
concern is if we only care and move kexec knob, or we have plan to try
to move all of them. If there's some background information or
discussion with a link, that would be helpful.

Thanks
Baoquan

> 
> with this maintenance.
> 
> > > Signed-off-by: yingelin <yingelin@huawei.com>
> > > ---
> > >   kernel/kexec_core.c | 20 ++++++++++++++++++++
> > >   kernel/sysctl.c     | 13 -------------
> > >   2 files changed, 20 insertions(+), 13 deletions(-)
> > > 
> > > diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
> > > index 68480f731192..e57339d49439 100644
> > > --- a/kernel/kexec_core.c
> > > +++ b/kernel/kexec_core.c
> > > @@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
> > >   struct kimage *kexec_image;
> > >   struct kimage *kexec_crash_image;
> > >   int kexec_load_disabled;
> > > +static struct ctl_table kexec_core_sysctls[] = {
> > > +	{
> > > +		.procname	= "kexec_load_disabled",
> > > +		.data		= &kexec_load_disabled,
> > > +		.maxlen		= sizeof(int),
> > > +		.mode		= 0644,
> > > +		/* only handle a transition from default "0" to "1" */
> > > +		.proc_handler	= proc_dointvec_minmax,
> > > +		.extra1		= SYSCTL_ONE,
> > > +		.extra2		= SYSCTL_ONE,
> > > +	},
> > > +	{ }
> > > +};
> > > +
> > > +static int __init kexec_core_sysctl_init(void)
> > > +{
> > > +	register_sysctl_init("kernel", kexec_core_sysctls);
> > > +	return 0;
> > > +}
> > > +late_initcall(kexec_core_sysctl_init);
> > >   /*
> > >    * No panic_cpu check version of crash_kexec().  This function is called
> > > diff --git a/kernel/sysctl.c b/kernel/sysctl.c
> > > index ae5e59396b5d..00e97c6d6576 100644
> > > --- a/kernel/sysctl.c
> > > +++ b/kernel/sysctl.c
> > > @@ -61,7 +61,6 @@
> > >   #include <linux/capability.h>
> > >   #include <linux/binfmts.h>
> > >   #include <linux/sched/sysctl.h>
> > > -#include <linux/kexec.h>
> > >   #include <linux/bpf.h>
> > >   #include <linux/mount.h>
> > >   #include <linux/userfaultfd_k.h>
> > > @@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
> > >   		.proc_handler	= tracepoint_printk_sysctl,
> > >   	},
> > >   #endif
> > > -#ifdef CONFIG_KEXEC_CORE
> > > -	{
> > > -		.procname	= "kexec_load_disabled",
> > > -		.data		= &kexec_load_disabled,
> > > -		.maxlen		= sizeof(int),
> > > -		.mode		= 0644,
> > > -		/* only handle a transition from default "0" to "1" */
> > > -		.proc_handler	= proc_dointvec_minmax,
> > > -		.extra1		= SYSCTL_ONE,
> > > -		.extra2		= SYSCTL_ONE,
> > > -	},
> > > -#endif
> > >   #ifdef CONFIG_MODULES
> > >   	{
> > >   		.procname	= "modprobe",
> > > -- 
> > > 2.26.2
> > > 
> > .
> 


^ permalink raw reply	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-24  2:35       ` Baoquan He
  0 siblings, 0 replies; 18+ messages in thread
From: Baoquan He @ 2022-02-24  2:35 UTC (permalink / raw)
  To: kexec

On 02/24/22 at 09:04am, yingelin wrote:
> 
> ? 2022/2/23 16:30, Baoquan He ??:
> > On 02/23/22 at 11:03am, yingelin wrote:
> > > This move the kernel/kexec_core.c respective sysctls to its own file.
> > Hmm, why is the move needed?
> > 
> > With my understanding, sysctls are all put in kernel/sysctl.c,
> > why is kexec special?
> 
> kernel/sysctl.c is a kitchen sink where everyone leaves their dirty dishes,
> 
> this makes it very difficult to maintain. The proc sysctl maintainers do not
> want to
> 
> know what sysctl knobs you wish to add for your own piece of code, we
> 
> just care about the core logic.
> 
> This patch moves the kexec sysctls to the place where they actually belong
> to help

That seems to be an issue everything related to sysctl are all added to
kernel/sysctl.c. Do you have a pointer that someone complained about it
and people agree to scatter them into their own component code?

I understand your concern now, I am personally not confused by that
maybe because I haven't got stuff adding or changing into sysctls. My
concern is if we only care and move kexec knob, or we have plan to try
to move all of them. If there's some background information or
discussion with a link, that would be helpful.

Thanks
Baoquan

> 
> with this maintenance.
> 
> > > Signed-off-by: yingelin <yingelin@huawei.com>
> > > ---
> > >   kernel/kexec_core.c | 20 ++++++++++++++++++++
> > >   kernel/sysctl.c     | 13 -------------
> > >   2 files changed, 20 insertions(+), 13 deletions(-)
> > > 
> > > diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
> > > index 68480f731192..e57339d49439 100644
> > > --- a/kernel/kexec_core.c
> > > +++ b/kernel/kexec_core.c
> > > @@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
> > >   struct kimage *kexec_image;
> > >   struct kimage *kexec_crash_image;
> > >   int kexec_load_disabled;
> > > +static struct ctl_table kexec_core_sysctls[] = {
> > > +	{
> > > +		.procname	= "kexec_load_disabled",
> > > +		.data		= &kexec_load_disabled,
> > > +		.maxlen		= sizeof(int),
> > > +		.mode		= 0644,
> > > +		/* only handle a transition from default "0" to "1" */
> > > +		.proc_handler	= proc_dointvec_minmax,
> > > +		.extra1		= SYSCTL_ONE,
> > > +		.extra2		= SYSCTL_ONE,
> > > +	},
> > > +	{ }
> > > +};
> > > +
> > > +static int __init kexec_core_sysctl_init(void)
> > > +{
> > > +	register_sysctl_init("kernel", kexec_core_sysctls);
> > > +	return 0;
> > > +}
> > > +late_initcall(kexec_core_sysctl_init);
> > >   /*
> > >    * No panic_cpu check version of crash_kexec().  This function is called
> > > diff --git a/kernel/sysctl.c b/kernel/sysctl.c
> > > index ae5e59396b5d..00e97c6d6576 100644
> > > --- a/kernel/sysctl.c
> > > +++ b/kernel/sysctl.c
> > > @@ -61,7 +61,6 @@
> > >   #include <linux/capability.h>
> > >   #include <linux/binfmts.h>
> > >   #include <linux/sched/sysctl.h>
> > > -#include <linux/kexec.h>
> > >   #include <linux/bpf.h>
> > >   #include <linux/mount.h>
> > >   #include <linux/userfaultfd_k.h>
> > > @@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
> > >   		.proc_handler	= tracepoint_printk_sysctl,
> > >   	},
> > >   #endif
> > > -#ifdef CONFIG_KEXEC_CORE
> > > -	{
> > > -		.procname	= "kexec_load_disabled",
> > > -		.data		= &kexec_load_disabled,
> > > -		.maxlen		= sizeof(int),
> > > -		.mode		= 0644,
> > > -		/* only handle a transition from default "0" to "1" */
> > > -		.proc_handler	= proc_dointvec_minmax,
> > > -		.extra1		= SYSCTL_ONE,
> > > -		.extra2		= SYSCTL_ONE,
> > > -	},
> > > -#endif
> > >   #ifdef CONFIG_MODULES
> > >   	{
> > >   		.procname	= "modprobe",
> > > -- 
> > > 2.26.2
> > > 
> > .
> 



^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
  2022-02-24  2:35       ` Baoquan He
@ 2022-02-26 20:20         ` Luis Chamberlain
  -1 siblings, 0 replies; 18+ messages in thread
From: Luis Chamberlain @ 2022-02-26 20:20 UTC (permalink / raw)
  To: Baoquan He
  Cc: yingelin, ebiederm, keescook, yzaikin, kexec, linux-kernel,
	linux-fsdevel, zengweilin, chenjianguo3, nixiaoming, qiuguorui1,
	young.liuyang

On Thu, Feb 24, 2022 at 10:35:21AM +0800, Baoquan He wrote:
> That seems to be an issue everything related to sysctl are all added to
> kernel/sysctl.c. Do you have a pointer that someone complained about it
> and people agree to scatter them into their own component code?

https://lkml.kernel.org/r/20220226031054.47DF8C340E7@smtp.kernel.org

> I understand your concern now, I am personally not confused by that
> maybe because I haven't got stuff adding or changing into sysctls. My
> concern is if we only care and move kexec knob, or we have plan to try
> to move all of them. If there's some background information or
> discussion with a link, that would be helpful.

We're moving them all out. Sorry, yingelin's commit log message sucks
and it needs to be fixed to account for the justification. All the
filesystem sysctls are already moved out. Slowly we are moving the other
ones out and also doing minor optimizations along the way.

  Luis

^ permalink raw reply	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-26 20:20         ` Luis Chamberlain
  0 siblings, 0 replies; 18+ messages in thread
From: Luis Chamberlain @ 2022-02-26 20:20 UTC (permalink / raw)
  To: kexec

On Thu, Feb 24, 2022 at 10:35:21AM +0800, Baoquan He wrote:
> That seems to be an issue everything related to sysctl are all added to
> kernel/sysctl.c. Do you have a pointer that someone complained about it
> and people agree to scatter them into their own component code?

https://lkml.kernel.org/r/20220226031054.47DF8C340E7 at smtp.kernel.org

> I understand your concern now, I am personally not confused by that
> maybe because I haven't got stuff adding or changing into sysctls. My
> concern is if we only care and move kexec knob, or we have plan to try
> to move all of them. If there's some background information or
> discussion with a link, that would be helpful.

We're moving them all out. Sorry, yingelin's commit log message sucks
and it needs to be fixed to account for the justification. All the
filesystem sysctls are already moved out. Slowly we are moving the other
ones out and also doing minor optimizations along the way.

  Luis


^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
  2022-02-24  2:35       ` Baoquan He
@ 2022-02-28  1:41         ` yingelin
  -1 siblings, 0 replies; 18+ messages in thread
From: yingelin @ 2022-02-28  1:41 UTC (permalink / raw)
  To: Baoquan He
  Cc: ebiederm, mcgrof, keescook, yzaikin, kexec, linux-kernel,
	linux-fsdevel, zengweilin, chenjianguo3, nixiaoming, qiuguorui1,
	young.liuyang


在 2022/2/24 10:35, Baoquan He 写道:
> On 02/24/22 at 09:04am, yingelin wrote:
>> 在 2022/2/23 16:30, Baoquan He 写道:
>>> On 02/23/22 at 11:03am, yingelin wrote:
>>>> This move the kernel/kexec_core.c respective sysctls to its own file.
>>> Hmm, why is the move needed?
>>>
>>> With my understanding, sysctls are all put in kernel/sysctl.c,
>>> why is kexec special?
>> kernel/sysctl.c is a kitchen sink where everyone leaves their dirty dishes,
>>
>> this makes it very difficult to maintain. The proc sysctl maintainers do not
>> want to
>>
>> know what sysctl knobs you wish to add for your own piece of code, we
>>
>> just care about the core logic.
>>
>> This patch moves the kexec sysctls to the place where they actually belong
>> to help
> That seems to be an issue everything related to sysctl are all added to
> kernel/sysctl.c. Do you have a pointer that someone complained about it
> and people agree to scatter them into their own component code?

I'm sorry to reply you too late, the link is

https://lkml.kernel.org/r/20220226031054.47DF8C340E7@smtp.kernel.org

> I understand your concern now, I am personally not confused by that
> maybe because I haven't got stuff adding or changing into sysctls. My
> concern is if we only care and move kexec knob, or we have plan to try
> to move all of them. If there's some background information or
> discussion with a link, that would be helpful.

Yeah, we are going to move all sysctls to their own places, in fact, all 
the filesystem

sysctls are moved out already. I'm sorry I didn't express it clearly. 
I'll fix it in v2 patch.

>
> Thanks
> Baoquan
>
>> with this maintenance.
>>
>>>> Signed-off-by: yingelin <yingelin@huawei.com>
>>>> ---
>>>>    kernel/kexec_core.c | 20 ++++++++++++++++++++
>>>>    kernel/sysctl.c     | 13 -------------
>>>>    2 files changed, 20 insertions(+), 13 deletions(-)
>>>>
>>>> diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
>>>> index 68480f731192..e57339d49439 100644
>>>> --- a/kernel/kexec_core.c
>>>> +++ b/kernel/kexec_core.c
>>>> @@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
>>>>    struct kimage *kexec_image;
>>>>    struct kimage *kexec_crash_image;
>>>>    int kexec_load_disabled;
>>>> +static struct ctl_table kexec_core_sysctls[] = {
>>>> +	{
>>>> +		.procname	= "kexec_load_disabled",
>>>> +		.data		= &kexec_load_disabled,
>>>> +		.maxlen		= sizeof(int),
>>>> +		.mode		= 0644,
>>>> +		/* only handle a transition from default "0" to "1" */
>>>> +		.proc_handler	= proc_dointvec_minmax,
>>>> +		.extra1		= SYSCTL_ONE,
>>>> +		.extra2		= SYSCTL_ONE,
>>>> +	},
>>>> +	{ }
>>>> +};
>>>> +
>>>> +static int __init kexec_core_sysctl_init(void)
>>>> +{
>>>> +	register_sysctl_init("kernel", kexec_core_sysctls);
>>>> +	return 0;
>>>> +}
>>>> +late_initcall(kexec_core_sysctl_init);
>>>>    /*
>>>>     * No panic_cpu check version of crash_kexec().  This function is called
>>>> diff --git a/kernel/sysctl.c b/kernel/sysctl.c
>>>> index ae5e59396b5d..00e97c6d6576 100644
>>>> --- a/kernel/sysctl.c
>>>> +++ b/kernel/sysctl.c
>>>> @@ -61,7 +61,6 @@
>>>>    #include <linux/capability.h>
>>>>    #include <linux/binfmts.h>
>>>>    #include <linux/sched/sysctl.h>
>>>> -#include <linux/kexec.h>
>>>>    #include <linux/bpf.h>
>>>>    #include <linux/mount.h>
>>>>    #include <linux/userfaultfd_k.h>
>>>> @@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
>>>>    		.proc_handler	= tracepoint_printk_sysctl,
>>>>    	},
>>>>    #endif
>>>> -#ifdef CONFIG_KEXEC_CORE
>>>> -	{
>>>> -		.procname	= "kexec_load_disabled",
>>>> -		.data		= &kexec_load_disabled,
>>>> -		.maxlen		= sizeof(int),
>>>> -		.mode		= 0644,
>>>> -		/* only handle a transition from default "0" to "1" */
>>>> -		.proc_handler	= proc_dointvec_minmax,
>>>> -		.extra1		= SYSCTL_ONE,
>>>> -		.extra2		= SYSCTL_ONE,
>>>> -	},
>>>> -#endif
>>>>    #ifdef CONFIG_MODULES
>>>>    	{
>>>>    		.procname	= "modprobe",
>>>> -- 
>>>> 2.26.2
>>>>
>>> .
> .

^ permalink raw reply	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-28  1:41         ` yingelin
  0 siblings, 0 replies; 18+ messages in thread
From: yingelin @ 2022-02-28  1:41 UTC (permalink / raw)
  To: kexec


? 2022/2/24 10:35, Baoquan He ??:
> On 02/24/22 at 09:04am, yingelin wrote:
>> ? 2022/2/23 16:30, Baoquan He ??:
>>> On 02/23/22 at 11:03am, yingelin wrote:
>>>> This move the kernel/kexec_core.c respective sysctls to its own file.
>>> Hmm, why is the move needed?
>>>
>>> With my understanding, sysctls are all put in kernel/sysctl.c,
>>> why is kexec special?
>> kernel/sysctl.c is a kitchen sink where everyone leaves their dirty dishes,
>>
>> this makes it very difficult to maintain. The proc sysctl maintainers do not
>> want to
>>
>> know what sysctl knobs you wish to add for your own piece of code, we
>>
>> just care about the core logic.
>>
>> This patch moves the kexec sysctls to the place where they actually belong
>> to help
> That seems to be an issue everything related to sysctl are all added to
> kernel/sysctl.c. Do you have a pointer that someone complained about it
> and people agree to scatter them into their own component code?

I'm sorry to reply you too late, the link is

https://lkml.kernel.org/r/20220226031054.47DF8C340E7 at smtp.kernel.org

> I understand your concern now, I am personally not confused by that
> maybe because I haven't got stuff adding or changing into sysctls. My
> concern is if we only care and move kexec knob, or we have plan to try
> to move all of them. If there's some background information or
> discussion with a link, that would be helpful.

Yeah, we are going to move all sysctls to their own places, in fact, all 
the filesystem

sysctls are moved out already. I'm sorry I didn't express it clearly. 
I'll fix it in v2 patch.

>
> Thanks
> Baoquan
>
>> with this maintenance.
>>
>>>> Signed-off-by: yingelin <yingelin@huawei.com>
>>>> ---
>>>>    kernel/kexec_core.c | 20 ++++++++++++++++++++
>>>>    kernel/sysctl.c     | 13 -------------
>>>>    2 files changed, 20 insertions(+), 13 deletions(-)
>>>>
>>>> diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
>>>> index 68480f731192..e57339d49439 100644
>>>> --- a/kernel/kexec_core.c
>>>> +++ b/kernel/kexec_core.c
>>>> @@ -936,6 +936,26 @@ int kimage_load_segment(struct kimage *image,
>>>>    struct kimage *kexec_image;
>>>>    struct kimage *kexec_crash_image;
>>>>    int kexec_load_disabled;
>>>> +static struct ctl_table kexec_core_sysctls[] = {
>>>> +	{
>>>> +		.procname	= "kexec_load_disabled",
>>>> +		.data		= &kexec_load_disabled,
>>>> +		.maxlen		= sizeof(int),
>>>> +		.mode		= 0644,
>>>> +		/* only handle a transition from default "0" to "1" */
>>>> +		.proc_handler	= proc_dointvec_minmax,
>>>> +		.extra1		= SYSCTL_ONE,
>>>> +		.extra2		= SYSCTL_ONE,
>>>> +	},
>>>> +	{ }
>>>> +};
>>>> +
>>>> +static int __init kexec_core_sysctl_init(void)
>>>> +{
>>>> +	register_sysctl_init("kernel", kexec_core_sysctls);
>>>> +	return 0;
>>>> +}
>>>> +late_initcall(kexec_core_sysctl_init);
>>>>    /*
>>>>     * No panic_cpu check version of crash_kexec().  This function is called
>>>> diff --git a/kernel/sysctl.c b/kernel/sysctl.c
>>>> index ae5e59396b5d..00e97c6d6576 100644
>>>> --- a/kernel/sysctl.c
>>>> +++ b/kernel/sysctl.c
>>>> @@ -61,7 +61,6 @@
>>>>    #include <linux/capability.h>
>>>>    #include <linux/binfmts.h>
>>>>    #include <linux/sched/sysctl.h>
>>>> -#include <linux/kexec.h>
>>>>    #include <linux/bpf.h>
>>>>    #include <linux/mount.h>
>>>>    #include <linux/userfaultfd_k.h>
>>>> @@ -1839,18 +1838,6 @@ static struct ctl_table kern_table[] = {
>>>>    		.proc_handler	= tracepoint_printk_sysctl,
>>>>    	},
>>>>    #endif
>>>> -#ifdef CONFIG_KEXEC_CORE
>>>> -	{
>>>> -		.procname	= "kexec_load_disabled",
>>>> -		.data		= &kexec_load_disabled,
>>>> -		.maxlen		= sizeof(int),
>>>> -		.mode		= 0644,
>>>> -		/* only handle a transition from default "0" to "1" */
>>>> -		.proc_handler	= proc_dointvec_minmax,
>>>> -		.extra1		= SYSCTL_ONE,
>>>> -		.extra2		= SYSCTL_ONE,
>>>> -	},
>>>> -#endif
>>>>    #ifdef CONFIG_MODULES
>>>>    	{
>>>>    		.procname	= "modprobe",
>>>> -- 
>>>> 2.26.2
>>>>
>>> .
> .


^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
  2022-02-26 20:20         ` Luis Chamberlain
@ 2022-02-28  1:43           ` yingelin
  -1 siblings, 0 replies; 18+ messages in thread
From: yingelin @ 2022-02-28  1:43 UTC (permalink / raw)
  To: Luis Chamberlain, Baoquan He
  Cc: ebiederm, keescook, yzaikin, kexec, linux-kernel, linux-fsdevel,
	zengweilin, chenjianguo3, nixiaoming, qiuguorui1, young.liuyang


在 2022/2/27 4:20, Luis Chamberlain 写道:
> On Thu, Feb 24, 2022 at 10:35:21AM +0800, Baoquan He wrote:
>> That seems to be an issue everything related to sysctl are all added to
>> kernel/sysctl.c. Do you have a pointer that someone complained about it
>> and people agree to scatter them into their own component code?
> https://lkml.kernel.org/r/20220226031054.47DF8C340E7@smtp.kernel.org
>
>> I understand your concern now, I am personally not confused by that
>> maybe because I haven't got stuff adding or changing into sysctls. My
>> concern is if we only care and move kexec knob, or we have plan to try
>> to move all of them. If there's some background information or
>> discussion with a link, that would be helpful.
> We're moving them all out. Sorry, yingelin's commit log message sucks
> and it needs to be fixed to account for the justification. All the
> filesystem sysctls are already moved out. Slowly we are moving the other
> ones out and also doing minor optimizations along the way.
I'm sorry I didn't express it clearly. I'll fix it in v2 patch.
>
>    Luis
> .

^ permalink raw reply	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-28  1:43           ` yingelin
  0 siblings, 0 replies; 18+ messages in thread
From: yingelin @ 2022-02-28  1:43 UTC (permalink / raw)
  To: kexec


? 2022/2/27 4:20, Luis Chamberlain ??:
> On Thu, Feb 24, 2022 at 10:35:21AM +0800, Baoquan He wrote:
>> That seems to be an issue everything related to sysctl are all added to
>> kernel/sysctl.c. Do you have a pointer that someone complained about it
>> and people agree to scatter them into their own component code?
> https://lkml.kernel.org/r/20220226031054.47DF8C340E7 at smtp.kernel.org
>
>> I understand your concern now, I am personally not confused by that
>> maybe because I haven't got stuff adding or changing into sysctls. My
>> concern is if we only care and move kexec knob, or we have plan to try
>> to move all of them. If there's some background information or
>> discussion with a link, that would be helpful.
> We're moving them all out. Sorry, yingelin's commit log message sucks
> and it needs to be fixed to account for the justification. All the
> filesystem sysctls are already moved out. Slowly we are moving the other
> ones out and also doing minor optimizations along the way.
I'm sorry I didn't express it clearly. I'll fix it in v2 patch.
>
>    Luis
> .


^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
  2022-02-28  1:43           ` yingelin
@ 2022-02-28  3:18             ` Baoquan He
  -1 siblings, 0 replies; 18+ messages in thread
From: Baoquan He @ 2022-02-28  3:18 UTC (permalink / raw)
  To: Luis Chamberlain, yingelin
  Cc: ebiederm, keescook, yzaikin, kexec, linux-kernel, linux-fsdevel,
	zengweilin, chenjianguo3, nixiaoming, qiuguorui1, young.liuyang

On 02/28/22 at 09:43am, yingelin wrote:
> 
> 在 2022/2/27 4:20, Luis Chamberlain 写道:
> > On Thu, Feb 24, 2022 at 10:35:21AM +0800, Baoquan He wrote:
> > > That seems to be an issue everything related to sysctl are all added to
> > > kernel/sysctl.c. Do you have a pointer that someone complained about it
> > > and people agree to scatter them into their own component code?
> > https://lkml.kernel.org/r/20220226031054.47DF8C340E7@smtp.kernel.org
> > 
> > > I understand your concern now, I am personally not confused by that
> > > maybe because I haven't got stuff adding or changing into sysctls. My
> > > concern is if we only care and move kexec knob, or we have plan to try
> > > to move all of them. If there's some background information or
> > > discussion with a link, that would be helpful.
> > We're moving them all out. Sorry, yingelin's commit log message sucks
> > and it needs to be fixed to account for the justification. All the
> > filesystem sysctls are already moved out. Slowly we are moving the other
> > ones out and also doing minor optimizations along the way.

Thanks for the explanation. Yeah, not like code chaging for fixing or
improvement, providing justification is very helpful and imporant for
this kind of code moving.


> I'm sorry I didn't express it clearly. I'll fix it in v2 patch.

That's OK, look forward to seeing the v2.


^ permalink raw reply	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-02-28  3:18             ` Baoquan He
  0 siblings, 0 replies; 18+ messages in thread
From: Baoquan He @ 2022-02-28  3:18 UTC (permalink / raw)
  To: kexec

On 02/28/22 at 09:43am, yingelin wrote:
> 
> ? 2022/2/27 4:20, Luis Chamberlain ??:
> > On Thu, Feb 24, 2022 at 10:35:21AM +0800, Baoquan He wrote:
> > > That seems to be an issue everything related to sysctl are all added to
> > > kernel/sysctl.c. Do you have a pointer that someone complained about it
> > > and people agree to scatter them into their own component code?
> > https://lkml.kernel.org/r/20220226031054.47DF8C340E7 at smtp.kernel.org
> > 
> > > I understand your concern now, I am personally not confused by that
> > > maybe because I haven't got stuff adding or changing into sysctls. My
> > > concern is if we only care and move kexec knob, or we have plan to try
> > > to move all of them. If there's some background information or
> > > discussion with a link, that would be helpful.
> > We're moving them all out. Sorry, yingelin's commit log message sucks
> > and it needs to be fixed to account for the justification. All the
> > filesystem sysctls are already moved out. Slowly we are moving the other
> > ones out and also doing minor optimizations along the way.

Thanks for the explanation. Yeah, not like code chaging for fixing or
improvement, providing justification is very helpful and imporant for
this kind of code moving.


> I'm sorry I didn't express it clearly. I'll fix it in v2 patch.

That's OK, look forward to seeing the v2.



^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
  2022-02-28  3:18             ` Baoquan He
@ 2022-04-15 22:59               ` Luis Chamberlain
  -1 siblings, 0 replies; 18+ messages in thread
From: Luis Chamberlain @ 2022-04-15 22:59 UTC (permalink / raw)
  To: Baoquan He
  Cc: yingelin, ebiederm, keescook, yzaikin, kexec, linux-kernel,
	linux-fsdevel, zengweilin, chenjianguo3, nixiaoming, qiuguorui1,
	young.liuyang

On Mon, Feb 28, 2022 at 11:18:01AM +0800, Baoquan He wrote:
> That's OK, look forward to seeing the v2.

yingelin, do you plan to post a v2? If splease base it on sysctl-testing [0]

[0] https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux.git/log/?h=sysctl-testing

  Luis

^ permalink raw reply	[flat|nested] 18+ messages in thread
* [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file
@ 2022-04-15 22:59               ` Luis Chamberlain
  0 siblings, 0 replies; 18+ messages in thread
From: Luis Chamberlain @ 2022-04-15 22:59 UTC (permalink / raw)
  To: kexec

On Mon, Feb 28, 2022 at 11:18:01AM +0800, Baoquan He wrote:
> That's OK, look forward to seeing the v2.

yingelin, do you plan to post a v2? If splease base it on sysctl-testing [0]

[0] https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux.git/log/?h=sysctl-testing

  Luis


^ permalink raw reply	[flat|nested] 18+ messages in thread
end of thread, other threads:[~2022-04-15 22:59 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-23  3:03 [PATCH sysctl-next] kernel/kexec_core: move kexec_core sysctls into its own file yingelin
2022-02-23  3:03 ` yingelin
2022-02-23  8:30 ` Baoquan He
2022-02-23  8:30   ` Baoquan He
2022-02-24  1:04   ` yingelin
2022-02-24  1:04     ` yingelin
2022-02-24  2:35     ` Baoquan He
2022-02-24  2:35       ` Baoquan He
2022-02-26 20:20       ` Luis Chamberlain
2022-02-26 20:20         ` Luis Chamberlain
2022-02-28  1:43         ` yingelin
2022-02-28  1:43           ` yingelin
2022-02-28  3:18           ` Baoquan He
2022-02-28  3:18             ` Baoquan He
2022-04-15 22:59             ` Luis Chamberlain
2022-04-15 22:59               ` Luis Chamberlain
2022-02-28  1:41       ` yingelin
2022-02-28  1:41         ` yingelin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.