Plan: zapit und controld zusammenlegen.

rhabarber1848

seife hat geschrieben:

Ja, der paintBackgroundBoxRel-Aufruf übergibt zu grosse Werte.

Die Frage ist nun, ist der Bildschirm deshalb schwarz oder wegen
der Änderungen im Framebuffer?

GetAway

Er schaltet zwischen 16Bit und 8Bit um. Deswegen wirds kurz schwarz. Besser fest auf 8Bit einstellen fürs CVS.
Den 16Bit Code natürlich mit übernehmen, nur halt nen festen Schalter intergrieren.

seife

Viel besser: er schaltet zwischen 8 und 8 bit um :-)

Ich hab's mal gefixt.

GetAway

Dank Deines "8Bit" fixes funktioniert der Aufruf der Plugins jetzt wieder wie gewohnt.
Kein Schwarzbild mehr und die kompatiblität der Farbpalette bleibt erhalten.

Schön!

seife

Hm. Dann mach mal bitte folgendes (zum testen):

Code:

--- a/tuxbox/neutrino/src/gui/plugins.cpp
+++ b/tuxbox/neutrino/src/gui/plugins.cpp
@@ -365,8 +365,9 @@ void CPlugins::startPlugin(int number,int param)
        if (plugin_list[number].fb)
        {
                /* most plugins are 8bit only -> switch FB to 8bit mode */
-               if (sizeof(fb_pixel_t) != 1)
+//             if (sizeof(fb_pixel_t) != 1)
                        frameBuffer->setMode(720, 576, 8);
+               frameBuffer->paletteSet();
                startparam = makeParam(P_ID_FBUFFER  , frameBuffer->getFileHandle()    , startparam);
        }
        if (plugin_list[number].rc)

Das Bild wird vermtulich trotzdem kurz schwarz werden, aber die Farbpalette könnte dann passen.

GetAway

Wieso, passt doch? :gruebel:

Edit:
Es flackert 1x im Millisekundenbereich.
Vorher hats mir besser gefallen. Hoffentlich bleibt es so.

rhabarber1848

rhabarber1848 hat geschrieben:

Im controld-zapit-merge-Branch funktioniert dieser Patch nicht mehr:
http://www.dbox2-tuning.net/for ... 82#p366182

Nach dem Beenden der Neutrino-Esound-Funktion (esd-Daemon war nicht einmal
gestartet, da nicht kompiliert) bleibt der Ton stumm, auch nach Kanalwechsel.

Gleiches Problem mit dem Audioplayer auf der Dbox2.

GetAway

Mp3's im Audioplayer und das Internetradio laufen ohne Probleme.
Nach beenden der selben und wechsel zum TV_Modus keine Probleme.

seife

GetAway hat geschrieben:

Mp3's im Audioplayer und das Internetradio laufen ohne Probleme.
Nach beenden der selben und wechsel zum TV_Modus keine Probleme.

Das könnte abhängig von der Lautstärkeregelmethode (AVS oder OST sein), evtl. mal beide probieren.

[edit] Ich habe beide mit dem Audioplayer probiert, und hinterher im TV sofort wieder Ton gehabt. Irgendwas muss bei dir speziell sein. Ist der Ton nach "pzapit -esb; pzapit -lsb" auch weg? [/edit]

GetAway hat geschrieben:

Wieso, passt doch? :gruebel:

Edit:
Es flackert 1x im Millisekundenbereich.
Vorher hats mir besser gefallen. Hoffentlich bleibt es so.

Ja, ich werde die Abfrage "wenn 8 bit, dann nicht umschalten" drinlassen, aber für den 16bit-modus werde ich das setPalette() reinmachen, damit es dort auch funktionieren könnte, wenn den jemand benutzen wollte.

rhabarber1848

seife hat geschrieben:

Das könnte abhängig von der Lautstärkeregelmethode (AVS oder OST sein), evtl. mal beide probieren.

[edit] Ich habe beide mit dem Audioplayer probiert, und hinterher im TV sofort wieder Ton gehabt. Irgendwas muss bei dir speziell sein. Ist der Ton nach "pzapit -esb; pzapit -lsb" auch weg? [/edit]

Folgendes habe ich in einem ungepachten Yadd getestet:

Nach "pzapit -esb; pzapit -lsb" kommt der TV-Ton sofort wieder.

Nach dem Audioplayer ist der TV-Ton stumm, egal ob mit avs
oder ost. Auch "pzapit -esb; pzapit -lsb" nach dem Audioplayer
bringt den TV-Ton nicht zurück.

PS: Habe den Grund gefunden, ein falscher Avia-Treiber...
Normalerweise nutze ich avia600_vb022, irgendwann muss
mir aber der avia600_vd030 ins Yadd gerutscht sein.
Letzterer macht Probleme, mit dem vb022 funktioniert
alles wieder problemlos. Sorry for the noise.

