100% CPU-Last beim sectionsd bis Uhrzeit ermittelt ist

[Npq]

Moin,

nach Hinweisen im IRC hatte ich mir gestern mal - kurz - den sectionsd angeschaut. Ich fand es ein wenig seltsam, daß eine Box, die keinen(!) Empfang hat, zwei Threads aushalten muß, die sich zusammen mal eben 100% der CPU-Leistung nehmen.

Die Ursache ist aber einfach, NIT und EIT-Thread hängen einfach in einer Endlosschleife bis die Uhrzeit ermittelt wird.

Code: Alles auswählen

for (;;)
{
    if (timeset) {
    ...
    }
}

Wobei es beim EIT noch etwas anders strukturiert zu sein scheint, aber der absolut unstrukturierte Aufbau der Routine macht das Analysieren schwierig. "Mal eben" drüberschauen ist dabei nämlich nicht mehr möglich, da man sich in den gewaltigen if- und else-Klauseln verliert.

Also guckt euch das mal bitte an.

[Houdini]

Jetzt weisst du ja warum der sectionsd suckt!
duck und

[Innuendo]

demnach müßten alle, die das bootproblem nicht haben, in der sectionsd die zeit per ntp abfragen. probier ich heut abend mal aus.

innu

[Nirvana]

@Npq

*mit der flachen Hand vor den Kopf hau*. Natürlich - das ist es!

[Nico 77]

demnach müßten alle, die das bootproblem nicht haben, in der sectionsd die zeit per ntp abfragen. probier ich heut abend mal aus.

innu

Bootzeit verkürzt sich hier beim starten.

Bootzeit DVB: 1:20sec (ARD)
Bootzeit NTP: 0:50sec (ARD)

[petgun]

Bootzeit verkürzt sich hier beim starten.
Bootzeit DVB: 1:20sec (ARD)
Bootzeit NTP: 0:50sec (ARD)

..dann war das wohl der Fehler und ich bin froh dass das damit sicher bald vom Tisch ist (Danke an NPQ)...auf den Wuergaround bin ich allerdings gespannt...nicht jeder hat seine Box staendig am Internet/einen NTP-Server im eigenen Netz laufen.

[Houdini]

auf den Wuergaround bin ich allerdings gespannt

Der ist billig: ein sleep(X) solange bis die Uhrzeit da ist wirkt Wunder.
Oder man könnte auch die Threads beim ersten mal alle gleich schlafen schicken und der timethread weckt sie dann alle auf, wenn die Zeit da ist.
Aber was ist wenn keine Zeit kommt...

Houdini

[Innuendo]

...auf den Wuergaround bin ich allerdings gespannt...

polemik hilft grad nicht weiter und wird nirvana auch sicher nicht motivieren.

ist im jtg image in der start_neutrino der eintrag
date -s 010101001970
enthalten?
ich meine mich daran zu erinnern, dass die sectionsd mal um

Code: Alles auswählen

// -- do we already have a valid(???) date/time?
if ((tmTime->tm_year + 1900) >= 2005) {
	first_time = false;
	timeset = true;
  	dprintf("we already have a time set\n");
}

und diesem date eintrag in start_neutrino erweitert wurde. wenn beim booten die zeit auf irgendwas gesetzt wird, bleibt timeset auf false, bis vom sat eine uhrzeit angekommen ist - und die schleife um if(timeset) wartet geduldig darauf.

innu

edit
@houdini: initialisier zum testen timeset = true;
allerdings gibt es dann mecker, dass die uhr falsch geht ;-)

[Nirvana]

polemik hilft grad nicht weiter und wird nirvana auch sicher nicht motivieren.

WAS? Wieso ich denn schon wieder?
Der Fehler hat da schon immer immer geschlummert. Ich habe ihn nur leider auch nicht behoben. Er tritt nur jetzt deutlicher zu Tage, weil es mehr Threads sind, die sich kloppen. Petgun wird mir noch ewig dankbar sein, weil seine Box in Zukunft 7 Sekunden schneller bootet, als je zuvor.

[Innuendo]

Petgun wird mir noch ewig dankbar sein, weil seine Box in Zukunft 7 Sekunden schneller bootet, als je zuvor.

morgen kommt der weihnachtsmann ... *summ*

[petgun]

Petgun wird mir noch ewig dankbar sein, weil seine Box in Zukunft 7 Sekunden schneller bootet, als je zuvor.

...wenn's wirklich so ist...sage ich jetzt schon mal _DANKE_!! ...noch mehr Danke, dass Du Dich von Polemik nicht demotivieren laesst.

@Innuendo
..ich will wirklich nicht polemisieren...habe im Moment nur Angst vor weiteren Schnellschuessen die zum Dauerzustand werden koennten.

[Metallica]

