Changeset 49 for trunk/BOOTCODE/REGULAR/DRIVEIO.ASM

Timestamp:

Apr 12, 2014, 12:13:57 PM (11 years ago)

Author:

Ben Rietbroek

Message:

Fixed a nasty bug when eCS phase1 is active [2012-05-13]

WARNING!!

All commits upto and including the commit of [2012-05-13] contain
a severe bug!! Building from these sources and then disabling
the 'force LBA' feature while also using the drive-letter feature or
editing the label can DESTROY THE MBR on ALL ATTACHED DISKS!!
DO NOT DISABLE 'FORCE LBA USAGE' WHEN BUILT FROM THE THESE COMMITS!!

Fixes

o Bug Description

When creating a partition in a free space between partitions for
eCS installation, the phase1 boot-through code would calculate the
wrong index for the new partition. As a result the next partition
would be auto-booted instead of the newly installed system.
This bug is also present in v1.07.

o Quick Fix

Run PARTSCAN_ScanForPartitions for a second time when eCS phase1
is active. This resyncs the PartitionXref table.

File:

• trunk/BOOTCODE/REGULAR/DRIVEIO.ASM (modified) (2 diffs)

trunk/BOOTCODE/REGULAR/DRIVEIO.ASM

@@ -107 +107 @@
         mov     es, ax
 
-        mov     ah, 00h                        ; Function 2 - Reset Drive
+        mov     ah, 00h                        ; Function 0 - Reset Drive
         xor     dl, dl
         int     13h
@@ -569 +569 @@
     IFDEF AUX_DEBUG
         ; show current partition location
-        pushf
-        pusha
+        ;~ pushf
+        ;~ pusha
         ;~ call    AuxIO_TeletypeNL
         ;~ mov     si,offset db_curlvmsec
         ;~ call    AuxIO_Print
-        mov     dx,bx
+        ;~ mov     dx,bx
         ;~ call    AuxIO_TeletypeHexDWord
         ;~ call    AuxIO_TeletypeNL
-        mov     si,offset LVMSector
+        ;~ mov     si,offset LVMSector
         ;~ call    AuxIO_DumpSector
         ;~ call    AuxIO_TeletypeNL
-        popa
-        popf
+        ;~ popa
+        ;~ popf
     ENDIF