Portierungen von Neutrino HD

[GetAway]

@Gaucho316

Schreibe bitte noch Deine EPG Einstellungen dazu. Es macht, soweit ich weiß, einen großen
unterschied aus, ob man nur 1000 oder 8000 Events durchsuchen muss. Ich gehe mal davon
aus, dass du mit den Standardeinstellungen getestet hast.

[rhabarber1848]

Meine Einstellungen sind

Code: Alles auswählen

epg_cache_time=21
epg_extendedcache_time=36
epg_max_events=25000

Ich werde sectionsd nochmal mit einem gcc-3.4.6-Image testen,
ob dort die selben Probleme wie mit gcc 4.1.2 auftreten.

[Gaucho316]

Bei mir sieht's so aus:

Code: Alles auswählen

epg_cache_time=7
epg_extendedcache_time=48
epg_max_events=10000

Und um die Ohren geflogen, ist mir der sectionsd bis jetzt nie. Ich muss aber dazusagen, dass ich beim Start des Movieplayers den sectionsd neu starten lasse.

[GetAway]

Und bei mir sieht es so aus:

Code: Alles auswählen

epg_cache_time=10
epg_extendedcache_time=24
epg_max_events=8000

Ich kann subjektiv keine Verschlechterung der Performance feststellen.

[rhabarber1848]

Damit keine Langeweile aufkommt, hab ich mal eine neue Version hochgeladen.

Bisher, auch mit gcc 4.1.2, keine Probleme feststellbar. Nur war die uptime meiner Box
in den letzten Tagen nicht allzu hoch Sieht aber gefühlt besser aus als die Vorversion.

[Gaucho316]

Heute gibt's mal wieder eine neue Version der portierten Änderungen am sectionsd. Rausgenommen habe ich den SVN-Commit 1422 und die dazugehörigen Korrekturen. Ich habe nämlich beobachtet, dass diese Änderung irgendwie den Speicherverbrauch des sectionsd erhöht, da öfter ungeänderte Events gelöscht und neu eingefügt werden. Warum der Speicherverbrauch ansteigt, weiß ich allerdings auch nicht. Ich habe das immer auf die Boost-Bibliothek geschoben, aber laut seife arbeitet die sauber. Ich beobachte das mit "ps". Außerdem habe ich noch meinen Aktualisierungsvorschlag aus dem DBox2World-Forum eingefügt.

Link zum Patch entfernt

[seife]

Welchen Wert aus "ps" deutest du denn als speicherverbrauch?

[Gaucho316]

%MEM

[Gaucho316]

Heute gibt's mal wieder eine neue Version. Neu ist vor allem der SVN-Commit 1511, der erfolgreich doppelte Events beseitigt, wie man sie vor allem auf sixx und Servus TV nach einiger Laufzeit der Box findet. Achtet beim Testen bitte auch darauf, ob durch diesen Patch irgendwelche "Löcher" im EPG entstehen.

Link zum Patch entfernt

[Striper]

Habe das auf der Coolstream bereits seit mehreren Tagen im Einsatz. Funktioniert hervorragend und endlich keine Doppelevents mehr auf ServusTV und Co.

"Löcher" gibts damit auch keine.

[rhabarber1848]

Heute gibt's mal wieder eine neue Version. Neu ist vor allem der SVN-Commit 1511

Seit Veröffentlichung getestet, keine Auffälligkeiten.

[Gaucho316]

@dbt

Ich war ein paar Tage nicht im Lande, so dass ich erst heute gesehen habe, dass du meine Portierungen für den sectionsd eingecheckt hast. Dabei ist aber leider einiges reichlich schief gelaufen. Zum Ersten hast du leider die alte Version des Patches vom 23.04.2011 eingecheckt. Zweitens passt der Log-Eintrag überhaupt nicht zum Patch. Und drittens habe ich weiter oben geschrieben, dass der Patch als Ganzes nicht zum Einchecken gedacht war, da ich noch alle Änderungen einzeln parat habe, damit wir vernünftige Log-Einträge bekommen. Vielleicht hätte ich das lieber zu jedem Patch nochmals dazuschreiben sollen. Es wäre wohl am besten, wenn du den CVS-Commit vom 14.06.2011 wieder rückgängig machen könntest. Ich würde dann einzelne, aufeinander aufbauende Patches hochladen, die du Stück für Stück einchecken könntest, wenn dir das nicht zu viel Arbeit ist.

[dbt]

Einzeln ist mir lieber.

Edit:

Ich würde dann einzelne, aufeinander aufbauende Patches hochladen, die du Stück für Stück einchecken könntest,

Hab das grade zurückgesetzt, kannst die Patches jetzt wie du sagst, hier ablegen. Ich hatte tatsächlich den Überblick verloren.

