Kleiner Fehler in VDRconvert-Paket

ah jetzt wo dus sagst seh ichs auch
thx

habs aber eigentlich nie verwendet, wenn ich mit der vdrconvert.env herumexperimentier tipp ich den befehl in di console wär zu umständlcih mich vorn vdr zu setzten

ja das klingt auch interessant, wieder ein schritt mehr in die richtige richtung!

Zitat

Original von atl
Nun ja, nachdem ich festgestellt habe, dass bei mir die MPlayer-Wiedergabe bei laufendem VDRconvert-Prozeß (IDLE-Mode) ruckelt, finde ich es praktisch, VDRconvert per Fernbedienung anhalten zu können.

Aber eigentlich bin ich am überlegen, wie ich das ganze so umbaue, das VDRconvert nur läuft, wenn es auch etwas zu tun hat. Das kann ja auch nicht schwer sein, da ja die ganzen VDRconvert-Listen "von Hand" gestartet werden. Es müsste also nur beim Start der Abarbeitung einer Liste der VDRconvert-Prozess gestartet werden und sich danach selbsttätig beenden.

Das ist ja seltsam. Im IDLE-Mode sollte vdrconvert eigentlich gar nicht stören. Es sieht doch bestimmt nur ab und zu nach, ob in seinen Auftragsdateien was steht und schläft zwischendurch die meiste Zeit. Das kann doch nicht so belastend für den Prozessor sein, dass es den MPlayer bremst.

Wenn doch, dann ist in vdrconvert wohl Optimierungsbedarf. Aber das Starten von vdrconvert im Hintergrund finde ich schon sehr sinnvoll. Es achtet ja auch darauf, dass nicht mehrere Jobs gleichzeitig gestartet werden. Das müsstest du dann wieder selbst berücksichtigen.

NeoTrace

Ich finde, das ist die falsche Richtung.

Tom

Schaut mal in den Mini-Thread "Mysteriöse Systembelastung". Vielleicht hilft auch euch der work-around.

Zitat

Es ist aber reproduzierbar, dass der MPlayer periodisch stockt, wenn VDRconvert im Hintergrund läuft - ohne etwas zu kodieren. Da scheint mir tatsächlich Optimierungsbedarf.

Dann kann ich mir nur vorstellen, dass es die Aufrufe von ps sind, die er macht, um festzustellen, ob schon ein Job läuft. Kannst du, bitte, folgenden Patch für /usr/bin/vdrconvert.sh ausprobieren? Dadurch schaut er nur nach laufenden Jobs, wenn wirklich was zu tun ist.

let run=0
-    if [ "$PARQ" = "no" ]
+    if [ $LINES -gt 0 -a "$PARQ" = "no" ]
     then
         run=`ps -aef|grep ${JOB}.sh|grep -v grep|wc -l`
     fi
-    if [ "$PARQ" = "none" ]
+    if [ $LINES -gt 0 -a "$PARQ" = "none" ]
     then
         for r in $CONVERT_JOBS
         do
             if [ $run -eq 0 ]
             then
                 run=`ps -aef|grep ${r}.sh|grep -v grep|wc -l`
             fi
         done
     fi


    if [ $LINES -gt 0 -a $run -eq 0 ]

Nach dem Patchen musst du vdrconvert neu starten. Vielleicht bringt's ja was.

Das Hochsetzen der sleep-Zeit wäre aber trotzdem sinnvoll. 5 Sekunden ist nun wirklich ganz schön kurz. Das macht man am besten in:
/etc/vdr/vdrconvert/etc/vdrconvert.env
mit
SLEEP_TIME=60

Tom

atl: Danke für den Hinweis!

Tobias

Kann, bitte, mal einer, der das MPlayer-Problem hat, ausprobieren, ob der obige Patch was bringt? Wenn ja, würde ich ihn an den Autor von VdrConvert schicken.

Tom

sag ma plz wie i des mach mim patch! dann test i sofort für dich

Du nimmst /usr/bin/vdrconvert.sh in den Editor, suchst die mit "-" gekennzeichneten Zeilen, ersetzt sie durch die entsprechende mit "+" gekennzeichnete Zeile (natürlich ohne "+") und speicherst das ab.

Nach dem Patchen musst du vdrconvert neu starten:
/etc/init.d/vdr-addon-vdrconvert restart

Lass zum Testen aber SLEEP_TIME=5, sonst können wir nicht unterscheiden, welche Änderung was gebracht hat.

Tom

das hätt ich auch zamgebracht, i hab dacht es gibt an befehl der diese änderungen automatisch vornimmt, so das ich das ned händisch reintippen muss

(für die 2 zeilen ändern wahrscheinlich a overkill, interessiert mi grad nur)

EDIT:
also ich hab jetzt kurz getestet die ruckler scheinen weg zu sein, kanns aber noch nicht 100% bestätigen, da jetzt meine freundin gilmore girls sehen will und ich nicht weiter irgendwelche avis laufen lassen darf
werd das aber am abend machen und bescheid geben

Das sieht doch gut aus! SLEEP_TIME steht wirklich noch auf 5?

Ich hätte es natürlich auch als Patch-Datei anhängen können, aber dann muss man evtl. erst erklären, wie der Patch-Aufruf aussieht. Da ist es bei zwei Zeilen einfacher und schneller, manuell zu editieren.

Tom

zerst hab i die sleeptime=60 rausglöscht und testet und dann hab is noamal reintan und auf 5 gstellt, man weis ja nie ob er sich das merk

Ja kann nur sagen schaut gut aus, werds aber auch noch ergibig testen!

Ich denke, auf schwächeren Systemen sollte man außerdem noch SLEEP_TIME hochsetzen. Wenn es mit 60 Sekunden nicht ruckelt, ist das ein guter Kompromiss.

Den Patch habe ich dimitri geschickt.

Tom

Hi,

Zitat

Original von TomG
Ich denke, auf schwächeren Systemen sollte man außerdem noch SLEEP_TIME hochsetzen. Wenn es mit 60 Sekunden nicht ruckelt, ist das ein guter Kompromiss.

Den Patch habe ich dimitri geschickt.

Tom

danke für den patch.
Leider kann ich damit nichts anfangen da ich nicht an der speziellen ct version arbeite.
Die reguläre Version 0.1.x hat schon seit langem Änderungen in diesem bereich erfahren
sodas da keine resourcen verbraucht werden da das ps kommando jetzt nur einmal aufgerufen wird.

Hat jemand auch mit der 0.1.x probleme in diesem bereich ?

Gruss
Dimitri

Wusste garnicht, dass es schon ne neue Version von vdrconvert gibt... hab ich irgendwie verschlafen.

Ich werd' mich morgen/heute Abend gleich mal ran setzen, das Paket zu updaten.

Gruß,

Tobias