VDR setzt eine Schreibsperre auf recordings, so lange externes Script läuft

MarkusE · 23. März 2024

Hi,

beim Anschauen des Codes ist mir aufgefallen, dass VDR das mit

--record

übergebene Script manchmal aufruft, während ein LOCK_RECORDINGS_WRITE gesetzt ist. Dieser LOCK wird erst wieder freigegeben, wenn das Script beendet ist.

Vermutlich ist das in der Praxis kein Problem, aber ich dachte, ich schreibe es auf mal auf. Falls es doch zu einem Problem führt. Betroffen sind: "copying", "copied" und "moved".

Hier noch der relevante Code:

recording.c, void cRecordingsHandler::Action(void):

Code

void cRecordingsHandler::Action(void)                                                                                                                                                      
{                                                                                                                                                                                          
  while (Running()) {                                                                                                                                                                      
        bool Sleep = false;                                                                                                                                                                
        {                                                                                                                                                                                  
          LOCK_RECORDINGS_WRITE;                                                                                                                                                           
          Recordings->SetExplicitModify();                                                                                                                                                 
          cMutexLock MutexLock(&mutex);                                                                                                                                                    
          if (cRecordingsHandlerEntry *r = operations.First()) {                                                                                                                           
             if (!r->Active(Recordings)) {                                                                                                                                                 
                error |= r->Error();                                                                                                                                                       
                r->Cleanup(Recordings);                                                                                                                                                    
                operations.Del(r);                                                                                                                                                         
                }                                                                                                                                                                          
             else                                                                                                                                                                          
                Sleep = true;                                                                                                                                                              
             }                                                                                                                                                                             
          else                                                                                                                                                                             
             break;                                                                                                                                                                        
        }                                                                                                                                                                                  
...                                                                                                                                                                       
}

Alles anzeigen

und in bool cRecordingsHandlerEntry::Active(cRecordings *Recordings):

Code

#define RUC_COPYINGRECORDING "copying"
#define RUC_COPIEDRECORDING  "copied"
#define RUC_MOVEDRECORDING   "moved"
...
cRecordingUserCommand::InvokeCommand(RUC_COPYINGRECORDING, FileNameDst(), FileNameSrc());
...
cRecordingUserCommand::InvokeCommand(RUC_COPIEDRECORDING, FileNameDst(), FileNameSrc());
...
cRecordingUserCommand::InvokeCommand(RUC_MOVEDRECORDING, FileNameDst(), FileNameSrc());
...

~ Markus

M-Reimer · 23. März 2024

In Fällen wo "dauerlaufende" Scripts ein Problem sind ist es zumindest bei einem Shellscript extrem einfach das zu vermeiden.

Der "Hack" mit "at" den man immer mal wieder sieht ist nämlich nie nötig gewesen.

Einfach den gesamten Shell-Code klammern (normale runde Klammern) und hinter die schließende Klammer ein "&" setzen. Das gesamte Script läuft damit im Hintergrund und die VDR-Prozesse, die Scripte nutzen, werden nicht blockiert.

kls · 23. März 2024

Siehe auch INSTALL:

Code

Executing commands before and after a recording:
------------------------------------------------

You can use the '-r' option to define a program or script that gets called
at various stages of handling recordings.

...
Within this program you can do anything you would like to do before and/or
after a recording or after an editing process. However, the program must return
as soon as possible, because otherwise it will block further execution of VDR.
Be especially careful to make sure the program returns before the watchdog
timeout you may have set up with the '-w' option! If the operation you want to
perform will take longer, you will have to run it as a background job.

Alles anzeigen

VDR setzt eine Schreibsperre auf recordings, so lange externes Script läuft

Jetzt mitmachen!

Teilen