[Gaucho316]

Ist überhaupt nicht schlimm, dass du den Überblick verloren hast. Du konntest es ja einfach rückgängig machen.

Hier sind nun also die versprochenen Patches alle einzeln aufeinander aufbauend. Thematisch zusammengehörende SVN-Commits habe ich zu einem Patch zusammengefasst. Den ersten Patch habe ich übrigens durch Vergleich der ersten Version von sectionsd.cpp aus dem Coolstream-SVN mit der dazu passenden Version aus dem Tuxbox-CVS zusammengebaut.

Ich sage auf alle Fälle schon mal danke fürs Einchecken.


Revision 27
Author: mrcolor
Log Message:

Code: Alles auswählen

sectionsd: disable unnecessary code and add some forgotten curly brackets

Link entfernt, da Patch im CVS


Revision 888
Author: seife
Log Message:

Code: Alles auswählen

SIsections: add parsing of PDC descriptor (VPS time)

Tdd the VPS time to the event if available. Not yet used for anything
useful ;-)

Link entfernt, da Patch im CVS


Revisions 1229 and 1261
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: improve CN thread's handling of non-broadcast events

* reduce EIT timeout to 3 seconds
* if there are no events broadcast, send an event to neutrino anyway
* when EIT has changed but events are the same, return to sleep faster

Link entfernt, da Patch im CVS


Revision 1253
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: improve CN EIT timeout detection

Link entfernt, da Patch im CVS


Revision 1254
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: improve eit_set_update_filter debug message

Link entfernt, da Patch im CVS


Revision 1255
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: improve getUTC function to decode TOT

Link entfernt, da Patch im CVS


Revision 1256
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: improve time acquisition from DVB stream

* during startup, get time from TDT, to get *any* time information quickly
* after startup, immediately (after 5 seconds) retry with TDT
* later calls only use TOT. TOT is not broadcast on all channels, but the
  channels that broadcast a TOT usually are much more "trustworthy" wrt
  time setting
Comments welcome, also reports if this works worse than before

Link entfernt, da Patch im CVS


Revision 1257
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: trigger time refetch after transponder change

This should ensure that the system time somehow matches the
transponder time. Comments and reports welcome on the developers
mailinglist. If this turns out to be not working well for all
setups, it could be made configurable.

Link entfernt, da Patch im CVS


Revision 1381
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: add table ID to event data

record the originating table ID with every event
if not set (e.g. for events read from saved data), it defaults
to 0xff to make sure it is bigger than a real existing table ID

Link entfernt, da Patch im CVS


Revision 1382
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: replace existing events with event from lower table ID

If we get a new epg event whose eventID is already existing, still
replace the existing event if the table ID of the new event is lower.
This comes from the observation that the lower table IDs have the
"more current" events:
Table 0x60-0x6f: other TS,   scheduled
      0x50-0x5f: current TS, scheduled
      0x4f:      other TS,   current/next
      0x4e:      current TS, current/next
So, at at least in theory, an event from table 0x4e should always be
"better" than one from 0x60.
If you see any regressions, don't hesitate to report them.

Link entfernt, da Patch im CVS


Revision 1392
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: don't take any chances in deleteEvent()

Just take the write lock right away, to make sure nothing changes the
event lists behind our backs and possibly corrupts the iterator.

Link entfernt, da Patch im CVS


Revision 1405
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: fix iterator type mismatch

Link entfernt, da Patch im CVS


Revisions 1406 and 1409
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: add timing info and number of read events to 'reading saved EPG' message

Link entfernt, da Patch im CVS


Revision 1408
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: simplify DMX::getSection code

Link entfernt, da Patch im CVS


Revisions 1410, 1473, 1474, 1475, 1479 and 1511
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: avoid duplicate events

When adding an event remove others overlapping the new one. This beats double
events on ServusTV and other broadcasters which do not honor the specs.

Link entfernt, da Patch im CVS


Revisions 1480 and 1506
Author: striper
Log Message:

Code: Alles auswählen

sectionsd: remove dup getName() call and refresh itemDescription, item and vps (thx Gaucho316)

Link entfernt, da Patch im CVS


Revision 1536
Author: striper
Log Message:

Code: Alles auswählen

sectionsd: remove superflous argument table_id in addEvent() (patch by Gaucho316)

Link entfernt, da Patch im CVS

[rhabarber1848]

committed to CVS:

