Permashift 1.0.4 für VDR 2.4, Betaversion

Ein Eike · January 28, 2021 at 9:43 PM

Hallo!

Ich habe nach Ewigkeiten mal wieder eine Version meines Plugins für permanenten Timeshift "Permashift" zusammengepackt.

Abgesehen von Codecleanup ist nur ein Patch für den aktuellen VDR 2.4.6 dazugekommen.

Da meine Testmöglichkeiten *hüstel* eingeschränkt sind, hoffe ich auf den einen oder anderen Neugierigen.

Hier gibt's den Quellcode: https://github.com/eikesauer/Perm…tag/v1.0.4-beta

Und eine Erklärung, worum es überhaupt geht: https://ein-eike.de/vdr-plugin-permashift/

Gesundheit!

Eike

carel · January 29, 2021 at 3:45 PM

Hallo Eike

Leider gibt’s eine Fehler beim Kompilieren:

Hab's versucht mit vdr-2.4.6 und vdr-2.5.1

Code

vdr.c: In function ‘int main(int, char**)’:
vdr.c:1371:16: error: jump to case label
 1371 |           case kPlayPause:
      |                ^~~~~~~~~~
In file included from args.h:15,
                 from vdr.c:42:
channels.h:266:48: note:   crosses initialization of ‘const cChannels* Channels’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:690:17: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  690 | const c##Class *Name __attribute__((unused)) = Name##_Lock.Name();
      |                 ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
channels.h:266:48: note:   crosses initialization of ‘cChannels_Lock Channels_Lock’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:689:16: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  689 | c##Name##_Lock Name##_Lock(false); \
      |                ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
vdr.c:1372:16: error: jump to case label
 1372 |           case kPause:
      |                ^~~~~~
In file included from args.h:15,
                 from vdr.c:42:
channels.h:266:48: note:   crosses initialization of ‘const cChannels* Channels’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:690:17: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  690 | const c##Class *Name __attribute__((unused)) = Name##_Lock.Name();
      |                 ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
channels.h:266:48: note:   crosses initialization of ‘cChannels_Lock Channels_Lock’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:689:16: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  689 | c##Name##_Lock Name##_Lock(false); \
      |                ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
vdr.c:1389:16: error: jump to case label
 1389 |           case kRecord:
      |                ^~~~~~~
In file included from args.h:15,
                 from vdr.c:42:
channels.h:266:48: note:   crosses initialization of ‘const cChannels* Channels’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:690:17: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  690 | const c##Class *Name __attribute__((unused)) = Name##_Lock.Name();
      |                 ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
channels.h:266:48: note:   crosses initialization of ‘cChannels_Lock Channels_Lock’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:689:16: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  689 | c##Name##_Lock Name##_Lock(false); \
      |                ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
vdr.c:1401:16: error: jump to case label
 1401 |           case kPower:
      |                ^~~~~~
In file included from args.h:15,
                 from vdr.c:42:
channels.h:266:48: note:   crosses initialization of ‘const cChannels* Channels’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:690:17: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  690 | const c##Class *Name __attribute__((unused)) = Name##_Lock.Name();
      |                 ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
channels.h:266:48: note:   crosses initialization of ‘cChannels_Lock Channels_Lock’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:689:16: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  689 | c##Name##_Lock Name##_Lock(false); \
      |                ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
vdr.c:1427:11: error: jump to case label
 1427 |           default: break;
      |           ^~~~~~~
In file included from args.h:15,
                 from vdr.c:42:
channels.h:266:48: note:   crosses initialization of ‘const cChannels* Channels’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:690:17: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  690 | const c##Class *Name __attribute__((unused)) = Name##_Lock.Name();
      |                 ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
channels.h:266:48: note:   crosses initialization of ‘cChannels_Lock Channels_Lock’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                                                ^~~~~~~~
tools.h:689:16: note: in definition of macro ‘USE_LIST_LOCK_READ2’
  689 | c##Name##_Lock Name##_Lock(false); \
      |                ^~~~
channels.h:266:29: note: in expansion of macro ‘USE_LIST_LOCK_READ’
  266 | #define LOCK_CHANNELS_READ  USE_LIST_LOCK_READ(Channels)
      |                             ^~~~~~~~~~~~~~~~~~
