Re: [ptxdist] [PATCH 28/48] [procps] Added installation of alternative /etc/sysctl.conf
On Fri, Jan 08, 2010 at 12:14:06AM +0100, Carsten Schlote wrote: Am Freitag, den 08.01.2010, 00:03 +0100 schrieb Ladislav Michl: As newer kernels emulates binary interface we probably do not want to support obsolete interfaces, right? http://lwn.net/Articles/361001/ Hi Thanks for the point. As sysctl was in our firmware for a long time and is still used for older kernels, I added this patch. I will add the comments to the patch, so everyone knows as well. And of course I will remove sysctl from our current dev-firmware asap. Nevertheless I need it for the old projects - all of them are maintained to work with the latest ptxdist version. Then it could be easy to read/write files in /proc/sys directly, unless you found some sysctl entry which does not have its equivalent in /proc/sys. Is it the case? Otherwise, it should be only matter of script(s) modification. ladis -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH 28/48] [procps] Added installation of alternative /etc/sysctl.conf
On Fri, Jan 08, 2010 at 12:14:45PM +0100, Carsten Schlote wrote: -Urspr?ngliche Nachricht- Then it could be easy to read/write files in /proc/sys directly, unless you found some sysctl entry which does not have its equivalent in /proc/sys. Is it the case? Otherwise, it should be only matter of script(s) modification. Sure, and i'm going to replace the stuff. But patch 28 just introduced the installation of sysctl.conf after installing sysctl. This should be no problem at all and either uses an empty generic file or a user version from project root. In case you install sysctl for your project, the startup system will check for this config file and call sysctl. I'm perfectly fine with that. I'd just like to see no PROCPS_SYSCTL in PTXdist config files at all, which is rather hard because 1) we are importing Busybox' version anyway 2) people won't understand it was removed for their own good ;-) So lets individuals decide and document it. ladis (thinking enlighted autocracy is the only right form of governement) -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH 28/48] [procps] Added installation of alternative /etc/sysctl.conf
carsten.schl...@gmx.net wrote: From: Carsten Schlote c.schl...@konzeptpark.de Signed-off-by: Carsten Schlote c.schl...@konzeptpark.de applied, tnx marc --- generic/etc/sysctl.conf |4 rules/procps.make |2 ++ 2 files changed, 6 insertions(+), 0 deletions(-) create mode 100644 generic/etc/sysctl.conf diff --git a/generic/etc/sysctl.conf b/generic/etc/sysctl.conf new file mode 100644 index 000..b10c21d --- /dev/null +++ b/generic/etc/sysctl.conf @@ -0,0 +1,4 @@ +# +# /etc/sysctl.conf - Configuration file for setting system variables +# See sysctl.conf (5) for information. +# diff --git a/rules/procps.make b/rules/procps.make index 57f8fdb..403714b 100644 --- a/rules/procps.make +++ b/rules/procps.make @@ -90,6 +90,8 @@ endif ifdef PTXCONF_PROCPS_SYSCTL @$(call install_copy, procps, 0, 0, 0755, -, \ /sbin/sysctl) + + @$(call install_alternative, procps, 0, 0, 0644, /etc/sysctl.conf, n) endif ifdef PTXCONF_PROCPS_PS @$(call install_copy, procps, 0, 0, 0755, -, \ -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions| Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917- | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | signature.asc Description: OpenPGP digital signature -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH 28/48] [procps] Added installation of alternative /etc/sysctl.conf
Ladislav Michl wrote: On Thu, Jan 07, 2010 at 09:28:07PM +0100, Marc Kleine-Budde wrote: carsten.schl...@gmx.net wrote: From: Carsten Schlote c.schl...@konzeptpark.de Signed-off-by: Carsten Schlote c.schl...@konzeptpark.de applied, tnx As newer kernels emulates binary interface we probably do not want to support obsolete interfaces, right? http://lwn.net/Articles/361001/ Kernel help says: | CONFIG_SYSCTL_SYSCALL: | | | | sys_sysctl uses binary paths that have been found challenging | | to properly maintain and use. The interface in /proc/sys | | using paths with ascii names is now the primary path to this | | information. | | | | Almost nothing using the binary sysctl interface so if you are | | trying to save some space it is probably safe to disable this, | | making your kernel marginally smaller. ... and your system will get even smaller without sysctl binary. Btw, 'sysctl' branch was merged 24 Dec 2009. Some people can really enjoy Christmas ;-) luckily, I haven't pushed that commit not yet. :) However it's probably interesting for olther kernels than the latest -rc :) Carsten can you please add a notice to PTXCONF_PROCPS_SYSCTL's helptext stating that this is a deprecated interface. cheers, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions| Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917- | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | signature.asc Description: OpenPGP digital signature -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH 28/48] [procps] Added installation of alternative /etc/sysctl.conf
Am Freitag, den 08.01.2010, 00:03 +0100 schrieb Ladislav Michl: As newer kernels emulates binary interface we probably do not want to support obsolete interfaces, right? http://lwn.net/Articles/361001/ Hi Thanks for the point. As sysctl was in our firmware for a long time and is still used for older kernels, I added this patch. I will add the comments to the patch, so everyone knows as well. And of course I will remove sysctl from our current dev-firmware asap. Nevertheless I need it for the old projects - all of them are maintained to work with the latest ptxdist version. -- Carsten Schlote carsten.schl...@gmx.net signature.asc Description: Dies ist ein digital signierter Nachrichtenteil -- ptxdist mailing list ptxdist@pengutronix.de
