Streamdev suspend. Gibts sowas - Plugin compiliert nicht unter 2.1.6

http://www.vdr-wiki.de/wiki/index.php/Suspendoutput-plugin
Zusätzlich solltest du dann noch den EPG-Scan des VDR deaktivieren.

Klarer Fall von: User nicht geeignet für Developer-Versionen?!

Der Fehler ist leicht zu beheben, wenn man weiß, wie. Ich packe das Plugin später mal bei VDR4Arch ins Repository, dann gibts auch einen Patch für 2.1.6

Da vergisst man mal die Smileys... So war das nicht gemeint.

Hier ist der Patch:
https://github.com/VDR4Arch/vd…utput-vdr2.1.2compat.diff

Als kleiner Nebeneffekt ist das Plugin jetzt auch für Arch Linux verfügbar. Vielleicht interessant für dich: Auch für ARMv6

Zitat von GTRDRIVER

Hallo

wie ich es in einem anderen Beitrag hier geschireben habe (sorry für den Doppel-Post) führt das Plugin (mit dem Patch) bei VDR 2.1.6 zu einem Speicherzugriffsfehler sobald der VDR in den Suspend geht

Hat irgendwer eine Idee was ich hier machen kann ?

(VDR+Streamdev+RPIHDDEVICE+RPI)
Auf dem RPI hat´s aber schon mal funktioniert (2.1.5er VDR+#ltere Version des RPIHDPlugin

CU
GTR

Alles anzeigen

Was heisst denn "schon mal funktioniert" genau ? Konnte man da tatsaechlich das vdr-logo standbild sehen ?
Habe jetzt keine aelteren versionen runtergeladen/compiliert zum vergleich, aber bei der 2.0.0 suspend sind bloss zwei .mpg files dabei (black und vdr logo), die der omxplayer beide nur als schwarz widergibt, und der StillPicture code im omxplayer des vdr-plugin-rpihddevice mag auch bloss PES Bilder abspielen, und die beiden Dateien sind nicht PES Bilder. Ist zwar nicht weiter schwer, das so zu patchen, das es nicht mehr abschmiert, aber so richtig Standbild abspielen braucht wohl PES .mpg files oder so.

Zitat von te36

[...] und der StillPicture code im omxplayer des vdr-plugin-rpihddevice mag auch bloss PES Bilder abspielen, und die beiden Dateien sind nicht PES Bilder. Ist zwar nicht weiter schwer, das so zu patchen, das es nicht mehr abschmiert, aber so richtig Standbild abspielen braucht wohl PES .mpg files oder so.

Suspendoutput funktioniert bei mir auf der Himbeere wunderbar, um das Plugin zu kompillieren, habe ich Zeile 315 in suspendoutput.c auskommentiert:

if ( !cRecordControls::Active() &&
//   	!cCutter::Active() &&
     	!Interface->HasSVDRPConnection()) {

Und cOmxDevice::StillPicture() benötigt PES- oder TS-Pakete, weil das im API so definiert ist:

virtual void StillPicture(const uchar *Data, int Length);
   	///< Displays the given I-frame as a still picture.
   	///< Data points either to a series of TS (first byte is 0x47) or PES (first byte
   	///< is 0x00) data of the given Length. The default implementation
   	///< converts TS to PES and calls itself again, allowing a derived class
   	///< to display PES if it can't handle TS directly.

Das beigelegte VDR-Logo lässt sich aber mit ffmpeg entsprechend konvertieren:

$ ffmpeg -i vdrlogo_720x576.mpg -f mpegts vdrlogo_720x576.pes
$ mv vdrlogo_720x576.pes vdrlogo_720x576.mpg

Damit erscheint auch das VDR-Logo, wenn der VDR in den Suspend geht.

Gruss
Thomas

Moin!

Hier der korrekte Patch für suspendoutput und vdr 2.1.6

--- a/suspendoutput.c
+++ b/suspendoutput.c
@@ -320,7 +320,11 @@
   if(cDummyPlayerControl::IsOpen()) {


     if ( !cRecordControls::Active() &&
+#if APIVERSNUM > 20101
+        !RecordingsHandler.Active() &&
+#else
         !cCutter::Active() &&
+#endif
         !Interface->HasSVDRPConnection()) {


 #define ACTIVITYTIMEOUT 600   // 10 min

Lars.

Zitat von reufer

Und cOmxDevice::StillPicture() benötigt PES- oder TS-Pakete, weil das im API so definiert ist:

Ja, ok. aber das gibt IMHO dem cOmxDevice::StillPicture() nicht das Recht den VDR mit segmentation fault zu crashen wenn ihm jemand non-PES daten uebergibt (so wie das suspendoutput plugin). Hier mal wie ich das gefixt habe.

*** omxdevice.c.orig    2014-12-15 12:31:30.888563046 +0100
--- omxdevice.c.fixed   2014-12-17 19:57:13.483865468 +0100
***************
*** 207,240 ****
                        ParseVideoCodec(Data, Length) == cVideoCodec::eMPEG2 ? 4 : 13;

                while (repeat--)
-               {
                        const uchar *data = Data;
                        int length = Length;

-                         printf("repeat: %d, length: %d\n", repeat, length);
-                         fflush(stdout);
                        // play every single PES packet, rise EOS flag on last
!                       while (length)
                        {
!                               int pktLen = PesHasLength(data) ? PesLength(data) : length;
!                                 printf("        length: %d, pktLen: %d\n", length, pktLen);
!                         fflush(stdout);

-                               // skip non-video packets as they may occur in PES recordings
                                if ((data[3] & 0xf0) == 0xe0) {
!                                         printf("        playing\n");
!                         fflush(stdout);
!                                       PlayVideo(data, pktLen, !repeat && (pktLen == length));
                                  } else {
!                                         printf("        not playing\n");
!                         fflush(stdout);
                                  }
- 
-                               data += pktLen;
-                               length -= pktLen;
                        }
                }
- 
                m_mutex->Unlock();
        }
  }
--- 207,242 ----
                        ParseVideoCodec(Data, Length) == cVideoCodec::eMPEG2 ? 4 : 13;

                while (repeat--)
                        const uchar *data = Data;
                        int length = Length;

                        // play every single PES packet, rise EOS flag on last
!                       while (length > 4)
                        {
!                               int pktLen;
!                                 int eos;

                                if ((data[3] & 0xf0) == 0xe0) {
!                                       pktLen = PesHasLength(data) ? PesLength(data) : length;
!                                         if(pktLen < 0 || pktLen > length)
!                                               pktLen = length;
!                                         eos = !repeat && (pktLen == length);
!                                       PlayVideo(data, pktLen, eos);
!                                       data += pktLen;
!                                       length -= pktLen;
                                  } else {
!                                         // skipping
!                                         int skip = 0;
!                                         while( length >= 4 &&
!                                                ( data[0] != 0x00 || data[1] != 0x00 ||
!                                                  data[2] != 0x01 || ((data[3] & 0xf0) != 0xe0))) {
!                                             length--;
!                                             data++;
!                                             skip++;
!                                         }
                                  }
                        }
                }
                m_mutex->Unlock();
        }
  }

