Sourcecode Fortschritt

Erhard Henkes

Rev. 572 (0.0.1.11):

neu in fat.h/c:
FS_ERROR FAT_remove (const char* fileName, FAT_partition_t* part)

@MrX: bitte einbinden, falls nicht schon erfolgt

Erhard Henkes

Rev. 573 (0.0.1.12):

- FAT_remove korrigiert
- FS_ERROR FAT_rewind(FAT_file_t* fileptr)

Anmerkungen:

FAT_remove: strcpy(fileptr->name, fileName); // must be 8+3 formatted first (muss das im FS manager passieren?) // strcpy muss evtl. noch angepasst werden auf strncpy mit 11 zeichen
FAT_rewind: // fileptr->seek = 0; // has to happen in FS manager
FAT_rename: noch nicht implementiert, muss erst noch diskutiert werden, ob dabei ein copy und delete erfolgen darf

@MrX: das endgültige Einbinden / Testen sollten wir zusammen durchführen

Mr X

Version 0.0.1.13:

- FAT_rewind gelöscht und durch rewind im fsmanager ersetzt, da rewind(file) äquivalent zu fseek(file, 0, SEEK_SET) ist und das im fsmanager erledigt werden kann und soll.
- FAT-remove nimmt nun einen partition_t* statt FAT_partition_t*
- FAT_remove in fsmanager "integriert"
- Kleinigkeiten

Erhard Henkes

Bei FAT_rewind bin ich mir nicht ganz sicher, es existiert immerhin ein
FS_ERROR FAT_fseek(file_t* file, int32_t offset, SEEK_ORIGIN whence).

remove geht nicht, weil kein 8+3 Filename "screen__txt" übergeben wurde, sondern "screen.txt". Das wird jetzt im Filemanager erledigt, da die Funktion FormatFileName(...) seit neuestem dort ist.

Jetzt gehts (ausprobiert mit screen.txt auf Floppy). Habe den "remove" Test in video.c belassen für eigene Versuche (auskommentiert ab Zeile 273):

Rev. 575 (0.0.1.14)

Mr X

Erhard Henkes schrieb:

Bei FAT_rewind bin ich mir nicht ganz sicher, es existiert immerhin ein
FS_ERROR FAT_fseek(file_t* file, int32_t offset, SEEK_ORIGIN whence).

Ich bin mir völlig sicher.

Denn, wie ich schon sagte, rewind(file) ist äquivalent zu fseek(file, 0, SEEK_SET), also kann man rewind so implementieren, das es den o.g. fseek-Aufruf durchführt, welcher dann u.a. FAT_fseek aufruft. Genau so hab ich es gemacht.

Erhard Henkes

@MrX: Ja, du hast völlig Recht. Siehe: http://openbook.galileocomputing.de/c_von_a_bis_z/016_c_ein_ausgabe_funktionen_014.htm#mjff798e62f1469fc3901b349f005d6547
Gut gemacht.

Erhard Henkes

Rev. 576 (0.0.1.15):
FS_ERROR FAT_rename(const char* fileNameOld, const char* fileNameNew, partition_t* part)
FS_ERROR FAT_fileRename (FAT_file_t* fileptr, const char* fileName)

Getestet mit Floppy, funktioniert.

Testcode siehe video.c, Zeile 273 ff.

//rename test 
    waitForKeyStroke();
    uint32_t error = rename(Pfad,"1:/scrnew.txt"); 
    printf("\nrename test: error: %u", error);

    // remove test 
    waitForKeyStroke();
    error = remove(Pfad); 
    printf("\nremove test: error: %u", error);
    waitForKeyStroke();

Erhard Henkes

Rev. 577 (0.0.1.16):

rename funktioniert nun auch mit rename(Pfad,"scrnew.txt") und mit Floppy und USB MSD.

Test in video.c ab Zeile 273 (auskommentiert)

getFilename angepasst, damit keine Pfadangabe vor dem Filenamen notwendig ist.

const char* getFilename(const char* path)
{
    if (strchr((char*)path,'/')==NULL && strchr((char*)path,'|')==NULL && strchr((char*)path,'\\')==NULL)
    {
        return path;
    }
    else
    {
        // ...
    }
}

Mr X

Version 0.0.1.17:

- FormatFileName wieder in fat.c (War doch besser dort aufgehoben)
- initrd etwas umgebaut

Mr X

Version 0.0.1.18

- fsmanager auf fgetc/fputc umgestellt
-- Bedingt durch fehlendes caching im FAT-Treiber ists sehr langsam
- FAT_fread/FAT_fwrite nur noch intern im FAT-Treiber genutzt
- getch im Kernel eingebaut -> CPU-Sparen mit hlt

Erhard Henkes

Erste Tests:

strg+s (screenshot auf floppy) und strg+u (screenshot auf usb-stick) gehen noch gleich schnell, Schreibvorgänge sind im FAT-Modul bereits gecacht.