vdr.c:1365:16: note: in expansion of macro ‘LOCK_CHANNELS_READ’
 1365 |                LOCK_CHANNELS_READ;
      |                ^~~~~~~~~~~~~~~~~~
make: *** [Makefile:145: vdr.o] Error 1

Display More

Gruss

Carel

Ein Eike · January 29, 2021 at 4:30 PM

Hallo Carel,

Quote from carel

Leider gibt’s eine Fehler beim Kompilieren:

Hab's versucht mit vdr-2.4.6 und vdr-2.5.1

dein Compiler scheint strenger zu sein als meiner.

Kannst du mal folgendes probieren?

In vdr.c, ca. Zeile 1360, sollte folgendes stehen:

Code

// Pausing live video:
case kFastRew:
  // test if there's a live buffer to rewind into...
  LOCK_CHANNELS_READ;
  if (cDevice::ActualDevice()->GetPreRecording(Channels->GetByNumber(cDevice::CurrentChannel())) == NULL) {
    break;
  }
// fall through to pause
case kPlayPause:

Ersetze das mal bitte durch dieses hier (also einfach zwei geschweifte Klammern mehr):

Code

// Pausing live video:
case kFastRew:
{
  // test if there's a live buffer to rewind into...
  LOCK_CHANNELS_READ;
  if (cDevice::ActualDevice()->GetPreRecording(Channels->GetByNumber(cDevice::CurrentChannel())) == NULL) {
    break;
  }
}// fall through to pause
case kPlayPause:

Gesundheit!

Eike

carel · January 29, 2021 at 4:48 PM

Bingo! Damit geht es jetzt

Danke für die rasche Lösung!

Carel

Ein Eike · February 1, 2021 at 10:22 AM

Hallo!

Wie sieht's aus, konnte der eine oder andere es inzwischen ausprobieren?

Kann ich das (mit der Patch-Änderung) releasen?

Gesundheit!

Eike

carel · February 2, 2021 at 5:07 PM

Hi Eike

Ja, für mich ist es OK!

Ein Eike · February 2, 2021 at 5:26 PM

Quote from carel

Ja, für mich ist es OK!

Super, danke!

Wer Probleme damit hatte, spreche jetzt oder schweige für immerdar.

... und bei wem es auch geklappt hat, der darf natürlich auch gerne sprechen.

kamel5 · February 2, 2021 at 7:21 PM

Quote from Ein Eike

Wer Probleme damit hatte, spreche jetzt oder schweige für immerdar.

Dann muss ich wohl mal was schreiben...

Bei mir ist es bei der neuen Version so:

1. Wenn ich auf Pause drücke, bekomme ich jetzt ein schwarzes Bild (das war vorher nicht so) und der Buffer wird abgespeichert. Warte ich jetzt bis das zu Ende ist, funktioniert permashift normal mit Start, Stop und Spulen.

2. Wenn ich nicht warte bis der Buffer abgespeichert ist und ich auf Play oder Spulen drücke, springe ich sofort zurück ins LiveBild. Die Timeshift-Aufnahme läuft aber im Hintergrund weiter.

Außerdem startet dann im Fall 1 die Wiedergabe nicht, wie erwartet, bei der aktuellen Stopzeit, sondern immer am Anfang der Aufnahme.

Ich bin allerdings noch nicht dazu gekommen, mal zu schauen, woran das liegen könnte.

Grüße

kamel5

kamel5 · February 2, 2021 at 7:34 PM

Hallo,

ich hab mal ein kurzes diff zur alten Version gemacht. Dabei ist mir das in menu.c cRecordControls::PauseLiveVideo von der alten auf die neue Version aufgefallen:

Code