Petgun wird mir noch ewig dankbar sein, weil seine Box in Zukunft 7 Sekunden schneller bootet, als je zuvor.

*LOL*

[petgun]

Petgun wird mir noch ewig dankbar sein, weil seine Box in Zukunft 7 Sekunden schneller bootet, als je zuvor.

*LOL*

;-) freut mich dass Dich das so sehr belustigt...ich kann 'ewig dankbar' sein...und ich werde dieses Thema (rund um secionsd) in guter Erinnerung behalten und nicht vergessen wer was/wie daran gemacht hat.

[Houdini]

Der Fehler hat da schon immer immer geschlummert.

stimmt

weil seine Box in Zukunft 7 Sekunden schneller bootet, als je zuvor

kann ich mir durchaus vorstellen

Also los, ran ans Werk.

[jmittelst]

Hätte keiner dran gebastelt, wäre nix passiert, so wird es besser und besser und ...

cu
Jens

[MOhlmann]

ist im jtg image in der start_neutrino der eintrag
date -s 010101001970
enthalten?

Ja das ist in der rcS drin

Code: Alles auswählen

/bin/date -s 0101120070 

habe mal meine start.neutrino dann geändert und das hier hinzugefügt.

Code: Alles auswählen

/bin/date -s 010112002005 

Was soll ich sagen, die Boot Zeit von 85 sec. auf 55 sec. runter.

Gruß Michael

[Torsten73]

demnach müßten alle, die das bootproblem nicht haben, in der sectionsd die zeit per ntp abfragen. probier ich heut abend mal aus.

innu

Stimmt ich hole auch die Zeit vom Satelitten.

Cu
Torsten

Nachtrag:
Bedenkt bitte auch dabei, dass es weiterhin sichergestellt sein muß mit der richtigen Uhrzeit zu arbeiten. Denn sonst haben wir wieder Probleme mit den Timern, die aufgrund falscher und fehlender Uhrzeit/Datum gelöscht werden!

[PauleFoul]

Tolle Arbeit Jungs!! Und wieder wird Neutrino ein ganzes Stück besser.

Dafür danke ich den "neuen" Dev es und allen Imagefuzies *vor Riker duck*


Gruß
____Paule

[zexma]

Petgun wird mir noch ewig dankbar sein, weil seine Box in Zukunft 7 Sekunden schneller bootet, als je zuvor.

...wenn's wirklich so ist...sage ich jetzt schon mal _DANKE_!! ...noch mehr Danke, dass Du Dich von Polemik nicht demotivieren laesst.

OT.
Wenn Du selbst weißt, dass es nur "(demotivierende) Polemik" ist, warum verzichtest Du zukünftig nicht einfach drauf?

[Torsten73]

Hi,
wie ich gerade sehe, hat Homar einen Fix zum obigen Fehler ins CVS gestellt!

Schon mal im Voraus vielen Dank!

Cu
Torsten

[rasc]

ich meine mich daran zu erinnern, dass die sectionsd mal um

Code: Alles auswählen

// -- do we already have a valid(???) date/time?
if ((tmTime->tm_year + 1900) >= 2005) {
	first_time = false;
	timeset = true;
  	dprintf("we already have a time set\n");
}

jo, da bin ich der Schuldige...

Aber eigentlich sollte das keine brute force, sondern irgendwo sollte ein sleep (1) dazwischen sein.

Da es ansonsten (worst case) bis 30 sec. dauern kann, bis die Zeit da ist, waere die Box ansonsten komplett dicht.

[Innuendo]

jo, da bin ich der Schuldige...

prima - auch wenn da schon ein "schneller-starten-patch" eingecheckt wurde - hauptsache wir haben einen schuldigen.
teeren, federn und durch dorf jagen sag i da

[BOFH]

Nicht hauen, aber könnte man dieses Zeitdingens nicht parallel auf NTP und DVB jagen?
Einer wird's schon packen.

*Duck* und wech........

Anonymous
BOFH

[mws]

ich weiss nicht, wieviele personen eine dbox2 mit linux und neutrino betreiben. aber von den dbox2 sind höchstens (meine einschätzung) 10% ständig am inet und könnten ntp nutzen.

da auf dem sat die zeit immer kommt - wenn sie falsch kommt, schlagt die provider ! - ist das die sinnigste methode.

ansonsten steht es jedem frei, das mal $neu zu implementieren. von mir aus auch per gui ein/aus/umschaltbar.
mit implementieren meine ich aber nicht reinfrickeln

gruss
mws

[DrStoned]

Mit dem letzten JtG-Snap vom 01.12.2005 22:22 Uhr läuft das Booten wieder wie gewohnt, ob mit oder ohne übers INET gesetzte Uhrzeit. Jetzt mal schauen wie sich der sectionsd verhält.

Greetz von DrStoned