Clarify things a bit.

datascientist1976 · Jun 8, 2016 · 8dde71a · 8dde71a
1 parent df0f38a
commit 8dde71a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/Docs/Kernel/Memory.md b/Docs/Kernel/Memory.md
@@ -0,0 +1 @@
+On initialization of the kernel, a GlobalInformationTable is setup. This contains the address of the first DataLookupEntry
diff --git a/source/Cosmos.Core/DataLookupTable.cs b/source/Cosmos.Core/DataLookupTable.cs
@@ -2,7 +2,7 @@
 
 namespace Cosmos.Core
 {
-    // The DataLookupTable (DLT) basically is a linked list.
+    // The DataLookupTable (DLT) basically is a double linked list.
     [StructLayout(LayoutKind.Explicit)]
     internal unsafe struct DataLookupTable
     {

diff --git a/source/Cosmos.Core/Heap.cs b/source/Cosmos.Core/Heap.cs
@@ -63,6 +63,7 @@ public static uint MemAlloc(uint aLength)
                 DataLookupTable* xCurrentTable = GlobalSystemInfo.GlobalInformationTable->FirstDataLookupTable;
                 DataLookupTable* xPreviousTable = null;
                 uint xResult;
+                #region Loop through existing tables and see if we find a free spot
                 while (xCurrentTable != null)
                 {
                     mDebugger.SendInternal($"Scanning DataLookupTable {xCurrentTableIdx}");
@@ -86,6 +87,7 @@ public static uint MemAlloc(uint aLength)
                     mLastTableIndex = xCurrentTableIdx;
                     mLastEntryIndex = 0;
                 }
+                #endregion Loop through existing tables and see if we find a free spot
 
                 // no tables found, lets create a new one, and use that
                 if (xPreviousTable == null)
@@ -143,6 +145,7 @@ private static bool ScanDataLookupTable(uint aTableIdx, DataLookupTable* aTable,
                 //mDebugger.Trace($"Item.Refcount", xCurrentEntry->Refcount);
                 if (xCurrentEntry->Size == 0)
                 {
+                    #region Found an uninitialized entry
                     mDebugger.SendInternal($"Found an entry at position {(uint)i}");
                     // found an entry now. Let's set it
                     if (aTable->Next != null)
@@ -188,11 +191,8 @@ private static bool ScanDataLookupTable(uint aTableIdx, DataLookupTable* aTable,
 
                     aHandle = (uint)xCurrentEntry;
                     //mDebugger.Trace($"Returning handle ", aHandle);
-                    if (aHandle == 0x0213D185)
-                    {
-                        mDebugger.SendInternal("Last known one");
-                    }
                     mLastEntryIndex = i;
+                    #endregion Found an uninitialized entry
                     return true;
                 }
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		On initialization of the kernel, a GlobalInformationTable is setup. This contains the address of the first DataLookupEntry