-  if (Start(Timers, NULL, true, &reused)) {
+  if (Start(Timers, NULL, true)) {

Kurz geändert und es geht wieder.

Was mir allerdings noch aufgefallen ist, auch schon bei der alten Version.

Die Timeshift-Aufnahme wird nicht immer zuverlässig beendet, wenn man den Modus verlässt.

Dazu habe ich aber noch keine Systematik erkannt.

Grüße

kamel5

Ein Eike · February 4, 2021 at 12:44 PM

Hallo!

Quote from kamel5

Dann muss ich wohl mal was schreiben...

Man muss nur drohen, dann klappt es auch!

Nein, im Ernst: Du hast deinem Namen keine Ehre gemacht - wenn der Fehlermelder die Lösung gleich mit findet, kann er kein Kamel sein...

Ich werd's in Release 1.4 einbauen.

Danke!

Quote from kamel5

Was mir allerdings noch aufgefallen ist, auch schon bei der alten Version.

Die Timeshift-Aufnahme wird nicht immer zuverlässig beendet, wenn man den Modus verlässt.

Dazu habe ich aber noch keine Systematik erkannt.

Ich bin da grad ehrlich gesagt nicht mal sicher, wie das Sollverhalten ist. Ich schau nochmal rein.

Die Debugausgaben sollten in der aktuellen Version erweitert sein, aber mir scheint, für diesen Fall gibt das noch nicht viel her. Kannst ja bei Gelegenheit trotzdem mal schauen, was das Log sagt.

Danke nochmal!

Eike

kamel5 · February 4, 2021 at 1:25 PM

Quote from Ein Eike

Nein, im Ernst: Du hast deinem Namen keine Ehre gemacht

Quote from Ein Eike

Ich bin da grad ehrlich gesagt nicht mal sicher, wie das Sollverhalten ist. Ich schau nochmal rein.

Aus meiner Sicht wäre das erwartete Verhalten: wenn man den Timeshift-Mode verlasst sollte auch alles bereinigt werden.

Das funktioniert ja auch manchmal, aber eben nicht immer und diese übrig bleibenden Aufnahmen beenden sich dann erst, wenn die eingestellte Zeit für Direktaufnahmen erreicht ist. So sammeln sich dann die nicht bereinigten Timeshift-Aufnahmen und müssen händisch gelöscht werden. Besondere Log-Meldungen gibt es dazu leider nicht.

Nachtrag:

Wünschenswert wäre ein Timeshift, bei dem nach Beenden des Modus der Buffer nicht verworfen wird, außer man wechselt den Kanal oder startet eine "normale" Wiedergabe. Also nach Beenden des Modus und erneutem Drücken von Pause sollte der ursprüngliche Buffer erneut in gesamter Länge zur Verfügung stehen.

Mir ist es schon öfters passiert, das ich aus Versehen (weil ich nicht mehr daran gedacht habe, das der Modus aktiv ist) auf Beenden gedrückt habe und das was ich noch sehen wollte, war dann weg. Hier half auch nur manchmal, das die Timeshift-Aufnahme im Hintergrund fälschlicherweise weiter lief.

Viele Grüße

kamel5

kamel5 · February 8, 2021 at 7:23 PM

Beim Testen für den skindesigner ist mir noch folgendes aufgefallen:

Die Zeiten die bei der Timeshift-Wiedergabe angezeigt werden, stimmen nicht.

Sie zeigen ca. das Doppelte von den tatsächlichen an und zählen auch bei normaler Wiedergabe doppelt so schnell hoch.

Grüße

kamel5

kamel5 · May 25, 2021 at 11:35 AM

Im Anhang mal ein Update für den permashift patch für VDR-2.5.4.

Bei mir gab es mit dem patch für VDR-2.4, angewendet auf VDR-2.5.4, einen segfault beim Beenden des VDR. Im Betrieb mit VDR-2.5.4 konnte ich vorerst keine Auffälligkeiten feststellen.

Der neue patch sollte diesen segfault beheben. Das Ganze aber ohne Gewähr.

Grüße

kamel5

md_berlin · February 6, 2022 at 7:10 PM

Hier der auf vdr-2.6.1 angepasste patch

vdr-2.6.1-patch-for-permashift.patch.gz

nvertigo · February 6, 2022 at 7:45 PM

md_berlin

Sollte es nicht

Diff

diff --git a/recorder.c b/recorder.c
index e6de8999..0e2435c7 100644
--- a/recorder.c
+++ b/recorder.c
@@ -164,7 +164,7 @@ void cFrameChecker::ReportBroken(void)
 cRecorder::cRecorder(const char *FileName, const cChannel *Channel, int Priority)
 :cReceiver(Channel, Priority)
 ,cThread("recording")
-,tsChecker(NULL), frameChecker(NULL), ringBuffer(NULL), frameDetector(NULL), fileName(NULL), recordingInfo(NULL), index(NULL), recordFile(NULL), recordingName(NULL)
+,tsChecker(NULL), frameChecker(NULL), recordingInfo(NULL), ringBuffer(NULL), frameDetector(NULL), fileName(NULL), index(NULL), recordFile(NULL), recordingName(NULL)
 {
   if (FileName != NULL) {
      InitializeFile(FileName, Channel);

Display More

sein, um

Code

make recorder.o
CC recorder.o
In Datei, eingebunden von recorder.c:10:
recorder.h: In Konstruktor »cRecorder::cRecorder(const char*, const cChannel*, int)«:
recorder.h:30:19: Warnung: »cRecorder::recordingInfo« wird initialisiert nach [-Wreorder]
   30 |   cRecordingInfo *recordingInfo;
      |                   ^~~~~~~~~~~~~
recorder.h:26:22: Warnung:   »cRingBufferLinear* cRecorder::ringBuffer« [-Wreorder]
   26 |   cRingBufferLinear *ringBuffer;
      |                      ^~~~~~~~~~
recorder.c:164:1: Warnung:   während es hier initialisiert wurde [-Wreorder]
  164 | cRecorder::cRecorder(const char *FileName, const cChannel *Channel, int Priority)

Display More

zu vermeiden?

Wegen: http://git.tvdr.de/?p=vdr.git;a=b…07c;hb=HEAD#l22

EDIT:

https://github.com/eikesauer/Permashift/issues/4

Ein Eike · August 10, 2022 at 8:19 PM

Danke, für eure Updates!

Ich hab sie nun endlich mal gepusht.

Mal schauen, vielleicht finde ich demnächst sogar mal Zeit, das gelegentlich fragwürdige Rückspulverhalten zu verbessern.

Ohne Gewähr.

kamel5 · August 11, 2022 at 2:00 PM

Schön, das Du da noch was hinsichtlich Rückspulen machen willst.

Ich habe da noch 2 andere Dinge, die mich stören:

- wenn eine Aufnahme mit der "roten Record-Taste"ausgelöst wird, wird immer eine Aufnahme mit der Dauer einer Direktaufzeichnung angelegt. Hier würde ich es besser finden, eine Aufnahme der gerade aktuellen Sendung zu machen, d.h. am Anfang den Permashift-Buffer mit zu nutzen, die Endzeit aber nach der aktuellen Sendung zu setzen. Möglicherweise kann das auch im Setup als wählbare Option angeboten werden.

- Zum Anzeigen des Live-Buffers von Permashift wird nicht nur die Pausentaste benutzt, sondern auch die Rückspultaste (kFastRew). Diese Taste nutze ich auch in TVGuide um Seitenweise in der Zeit rückwärts zu navigieren. Das geht nicht mehr, wenn man den Permashift-Patch in vdr.c anwendet. Hier würde ich mir wünschen, das die Rückspultaste nur dann von Permashift ausgewertet wird, wenn kein OSD offen ist. Der angehängte 2-zeilige Patch löst dieses Problem.

Grüße

kamel5

SurfaceCleanerZ · August 11, 2022 at 5:47 PM

Hi,

Nur mal reingegrätscht: wenn die Abfrage auf offenes OSD so einfach ist, warum kann man dann im VDR selbst nicht auch m und s abfangen, so dass die nur dann Menü bzw. Mute bei Keyboard Bedienung machen, wenn kein OSD offen ist? So kann man nämlich keine Worte damit eintippen, ohne dass er sofort das Menü öffnet und man von vorne anfängt. Uralter VDR Bug der Standardtastaturbelegung. Ja ich weiß dass man die für sich ändern kann...

MfG Stefan

M-Reimer · August 11, 2022 at 5:54 PM

Was ist denn die "VDR Standardtastaturbelegung"? Wird die standardmäßig nicht wie eine Fernbedienung angelernt?

SurfaceCleanerZ · August 11, 2022 at 5:59 PM

Nö, wird bei Distris mitgeliefert, anlernen geht natürlich. M öffnet das Menü und S deaktiviert den Ton. Da hatte ich vor Jahren schon mit Klaus drüber geschrieben, wurde aber nie angepasst, da ja wenige VDR mit Tastatur bedienen.

MfG Stefan

Permashift 1.0.4 für VDR 2.4, Betaversion

Participate now!

Tags