Changeset 148 for trunk/bootcode/setup/part_set.asm

Timestamp:

Apr 8, 2017, 12:27:52 AM (8 years ago)

Author:

Ben Rietbroek

Message:

Corrected check for successful LVM sector load [v1.1.1-testing]

The 'DriveIO_LoadLVMSector' function now uses disk i/o semantics,
meaning CY indicates failure and NC indicates success.

CAUTION:
This is a testbuild !
AirBoot uses the BIOS to access disks and a small coding error can trash
partition tables or other vital disk structures. You are advised to make
backups of TRACK0 and EBRs before using this testbuild. More info at:
​https://rousseaux.github.io/netlabs.air-boot/pdf/AirBoot-v1.1.0-manual.pdf

File:

• trunk/bootcode/setup/part_set.asm (modified) (1 diff)

trunk/bootcode/setup/part_set.asm

@@ -678 +678 @@
     mov     di, si                        ; Put SI into DI
     call    DriveIO_LoadLVMSector
-    jnc     PSCPN_LVMGotError             ; Security again, if problem -> halt
+    jc     PSCPN_LVMGotError             ; Security again, if problem -> halt
     push    dx
       mov     ax, [di+LocIPT_AbsoluteBegin]