Changeset 114 for trunk/bootcode/regular/driveio.asm

Timestamp:

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

Author:

Ben Rietbroek

Message:

Make checking for valid LVM-sector use CF instead of ZF [v1.1.1-testing]

Both the checks for LVM Signature and CRC already use CF on return.
Doing the same for complete LVM-sector validation is more consistent.

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/regular/driveio.asm (modified) (2 diffs)

trunk/bootcode/regular/driveio.asm

@@ -1303 +1303 @@
 
         ; See if this is a valid LVM-sector
-        ; ZF=0 if valid
+        ; CY if valid
         call    LVM_ValidateSector
 
@@ -1327 +1327 @@
 
         ; Yep, we found the master LVM-sector
-        jnz     DriveIO_LoadMasterLVMSector_Found
+        jc      DriveIO_LoadMasterLVMSector_Found
+
         ; Try next location
         loop    DriveIO_LoadMasterLVMSector_NextTry