Menu selection patch

PeterD · 28. Februar 2004

Hallo,

Hier ne neue version des menu selection patch:
menu_sel-0.3.diff

Neu ist jetz das die eingegebenen zahlen unten in der status leiste als "1-" eingeblendet werden. Ausserdem wird der selektionsbalken auf den eintrag mit der entspechenden nummer gesetzt.

Ist zwar noch unter Komplettpatch-3E + improved OSD entwickelt, sollte aber auch mit neueren patches laufen (aber noch nicht getestet).
Über die verwendung bei 1.3.x kann ich nichts sagen, da ich noch 1.2.6 verwende.

gruss Peter

P.S. habe noch einen heftigen bug in version 0.2a entdeckt. Das diff enthält die änderungen an osd.h nicht. Wer 0.1a nicht vorher probiert hatte hätte eigentlich compilier probleme haben müssen

gestein · 28. Februar 2004

Funktioniert bei mir wirklich toll. Danke dafür.

Mal sehen, ob ich das in den KP-f einbauen kann.

lg, Gerhard

steffx · 28. Februar 2004

Hallo Zusammen,

hat sich schon jemand die Mühe gemacht den Patch auf 1.3.x anzupassen ?

steffx

viking · 4. März 2004

Hallo,

ist dir da nicht einen fehler unterlaufen ?

Der patch is komplett für osd.h !?

Es sollte vermutlich nur der anfang in osd.h und der rest in osd.c

Ansonsten tolle idee dein patch

Gruß
Viking

brst · 4. März 2004

Hallo zusammen,

sorry, aber irgendwie bekomme ich das mit dem menu_sel-0.3.diff Patch nicht gebacken.
Ich habe vdr-1.2.6 mit dem vdr-1.2.6-ElchiAIO4c.diff.gz versehen. All meine Versuche haben leider nicht zum erflogreichen patchen geführt. Kann mir bitte jemand sagen, wie ich den menu Patch anwenden muss. cat menu_sel-0.3.diff | patch führt leider nicht zum Ziel. Auch nicht mir dem p1 Parameter.

Für jeden Tipp bin ich sehr dankbar, da ich den älteren menu Patch bereits hatte und ihn super finde.

Bis dahin

Ciao Stefan

gestein · 4. März 2004

Da ich meinen eigenen KP zusammen gestellt habe, habe ich einfach die paar Zeilen per Hand eingetragen.

Ist nicht wirklich schwer und funktioniert auch.

lg, Gerhard

brst · 5. März 2004

Hallo Gerhard,

danke für den Tipp. Werde ich heute Abend mal ausprobieren.

Ciao Stefan

RaK · 5. März 2004

In dem Patch ist irgendwie der Wurm drin. Er patcht nur osd.h und versucht dei Aenderungen die eigentlich nach osd.c muessen dort einzubauen. Das gibt natuerlich ein reject.

Ich hab die Aenderungen manuell in osd.c eingebaut und jetzt funktioniert das. Nur wenn ich ein Diff mache hilft das keinem, da ich einen heavy gepatchtes 1.2.5 betreibe ;).

Aber vielleicht kann sich der Author das nochmal genau anschauen.

Gruss

viking · 5. März 2004

Hallo,

ist eigentlich ganz einfach.

vor dem Zweiten @@ :

Code

@@ -345,6 +328,7 @@

müsst ihr dies einfügen - dann gehts :

Diff

--- ../vdr-1.2.6-test/osd.c	2004-01-15 21:56:43.000000000 +0100
+++ osd.c	2004-01-12 21:50:16.000000000 +0100

ach ja und mit "patch -p2" einspielen

Gruß
Viking

niklot · 7. März 2004

Hi,

prinzipiell finde ich den Patch sehr gut, aber leider hat er bei mir (vielleicht auch bei anderen?) eine
unangenehme Nebenwirkung. Die CPU-Last (Athlon 900MHz) steigt bei Anzeige des OSD auf 25%. Ohne den Patch gibt es bei der CPU-Auslastung keinen Unterschied, egal ob ich gerade eine OSD-Anzeige habe oder nicht. Kann man da am Patch noch irgendetwas ändern ?

Gerade hae ich mal menu_sel0.2a probiert. Der hat die Probleme nicht

gruesse
michael

viking · 8. März 2004

Hallo,

habe nachgetestet und kann die beobachtung von niklot bestätigen !

Bei mir sind das ca. 6% bei einen P3 933Mhr.

Gruß
Viking

PeterD · 11. März 2004

Hab's auch bei mir mal nachgesehen: liegt so bei 5..10% im menu gegenüber 2..5% ausserhalb (TOP / VIA C3N 1GHz).

Die menu funktion wird halt 1x pro sekunde aufgerufen. Checked auch die systemzeit + ein paar printf ...
Eventuell zu viele OSD updates ...
Durch die timeouts eben nicht für lau zu realisieren.
Vielleicht kann ich noch ein bischen herausquetschen.

Ich weis nur nicht ob es das wert ist.
VDRCONVERT reduziert bei mir den idle wert um 60..70% und bringt sogar bei der CPU 5..10° temperaturerhöhung.

Aber 25% sind echt nicht normal. Hast du's mal ohne den patch versucht?

gruss Peter

P.S. habe gerade noch mal ps ausprobiert:
1. ist anschenend der vdr root thread
2. TOP sagt ~10%; ps sagt 0.3%; wer hat nun recht

EDIT
schau mal in cOsdMenu::HotKey(eKeys Key):

if (Key == kNone) {
if (key_nr == -1) {
// SetStatus(NULL);
// Display();
}
if ((time(NULL) - lastActivity) >= MENUKEYTIMEOUT) {
Key = kOk;
}
else {
return osContinue;
}
}

ich glaub die zwei auskommentierten zeilen verursachen das.
Wird halt 1x pro sekunde ausgeführt, ob nötig oder nicht.
Muss halt mal sehen ob das sparsamer geht ...
/EDIT

PeterD · 11. März 2004

Hier mal ein erster korrektur patch (ist noch nicht perfekt):

in cOsdMenu::HotKey(eKeys Key):

if (Key == kNone) {
- if (key_nr == -1) {
+ if ((key_nr == -1) && (status != NULL)) {
SetStatus(NULL);
Display();
}

und

- if (!match) {
+ if ((!match) && (Key != kNone)) {
key_nr = -1;
SetStatus(NULL);
Display();
}

das sollte die CPU time runter kriegen

Peter

PeterD · 11. März 2004

hier nun ein endgültiger patch:

menu_sel-0.3a.diff

Das CPU problem sollte jetzt weg sein.
(Während der eingabe geht's aber mal für ne sekunde auf ~5%)

Habe die anderen links auf 0.3 entsprechend angepasst.

gruss Peter

niklot · 12. März 2004

Hi Peter,

erst mal vielen Dank für deine Mühen. Also meine 25 % beziehen sich auf top
und ich weiss auch aus eigener leidiger Erfahrung, dass die Anzeige von top
zumindest ungenau ist . Was halt auffiel war eine deutliche Erhöhung der CPU-Last.

Leider komme ich erst heute abend zum testen, aber dann gibts auf jeden Fall ein Feedback.

erstmal besten Dank für deine Bemühungen

gruesse
michael

Menu selection patch

Jetzt mitmachen!

Teilen