Zitat von reufer

Das beigelegte VDR-Logo lässt sich aber mit ffmpeg entsprechend konvertieren:

Supi, vielen Dank. Hatte schon versucht das selbst zu machen, war bisher aber erst erfolglos mit mencoder am probieren Ich schick mal eine Mail an Petri das er das mal im module fixen kann.

Zitat von te36

Quoted from "reufer"



Das beigelegte VDR-Logo lässt sich aber mit ffmpeg entsprechend konvertieren:

Zitat von te36

Supi, vielen Dank. Hatte schon versucht das selbst zu machen, war bisher aber erst erfolglos mit mencoder am probieren Ich schick mal eine Mail an Petri das er das mal im module fixen kann.

Hat funktioniert mit dem ffmpeg. Allerdings war das VDRlogo ziemlich ploede. Bloss vdrlogo ohne irgendein Hinweis. Ich hatte da das gleiche Logo erwartet wie beim suspend im streamdev-server - das hat den netten hinweis das vdr suspended ist. Habe also das genommen. Das war natuerlich auch kein richtiges PES. Musste ich also auch erst konvertieren bevor ich es im suspendoutput compiliet habe. Denke mal dass streamdev-server auch absemmelt unter rpihddevice solange man das dort nicht auch fixed.

Ich habe StillImage() im rpihddevice-Plugin nun dahingehend geändert, dass auch normales MPEG ohne PES-Header angezeigt wird. Damit zeigt nun suspendoutput ohne zu patchen das VDR-Logo an und auch der Hinweis in streamdev-server sollte funktionieren - letzteres habe ich allerdings nicht getestet.

Fix ist in Git eingecheckt .

Gruss
Thomas

Bekomme jetzt leider einen Build Fehler

omxdevice.c:37:1: error: expected ‘}’ before ‘const’
omxdevice.c:37:1: error: expected ‘,’ or ‘;’ before ‘const’
make[1]: *** [omxdevice.o] Fehler 1

Gruß Patrick

Konnte es selbst beheben
Vor Zeile 37 fehlt ein

};

Gruß Patrick

Hallo Patrick

Zitat von vectra130

Bekomme jetzt leider einen Build Fehler

Danke für den Hinweis - kleiner Fehler beim Auseinanderfriemeln der einzelnen Hunks. Ist nun gefixt .

Gruss
Thomas