[v3,7/8] root_file_system: Remove sysctl.conf

Submitted by Adrian Schmutzler on Aug. 4, 2018, 2:34 p.m.

Details

Message ID 20180804143427.18488-8-freifunk@adrianschmutzler.de
State Rejected
Headers show

Commit Message

Adrian Schmutzler Aug. 4, 2018, 2:34 p.m.
OpenWRT changed their structure to use sysctl.d subfiles.

The default sysctl.conf is empty and kernel.panic=3 is defined
elsewhere. Thus, we do not need sysctl.conf anymore.

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 bsp/default/root_file_system/etc/sysctl.conf | 1 -
 1 file changed, 1 deletion(-)
 delete mode 100644 bsp/default/root_file_system/etc/sysctl.conf

Patch hide | download patch | download mbox

diff --git a/bsp/default/root_file_system/etc/sysctl.conf b/bsp/default/root_file_system/etc/sysctl.conf
deleted file mode 100644
index 34ce7087..00000000
--- a/bsp/default/root_file_system/etc/sysctl.conf
+++ /dev/null
@@ -1 +0,0 @@ 
-kernel.panic=3

Comments

Tim Niemeyer Aug. 5, 2018, 3:32 p.m.
Am Samstag, den 04.08.2018, 16:34 +0200 schrieb Adrian Schmutzler:
> OpenWRT changed their structure to use sysctl.d subfiles.
> 
> The default sysctl.conf is empty and kernel.panic=3 is defined
> elsewhere.
Ok, ich glaub das einfach mal.

>  Thus, we do not need sysctl.conf anymore.
Ok.

Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  bsp/default/root_file_system/etc/sysctl.conf | 1 -
>  1 file changed, 1 deletion(-)
>  delete mode 100644 bsp/default/root_file_system/etc/sysctl.conf
> 
> diff --git a/bsp/default/root_file_system/etc/sysctl.conf
> b/bsp/default/root_file_system/etc/sysctl.conf
> deleted file mode 100644
> index 34ce7087..00000000
> --- a/bsp/default/root_file_system/etc/sysctl.conf
> +++ /dev/null
> @@ -1 +0,0 @@
> -kernel.panic=3
Adrian Schmutzler Aug. 5, 2018, 9:22 p.m.
Hallo Tim,

siehe hier:

https://github.com/openwrt/openwrt/search?q=sysctl.d&type=Commits

Dabei ist besonders zu beachten, dass openwrt jetzt selbst Dateien im Ordner /etc/sysctl.d nutzt.

Diese werden also nicht wie bisher durch Überschreiben der Datei sysctl.conf gelöscht, sondern wir müssen die einzeln in unseren /etc/sysctl.d Dateien überschreiben. Ich habe mir das damals einmal im Detail angeschaut und habe in Erinnerung, das wir hier aber tatsächlich alle relevanten Werte überschreiben.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Tim Niemeyer
> Sent: Sonntag, 5. August 2018 17:32
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> dev@freifunk.net
> Subject: Re: [PATCH v3 7/8] root_file_system: Remove sysctl.conf
> 
> Am Samstag, den 04.08.2018, 16:34 +0200 schrieb Adrian Schmutzler:
> > OpenWRT changed their structure to use sysctl.d subfiles.
> >
> > The default sysctl.conf is empty and kernel.panic=3 is defined
> > elsewhere.
> Ok, ich glaub das einfach mal.
> 
> >  Thus, we do not need sysctl.conf anymore.
> Ok.
> 
> Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> >
> > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > ---
> >  bsp/default/root_file_system/etc/sysctl.conf | 1 -
> >  1 file changed, 1 deletion(-)
> >  delete mode 100644 bsp/default/root_file_system/etc/sysctl.conf
> >
> > diff --git a/bsp/default/root_file_system/etc/sysctl.conf
> > b/bsp/default/root_file_system/etc/sysctl.conf
> > deleted file mode 100644
> > index 34ce7087..00000000
> > --- a/bsp/default/root_file_system/etc/sysctl.conf
> > +++ /dev/null
> > @@ -1 +0,0 @@
> > -kernel.panic=3