softhddevice: Vorschlag für geändertes Umschaltverhalten (mit Code)

johns · 15. November 2013

Dies ist ja inzwischen im GIT gelandet. Beim Umschalten war noch das Problem, daß das Schwarzbild nicht angezeigt wurde.

Diff

diff --git a/video.c b/video.c
index 6893963..45b2c57 100644
--- a/video.c
+++ b/video.c
@@ -4979,7 +4979,10 @@ static void VaapiSyncDecoder(VaapiDecoder * decoder)
                _("video: decoder buffer empty, "
                    "duping frame (%d/%d) %d v-buf\n"), decoder->FramesDuped,
                decoder->FrameCounter, VideoGetBuffers(decoder->Stream));
-           if (decoder->Closing < -300) {
+           // some time no new picture or black video configured
+           if (decoder->Closing < -300 || (VideoShowBlackPicture
+                   && decoder->Closing)) {
+               // clear ring buffer to trigger black picture
                atomic_set(&decoder->SurfacesFilled, 0);
            }
        }

Alles anzeigen

Hilft dieser Patch?
Johns

MegaV0lt · 15. November 2013

Hab hier immer noch kurz Satndbild beim Umschalten. Bei längeren wechseln zwischen verschlüsselten Kanälen erst einen Sekund Standbild dann schwarz... Im Log wird auf ARDHD (51) und dann aud ZDFHD (52) geschaltet:

Code