ttt.elf laden:

a1) Floppy real: wird jedes byte einzeln per sector lesen geladen (lese-cache notwendig), real abgebrochen zum Schutz des FDD

a2) Floppy Qemu: geht (etwas langsamer ^^)

b1) USB-Stick real: #PF (schreiben in read-only area ??), entweder bug eingebaut oder technik geht so nicht (MrX: bitte prüfen und kommentieren)

b2) USB-Stick VMWare-Player: wird jedes byte einzeln per sector lesen geladen (lese-cache notwendig), interessanter Dauertest für OS und Stick (auch nett: Stick ziehen, dauer-rot auf screen ^^)

Ein interessanter Trümmerhaufen, aus dem Phoenix aus der Asche wieder erstehen soll.

Die Grundfrage ist die, ob nach erfolgreichem Read-Cache-Einbau die Performance nicht doch leidet durch die vielen zusätzlichen Abläufe zwischen dem Lesen der Bytes mit fgetc.

Erhard Henkes

0.0.1.19 (SVN Rev. 580)

Read Cache für readSector funktioniert wie gewünscht.

Leider existiert ein Problem beim Laden von usb-stick (Ursache noch nicht untersucht)

Erhard Henkes

0.0.1.20 (Rev. 581)

Problem: malloc spinnt! (seit 0.0.1.18) keiner weiß warum ^^

Es passiert, wenn ein programm einmal geladen wird (muss nicht ausgeführt werden)

file->name von pointer auf array umgestellt, wegen Analogie zu FAT_file->name (dort exakt 8+3), hilft aber nicht beim aktuellen Problem, sollte aber ansonsten fehlersicherer sein, da man dabei nicht vergessen kann malloc(...) auszuführen.

Erhard Henkes

malloc/free beginnend vom Start verfolgt:

void* malloc(uint32_t size, uint32_t alignment)
{
    //...

    // debug
    textColor(0x0E);    
    printf("\nmalloc: %X",address);
    textColor(0x0F);
    waitForKeyStroke();

    return address;
}

void free(void* address)
{
    // debug
    textColor(0x0E);
    printf("\nfree: %X",address);
    textColor(0x0F);
    waitForKeyStroke();

    //...
}

Fazit: Da passt nix zusammen!

Beweis-Foto: http://www.henkessoft.de/OS_Dev/Bilder/0_0_1_20_malloc_free_problem.PNG

malloc    free
--------------------------------------
C0000000   
          C0000048
          00000000
C0004CC0  
          C0204CC0          
          C0000568
          C0000690
          C0000038

usw.

Absoluter Nonsens zur Zeit.

Mr X

Version 0.0.1.21

- malloc auf Placement umgestellt (HACK), free auskommentiert (HACK)
-> PrettyOS funktioniert wieder, Fehler in malloc
- Memory-Leak in executeFile behoben (durch HACK von oben wirkungslos) und unsinnigen Code dort weggemacht
- Ehenkes Umbau von file_t::name auf statisches Array zurückgebaut

Erhard Henkes

PrettyOS funktioniert wieder

stimmt leider nicht!

Ich werde versuchen, die Ursachen zu finden und zu korrigieren. Bitte keine Änderungen vornehmen, bis ich das Committen wieder frei gebe. Hinweise auf den konkreten Haupt-Fehler sind natürlich gerne gesehen.

Erhard Henkes

Rev. 583: 0.0.1.22

zwischenstand: log in malloc zum aufspüren von fehlern
heap expansion evtl. nicht ok

Erhard Henkes

Rev. 584 (0.0.1.24)

Zwischenstand zur Fehlersuche beim Expandieren des Heaps

Committen wieder frei gegeben

Fragen:

Wo liegt der Fehler im MM (Heap)?
Warum löst 1:\... das aus?

Erhard Henkes

Rev. 585 (0.0.1.25)

Zwischenstand Heap-Erweiterungs-Problematik:
Heap-Logger zur Analyse zeigt die Arbeitsweise des Heaps.

http://www.henkessoft.de/OS_Dev/Bilder/0_0_1_25_heap_problem.PNG <--- OK ?
http://www.henkessoft.de/OS_Dev/Bilder/0_0_1_25_heap_problem_PF.PNG <--- not OK

Erhard Henkes

Rev. 586 (0.0.1.26):

HEAP_MIN_GROWTH = 0x40000;

#define PLACEMENT_BEGIN ((uint8_t*)0x1200000) // 18 MiB (vorher 16 MiB)

(Tobiking meint, das die Veränderung der Liste von außen erfolgen könnte, daher Tests mit 16/17/18/19 MiB als Placement-Untergrenze. Hatte hohen Einfluss.)

bei mir unter qemu bei 1:\... oder 1:|... Division by zero mit eip 0x104504 (kernel.map: 0x00105499 _FAT_fread)

Die Fehlersuche ist bisher über das Eingrenzen der Symptome nicht hinaus gewachsen.