Changeset 623 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Src/VariablesAndDPTs

Timestamp:

Jun 15, 2022, 2:48:59 PM (3 years ago)

Author:

Krister Nordvall

Message:

Changes:

• Reversed the change to IdeDPT.asm in r622 as it didn't work as intended.
• Reordered some procedures to reduce alignment padding.
• Added two new defines (EXTRA_LOOP_UNROLLING_SMALL and EXTRA_LOOP_UNROLLING_LARGE) that should improve transfer speeds for some hardware combinations, specifically 808x CPUs with any IDE controller using port I/O and any CPU with XT-IDE controllers.
• Added a new define (USE_086) for use with 8086 and V30 CPUs only. Unlike the other USE_x86 defines, this define will not change the instruction set used and is therefore compatible with all CPUs. However, it will apply padding to make jump destinations WORD aligned which should improve performance on 8086/V30 CPUs but on 8088/V20 CPUs there is no benefit and, in addition to wasting ROM space, it might in fact be slower on these machines. Since the vast majority of XT class machines are using 8088/V20 CPUs this define is not used in the official XT builds - it's primarily intended for custom BIOS builds.
• XTIDECFG: The URL to the support forum has been updated.

Location:

trunk/XTIDE_Universal_BIOS/Src/VariablesAndDPTs

Files:

• FindDPT.asm (modified) (4 diffs)
• RamVars.asm (modified) (2 diffs)

trunk/XTIDE_Universal_BIOS/Src/VariablesAndDPTs/FindDPT.asm

@@ -65 +65 @@
 
     cmp     ah, dl                              ; Check second drive if two, first drive if only one
-    jz      SHORT .CalcDPTForDriveNumber
+    je      SHORT .CalcDPTForDriveNumber
     cmp     al, dl                              ; Check first drive in all cases, redundant but OK to repeat
-    jnz     SHORT .DiskIsNotHandledByThisBIOS
+    jne     SHORT .DiskIsNotHandledByThisBIOS
 %else
     cmp     dl, ah                              ; Above last supported?
@@ -100 +100 @@
 
     test    dl, dl
-    js      .harddisk
+    js      SHORT .Harddisk
 
     call    RamVars_UnpackFlopCntAndFirstToAL
@@ -106 +106 @@
 
 ALIGN JUMP_ALIGN
-.harddisk:
+.Harddisk:
     sub     dl, al                      ; subtract off beginning of either hard disk or floppy list (as appropriate)
 %else
@@ -114 +114 @@
 .CalcDPTForNewDrive:
     mov     al, LARGEST_DPT_SIZE
-
     mul     dl
     add     ax, RAMVARS_size            ; Clears CF (will not overflow)

trunk/XTIDE_Universal_BIOS/Src/VariablesAndDPTs/RamVars.asm

@@ -135 +135 @@
 
 
-;--------------------------------------------------------------------
-; RamVars_GetCountOfKnownDrivesToAX
-;   Parameters:
-;       DS:     RAMVARS segment
-;   Returns:
-;       AX:     Total hard disk count
-;   Corrupts registers:
-;       None
-;--------------------------------------------------------------------
-ALIGN JUMP_ALIGN
-RamVars_GetCountOfKnownDrivesToAX:
-    mov     ax, [RAMVARS.wFirstDrvAndCount]
-    add     al, ah
-    and     ax, BYTE 7fh
-    ret
-
-;--------------------------------------------------------------------
-; RamVars_GetIdeControllerCountToCX
-;   Parameters:
-;       Nothing
-;   Returns:
-;       CX:     Number of IDE controllers to handle
-;   Corrupts registers:
-;       Nothing
-;--------------------------------------------------------------------
-ALIGN JUMP_ALIGN
-RamVars_GetIdeControllerCountToCX:
-    eMOVZX  cx, [cs:ROMVARS.bIdeCnt]
-    ret
-
-
 %ifdef MODULE_SERIAL_FLOPPY
 ;--------------------------------------------------------------------
@@ -184 +153 @@
     ret
 %endif
+
+
+;--------------------------------------------------------------------
+; RamVars_GetIdeControllerCountToCX
+;   Parameters:
+;       Nothing
+;   Returns:
+;       CX:     Number of IDE controllers to handle
+;   Corrupts registers:
+;       Nothing
+;--------------------------------------------------------------------
+ALIGN JUMP_ALIGN
+RamVars_GetIdeControllerCountToCX:
+    eMOVZX  cx, [cs:ROMVARS.bIdeCnt]
+    ret
+
+
+;--------------------------------------------------------------------
+; RamVars_GetCountOfKnownDrivesToAX
+;   Parameters:
+;       DS:     RAMVARS segment
+;   Returns:
+;       AX:     Total hard disk count
+;   Corrupts registers:
+;       None
+;--------------------------------------------------------------------
+ALIGN JUMP_ALIGN
+RamVars_GetCountOfKnownDrivesToAX:
+    mov     ax, [RAMVARS.wFirstDrvAndCount]
+    add     al, ah
+    and     ax, BYTE 7Fh
+    ret