1 Frage + 1 Fix zum Extension Patch

helau · 5. Januar 2008

Hallo
Zu den Anmerkungen des Extensions Patches (vdr 1.4.7):

Zitat

Wenn LIVEBUFFER und SUBTITLES zusammen verwendet werden, kommt es zu Abstürzen beim Aufnehmen!

Gibts die Abstuerze nur wenn auch beides aktiv ist, oder gibts auch Abstuerze wenn weder Livebuffer aktiviert ist, noch das subtitle Plugin laeuft ?!

Zitat

Mit LIVEBUFFER und SWITCHTIMER werden unter bestimmten Umständen Timer nur als Umschalt-Timer behandelt!

Das behebt dieser Patch:

Diff

--- timers.h.org        2008-01-05 15:28:13.000000000 +0100
+++ timers.h    2008-01-05 15:39:35.000000000 +0100
@@ -21,7 +21,7 @@
                    tfVps       = 0x0004,
                    tfRecording = 0x0008,
 #ifdef USE_SWITCHTIMER
-                   tfSwitchOnly= 0x0010, ///< wert eventuell noch aendern!
+                   tfSwitchOnly= 0x0020, ///< wert geaendert
 #endif /* SWITCHTIMER */
 #ifdef USE_LIVEBUFFER
                    tfhasLiveBuf= 0x0012, // ???

Alles anzeigen

zulu · 5. Januar 2008

Hallo,

Zitat

Gibts die Abstuerze nur wenn auch beides aktiv ist, oder gibts auch Abstuerze wenn weder Livebuffer aktiviert ist, noch das subtitle Plugin laeuft ?!

Weiß ich nicht mehr so genau. Wenn du da an einer Lösung arbeitest probier ich aber gerne nochmal...

Zitat

Das behebt dieser Patch:...

Super !

original im LB ist so:

Code

tfRecording = 0x0008,
+                   tfhasLiveBuf= 0x0010,
                    tfAll       = 0xFFFF,

Kann ich das im Extensions-Patch dann so machen:

Code

tfInstant   = 0x0002,
                    tfVps       = 0x0004,
                    tfRecording = 0x0008,
+#ifdef USE_LIVEBUFFER
+                   tfhasLiveBuf= 0x0010,
+#endif /* LIVEBUFFER */
+#ifdef USE_SWITCHTIMER
+                   tfSwitchOnly= 0x0012,
+#endif /* SWITCHTIMER */
                    tfAll       = 0xFFFF,
                  };
 enum eTimerMatch { tmNone, tmPartial, tmFull };

Alles anzeigen

Gruß
Marc

zulu · 5. Januar 2008

Hallo nochmal,

ich habe beides ausprobiert.

Der Fix funktioniert

und :

vdr mit LIVEBUFFER und SUBTITLES übersetzt.
vdr gestartet und den Livebuffer aktiviert.
Dann Aufnahme gestartet.
Alles Ok.
Aufnahme beendet

ttxtsubs-plugin aktiviert und vdr neustart.
Neue Aufnahme gestartet.
Absturz.

Gruß
Marc

helau · 5. Januar 2008

Zitat

Original von zulu
Hallo,
original im LB ist so:

Code

tfRecording = 0x0008,
+                   tfhasLiveBuf= 0x0010,
                    tfAll       = 0xFFFF,

Kann ich das im Extensions-Patch dann so machen:

Code

tfInstant   = 0x0002,
                    tfVps       = 0x0004,
                    tfRecording = 0x0008,
+#ifdef USE_LIVEBUFFER
+                   tfhasLiveBuf= 0x0010,
+#endif /* LIVEBUFFER */
+#ifdef USE_SWITCHTIMER
+                   tfSwitchOnly= 0x0012,
+#endif /* SWITCHTIMER */
                    tfAll       = 0xFFFF,
                  };
 enum eTimerMatch { tmNone, tmPartial, tmFull };

Alles anzeigen

Nein so gehts nicht - das sind bitsettings, d.h. jede Option darf nur ein Bit auf 1 haben der Rest muss Null sein, ich hatte mich schon ueber die 12 gewundert ...
Dann muss es also richtig so aussehen:

Code

tfInstant   = 0x0002,
                    tfVps       = 0x0004,
                    tfRecording = 0x0008,
