Changeset 589 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Src/Device/IDE/IdeTransfer.asm

Timestamp:

May 22, 2016, 12:26:57 PM (9 years ago)

Author:

Krister Nordvall

Message:

Changes:

• BIOS: Fixed a purely cosmetic bug from r542 where, in builds containing MODULE_EBIOS, the boot menu would display an incorrect drive size (0.4 kB with MODULE_STRINGS_COMPRESSED or 0.5 kB without) for old drives with no support for LBA.
• Fixed a bug from r392 where Vision_DetectAndReturnIDinAXandPortInDXifControllerPresent would return the ID in AL instead of AH (if DANGEROUS_DETECTION had been defined).
• Fixed a bug from r587 in AdvAtaInit.asm that would prevent detection of QDI Vision controllers.
• Also changed how the QDI Vision IDs are defined (removed the need for shifting) to avoid confusion. This fixed a potential bug from r587 in AdvAtaInit.asm where some IDs were not being shifted.
• Fixed a bug in PDC20x30.asm from r587 where GetPdcIDtoAX would not return with the IDE base port in DX so DisablePdcProgrammingMode would fail.
• Made some changes to ModuleDependency.inc and other files so that MODULE_ADVANCED_ATA now requires USE_386. Consequently it is no longer included in the regular AT-builds, only in the 386_8k-build.
• Moved the UNROLL_SECTORS_IN_CX_TO_xWORDS macros from IDE_8bit.inc to IdeIO.inc which means it's now possible to build a BIOS without MODULE_8BIT_IDE.
• XTIDECFG: Added a minimum DOS version check (since it needs DOS version 2+) to allow the program to quit gracefully in the unlikely scenario where someone tries to run it under DOS version 1.
• Made some changes to Drive.asm to improve drive enumeration. The old method using GET_DOS_DRIVE_PARAMETER_BLOCK_FOR_SPECIFIC_DRIVE worked well in Windows XP but not in Windows 98 SE (in Windows or in DOS mode). The two problems were; 1) The function call would access the drives which on single floppy drive systems would cause Windows to swap between A: and B: (throwing a blue screen asking the user to insert a disk etc). 2) Only floppy drives and FAT16 drives would be available in the list of drives, no FAT32/optical/network drives.
• Improved code in IdeControllerMenu.asm so that the default port addresses for all IDE interfaces are now restored when (re-)selecting the (same) type of IDE device.
• Also made it impossible to select a device type unless the required module is included in the loaded BIOS.
• The version check done when loading a BIOS now uses the FLASH_SIGNATURE definition from Version.inc. Any changes affecting RomVars now only requires updating that definition. This means that changes to RomVars must be implemented in both the BIOS and XTIDECFG before being committed to the repository.
• Added a compatibility fix for 3Com 3C503 cards to the ROM checksumming code in Buffers.asm (Buffers_GenerateChecksum).
• SerDrive: Made some minor changes to file names and paths to improve compatibility with case sensitive environments.
• BIOSDRVS: Made a minor size optimization which as a side effect also makes it compatible with all DOS versions including DOS version 1.
• Library: Renamed the WAIT_RETRACE_IF_NECESSARY_THEN macro to CALL_WAIT_FOR_RETRACE_IF_NECESSARY_THEN and made a tail-call-optimized version of it (JMP_WAIT_FOR_RETRACE_IF_NECESSARY_THEN).
• A speed optimization to the eRCL_IM macro for 386 and higher. This change breaks emulation in the sense that the macro will fail when given a memory operand as the first parameter.
• Other minor optimizations and fixes.

File:

• trunk/XTIDE_Universal_BIOS/Src/Device/IDE/IdeTransfer.asm (modified) (7 diffs)

trunk/XTIDE_Universal_BIOS/Src/Device/IDE/IdeTransfer.asm

@@ -173 +173 @@
     cmp     [bp+PIOVARS.bSectorsLeft], cl
     jbe     SHORT .WriteLastBlockToDrive
+    push    ds
+    push    es
+    pop     ds
     call    [bp+PIOVARS.fnXfer]
-
+    pop     ds
     ; Wait until ready for next block and check for errors
     call    IdeWait_IRQorDRQ                    ; Wait until ready to transfer
@@ -189 +192 @@
     mov     cl, [bp+PIOVARS.bSectorsLeft]       ; CH is already zero
     push    cx
+    push    ds
+    push    es
+    pop     ds
     call    [bp+PIOVARS.fnXfer]                 ; Transfer possibly partial block