Nov 15 10:56:42 [vdr] [16394] switching to channel 51
Nov 15 10:56:42 [vdr] [16394] [softhddev]SetPlayMode: 0_
Nov 15 10:56:42 [vdr] [16394] [softhddev]SetVideoDisplayFormat: 0_
Nov 15 10:56:42 [vdr] [16394] [softhddev]GetSpuDecoder:_
Nov 15 10:56:42 [vdr] [21261] dbus2vdr: System: emit signal /Status de.tvdr.vdr.status ChannelSwitch (5, 0, true)
Nov 15 10:56:42 [vdr] audio/alsa: using device 'hw:NVidia,0'_
Nov 15 10:56:42 [vdr] [21108] osdteletext-receiver thread ended (pid=16394, tid=21108)
Nov 15 10:56:42 [vdr] [16394] buffer stats: 0 (0%) used
Nov 15 10:56:42 [vdr] [21261] dbus2vdr: System: emit signal /Status de.tvdr.vdr.status ChannelSwitch (1, 0, false)
Nov 15 10:56:42 [vdr] [21261] dbus2vdr: System: emit signal /Status de.tvdr.vdr.status ChannelSwitch (1, 51, false)
Nov 15 10:56:42 [vdr] audio/alsa: start delay 361ms_
Nov 15 10:56:42 [vdr] video: slow down video, duping frame_
Nov 15 10:56:42 [vdr] video: decoder buffer empty, duping frame (1716/2508) 0 v-buf_
Nov 15 10:56:42 [vdr] video: --:--:--.---   +0	0 240/\ms   0+5 v-buf_
Nov 15 10:56:42 [vdr] [21109] TS buffer on device 1 thread ended (pid=16394, tid=21109)
Nov 15 10:56:42 [vdr] [21107] buffer stats: 127276 (0%) used
Nov 15 10:56:42 [vdr] [21107] receiver on device 1 thread ended (pid=16394, tid=21107)
Nov 15 10:56:42 [vdr] [21262] receiver on device 1 thread started (pid=16394, tid=21262, prio=high)
Nov 15 10:56:42 [vdr] [21264] TS buffer on device 1 thread started (pid=16394, tid=21264, prio=high)
Nov 15 10:56:42 [vdr] [21261] dbus2vdr: System: emit signal /Status de.tvdr.vdr.status ChannelSwitch (5, 51, true)
Nov 15 10:56:42 [vdr] [21263] osdteletext-receiver thread started (pid=16394, tid=21263, prio=high)
Nov 15 10:56:42 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/sd3.png
Nov 15 10:56:42 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/sd3.png sucessfully loaded
Nov 15 10:56:42 [vdr] [16394] [softhddev]SetPlayMode: 1_
Nov 15 10:56:42 [vdr] audio/alsa: using device 'hw:NVidia,0'_
Nov 15 10:56:42 [vdr] audio/alsa: start delay 361ms_
Nov 15 10:56:42 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/sd3.png
Nov 15 10:56:42 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/sd3.png sucessfully loaded
Nov 15 10:56:43 [vdr] video: decoder buffer empty, duping frame (1762/22) 0 v-buf_
Nov 15 10:56:43 [vdr] video: slow down video, duping frame_
Nov 15 10:56:43 [vdr] video: 20:45:41.935 +969  420   0/\ms   5+0 v-buf_
Nov 15 10:56:44 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png
Nov 15 10:56:44 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png sucessfully loaded
Nov 15 10:56:44 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png
Nov 15 10:56:44 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png sucessfully loaded
Nov 15 10:56:45 [vdr] video/vdpau: synced after 123 frames_
Nov 15 10:56:45 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png
Nov 15 10:56:45 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png sucessfully loaded
Nov 15 10:56:45 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png
Nov 15 10:56:45 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png sucessfully loaded
Nov 15 10:56:47 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/logos/zdf hd.png
Nov 15 10:56:48 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/logos/zdf hd.png sucessfully loaded
Nov 15 10:56:48 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/uncrypted.png
Nov 15 10:56:48 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/uncrypted.png sucessfully loaded
Nov 15 10:56:48 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/logos/zdf hd.png
Nov 15 10:56:48 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/logos/zdf hd.png sucessfully loaded
Nov 15 10:56:48 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/uncrypted.png
Nov 15 10:56:48 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/uncrypted.png sucessfully loaded
Nov 15 10:56:48 [vdr] [16394] switching to channel 52
Nov 15 10:56:48 [vdr] [16394] [softhddev]SetPlayMode: 0_
Nov 15 10:56:48 [vdr] [16394] [softhddev]SetVideoDisplayFormat: 0_
Nov 15 10:56:48 [vdr] [16394] [softhddev]GetSpuDecoder:_
Nov 15 10:56:48 [vdr] [21261] dbus2vdr: System: emit signal /Status de.tvdr.vdr.status ChannelSwitch (5, 0, true)
Nov 15 10:56:48 [vdr] audio/alsa: using device 'hw:NVidia,0'_
Nov 15 10:56:48 [vdr] [21263] osdteletext-receiver thread ended (pid=16394, tid=21263)
Nov 15 10:56:48 [vdr] [16394] buffer stats: 0 (0%) used
Nov 15 10:56:48 [vdr] [21261] dbus2vdr: System: emit signal /Status de.tvdr.vdr.status ChannelSwitch (1, 0, false)
Nov 15 10:56:48 [vdr] [21261] dbus2vdr: System: emit signal /Status de.tvdr.vdr.status ChannelSwitch (1, 52, false)
Nov 15 10:56:48 [vdr] audio/alsa: start delay 361ms_
Nov 15 10:56:48 [vdr] video: slow down video, duping frame_
Nov 15 10:56:48 [vdr] video: decoder buffer empty, duping frame (1809/195) 0 v-buf_
Nov 15 10:56:48 [vdr] video: --:--:--.---   +0	0   0/\ms   0+0 v-buf_
Nov 15 10:56:48 [vdr] [21264] TS buffer on device 1 thread ended (pid=16394, tid=21264)
Nov 15 10:56:48 [vdr] [21262] buffer stats: 196648 (1%) used
Nov 15 10:56:48 [vdr] [21262] receiver on device 1 thread ended (pid=16394, tid=21262)
Nov 15 10:56:48 [vdr] [21284] receiver on device 1 thread started (pid=16394, tid=21284, prio=high)
Nov 15 10:56:48 [vdr] [21286] TS buffer on device 1 thread started (pid=16394, tid=21286, prio=high)
Nov 15 10:56:48 [vdr] [21285] osdteletext-receiver thread started (pid=16394, tid=21285, prio=high)
Nov 15 10:56:48 [vdr] [21261] dbus2vdr: System: emit signal /Status de.tvdr.vdr.status ChannelSwitch (5, 52, true)
Nov 15 10:56:48 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png
Nov 15 10:56:48 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png sucessfully loaded
Nov 15 10:56:48 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/(null).png
Nov 15 10:56:48 [vdr] [16394] [softhddev]SetPlayMode: 1_
Nov 15 10:56:48 [vdr] audio/alsa: using device 'hw:NVidia,0'_
Nov 15 10:56:48 [vdr] audio/alsa: start delay 361ms_
Nov 15 10:56:48 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png
Nov 15 10:56:48 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png sucessfully loaded
Nov 15 10:56:49 [vdr] video: decoder buffer empty, duping frame (1849/2) 6 v-buf_
Nov 15 10:56:49 [vdr] video: slow down video, duping frame_
Nov 15 10:56:49 [vdr] video: 24:35:19.316 +699  446   0/\ms   5+1 v-buf_
Nov 15 10:56:49 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png
Nov 15 10:56:49 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png sucessfully loaded
Nov 15 10:56:50 [vdr] [16394] imageloader: trying to load: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png
Nov 15 10:56:50 [vdr] [16394] imageloader: /usr/share/vdr/plugins/skinflatplus/icons/MVGrapefruit/hd3.png sucessfully loaded
Nov 15 10:56:50 [vdr] video/vdpau: synced after 74 frames_

