Änderung in Zapit macht zappen träge...

[PauleFoul]

Hallo zusammen,

anscheinend führen diverse Änderungen an der Zapit dazu das Neutrino beim Zappen
träger geworden ist bzw. die Infobar später angezeigt wird.
Offensichtlich wird die Infobar jetzt erst angezeigt wenn der Sender gelockt ist.

Kann man da bitte wieder zurück bauen bzw. welche Vorteile soll diese Änderung bringen??


Gruß
____Paule

[seife]

- welche Änderung?
- sicher, dass es nicht der sectionsd ist?
- bist du einer von denen, die den sectionsd neu starten, ohne neutrino neu zu starten? (Hint: das ist zwar weitverbreitet, aber halt ein extrem dummer Fehler ;-))

[PauleFoul]

- welche Änderung?

Kann ich Dir leider nicht genau sagen... Das Verhalten der Infobar hat sich jedenfalls geändert.

- sicher, dass es nicht der sectionsd ist?

Möglich...

- bist du einer von denen, die den sectionsd neu starten, ohne neutrino neu zu starten? (Hint: das ist zwar weitverbreitet, aber halt ein extrem dummer Fehler ;-))

Nö, nichts neu starten. Einfach nur zappen.


Gruß

[rhabarber1848]

anscheinend führen diverse Änderungen an der Zapit dazu das Neutrino beim Zappen
träger geworden ist bzw. die Infobar später angezeigt wird.
Offensichtlich wird die Infobar jetzt erst angezeigt wenn der Sender gelockt ist.

Ich kann die beschriebene Verhaltensänderung bestätigen. Im März 08 wurden im zapit-
Bereich einige Patches ins CVS eingecheckt: http://cvs.tuxbox-cvs.sourceforge.net/l ... reads.html

Welcher Patch für die Änderung verantwortlich ist, entzieht sich meiner Kenntnis, vielleicht hat
es etwas mit den "lost lock"-patches zu tun:
http://cvs.tuxbox-cvs.sourceforge.net/l ... 00044.html

Dirty hack for lost lock checking. Works only with PMT update enabled.
Disable with "zapit -l".

http://cvs.tuxbox-cvs.sourceforge.net/l ... 00064.html

Fix "lost lock" check to not interfere with transponder scan.

@PauleFoul: Scheinbar kann der neue Code durch "zapit -l" auf der dbox-Konsole
abgeschaltet werden. Kannst Du mal testen, ob es hilft?

[PauleFoul]

@PauleFoul: Scheinbar kann der neue Code durch "zapit -l" auf der dbox-Konsole
abgeschaltet werden. Kannst Du mal testen, ob es hilft?

Nein das hillft leider auch nichts. Dadurch kommt noch hinzu das beim schnellen zappen
der Sender nicht mehr richtig mit umgeschaltet wird.

Man zappt z.B. schnell ARD>>ZDF>>RTL>>VOX
Die Infobar macht das auch aber Sender wird nur auf ZDF umgeschaltet.


Gruß
____Paule

[mb405]

seifes änderungen haben meiner meinung das zappen verbessert, denn es kommt wesentlich weniger kanal nicht verfügbar.

[seife]

Ich hatte auch keine negativen Nebenwirkungen gefunden, und ich hatte den Code auf verschiedenen Umgebungen getestet (Kernel 2.4/linuxthreads, Kernel 2.6/NPTL, dreambox).
Insofern müste da IMHO etwas mehr debugging betrieben werden. Wenn z.B. das genaue CVS-Datum, an dem es kaputt ging, bekannt wäre (und die Umgebungsbedingungen wie: Konfiguration, andere auf der Box laufende Dienste, zusätzliche Patches...), dann könnte man da was zu sagen.

[rhabarber1848]

Meine Beobachtung ist die, dass mit dem neuen Code die dBox
beim mehrfachen Drücken der Kanal +/- Tasten erst dann einen
Kanal weiterschaltet, wenn der aktuelle Kanal gelockt wurde.

