Statt sleeps sollte einfach eine Mutex o.ä. benutzt werden. Das ist immer die leichteste Variante, um zwei Threads zu synchronisieren.
Der vdr bringt da eine vorbereitete Klasse cMutex mit. Und die ist auch vollkommen ausreichend, wenn es keine performancekritische Sache ist.
Lars.
Ich habe SoftHDDEvice mit originaler audio.c und dem Patch von TomJoad gebaut (ohne Debug). Leider geht da der Ton nach DETA ATTA auch wieder nicht:
Kein Ton:
In meinem Patch war noch ein kleines Problem, dass die Variable nicht initialisiert wird, wenn -w alsa-no-close-open verwendet wird. Ob jemand das gleiche Verhalten wie ich sieht, erkennt man dann daran, dass "AudioExit while in Setup" ausgegeben wird.
MegaVOlt scheint ein anderes Problem zu haben.
mini73: Mir wäre auch lieber, wenn softhddevice vdr-threads verwenden würde
Die Frage ist, liegt es nur daran, daß der Thread im Open ist und dann gecancelt wird?
Es gibt mehrere Abhilfen. Ich hatte gehoffet, daß der Handle = NULL wie ein Lock funktioniert.
Wenn nicht wäre ein Lock ja sehr einfach einzubauen.
Oder man macht den Thread an der Stelle nicht cancelbar.
zu den C++ Klassen. Threads sind so einfach, da muß man nicht extra eine Klasse für machen.
Ich finde dieser C++ Klassenscheiß macht den Sourcecode unlesbar genauso wie die C #defines.
Es mag einfach und bequem für den Author zusein, aber für einen Fremdleser wird der Code unlesbar.
Threads versteht jeder sofort, wenn die in eine Klasse gezpackt sind, dann muß man noch an mindestens
zwei Stellen nachlesen, was der Code macht.
Johns
Moin!
Welche Thread-Implementierung benutzt wird, ist mir eigentlich egal, ob nun die Klasse des vdr oder direkt pthread macht keinen Unterschied.
Ich hab nur von der Mutex gesprochen. Natürlich kann man die auch direkt initialisieren und benutzen, aber wenn das schon praktischerweise eine Klasse im vdr macht, sehe ich keinen Grund, die nicht zu benutzen. Die meisten Fremdleser werden mit den Aufrufen von Lock/Unlock eines cMutex-Objektes schon verstehen, was gemeint ist.
http://projects.vdr-developer.…vdr.git/tree/thread.h#n63
http://projects.vdr-developer.…dr.git/tree/thread.c#n175
Beim Unlock würde ich im else-Zweig noch eine Fehlermeldung ins Log schreiben, denn das bedeutet, dass da einer Unlock häufiger aufruft als den Lock, aber das sind nicht so wichtige Details, solange man dafür sorgt, dass Lock/Unlock immer im Paar aufgerufen werden. Am besten benutzt man cMutexLock:
http://projects.vdr-developer.…dr.git/tree/thread.h#n129
http://projects.vdr-developer.…dr.git/tree/thread.c#n354
Lars.
Viel Spass C++ im C zuverwenden.
Johns
Ich hatte mir den Code an der Stelle nicht so genau angesehen. Wenn das reines C ist, dann ist es natürlich was anderes. 
Ich hab beruflich so viel mit C++ zu tun, dass ich gar nicht auf die Idee gekommen bin, ein vdr-Plugin in C zu entwickeln, insbesondere, weil der vdr ja auch in C++ geschrieben ist. Dann muss man sich eben einfach die passenden init/lock/unlock-Teile rauskopieren, ist ja auch nicht so schlimm.
Lars.
Erster kurzer Test mit dem Patch von johns sieht gut aus:
Mit ClearOnSwitch=1 und ohne DEBUG gebaut geht nun der Ton!
Gestartet ohne jegliche -w Switches
Erster kurzer Test mit dem Patch von johns sieht gut aus:
Du meinst den FixDetach2.diff von TomJoad?
Nein; ich habe den von johns genommen. Hatte ich doch eigentlich geschrieben...
Habe mal eine saubere Version gebaut:
Es wird der Thread nicht mehr von aussen abgeschossen, sondern ihm höflichst mitgeteilt sich zu beenden.
Johns
Habe nun auch den close_fix.diff von johns getestet und er geht. Habe Ton auch wenn ClearOnSwitch=1. Vielen Dank!
Haben den Patch noch mehr Leute getestet? Wenn es so geht, könnte er ja übernommen werden?
Patch von Johns fkt hier auch
Hab den Patch von Johns auch getestet und auch bei mir klappt es jetzt.
Vielen Dank an Johns und allen die mitgeholfen haben.
Gruß
SieDu
Dann werde ich den mal ins GIT packen. Bin am Überlegen ob ich die gleiche Lösung nicht auch für Video nehmen soll.
Irgentwie war ich der Meinung ich muß pthread_cancel nehmen.
Johns
Ich habe noch das Problem, dass manchmal der Ton weg ist. (Nach Play oder Stop; gelegentlich nach Umschalten). Das Log ist leider nicht sehr aussagekräftig:
Sep 24 20:15:02 [vdr] audio/alsa: using device 'hw:NVidia,7'_
Sep 24 20:15:02 [vdr] audio/alsa: playback open 'hw:NVidia,7' error: Das Gerät oder die Ressource ist belegt_
Sep 24 20:15:02 [vdr] audio: can't set channels 2 sample-rate 48000Hz_
Sep 24 20:15:02 [G2V vdr_record] 2014-09-24 20:15 /tmp/vdr/vdr_record before /video/Once_Upon_A_Time_In_Wonderland/Folgen_wir_dem_weißen_Kaninchen!/2014-09-24.20.08.74-0.rec
Sep 24 20:15:02 [G2V vdr_record] Markad activated - noad ignored
Sep 24 20:15:02 [vdr] [30388] record /video/Once_Upon_A_Time_In_Wonderland/Folgen_wir_dem_weißen_Kaninchen!/2014-09-24.20.08.74-0.rec
Sep 24 20:15:03 [G2V g2v_rec_msg.sh] Aufnahme: Once Upon A Time In Wonderland~Folgen wir dem weißen Kaninchen!
Sep 24 20:15:03 [G2V g2v_rec_msg.sh] svdrpsend MESG Aufnahme: Once Upon A Time In Wonderland~Folgen wir dem weißen Kaninchen!
Sep 24 20:15:03 [vdr] [30388] connect from 127.0.0.1, port 50949 - accepted
Sep 24 20:15:03 [vdr] [30388] SVDRP message: 'Aufnahme: Once Upon A Time In Wonderland~Folgen wir dem weißen Kaninchen!'
Sep 24 20:15:03 [vdr] [30388] switching to channel 74
Sep 24 20:15:04 [vdr] [30388] closing SVDRP connection
Sep 24 20:15:04 [logger] 220 hdvdr01 SVDRP VideoDiskRecorder 2.1.6; Wed Sep 24 20:15:03 2014; UTF-8_
Sep 24 20:15:04 [logger] 250 Message queued_
Sep 24 20:15:04 [logger] 221 hdvdr01 closing connection_
Sep 24 20:15:04 [vdr] [30388] warning: 4 Timer-Konflikte! Erster am 24.09. 20:08. Anzeigen?
Sep 24 20:15:04 [vdr] audio: can't set channels 2 sample-rate 0Hz_
Sep 24 20:15:04 [vdr] audio: can't set channels 2 sample-rate 48000Hz_Beim Umschalten dürfte es keine Thread Probleme geben.
Aber die Debug Funktionen sind im Source:
Debug(3, "audio: %s [\n", __FUNCTION__);
und
Debug(3, "audio: %s ]\n", __FUNCTION__);Einfach die "//" davor entfernen.
Ansonsten noch die -w alsa-no-close-open und -w alsa-close-open-delay versuchen.
Johns
Sie haben noch kein Benutzerkonto auf unserer Seite? Registrieren Sie sich kostenlos und nehmen Sie an unserer Community teil!