Alles anzeigen

Viel wichtiger wäre mit das Schwarzbild bei ende der Wiedergabe, da ich nie Live TV schaue.

Dr. Seltsam · 15. November 2013

dass das Schwarzbild gar nicht angezeigt wurde, kann ich zumindest nicht bestätigen. Es kommt bloß nicht sofort beim Umschalten, sondern wird einfach kurz vor dem neuen Bild eingefügt.

Mein Eindruck ist, dass es mit dem Patch etwas fixer geht, aber wie Megavolt richtig schreibt: Beim Umschalten bleibt erst das alte Bild noch einen Moment stehen (oder läuft ohne meinen Patch sogar noch 1 s nach ehe es stehen bleibt), und erst dann kommt das Schwarzbild.

Richtig wäre, wenn case pmNone in SetPlayMode sofort auf Schwarzbild schaltet, so dass man im Moment des Drückens der FB-Taste (Zappen oder Beendne einer Wiedergabe) sofort ein Schwarzbild hat. Das Schwarzbild soll die noch zum Umschaltvorgang gehörenden Standbilder des alten Kanals quasi überblenden, und nicht zusätzlich an deren Anschluss eingefügt werden.

johns · 15. November 2013

Ihr habt komische Geschmäcker.

Bei Aufnahmen müsste die neue Config Option auch greifen.
Der obige Patch war nicht vollständig, der dürfte nur bei VA-API greifen.
Also dürfte sich am bisherigen Verhalten nichts geändert haben.

Anbei eine hoffentlich bessere Version. Wie oben aber nun auch für VDPAU.

Johns

MegaV0lt · 15. November 2013

Mit der V2 habe ich nun sofort Schwarzbild - Genau so wie es alle anderen auch machen

Bei der Wiedergabe am Ende kommt nun auch Schwarz.

Bitte so in das GIT aufnehmen...

Dr. Seltsam · 15. November 2013

Bingo!!

in Verbindung mit meinem Patch "Decoder bei Kanalwechsel leeren" ist das Umschaltverhalten nun nicht mehr von einem Digitalreceiver zu unterscheiden.

Unbedingt einchecken!!

Vielen Dank für Deine Mühe!

fnu · 15. November 2013

Zitat von johns

Ihr habt komische Geschmäcker.

In der Tat ...

Zitat von MegaV0lt

Genau so wie es alle anderen auch machen

Wenn die aus dem Fenster springen, springst Du hinterher ... ?

Zitat von Dr. Seltsam

Unbedingt einchecken!!

Ich finde das nicht in Ordnung das der der seltsame Geschmack einiger ganz weniger die ganze Entwicklung beeinflussen soll, ich bin mit SHD zufrieden wie es ist und habe ab dem ersten Monat mitgetestet und genutzt.

Gut, ist vllt. auch unglücklich von johns nur auf das Geschrei einiger weniger einzugehen, während die große schweigende Mehrheit zufrieden ist.

Ich habe für die hier genannten Änderungen nix übrig, ausser den Fehlerbehebungen und bin gegen das Einfliessen in den GIT Stand!

Regards
fnu

TheChief · 15. November 2013

Alternativ mach es konfigurierbar, dann ist jeder zufrieden.

fnu · 15. November 2013

Zitat von TheChief

Alternativ mach es konfugurierbar, dann ist jeder zufrieden.

Erschreckend wenn "jeder" nur 2 Nutzer sind die es haben wollen ...

TheChief · 15. November 2013

Zitat von fnu

Erschreckend wenn "jeder" nur 2 Nutzer sind die es haben wollen ...

Jeder bist auch Du, der das dann deaktivieren kann.

Copperhead · 15. November 2013

Ich will es auch. Dann sind wir schon mindestens drei.

fnu · 15. November 2013

Zitat von TheChief

Jeder bist auch Du, der das dann deaktivieren kann.

Schon, aber weder ich noch Du haben die Mühen damit ...