Vorher konnte man schneller mehrere Kanäle überspringen,
z.B. von Programm1 nach Programm5 mit fünfmal Kanal +
drücken, da die Programme2-4 nicht gelockt wurden.

[Houdini]

Full ack, da ist was bei Seifes Fastzap Implementation in die Hose gegangen.

Da ich aber nicht recht weiß was Seife damit vorhatte sollte er die Änderung unten noch abnicken.
Tatsache ist das bei der DBOX das zapNOWAIT nicht mehr gemacht wird.

So gehts bei mir:

Code: Alles auswählen

Index: lib/zapitclient.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/dvb/zapit/lib/zapitclient.cpp,v
retrieving revision 1.116
diff -b -B -u -r1.116 zapitclient.cpp
--- lib/zapitclient.cpp	21 Mar 2008 12:21:19 -0000	1.116
+++ lib/zapitclient.cpp	23 Apr 2008 21:05:08 -0000
@@ -176,13 +176,13 @@
 /* zaps to channel, does NOT wait for completion (uses event) */
 void CZapitClient::zapTo_serviceID_NOWAIT(const t_channel_id channel_id)
 {
-	(void)zapTo_serviceID(channel_id, false);
+	(void)zapTo_serviceID(channel_id, true);
 }
 
 /* zaps to subservice, does NOT wait for completion (uses event) */
 void CZapitClient::zapTo_subServiceID_NOWAIT(const t_channel_id channel_id)
 {
-	(void)zapTo_subServiceID(channel_id, false);
+	(void)zapTo_subServiceID(channel_id, true);
 }
 
 
Index: src/zapit.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/dvb/zapit/src/zapit.cpp,v
retrieving revision 1.409
diff -b -B -u -r1.409 zapit.cpp
--- src/zapit.cpp	21 Mar 2008 19:17:15 -0000	1.409
+++ src/zapit.cpp	23 Apr 2008 21:05:10 -0000
@@ -2765,6 +2766,8 @@
 #endif
 #ifdef HAVE_DREAMBOX_HARDWARE
 	setFastZap(fastzap);
+#else
+	fastzap = 0;
 #endif
 
 	time_t lastlockcheck = 0;

[seife]

Ups.

Sieht richtig aus, ich kann es aber momentan nicht testen. Hat es bei dir geholfen?
Ich würde das in der zapit.cpp so machen:

Code: Alles auswählen

RCS file: /cvs/tuxbox/apps/dvb/zapit/src/zapit.cpp,v
retrieving revision 1.409
diff -u -p -r1.409 zapit.cpp
--- src/zapit.cpp       21 Mar 2008 19:17:15 -0000      1.409
+++ src/zapit.cpp       24 Apr 2008 06:02:37 -0000
@@ -157,8 +157,12 @@ extern std::map<t_satellite_position, ui
 extern std::map<string, t_satellite_position> satellitePositions;

 bool standby = true;
+#ifdef HAVE_DREAMBOX_HARDWARE
 /* on dreambox: use FASTZAP ioctl? */
 int fastzap = 1;
+#else
+int fastzap = 0;
+#endif

 uint32_t lastChannelRadio;
 uint32_t lastChannelTV;

Das finde ich eindeutiger / weniger verwirrend als erst oben fastzap als 1 zu definieren und dann später zu 0 zu ändern.

[Houdini]

committed!

[seife]

Danke. Komisch, dass das bei mir daheim niemand aufgefallen ist. Meine Betatester sind normalerweise recht kritisch... ;-)

[PauleFoul]

Danke. Komisch, dass das bei mir daheim niemand aufgefallen ist. Meine Betatester sind normalerweise recht kritisch... ;-)

Wohl nicht kritisch genug...

committed!

Danke!

[Boardgeist]

War schon kritisch und nervig das ganze.
Funktioniert nun wieder flüssig und einwandfrei!

Danke!

gruß boardgeist