cDvbPlayer::Action - PlayTs() möglicher Bug

rell · December 10, 2025 at 9:02 PM

Hallo zusammen,

ich bin mir nicht sicher, ob ich einen Bug gefunden habe. Tritt sporadisch beim Wechsel von pmPause nach pmPlay auf.

Ein cDvbPlayer::Freeze() setzt ein lock auf den thread, d.h. die Dauerschleife stoppt hier.

Nachdem Freeze() den playmode auf pmPause gesetzt hat, geht es mit der else-Anweisung weiter, d.h. Sleep wird true, damit beim nächsten Schleifendurchgang geschlafen wird.

Trotzdem gehts weiter, aus dem Ringbuffer werden Daten geholt und mit PlayTs() verarbeitet. VideoOnly ist jetzt true. Falls gerade jetzt ein Audiopaket ansteht, heißt das aber wiederum, dass cDevice::PlayTs() die Daten wegen VideoOnly hier nicht verarbeitet, trotzdem aber Played um TS_SIZE hochzählt und den Wert zurückliefert, als ob sie angenommen wurden.

Das führt zu dem Problem, dass dem Ausgabeplugin diese Daten fehlen und ggfs. einen Sync mit framedrops erzeugt.

Liege ich damit richtig?

Ein relativ einfacher Fix wäre, sich hier mit einem continue aus dem Schleifendurchgang zu verabschieden. Funktionier hier, aber ich weiß nicht, ob das an anderer Stelle Nebenwirkungen hat. Was meint ihr bzw. kls ?

Gruß Andreas

kls · December 13, 2025 at 11:25 AM

Schau ich mir an.

SHofmann · December 13, 2025 at 2:30 PM

Quote from rell

Das führt zu dem Problem, dass dem Ausgabeplugin diese Daten fehlen und ggfs. einen Sync mit framedrops erzeugt.

Ob das auch erklärt. warum (zumindest bei mir) vor allem nach dem Anspringen einer Schnittmarke die Tonspur meist erst einmal Aussetzer hat und sich erst nach ein paar Sekunden wieder fängt?

Wenn das endlich Aufhören würde, fände ich das richtig klasse!

rell · December 13, 2025 at 7:14 PM

Bin mir mittlerweile ziemlich sicher, dass da ein Fehler ist. Ist uns während der Arbeit an softhddevice-drm-gles aufgefallen, weil wir da für die Fehlersuche alle Pakete geloggt haben, die vom VDR mit PlayAudio kommen. Und ab und zu ist beim ersten Paket der Unterschied vom PTS zum vorherigen PTS genau doppelt so groß wie er sonst ist. Länge des Pakets ist aber gleich -> VDR verschluckt das Paket.

kls · December 31, 2025 at 3:52 PM

rell Nachdem du das wahrscheinlich eingehender untersucht hast als ich es auf die Schnelle überblicke, und auch den positiven Effekt der Änderung bestätigen kannst (das nehme ich zumindest an), sehe ich keinen Einwand. Das würde dann so aussehen, oder?

Diff

--- dvbplayer.c 2025/07/23 08:56:52     5.12 
+++ dvbplayer.c 2025/12/31 14:45:36 
@@ -616,8 +616,10 @@ 
                   Sleep = true; 
                } 
             } 
-          else 
+          else { 
             Sleep = true; 
+             continue; 
+             } 

          if (dropFrame) { 
             if (!eof || (playDir != pdForward && dropFrame->Index() > 0) || (playDir == pdForward && dropFrame->Index() < readIndex)) {

Display More

SHofmann Kannst du bitte testen, ob das auch dein Problem aus #3 behebt?

rell · December 31, 2025 at 3:56 PM

Ja. Genau mit diesem Patch läuft VDR jetzt schon eine Weile und das Problem tritt nicht mehr auf. Natürlich auch kein Neues

Participate now!