4. PrettyOS Fehler-/Testthread

PrettyOS Fehler-/Testthread

  • M

    Das Problem ist bekannt und auch mit Microsoft Virtual PC läuft PrettyOS derzeit nicht, anscheinend gibt es auch Probleme mit einigen echten PCs, die damit vlt. zusammenhängen.

    0
  • E
    Mod

    00:00:03.394 PIT: mode=3 count=0x2e9b (11931) - 100.00 Hz (ch=0)
    00:00:05.612 fatal error in recompiler cpu: triple fault
    00:00:05.612 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
    00:00:05.612 !!
    00:00:05.612 !! Guru Meditation -2301 (VERR_REM_VIRTUAL_CPU_ERROR)

    Leider versteht niemand bisher diesen "fatal error".

    0
  • M

    Sooo... Ich hab auch Fehler im Angebot 😉

    1. Page-Fault unter Qemu 0.12.2. Screenshot: http://kloke-witten.dyndns.org/~philipp/Temp/Fehler_Rev125.png

    2. Uhr läuft ca. 2x zu schnell unter Virtual Box 3.1.4

    3. Dieser Fehler ist etwas komplizierter zu beschreiben und ich hab ihn auch schon geschildert, wobei Erhard und ich nicht zu einer Lösung gekommen sind (Im IRC). Bevor das in Vergessenheit gerät, kommt hier die Fehlerbeschreibung (Ich hab das Verhalten auch noch etwas genauer Analysiert):
    Das Problem tritt in Userprogrammen bei der Texteingabe auf. Die Backspace-Taste funktioniert nicht richtig, wenn man versucht, das erste Zeichen der Zeile zu löschen: Es wird das erste Zeichen dieser Zeile und das letzte Zeichen der vorigen Zeile gelöscht. Der Schreibcursor springt dann auch in die vorige Zeile. Testsystem: Virtual Box 3.1.4; Der Fehler tritt meines Wissens nach auch mit anderen Simulationen auf, komischerweise hab aber bislang nur ich ihn reproduzieren können 😕 .

    mfg und in der Hoffnung, das die Fehler gefunden und behoben werden 🙂
    Mr X

    0
  • E
    Mod

    zu 3)
    Das ist der Code von gets:

    char* gets(char* s)
{
    int i=0;
    char c;

    //settaskflag(0);

    do
    {
        c = getch();
        putch(c);
        if(c==8)  // Backspace
        {
           if(i>0)
           {
              s[i-1]='\0';
              --i;
           }
        }
        s[i] = c;
        i++;
    }
    while(c!=10); // Linefeed
    s[i]='\0';

    //settaskflag(1);

    return s;
}

    Vielleicht fällt jemand da etwas auf? MrX und Cuervo mäkeln daran herum.

    //settaskflag(0); <--- warum auskommentiert?

    0
  • M

    Nachdem ich mir gets() näher beguckt hab, bin ich zu dem Schluss gekommen, das der Fehler mglw. eher in getch() o.ä. zu suchen ist.

    Abgesehen davon denke ich, das z.20 und 21 von einem else-Block umschlossen werden sollten; Es ist imho wohl eher nicht sinnvoll, Die Felder, wo gelöscht wurde mit dem Wert 8 zu füllen, nachdem man sie grad zuvor mit 0 gefüllt hat.

    0
  • E
    Mod

    Damit geht es nun:

    Testversion:

    char* gets(char* s)
{
    int i=0,flag=0;
    char c;
    //settaskflag(0);
    do
    {
        c = getch();
        //settextcolor(i,0);///TEST

        if(c=='\b')  // Backspace
        {
           if(i>0)
           {
              putch(c);
              s[i-1]='\0';
              --i;
           }
           else
           {
               beep(50,20);
               if(flag==false)
               {
                   putch('\n');
                   flag=true;
               }
           }
        }
        else
        {
            s[i] = c;
            putch(c);
            flag=false;
            i++;
        }
    }
    while(c!=10); // Linefeed
    s[i]='\0';

    settextcolor(15,0);
    int j;
    for(j=0;j<15;j++)
    {
        if(s[j]=='\b')
        {
            puts("backspace");
        }
        else if(s[j]=='\0')
        {
            puts("EOS");
            putch('\n');
            break;
        }
        else if(s[j]=='\n')
        {
            puts("NL");
        }
        else
        {
            putch(s[j]);
        }
        putch('\n');
    }
    //settaskflag(1);

    return s;
}

    Ich kommentiere das aus und stelle es hoch. Liegt nicht am TTT, habe es separat mit einem anderen Programm getestet:

    #include "userlib.h"