+#ifdef USE_LIVEBUFFER
+                   tfhasLiveBuf= 0x0010,
+#endif /* LIVEBUFFER */
+#ifdef USE_SWITCHTIMER
+                   tfSwitchOnly= 0x0020,
+#endif /* SWITCHTIMER */
                    tfAll       = 0xFFFF,
                  };
 enum eTimerMatch { tmNone, tmPartial, tmFull };

Alles anzeigen

Ansonsten setzt du mit:
SetFlags( tfSwitchOnly ) auch gleichzeitig tfInstant auf 1.

P.S. Kommt der Absturz mit LB + Subtitles auch mit dem geanderten Patch ?
Falls ja muesste man mal nen Backtrace ziehen.
Da ich beides nicht nutze komme ich da aber sobald nicht dazu, ausser einer meiner User meldet mal den Fehler

zulu · 5. Januar 2008

Zitat

P.S. Kommt der Absturz mit LB + Subtitles auch mit dem geanderten Patch ?

Habe es jetzt nochmal damit übersetzt

Code

tfInstant   = 0x0002,
                    tfVps       = 0x0004,
                    tfRecording = 0x0008,
+#ifdef USE_LIVEBUFFER
+                   tfhasLiveBuf= 0x0010,
+#endif /* LIVEBUFFER */
+#ifdef USE_SWITCHTIMER
+                   tfSwitchOnly= 0x0020,
+#endif /* SWITCHTIMER */
                    tfAll       = 0xFFFF,
                  };
 enum eTimerMatch { tmNone, tmPartial, tmFull };

Alles anzeigen

und nehme grade mit aktiviertem Livebuffer den Tatort samt Untertitel auf

helau · 6. Januar 2008

Somit bin ich nicht der einzige der durch Gen2VDR V2 nun mehr Arbeit hat

zulu · 7. Januar 2008

Zitat

P.S. Kommt der Absturz mit LB + Subtitles auch mit dem geanderten Patch ?

Nach etwas ausgiebigeren Tests muss ich leider vermelden, das das Problem nach wie vor besteht .

Das fiese ist, das die Abstürze erst nach einem Neustart des VDR auftreten.
Also:
Livebuffer aktivieren ->
Aufnahme starten (und freuen das es geht) ->
Aufnahme beenden und VDR neustarten ->
Aufnahme starten -> Absturz

Code

Jan  7 18:26:31 siduxbox kernel: vdr[3472]: segfault at 20202028 eip 080ea5e3 esp abdbb360 error 4

Hierbei spielt es keine Rolle ob das ttxtsub-plugin aktiviert ist oder nicht.
Es reicht, das der VDR mit SUBTITLES und LIVEBUFFER übersetzt und der LB aktiviert wird.

Gruß
Marc

helau · 7. Januar 2008

Hallo Marc,
Dann zieh doch mal nen Backtrace, wenn es sich reproduzieren laesst

zulu · 7. Januar 2008

Backtrace zieh ich nachher. Hab jetzt nochmal ohne SUBTITLES gegengeprüft -> Auch mit aktiviertem LB kein Absturz beim Aufnehmen.

zulu · 7. Januar 2008

Code

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0xab5fcb90 (LWP 19454)]
cFileWriter::Action (this=0x8629c28) at recorder.c:144
144                      if (ttxtSubsRecorder->GetPacket(&subsp, &len)) {
Current language:  auto; currently c++
(gdb) bt
#0  cFileWriter::Action (this=0x8629c28) at recorder.c:144
#1  0x081103bc in cThread::StartThread (Thread=0x8629c28) at thread.c:244
#2  0xb7f8b4fb in start_thread () from /lib/i686/cmov/libpthread.so.0
#3  0xb7deb75e in clone () from /lib/i686/cmov/libc.so.6

helau · 9. Januar 2008

Hallo,
So richtig hilft mir der Backtrace nicht wirklich.
Sowie das aussieht ist ttxtSubsRecorder ungueltig und der Aufruf geht ins Nirwana.
Aber das muesste sich mal jemand anschauen, der den Code besser kennt.

zulu · 9. Januar 2008

Hi,

ich vermute, es liegt an den fehlenden Erweiterungen für die SUBTITLES bzw TTXTSUBS in livebuffer.c + .h
IMHO muss "nur" der Code aus recorder.c + .h dort auch eingebaut werden.

Zitat

Aber das muesste sich mal jemand anschauen, der den Code besser kennt.

wer macht's?

Gruß
Marc

1 Frage + 1 Fix zum Extension Patch

Jetzt mitmachen!

Teilen