louis · 15. November 2013

Zitat von fnu

Schon, aber weder ich noch Du haben die Mühen damit ...

Welche Mühen? So unübersichtlich ist der zusätzliche Code jetzt auch nicht, dass das Plugin mit dem Code nicht mehr wartbar wäre.

Aktuell ist die Setup Einstellung "Schwarzbild beim Umschalten" (oder so ähnlich) ja nicht wirklich das, was man erwartet. Ich habe übrigens keine Ahnung, ob ich das will oder nicht, dazu muss ich es erst mal testen Wobei ich befürchte, dass damit die Umschaltzeiten "gefühlt" langsamer sind, da man bisher ja irgendwas sieht.

Ciao Louis

TheChief · 15. November 2013

Zitat von fnu

Schon, aber weder ich noch Du haben die Mühen damit ...

Naja, weiss nicht, wo jetzt das Problem liegt. johns hat sich dem Wunsch angenommen und einen Patch erstellt. Wenns ins git soll, dann konfigurierbar...damit Du zufrieden bist. Wenns nicht ins git soll, dann müssen halt die bisher 3 Leute selbst patchen. Ist mir eigentlich wurscht.

fnu · 15. November 2013

Zitat von louis

Wobei ich befürchte, dass damit die Umschaltzeiten "gefühlt" langsamer sind, da man bisher ja irgendwas sieht.

Ja, genau das sehe ich ebenso, geht gar nicht. Was'n Quark das Verhalten von Kaufreceivern zu simulieren oder weil man das von vdr-plugin-xine so gewohnt ist ...

Zitat von TheChief

Wenns nicht ins git soll, dann müssen halt die bisher 3 Leute selbst patchen.

FullAck.

Regards
fnu

iNOB · 15. November 2013

Versteh den Aufriss nicht, den hier Einige machen, nur weil es beim Umschalten mal kurz schwarz wird... kopfschüttel. Sofern das über das Setup einstellbar ist, kann es denen mit dem "guten" Geschmack doch egal sein.

johns · 15. November 2013

Die Option beim Umschalten schwarz zuschalten gibts schon immer. Bisher hatte die aber scheinbar nie so richtig funktioniert.
War schwierig zu testen, da immer die Puffer noch abgespielt wurden und dann das neue Video langsam anlief.

Beim Umschalten gibt es nun drei Optionen:
softhddevice.ClearOnSwitch
Schaltet das Nachlaufen aus.

softhddevice.BlackPicture
Löscht das letzte Bild des alten Kanals.

softhddevice.SoftStartSync
Lässt das neue Bild sofort anlaufen, ist aber verlangsamt bis die Puffer gefüllt sind.

Bei "nun" nur die erste Option ist neu.

Johns

urknall · 15. November 2013

Ok, machen wir dann in Zukunft immer so? Bei neuen Feature Requests wird erst einmal im Portal abgestimmt, kommen nicht mindestens 100 Portal User zusammen, die das Feature wollen kommt es nicht ins Git.???!!!

Ich kann da echt nur den Kopf schütteln und damit ist das Thema für mich erledigt....

MegaV0lt · 15. November 2013

Ich verstehe die Aufregung nicht. Es ist doch eine zuschaltbare Option. Wer das nicht will, kann es deaktiviert lassen.
Ich bin erst darauf aufmerksam geworden, weil es eine Option Schwarzbild gab, die nicht funktionierte. Außerdem bin ich schon der Meinung, dass man das Verhalten, das alle anderen Geräte einschließlich der FullFeature-Karten (SD und HD) beim VDR haben nach zu stellen.

gda · 15. November 2013

Zitat von urknall

Ok, machen wir dann in Zukunft immer so? Bei neuen Feature Requests wird erst einmal im Portal abgestimmt, kommen nicht mindestens 100 Portal User zusammen, die das Feature wollen kommt es nicht ins Git.???!!!
Ich kann da echt nur den Kopf schütteln und damit ist das Thema für mich erledigt....

Ich weiß nicht so ganz genau worauf du hinaus willst. Du weißt ja, meiner Meinung nach darf ein Entwickler alles.
Wenn ihn die Wünsche seiner User am Arsch vorbei gehen ist das völlig okay.
Geht er auf diese Wünsche ein ist das ebenso seine Angelegenheit und ich habe das zu akzeptieren, ob es mir nun passt oder nicht.

Gerald

softhddevice: Vorschlag für geändertes Umschaltverhalten (mit Code)

Jetzt mitmachen!

Teilen