sweb

Post from Markus Steinberger:

Hi,
Falls es noch jemanden da draußen gibt, der bei seinen Tests viele Files
erzeugt/aufmacht und wieder zumacht und dann irgendwann einen random
PageFault erhält, der sei gewarnt!

Denn die File Descriptoren werden nicht aus global_fd entfernt, nachdem sie
gelöscht wurden! Somit zeigt der Pointer ins Leere! Beim Durchsuchen der
global_fd Liste wird dann eine Methode auf einen Pointer ins Leere
aufgerufen. Man kann sich vorstellen, dass das vielleicht irgendwann einmal
zu einem Fehler führt, der genau alle 50 bis 100 Versuche auftritt und
SEHR lustig zum debuggen ist.

Nach 10h Suchen, großem Wundern und Durchforsten des ganzen FS Gewirrs
kommt man dann drauf, dass eine kleine Zeile in allen SuperBlock
Implementierungen unter removeFD auskommentiert ist:

//global_fd.remove(fd);

So einfach, nur leider schreit diese Zeile nicht:

"IM SWEB FS IST DER WURM DRIN UND NIRGENDS IST EIN KOMMENTAR, DASS DA WAS
NICHT STIMMT, ALSO SCHAU DIR ALLES DURCH, NACH AUSKOMMENTIERTEN ZEILEN!"

Ist es so schwer ein TODO oder ein ATTENTION rein zu schreiben?

*Frust abgelassen hab*

Weiter in der Geschichte:

Kommentiert man diese Zeile ein, kommt es zu noch schnellerem Chaos! - Was
wohl auch der Grund ist, warum diese Zeile auskommentiert ist!


Kleine Nachforschungen und Herumprobieren später:

In PointList.h, PointList<ContentType>::remove
verursacht die Zeile

element = find ( entry );

das Problem. Woran das liegt, weiß ich leider nicht. Naiver Weise rate ich
mal und sage, dass irgendwas mit der globalen Variable nicht richtig
hinhaut. Entweder beim Kompilieren oder bei BOCHS. Denn wenn die PointList
als Member verwendet wird geht alles ( z.B. SuperBlock::s_files_).

So ein schneller Fix sieht wie folgt aus (einfach das find in die Methode
remove kopieren:

//element = find ( entry );
//copy begin
if ( first_ == 0 )
return -1;

element = first_;
bool found = false;
do
{
if ( element->getItem() == entry )
{
found = true;
break;
}
}
while ( element != last_ && (element = element->getNext()) );
if(!found)
return -1;
//copyend


lg,
Markus Steinberger