int main()
{
    puts("********************************************************************************\n");
    puts("*                                  Eingabe-Test                                *\n");
    puts("********************************************************************************\n");

    char str[100];
	for(int i = 0; i < 5; ++i)
	{
	    puts("Bitte Eingabe:\n");
	    gets(str);
	    puts(str);
	}
	return 0;
}
    0
  • E
    Mod

    zu 1) das liegt an der in Paging neu eingebauten Funktion, die momentan nur ab 0xF0000000 funktioniert. Badestrand untersucht diesen Punkt bereits.

    zu 3) MS VPC läuft ok; Sun VB wirklich merkwürdig mit den Sekunden. 😃

    Man schafft es auch eher, bei Sun VB durch heftiges Drücken der Backspace-Taste in die obere Zeile zu geraten. Bei MS VPC gelingt mir das nicht.

    MS VPC ist für Tests offenbar gar nicht so schlecht geeignet.

    0
  • E
    Mod

    An gets(...) wird immer noch herum gemeckert. 🙄
    Bei mir klappt es soweit. Grund für Probleme liegt vermutlich an anderer Stelle. Eingegebener String kommt sauber an. Rücksprung auf obere Zeile nur selten (muss mit dem Taskwechsel zusammen hängen, den man ja ausschalten kann in der Funktion <--- wurde von MrX getestet: brachte nichts). Mysteriös.

    0
  • M

    Ich hab PrettyOS mal durch cppcheck gejagt. Folgendes Ergebnis kam raus (Irrelevantes à la "use fgets instead of gets" entfernt):

    [elf.c:74]: (style) struct or union member 'elf_header_t::shoff' is never used
