Changeset 505 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h

Timestamp:

Feb 25, 2013, 4:23:09 PM (12 years ago)

Author:

krille_n_@…

google:author:

krille_n_@hotmail.com

Message:

Changes:

• Reverted the changes to MenuEvents.inc done in r492 since they broke the F1 key function in XTIDECFG.
• Added a tail-call optimized variant of the CALL_DISPLAY_LIBRARY macro (JMP_DISPLAY_LIBRARY).
• Put a block size limit in AH1Eh_ChangeXTCFmodeBasedOnControlRegisterInAL. I think it's needed but if not, it's easy to remove.
• Other optimizations and fixes.

Location:

trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h

Files:

• AH0h_HReset.asm (modified) (1 diff)
• AH1Eh_XTCF.asm (modified) (2 diffs)
• AH9h_HInit.asm (modified) (1 diff)

trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h/AH0h_HReset.asm

@@ -39 +39 @@
 AH0h_HandlerForDiskControllerReset:
     ; Reset foreign Floppy and Hard Drives (those handled by other BIOSes)
-    xor     bx, bx                                      ; Zero BH to assume no errors
-    or      bl, dl                                      ; Copy requested drive to BL
+    eMOVZX  bx, dl                                      ; Copy requested drive to BL and zero BH to assume no errors
+
     xor     ah, ah                                      ; Disk Controller Reset
     call    Int13h_CallPreviousInt13hHandler            ; Reset floppy drives only or floppy drives and foreign hard disks

trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h/AH1Eh_XTCF.asm

@@ -102 +102 @@
     out     dx, al
 
+    ; We always need to enable 8-bit mode since 16-bit mode is restored
+    ; when controller is reset (AH=00h or 0Dh)
+    ePUSH_T bx, AH23h_Enable8bitPioMode
+
     ; Convert Control Register Contents to device code
     test    al, al
@@ -108 +112 @@
     jae     SHORT .SetMemoryMappedMode
 
+; We need to limit block size here. Consider this scenario;
+; 1. While in PIO mode or memory mapped mode, the drive is set to do
+;    block transfers larger than XTCF_DMA_MODE_MAX_BLOCK_SIZE.
+; 2. A call is subsequently made to change device mode to DEVICE_8BIT_XTCF_DMA.
+; 3. The call to AH24h_SetBlockSize fails but the change in device mode has been made.
+
     ; Set DMA Mode
     mov     BYTE [di+DPT_ATA.bDevice], DEVICE_8BIT_XTCF_DMA
     mov     al, [di+DPT_ATA.bBlockSize]
-    call    AH24h_SetBlockSize  ; AH=24h limits block size if necessary
-    jmp     SHORT .Enable8bitPioMode
+    MIN_U   al, XTCF_DMA_MODE_MAX_BLOCK_SIZE
+    jmp     SHORT AH24h_SetBlockSize    ; Returns via AH23h_Enable8bitPioMode
 
 .SetMemoryMappedMode:
-    mov     BYTE [di+DPT_ATA.bDevice], DEVICE_8BIT_XTCF_MEMMAP
-    jmp     SHORT .Enable8bitPioMode
+    mov     al, DEVICE_8BIT_XTCF_MEMMAP
+    SKIP2B  bx
 
 .Set8bitPioMode:
-    mov     BYTE [di+DPT_ATA.bDevice], DEVICE_8BIT_XTCF_PIO8
-    ; Fall to .Enable8bitPioMode
-
-    ; We always need to enable 8-bit mode since 16-bit mode is restored
-    ; when controller is reset (AH=00h or 0Dh)
-.Enable8bitPioMode:
-    jmp     AH23h_Enable8bitPioMode
+    mov     al, DEVICE_8BIT_XTCF_PIO8
+    mov     [di+DPT_ATA.bDevice], al
+    ret     ; Via AH23h_Enable8bitPioMode
 
 

trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h/AH9h_HInit.asm

@@ -101 +101 @@
     call    Device_SelectDrive
     STORE_ERROR_FLAG_TO_DPT     FLG_INITERROR_FAILED_TO_SELECT_DRIVE
+%ifdef USE_386
+    jc      .ReturnWithErrorCodeInAH
+%else
     jnc     SHORT .ContinueInitializationSinceDriveSelectedSuccesfully
     jmp     .ReturnWithErrorCodeInAH
 .ContinueInitializationSinceDriveSelectedSuccesfully:
+%endif