http://article.gmane.org/gmane.comp.vid ... x.scm/2985
http://article.gmane.org/gmane.comp.vid ... x.scm/2986
http://article.gmane.org/gmane.comp.vid ... x.scm/2987
http://article.gmane.org/gmane.comp.vid ... x.scm/2988
http://article.gmane.org/gmane.comp.vid ... x.scm/2989
http://article.gmane.org/gmane.comp.vid ... x.scm/2990
http://article.gmane.org/gmane.comp.vid ... x.scm/2991
http://article.gmane.org/gmane.comp.vid ... x.scm/2992
http://article.gmane.org/gmane.comp.vid ... x.scm/2993
http://article.gmane.org/gmane.comp.vid ... x.scm/2994
http://article.gmane.org/gmane.comp.vid ... x.scm/2995
http://article.gmane.org/gmane.comp.vid ... x.scm/2996
http://article.gmane.org/gmane.comp.vid ... x.scm/2997
http://article.gmane.org/gmane.comp.vid ... x.scm/2998
http://article.gmane.org/gmane.comp.vid ... x.scm/2999
http://article.gmane.org/gmane.comp.vid ... x.scm/3000
http://article.gmane.org/gmane.comp.vid ... x.scm/3001

[Gaucho316]

Ich habe jetzt noch einen weiteren Patch für den sectionsd aus dem SVN portiert.

Revision 1543
Author: seife
Log Message:

Code: Alles auswählen

sectionsd: improve deletion of surplus events in addEvent()

If the number of events is bigger than the configured maximum, first
remove old events and only if there are none left start removing the
most future events.

Link entfernt, da Patch im CVS

[Gaucho316]

Heute habe ich mich zum wiederholten Male dabei erwischt, wie ich in einem Menü eine der Seitenscrolltasten (im Normalfall Vol+ und Vol-) gedrückt habe, um eine Seite weiterzuscrollen. Dabei habe ich natürlich nur die Lautstärke verstellt. Passiert ist mir das übrigens in der Kanalauswahl beim händischen Anlegen eines Timers. Deshalb habe ich mal schamlos den entsprechenden Code aus Neutrino HD rüberportiert, damit das in Zukunft nicht mehr vorkommt.

Link zum Patch entfernt

[seife]

Halte ich für fragwürdig. Boxen mit neutrino-HD haben mehr Tasten auf der Fernbedienung und eben nicht die Volume-Tasten als Page up/down. Somit kann man dort auch im Menü noch die Lautstärke einstellen.
Ich fände es eher unpraktisch, immer erst aus den Menüs rausgehen zu müssen, nur um leiser zu machen.

Dann lieber die Menüs aufräumen, dass man nicht so viel hoch und runterscrollen muss ;-)

[Gaucho316]

An anderen Stellen in Neutrino ist es aber auch so, dass man die Lautstärke nicht einstellen kann und stattdessen mit Vol+ und Vol- scrollt. Als Beispiel nenne ich mal die Kanalliste und den Moviebrowser. Es wäre also nur konsequent, dass in den Menüs auch so zu machen. Ich hatte übrigens noch nie den Bedarf, die Lautstärke zu ändern, während ich in einem Menü war.

[seife]

In Kanalliste und Moviebrowser gibt es aber auch mehr durchzuscrollen ;-)

Egal, mich betrifft es nicht, insofern ist es mir egal.

[Gaucho316]

Gibt es hierzu noch andere Meinungen? Im Moment steht es ja 1:1. Ich würde die Änderung schon gerne im CVS sehen.

[MTM]

Hallo,
ich bin dafür es einheitlich zu machen, mir passiert das nämlich auch manchmal. Also pro Gaucho316.

MfG,
MTM.

[rhabarber1848]

Heute habe ich mich zum wiederholten Male dabei erwischt, wie ich in einem Menü eine der Seitenscrolltasten (im Normalfall Vol+ und Vol-) gedrückt habe, um eine Seite weiterzuscrollen.

Das habe ich noch nie versucht, warum auch?

Dabei habe ich natürlich nur die Lautstärke verstellt.

Ich ändere während einer Menünavigation schon mal absichtlich die Lautstärke und möchte das weiterhin können.

Ich fände es eher unpraktisch, immer erst aus den Menüs rausgehen zu müssen, nur um leiser zu machen.

[MTM]

Hallo,

Ich ändere während einer Menünavigation schon mal absichtlich die Lautstärke und möchte das weiterhin können.

Und was ist während einer Bouquetnavigation?

Zugegebenermaßen gibt es nicht viele mehrseitige Menüs, aber falls dich die Lautstärke dann dort drin stört, gibts ja immernoch Mute. Wie auch immer, es ist uneinheitlich. Leider hat die dbox2-FB ja keine Doppelpfeiltasten.

MfG,
MTM.

[seife]

meine (unmaßgebliche ;-)) Meinung: bei den Bouquets muss man eher mal viele Seiten scrollen als in den menüs.

...aber meine boxen haben alle PgUp/PgDown, deswegen betrifft es mich nicht ...