Plan: zapit und controld zusammenlegen.

[rhabarber1848]

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: Alles auswählen

--- 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?

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

[rhabarber1848]

Im controld-zapit-merge-Branch funktioniert dieser Patch nicht mehr:
http://forum.tuxbox-cvs.sourceforge.net ... 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]

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?

Wieso, passt doch?

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]

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?

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]

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]

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

controld_zapit1.diff

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

controld_zapit2.diff

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]

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]

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]

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]

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.