rhabarber1848

seife hat geschrieben:

Zapit braucht jetzt ein paar sachen, die vorher erst später gebraucht wurden (nämlich vom controld).

Dazu dieser Patch, der zudem controld aus start_neutrino entfernt und
für Enigma den geänderten Speicherort von scart.conf berücksichtigt.
controld_zapit_cdk.diff

seife

Barf ist ja der (IMHO richtigen) Meinung, dass die scart.conf überhaupt nicht installiert werden sollte, insofern würde ich eher in diese Richtung gehen.

rhabarber1848

Ok, über scart.conf lässt sich reden, afaik gibt es schon einen Thread
dazu. Solange das Thema aber nicht entschieden ist, sollte obiger
Patch ins CVS, damit es beim Kompilieren keine Probleme gibt.

Meinerseits steht einem commit des controld-zapit-merge ins CVS
nichts im Wege.

rhabarber1848

seife hat geschrieben:

Ja. Ich würde erstmal nur die Sachen in apps/dvb/zapit

controld_zapit1.diff

seife hat geschrieben:

und das Entfernen des controld in apps/tuxbox/neutrino, sowie das "controld->shutdown() -> timerd->shutdown()" in neutrino ins CVS übernehmen.

controld_zapit2.diff

rhabarber1848 hat geschrieben:

seife hat geschrieben:

Zapit braucht jetzt ein paar sachen, die vorher erst später gebraucht wurden (nämlich vom controld).

Dazu dieser Patch, der zudem controld aus start_neutrino entfernt und
für Enigma den geänderten Speicherort von scart.conf berücksichtigt.
controld_zapit_cdk.diff

Dazu noch dieser Patch, dann dürfte controld der Geschichte angehören.

@seife: Schaust Du nochmal drüber? Wenn das so in Ordnung ist,
committe ich es ins CVS.

seife

Das sieht gut aus. Und wenn es fallout gibt, dann fixen wir den halt (am meisten Bedenken habe ich, dass etwas bei der AVS/OST-Lautstärkeregelung nicht ganz 100%ig ist, aber andererseits funktioniert es bei mir seit Wochen).

Wenn dann die ersten Imagebauer damit bauen, werden wir schon noch reports bekommen

rhabarber1848

seife hat geschrieben:

Das sieht gut aus.

committed to CVS * duck und weg *

seife

Danke. Schaun' wir mal, was alles gefunden wird; die Woche ab dem 31.3. werde ich vermutlich relativ viel Zeit haben, mich drum zu kümmern.

GetAway

Hab mal eben die üblichen Sachen getestet. Keine Auffälligkeiten festgestellt.

Der Framebufferfix (Clipping) sollte so schnell wie möglich mit ins CVS wandern.
Dort tritt der Fehler auch auf und ist jetzt dauerhaft reproduzierbar.
Kann natürlich auch sein das es im Yadd "anders(öfter)" auftritt als aus dem Flash.

seife

Mach ich, sobald mein git cvsimport durchgelaufen ist (irgendwie hat Obi's git den controld-merge-commit verschusselt, evtl. weil Houdini gleichzeitig den LCD fix committed hat).

rhabarber1848

seife hat geschrieben:

verbessertem ringbuffer-Code in stream2file (Aufnahme auf NFS/Festplatte)

Hier der Patch zum Testen: EDIT: Patch ist im CVS

GetAway

Da es ja schon im, ich sag mal "git-branch" drin war, hatte ich schon Testaufnahmen damit
gemacht. Funktionieren tut's. Problematisch könnte die 8MB-Einstellung werden. Wie sieht es
damit aus, wenn man auf HDD swapped.

seife

Die ist auch nicht problematischer, als ringbuffers > 61. Nur sieht jetzt jeder gleich, dass es Blödsinn ist

rhabarber1848

GetAway hat geschrieben:

Problematisch könnte die 8MB-Einstellung werden.

Vielleicht könnte dieser Wert nur auf Boxen >= 64MB angeboten werden.
Ich habe eine solche Box und bin an der 8MB-EInstellung interessiert,
evtl. sogar 12MB.

GetAway

Also ist swap uninteressant, weil ja auch auf Platte geschrieben werden muss.

seife

rhabarber1848 hat geschrieben:

Ich habe eine solche Box und bin an der 8MB-EInstellung interessiert,
evtl. sogar 12MB.

12 MB ist keine Zweierpotenz. 16MB ist die nächste Möglichkeit.
Allerdings weiss ich nicht, was der Kernel dir erzählt, wenn du 16 MB mit malloc() allozieren willst... Ich würde nicht mal wetten, dass 8MB zuverlässig funktionieren.

Tuxbox-Forum

Plan: zapit und controld zusammenlegen.

Wer ist online?