[Patch] VDR 1.6.0 wird nicht korrekt entladen

Zitat

Original von TomJoad
Ihr habt doch beide Recht. Wenn vdr mit

Code

export LD_ASSUME_KERNEL=2.4.1

startet wie z.B. bei easyvdr, bekommt jeder thread eine eigene PID.

LD_ASSUME_KERNEL=.. macht eigentlich keinen Sinn, da der VDR seit Version 1.4.x NPTL kompatibel ist. Vermutlich gibts aber irgendein Plugin, das unbedingt unterstützt werden muß und das eine Library verwendet, zu der man keine Sourcen hat und die ohne NPTL Support kompiliert wurde. Da sollte man mal fragen, wie es denn mit der Lizenz von EasyVdr und den verwendeten Programmen aussieht, GPL bzw. LGPL konform ist das sicherlich nicht.

Gruß
e9hack

Zitat

Original von ardi
ich habe jetzt herausgefunden, dass Easyvdr 10x (im Sekundentakt) versucht den vdr zu beenden.

In runvdr gibts bei EasyVdr folgend Sequenz:

down)
    ##### VDR beenden #####
    WAIT=$KILLWAIT
    PID=$(pidof vdr)
    if [ -n "$PID" ]; then
      while [ -z "$WAIT" ] || [ "$WAIT" -gt 0 ] ; do
        ps $PID >&- || { echo done ; break ; }
        killall vdr 2>&1 >/dev/null
        sleep 1
        [ -n "$WAIT" ] && let WAIT=WAIT-1
      done    
      killall -9 vdr 2>&1 >/dev/null
    fi

Da hatten die EasyVdr-Entwickler nicht unbedingt eine Sternstunde. Man sendet nicht einfach SIGTERM im Sekundentakt an eine Anwendung. Normalerweise sendet man einmal SIGTERM. Wenn sich das Programm nicht innerhalb wenigen Sekunden beendet, wird ein SIGKILL hinterher geschickt. Für sowas verwendet man killproc. Da kann man auch die Wartezeit angeben. Die Schleife entfällt dann.

Gruß
e9hack

Zitat

Original von ardi
Ob die vorgehnsweise von Easyvdr, dem VDR im Sekundentakt ein SIGTERM zu verpassen, sinnvoll ist oder nicht vermag ich nicht zu beurteilen.

Ich halte das für falsch. Wenn Du den Timeoutwert auf 5sec hochsetzt, benötigst Du den Patch nicht mehr.

Zitat

Ich bin aber der Meinung, das die VDR Entwickler mit

Code

// Reset all signal handlers to default before Interface gets deleted:
  signal(SIGHUP,  SIG_DFL);
  signal(SIGINT,  SIG_DFL);
  signal(SIGTERM, SIG_DFL);
  signal(SIGPIPE, SIG_DFL);
  signal(SIGALRM, SIG_DFL);

etwas übers Ziel hinausgeschossen sind. Schließlich soll ja nur verhindert werden, dass das "Interface" (durch den Signalhandler) angefasst wird, wenn es nicht mehr existiert. Dies ist durch meinen Patch, glaube ich, besser gelöst.

Normalerweise wird ja auch nur 1x SIGTERM gesendet. Daher ist es völlig korrekt, die Signalhandler am Ende auf Default zurückzusetzen.

Gruß
e9hack