[elf.c:75]: (style) struct or union member 'elf_header_t::flags' is never used
[elf.c:76]: (style) struct or union member 'elf_header_t::ehsize' is never used
[elf.c:79]: (style) struct or union member 'elf_header_t::shentrysize' is never used
[elf.c:80]: (style) struct or union member 'elf_header_t::shnum' is never used
[elf.c:81]: (style) struct or union member 'elf_header_t::shstrndx' is never used
[elf.c:101]: (style) struct or union member 'program_header_t::paddr' is never used
[elf.c:104]: (style) struct or union member 'program_header_t::flags' is never used
[elf.c:105]: (style) struct or union member 'program_header_t::align' is never used
[fat12.c:676]: (style) The scope of the variable j can be reduced
[paging.c:33]: (style) struct or union member '__attribute__::ext' is never used
[video.c:177]: (style) The scope of the variable temp can be reduced
[../tools/src/CreateFloppyImage.cpp:30]: (style) struct or union member 'BootSector::ignore1' is never used
[../tools/src/CreateFloppyImage.cpp:31]: (style) struct or union member 'BootSector::oem_name' is never used
[../tools/src/CreateFloppyImage.cpp:38]: (style) struct or union member 'BootSector::media_destriptor_byte' is never used
[../tools/src/CreateFloppyImage.cpp:40]: (style) struct or union member 'BootSector::sectors_per_track' is never used
[../tools/src/CreateFloppyImage.cpp:41]: (style) struct or union member 'BootSector::head_count' is never used
[../tools/src/CreateFloppyImage.cpp:42]: (style) struct or union member 'BootSector::hidden_sectors' is never used
[../tools/src/CreateFloppyImage.cpp:43]: (style) struct or union member 'BootSector::sector_count_ex' is never used
[../tools/src/CreateFloppyImage.cpp:44]: (style) struct or union member 'BootSector::drive_number' is never used
[../tools/src/CreateFloppyImage.cpp:45]: (style) struct or union member 'BootSector::ignore2' is never used
[../tools/src/CreateFloppyImage.cpp:46]: (style) struct or union member 'BootSector::boot_signature' is never used
[../tools/src/CreateFloppyImage.cpp:47]: (style) struct or union member 'BootSector::volume_serial' is never used
[../tools/src/CreateFloppyImage.cpp:48]: (style) struct or union member 'BootSector::fs_name' is never used
[../tools/src/CreateFloppyImage.cpp:50]: (style) struct or union member 'BootSector::bootloader' is never used
[../tools/src/CreateFloppyImage.cpp:51]: (style) struct or union member 'BootSector::bootloader_sig' is never used
[../user/user_program_c/userlib.c:418]: (style) The scope of the variable i can be reduced
[../user/user_program_c/userlib.c:419]: (style) The scope of the variable temp1 can be reduced
[../user/user_program_c/userlib.c:419]: (style) The scope of the variable temp2 can be reduced
[../user/user_program_c/userlib.c:419]: (style) The scope of the variable temp3 can be reduced
[../user/user_test_c/userlib.c:418]: (style) The scope of the variable i can be reduced
[../user/user_test_c/userlib.c:419]: (style) The scope of the variable temp1 can be reduced
[../user/user_test_c/userlib.c:419]: (style) The scope of the variable temp2 can be reduced
[../user/user_test_c/userlib.c:419]: (style) The scope of the variable temp3 can be reduced

    Die Scope-Warnungen hab ich schonmal gefixt:
    fat12.c:

    void parse_dir(uint8_t* a, int32_t in, struct dir_entry* rs)
{
   int32_t i = (in %DIR_ENTRIES) * 32;

   for(int32_t j=0;j<8;j++,i++)
   {
       rs->Filename[j] = a[i];
   }

   for(int32_t j=0;j<3;j++,i++)
   {
       rs->Extension[j] = a[i];
   } //...

    video.c:

    void scroll()
{
    uint32_t blank = 0x20 | (attrib << 8);
    if(csr_y >= SCROLL_LINE)
    {
		uint32_t temp = csr_y - SCROLL_LINE + 1;
        memcpy (vidmem, vidmem + temp * COLUMNS, (SCROLL_LINE - temp) * COLUMNS * 2);
        memsetw (vidmem + (SCROLL_LINE - temp) * COLUMNS, blank, COLUMNS);
        csr_y = SCROLL_LINE - 1;
    }
}

    userlib.c:

    void showInfo(signed char val)
{
    static char* line1 = "   _______                _______      <>_<>                                    "    ;
    static char* line2 = "  (_______) |_|_|_|_|_|_|| [] [] | .---|'\"`|---.                                "   ;
    static char* line3 = " `-oo---oo-'`-oo-----oo-'`-o---o-'`o\"O-OO-OO-O\"o'                               "  ;

    if(val==1)
    {
        char temp1 = line1[79];
        char temp2 = line2[79];
        char temp3 = line3[79];

        for(int i=79;i>0;--i)
        {
            line1[i] = line1[i-1];
            line2[i] = line2[i-1];
            line3[i] = line3[i-1];
        }
        line1[0] = temp1;
        line2[0] = temp2;
        line3[0] = temp3;
        printLine(line1,46,0xE);
        printLine(line2,47,0xE);
        printLine(line3,48,0xE);
    }
}
    0
  • E
    Mod

    Danke! Wird demnächst implementiert.

    Irrelevantes à la "use fgets instead of gets" entfernt

    Ist schon berechtigt, denn momentan kann man locker über den Puffer hinaus schreiben. Damit kann man sich in unser OS "hinein hacken". Das gets sollten wir wirklich implizit in ein "fgets" mit stdin als file verwandeln.

    0
  • ?

    Revision 151

    Ja, hallo erstmal!

    Die "FloppyImage.bin" funktioniert unter/auf "MS Virtual-PC" einigermaßen perfekt. Allerdings gibt es ein Problem in der "ckernel.c" bzw. in der "time.c":

    // ckernel.c Revision 151

 if (CurrentSeconds!=CurrentSecondsOld)
 {
   itoa(CurrentSeconds, timeBuffer);
   getCurrentDateAndTime(DateAndTime); // <- arbeitet nicht korrekt
   strcat(DateAndTime, "     ");
   strcat(DateAndTime, timeBuffer);
   strcat(DateAndTime, " seconds since start.");

   // output in status bar
   printf(DateAndTime, 49, 0xC);
 }

    "getCurrentDateAndTime" arbeitet wie ein "strcat". Darum wird der String pro Aufruf immer länger. Nach spätestens drei Aufrufen blockiert die VM, was heißt, daß "nichts mehr geht".

    Leider bin ich zu doof dafür um mir ein PrettyOS selbst zu kompilieren. Deshalb mußte ich die "FloppyImage.bin" reversen. 😞

    0
  • B

    +gjm+ schrieb:

    Leider bin ich zu doof dafür um mir ein PrettyOS selbst zu kompilieren. Deshalb mußte ich die "FloppyImage.bin" reversen. 😞

    Woran hapert's denn? Wir können da bestimmt helfen 😉

    0
  • M

    Das könnte inzwischen auch einfacher geworden sein: Wir haben ein neues makefile. Das hilft, wenn Du es unter Windows versuchst, da msys nicht mehr nötig ist.

    0
  • E
    Mod

    "getCurrentDateAndTime" arbeitet wie ein "strcat". Darum wird der String pro Aufruf immer länger. Nach spätestens drei Aufrufen blockiert die VM, was heißt, daß "nichts mehr geht".

    Danke für den Hinweis. Sehr wichtig, da wir Sun VB für USB EHCI Tests brauchen! 🙂

    0
  • ?

    "doof" ist bei mir ein Synonym für "faul". Außerdem bin ich als "Windowser" nun mal verwöhnt (und verweichlicht). 🙂

    Revision 146

    Die Revision 146 läuft unter/auf "MS Virtual-PC" auch, sofern man die o.g. if-Abfrage in der ckernel.c "rauswirft":

    // FloppyImage.bin

 Offset    0  1  2  3  4  5  6  7  8  9  A  B  C  D  E  F
000049B0  00 00 89 C3 39 C7 [b]74[/b] 62 83 EC 08 8D 45 D4 50 53

Offset 49B6:

74 (JE) ersetzen durch EB (JMP)

    Allerdings wundert es mich, warum die Revision 146 auf "realen" PCs gelaufen sein soll. Eigentlich hätte es eine Art "Buffer Overflow" geben müssen.

    0
  • M

    Das kompilieren unter Windows sollte eig. auch nicht mehr so schwer sein 😉

    Das Problem mit dem Absturz unter VPC und VB hat ehenkes übrigens soeben behoben.

    Dafür hab ich folgendes entdeckt:
    Wir haben einen Schreibcursor. Dies hab ich erst jetzt gemerkt, da er bei mir nur unter Microsoft Virtual PC funktioniert (Nutze ich selten). Qemu und Virtual Box zeigen ihn schlichtweg nicht an.
    Das dürfte ein Fehler sein.

    Die zu schnelle Uhr unter Virtual Box besteht auch weiterhin.

    0
  • C

    So, hier alle meine Test-PCs für PrettyOS:

    http://prettyos.fanofblitzbasic.de/Computers/

    0
  • A
     

    D:\PrettyOS\trunk\Source>tools\mingw32-make OS=WINDOWS
nasmw -f bin stage1_bootloader/boot.asm -Istage1_bootloader/ -o stage1_bootloade
r/boot.bin
nasmw -f bin stage2_bootloader/boot2.asm -Istage2_bootloader/ -o stage2_bootload
er/BOOT2.BIN
del *.o
nasmw -O32 -f elf user/user_program_c/start.asm -Iuser/user_program_c/ -o start.
o
i586-elf-gcc user/user_tools/userlib.c -c -Iuser/user_tools -m32 -std=c99 -Wshad
ow -march=i386 -mtune=i386 -m32 -fno-pic -Werror -Wall -O -ffreestanding -fleadi
ng-underscore -nostdlib -nostdinc -fno-builtin -fno-stack-protector -Iinclude
process_begin: CreateProcess(NULL, i586-elf-gcc user/user_tools/userlib.c -c -Iu
ser/user_tools -m32 -std=c99 -Wshadow -march=i386 -mtune=i386 -m32 -fno-pic -Wer
ror -Wall -O -ffreestanding -fleading-underscore -nostdlib -nostdinc -fno-builti
n -fno-stack-protector -Iinclude, ...) failed.
make (e=2): Das System kann die angegebene Datei nicht finden.
mingw32-make: *** [initrd] Error 2

    Kann jemand damit was anfangen???

    Cygwin ist aus dem Pfad herausgenommen.

    0
  • A

    Nochmal ich: Revision 163 läuft unter VBox mehr oder wenige in einer Endlosschleife. Ständig wiederholt sich der USB-Status. Befehle scheinen aber abgearbeitet zu werden. Unter Qemu hingegen läuft das System normal.

    0
  • E
    Mod

    Ständig wiederholt sich der USB-Status. <--- wenn dich das stört, hier abschalten:

    initEHCIHostController(); // used only for first tests
    Zeile 188 pci.c auskommentieren

    oder:
    in Sun VBox einfach unter Änderungen / USB / USB 2.0 checkbox deaktivieren 😉

    0
Anmelden zum Antworten