[PATCH] fff-hoods: remove dependency to /tmp/started

Adrian Schmutzler mail at adrianschmutzler.de
Mi Jun 13 13:37:33 CEST 2018


Hallo Fabian/Tim,

bitte meinen alten Kommentar beachten:

> weiß nicht mehr, ob wir das schon diskutiert haben, aber:
> Das Locking verhindert, dass das File zweimal gleichzeitig läuft.
> Micrond wird aber mit S50 gestartet, configurenetwork mit S95.

> D.h. es kann ohne /tmp/started dazu kommen (wenn Zeitfenster blöd), dass
> durch micrond configurehood VOR configurenetwork ausführt, was definitiv
> nicht gut ist.

> Dies betrifft nur das Entfernen der /tmp/started Bedingung, das Lock selbst
> ist davon ja nicht betroffen.

Ich habe kein Problem mit dem Locking, ich würde nur das /tmp/started trotzdem im micrond belassen, dann kann gar nichts mehr passieren!

Solange das /tmp/started entfernt wird, ist das daher für mich NAK.

Grüße

Adrian


> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Fabian Bläse
> Sent: Mittwoch, 13. Juni 2018 09:16
> To: Tim Niemeyer <tim at tn-x.org>
> Cc: franken-dev <franken-dev at freifunk.net>
> Subject: Re: [PATCH] fff-hoods: remove dependency to /tmp/started
> 
> Ich wollte an dieser Stelle nochmal an dieses Patch erinnern.
> Imo will man das unbedingt noch im Release haben.
> 
> Soweit ich das mit dem trap richtig überblicke, müsste das Lockfile auch
> immer sauber gelöscht werden.
> 
> Grade auch gesehen, dass noch was fehlt:
> Reviewed-by: Fabian Bläse <fabian at blaese.de>
> 
> > On 20. Jan 2018, at 23:03, Tim Niemeyer <tim at tn-x.org> wrote:
> >
> > The dependency was only to avoid duplicated script instances.
> > Therefore, this patch implements a solution without the need to depend
> > on the /tmp/started file.
> >
> > Signed-off-by: Tim Niemeyer <tim at tn-x.org>
> > ---
> >
> > src/packages/fff/fff-hoods/Makefile                         | 2 +-
> > src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods | 2 +-
> > src/packages/fff/fff-hoods/files/usr/sbin/configurehood     | 7 +++++++
> > 3 files changed, 9 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/packages/fff/fff-hoods/Makefile
> > b/src/packages/fff/fff-hoods/Makefile
> > index 11ab6d1..1a616ff 100644
> > --- a/src/packages/fff/fff-hoods/Makefile
> > +++ b/src/packages/fff/fff-hoods/Makefile
> > @@ -1,7 +1,7 @@
> > include $(TOPDIR)/rules.mk
> >
> > PKG_NAME:=fff-hoods
> > -PKG_VERSION:=0.0.1
> > +PKG_VERSION:=2
> > PKG_RELEASE:=1
> >
> > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
> > diff --git
> > a/src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods
> > b/src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods
> > index 39e800e..ca8d798 100644
> > --- a/src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods
> > +++ b/src/packages/fff/fff-hoods/files/usr/lib/micron.d/fff-hoods
> > @@ -1 +1 @@
> > -*/5 * * * * [ -f /tmp/started ] && /usr/sbin/configurehood
> > +*/5 * * * * /usr/sbin/configurehood
> > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > index 2aaa245..e14995b 100755
> > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > @@ -1,5 +1,12 @@
> > #!/bin/sh
> >
> > +lockfile="/var/lock/${0##*/}.lock"
> > +if ! lock -n "$lockfile"; then
> > +	echo "Only one instance of $0 allowed."
> > +	exit 1
> > +fi
> > +trap "lock -u \"$lockfile\"" INT TERM EXIT
> > +
> > . /usr/share/libubox/jshn.sh
> > . /lib/functions/fff/keyxchange
> > . /lib/functions/fff/network
> > --
> > 2.11.0
> >
> > --
> > franken-dev mailing list
> > franken-dev at freifunk.net
> > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net




Mehr Informationen über die Mailingliste franken-dev