+    pop     ds
     jmp     SHORT CheckErrorsAfterTransferringLastBlock
 
@@ -237 +244 @@
     ; Convert ES:SI to physical address
 %ifdef USE_386
-
     mov     dx, es
     xor     ax, ax
@@ -247 +253 @@
 
 %elifdef USE_186
-                        ; Bytes EU Cycles(286)
-    mov     ax, es      ; 2     2
-    rol     ax, 4       ; 3     9
-    mov     dx, ax      ; 2     2
-    and     ax, BYTE 0Fh; 3     3
-    xor     dx, ax      ; 2     2
-    add     si, dx      ; 2     2
-    adc     al, ah      ; 2     2
-    mov     es, ax      ; 2     2
-    ;------------------------------------
-                        ; 18    24
+    mov     ax, es
+    rol     ax, 4
+    mov     dx, ax
+    and     ax, 0Fh
+    xor     dx, ax
+    add     si, dx
+    adc     al, ah
+    mov     es, ax
+
 %else ; 808x
-                        ; Bytes EU Cycles(808x)
-    mov     al, 4       ; 2     4
-    mov     dx, es      ; 2     2
-    xchg    cx, ax      ; 1     3
-    rol     dx, cl      ; 2     24
-    mov     cx, dx      ; 2     2
-    xchg    cx, ax      ; 1     3
-    and     ax, BYTE 0Fh; 3     4
-    xor     dx, ax      ; 2     3
-    add     si, dx      ; 2     3
-    adc     al, ah      ; 2     3
-    mov     es, ax      ; 2     2
-    ;------------------------------------
-                        ; 21    53
-;
-; Judging by the Execution Unit cycle count the above block of code is
-; apparently slower. However, the shifts and rotates in the block below
-; execute faster than the Bus Interface Unit on an 8088 can fetch them,
-; thus causing the EU to starve. The difference in true execution speed
-; (if any) might not be worth the extra 5 bytes.
-; In other words, we could use a real world test here.
-;
-%if 0
-                        ; Bytes EU Cycles(808x/286)
-    xor     dx, dx      ; 2     3/2
-    mov     ax, es      ; 2     2/2
-%rep 4
-    shl     ax, 1       ; 8     8/8
-    rcl     dx, 1       ; 8     8/8
-%endrep
-    add     si, ax      ; 2     3/2
-    adc     dl, dh      ; 2     3/2
-    mov     es, dx      ; 2     2/2
-    ;------------------------------------
-                        ; 26    29/26
-%endif ; 0
+    mov     al, 4
+    mov     dx, es
+    xchg    cx, ax
+    rol     dx, cl
+    mov     cx, dx
+    xchg    cx, ax
+    and     ax, 0Fh
+    xor     dx, ax
+    add     si, dx
+    adc     al, ah
+    mov     es, ax
+
 %endif
 
@@ -334 +314 @@
 %ifdef USE_AT       ; CF is always clear for XT builds
     ; AH = RET_HD_INVALID (01) if CF set, RET_HD_SUCCESS (00) if not. CF unchanged.
+%ifdef USE_386
+    setc    ah
+%else
     sbb     ah, ah
     neg     ah
 %endif
+%endif ; USE_AT
     ret
 
@@ -345 +329 @@
 g_rgfnPioRead:
         dw      IdePioBlock_ReadFrom16bitDataPort   ; 0, DEVICE_16BIT_ATA
+%ifdef MODULE_ADVANCED_ATA
         dw      IdePioBlock_ReadFrom32bitDataPort   ; 1, DEVICE_32BIT_ATA
+%elifdef MODULE_8BIT_IDE
+        dw      NULL
+%endif ; MODULE_ADVANCED_ATA
 %ifdef MODULE_8BIT_IDE
         dw      IdePioBlock_ReadFrom8bitDataPort    ; 2, DEVICE_8BIT_ATA
@@ -361 +349 @@
 g_rgfnPioWrite:
         dw      IdePioBlock_WriteTo16bitDataPort    ; 0, DEVICE_16BIT_ATA
+%ifdef MODULE_ADVANCED_ATA
         dw      IdePioBlock_WriteTo32bitDataPort    ; 1, DEVICE_32BIT_ATA
+%elifdef MODULE_8BIT_IDE
+        dw      NULL
+%endif ; MODULE_ADVANCED_ATA
 %ifdef MODULE_8BIT_IDE
         dw      IdePioBlock_WriteTo8bitDataPort     ; 2, DEVICE_8BIT_ATA