Schon mal Debug Version gebaut? Und vielleicht in paar mehr Ausgaben.
Sieht mir nach einer Thread Race Condition aus.
Johns
-
-
Habe das softhddevice mit Debug gebaut und beim ersten mal hat es mit dem Ton geklappt. Beim zweiten Versuch jedoch kein Ton in XBMC:
Beim zurückschalten auf VDR auch kein Ton:
Nach einem killall vdr wieder Ton wieder da im VDR
Ein komlettes Logset: https://dl.dropboxusercontent.…g/g2v_log_09120801.tar.xzPS: mit "-w alsa-close-open-delay"
-
Also da geht schon was am Anfang schief.
CodeSep 12 07:59:20 [root] 900 SoftHdDevice already detached_Das Plugin meint das es bereits detached ist.
Leider sind im Audio Close keine Meldungen drin, Diese müssten ergänzt werden.
Um das Umschaltscript auszuschliessen:
Codewhile true; do svdrpsend plug softhddevice deta; fuser /dev/snd/*; svdrpsend plug softhddevice atta; doneVielleicht noch >/dev/null bei den svdrpsend's
Johns
-
Das DETA im zweiten Log bei 7:59:20 ist da wo von XBMC (bereits ohne Ton) in den Launcher gewechselt wird. Sollte eigentlich nichts ausmachen, wenn es (unnötigerweise) erneut aufgerufen wird.
Im ersten Log, wo die Schaltung von VDR -> XBMC abläuft. Sieht man nur ein DETA bei 7:56:56 und die Erfolgsmeldung bei 7:56:57.
Wo muss ich da welche Ausgaben einbauen?
-
Ich habe die While-Schleife ein paar Sekunden laufen lassen und dann mit STRG-C abgebrochen. Die Ausgaben waren diese:
Daraus werde ich nicht schlau... Nach dem STRG-C habe ich am VDR keinen Ton
-
Laut Log hat es schon beim ersten Durchgang gehakt:
-
Moin,
Code/dev/snd/pcmC1D7p: 17383msollte nicht erscheinen. Ist also einfach reproduzierbar.
Codesofthddevice.ClearOnSwitch = 1Ist die entscheidene Option ? Mit der es geht oder nicht?
Ansonsten:
Diff
Alles anzeigendiff --git a/audio.c b/audio.c index 6ca3df1..d0ea7de 100644 --- a/audio.c +++ b/audio.c @@ -2169,6 +2169,8 @@ static void AudioExitThread(void) { void *retval; + Debug(3, "audio: %s\n", __FUNCTION__); + if (AudioThread) { if (pthread_cancel(AudioThread)) { Error(_("audio: can't queue cancel play thread\n")); @@ -2994,6 +2996,8 @@ void AudioExit(void) { const AudioModule *module; + Debug(3, "audio: %s\n", __FUNCTION__); + #ifdef USE_AUDIO_THREAD if (AudioUsedModule->Thread) { // supports threads AudioExitThread();und mehr mal einbauen. Damit sieht ob und wann die aufgerufen werden.
Und Plugin mit DEBUG bauen, sonst kommen die nicht.Johns
-
Vielen Dank für die Unterstützung.
Ich habe die beiden Debugausgeben mal eingebaut und die While Schleife gestartet.
Die Einstellung
Codesofthddevice.ClearOnSwitch = 1ist diejenige, wo es nicht geht.
Habe manuell svdrpsend plug softhddevice deta und atta gemacht: Kein Ton:
Wenn ich das gleiche (wieder Manuell) mit ClearOnSwitch=0 mache geht es:
Wäre es nicht am einfachsten, wenn das Plugin den ClearOnSwitch beim DETA einfach ignorieren wüdre?
-
Also mit
Codesofthddevice.ClearOnSwitch = 1habe ich getestet, bei mir funktioniert es.
Habe alsa-lib-1.0.28 und Stereo Ausgabe.Ich sehe keinen großen Unterschied zwischen den beiden Varianten.
Mit Debugger könnte man den Exit tracen, aber ich kann den Fehler leider nicht nachstellen.ZitatWäre es nicht am einfachsten, wenn das Plugin den ClearOnSwitch beim DETA einfach ignorieren würde?
An der Stelle weiß man nicht ob es ein einfacher Kanalwechsel oder Detach ist.Verwende einfach kein ClearOnSwitch :).
Ansonsten teste mal ob Stereo / HDMI / SPDIF ein Untersched macht. Und welche alsa-lib und Kernel?
Johns
-
Verwende einfach kein ClearOnSwitch :).
Das ist nicht wirklich eine Lösung. Es ist grausam, wenn das Bild nachläuft und das Menü schon da ist.
Ansonsten teste mal ob Stereo / HDMI / SPDIF ein Untersched macht. Und welche alsa-lib und Kernel?
Ok, gerne. Wie muss ich das machen. Habe nur via HDMI-Kabel am Reciever angeschlossen und -a hw:Nvidia,7 in der Konfig stehen. Die AC3's habe ich im Plugin auf PassThru stehen.
Kernel/Alsa
Wie wäre es, wenn das Plugin wenn ein DETA kommt einfach selbst die Konfig in ClearOnSwitch ändert und bei ATTA wie der auf 1 stellt?
-
Mir fällt gerade noch ein, dass manchmal (selten) beim betätigen von Wiedergabe (Aufnahme) kein Ton kommt und das Bild dann nach ca. 1 Sekunde stehen bleibt. Wenn ich dann auf Stop und wieder auf Wiedergabe gehe geht es dann...
Kann man im Plugin eventuell eine Abfrage einbauen, ob das Sounddevice freigegeben wurde und falls nicht dieses noch mal schließen?
-
MegaV0lt:
Kannst du denn mit -w alsa-no-close-open beim Plugin-Aufruf leben? Ohne diesen Parameter hatte ich beim Starten von xbmc aus vdr heraus Probleme.
Die hatte ich soweit analysiert, dass manchmal AlsaExit keinen pcm_close macht, weil der letzte PCMOpen noch in Arbeit war, dann aber der AudioThread beendet wurde. Danach ist das Ausgabegerät blockiert. -
Code
-w alsa-no-close-open
Damit hatte ich letztens das Problem, dass der Ton (Aufnahme) zwar bei der Werbung da war, aber in der Serie nicht! Ich habe das dann nicht weiter getestet.Also keine OptionUnd der mit delay bringt auch nichts. Habe sogar im Quellcode schon von 50 auf 100ms erhöht...
ZitatDie hatte ich soweit analysiert, dass manchmal AlsaExit keinen pcm_close macht, weil der letzte PCMOpen noch in Arbeit war, dann aber der AudioThread beendet wurde. Danach ist das Ausgabegerät blockiert.
Edit:
Das wäre doch schon mal eine Erklärung, bzw. ein Ansatzpunkt für johns? -
Die hatte ich soweit analysiert, dass manchmal AlsaExit keinen pcm_close macht, weil der letzte PCMOpen noch in Arbeit war, dann aber der AudioThread beendet wurde. Danach ist das Ausgabegerät blockiert.Sowas in der Art ist auch meine Vermutung. Nur ohne Fehler ist es schwierig zusuchen.
Diff
Alles anzeigen--- a/audio.c +++ b/audio.c @@ -1217,6 +1217,7 @@ static int AlsaSetup(int *freq, int *channels, int passthrough) handle = AlsaPCMHandle; // FIXME: need lock + Debug(3, "audio: %s [\n", __FUNCTION__); AlsaPCMHandle = NULL; // other threads should check handle snd_pcm_close(handle); if (AudioAlsaCloseOpenDelay) { @@ -1227,6 +1228,7 @@ static int AlsaSetup(int *freq, int *channels, int passthrough) return -1; } AlsaPCMHandle = handle; + Debug(3, "audio: %s ]\n", __FUNCTION__); } for (;;) {Da steht schon brauche "lock", aber bisher klappte es ohne.
Johns
-
Das ist nicht wirklich eine Lösung. Es ist grausam, wenn das Bild nachläuft und das Menü schon da ist.
Ok, gerne. Wie muss ich das machen. Habe nur via HDMI-Kabel am Reciever angeschlossen und -a hw:Nvidia,7 in der Konfig stehen. Die AC3's habe ich im Plugin auf PassThru stehen.
Wie wäre es, wenn das Plugin wenn ein DETA kommt einfach selbst die Konfig in ClearOnSwitch ändert und bei ATTA wie der auf 1 stellt?
Einfach den Stereoausgang verwenden, du hörst zwar nichts, aber es werden andere Treiber verwendet.
Wenn es mit Stereo bei dir geht, dann kann ich es vergessen, hier mit Stereo den Fehler zu produzieren.alsa-lib-1.0.27.2 könnte ich testen.
Johns
-
Ok, eingebaut und
Codehdvdr01 softhddevice # svdrpsend plug softhddevice deta 220 hdvdr01 SVDRP VideoDiskRecorder 2.1.6; Tue Sep 16 10:44:04 2014; UTF-8 900 SoftHdDevice is detached 221 hdvdr01 closing connection hdvdr01 softhddevice # svdrpsend plug softhddevice atta 220 hdvdr01 SVDRP VideoDiskRecorder 2.1.6; Tue Sep 16 10:44:14 2014; UTF-8 900 SoftHdDevice is attached 221 hdvdr01 closing connectionMit ClearOnSwitch = 0
Mit ClearOnSwitch=1Jetzt nach mehreren Tests mit Ton! Sind da die Debugausgaben schuld?
-
Jetzt nach mehreren Tests mit Ton! Sind da die Debugausgaben schuld?
Sowas kann schon mal passieren, weil jetzt das Timing etwas anders ist und die race condition dadurch seltener ausgelöst wird.Lars.
-
Und das obwohl ich ja schon den -w alsa-close-open-delay switch verwende und im Code von 50 auf 100ms gegangen bin...
Hab mal eben schnell ohne den -w alsa-close-open-delay getestet. Geht jetzt auch mit Ton...
-
Race conditions suchen ist spaßig...
Lars.
-
Bei mir geht der DETA regelmäßig schief, wenn ich ohne alsa-no-close.open arbeite. Einfachste Abhilfe wäre in softhdevice.cpp in der DETA-Rountine zwischen ::Attach und Suspend einen sleep von mindestens 50ms einzubauen, besser aber die close-open-routine wie im anhängenden patch zu blocken.
close-open-delay spielt dabei keine rolle, weil erst der open von thread-cancel gekillt wird
Jetzt mitmachen!
Sie haben noch kein Benutzerkonto auf unserer Seite? Registrieren Sie sich kostenlos und nehmen Sie an unserer Community teil!