Changeset 617 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Src/Handlers

Timestamp:

Aug 10, 2021, 7:27:30 AM (3 years ago)

Author:

Tomi Tilli

Message:

• Verify functions did not return correctly the number of succesfully verified sectors. That should be now fixed except for serial device. Serial device do not have error handling implemented yet.

Location:

trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h

Files:

• AH4h_HVerify.asm (modified) (2 diffs)
• EBIOS/AH44h_ExtendedVerifySectors.asm (modified) (1 diff)

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

@@ -29 +29 @@
 ;       DL:     Translated Drive number
 ;       DS:DI:  Ptr to DPT (in RAMVARS segment)
-;       SS:BP:  Ptr to INTPACK
+;       SS:BP:  Ptr to IDEPACK
 ;   Parameters on INTPACK in SS:BP:
 ;       AL:     Number of sectors to verify (1...128)
@@ -46 +46 @@
     call    Prepare_ByValidatingSectorsInALforOldInt13h ; Preserves AX
     mov     bx, TIMEOUT_AND_STATUS_TO_WAIT(TIMEOUT_DRQ, FLG_STATUS_DRDY)
-%ifdef USE_186
-    push    Int13h_ReturnFromHandlerAfterStoringErrorCodeFromAHandTransferredSectorsFromCL
-    jmp     Idepack_TranslateOldInt13hAddressAndIssueCommandFromAH
-%else
+    push    ax          ; Store number of sectors to verify
     call    Idepack_TranslateOldInt13hAddressAndIssueCommandFromAH
+    pop     cx          ; Number of sectors verified if successful
+    jnc     SHORT .NoErrors
+
+; TODO: For now we assume serial device do not produce verify errors
+    call    AH4h_CalculateNumberOfSuccessfullyVerifiedSectors
+.NoErrors:
     jmp     Int13h_ReturnFromHandlerAfterStoringErrorCodeFromAHandTransferredSectorsFromCL
-%endif
+
+
+;--------------------------------------------------------------------
+; Calculates number of succesfully verified sectors. This function works only
+; if verify command stopped to an device error (such as bad sector) since IDE
+; register contents are not valid unless error.
+;
+; AH4h_CalculateNumberOfSuccessfullyVerifiedSectors
+;   Parameters:
+;       AH:     INT 13h error code
+;       CX:     Number of sectors that was meant to we verified
+;       DS:DI:  Ptr to DPT (in RAMVARS segment)
+;       SS:BP:  Ptr to IDEPACK
+;   Returns with INTPACK in SS:BP:
+;       CX:     Number of sectors succesfully verified
+;       CF:     1
+;--------------------------------------------------------------------
+ALIGN JUMP_ALIGN
+AH4h_CalculateNumberOfSuccessfullyVerifiedSectors:
+    xchg    cx, ax          ; Store error code to CL
+    call    Device_ReadLBAlowRegisterToAL
+    mov     ah, [bp+IDEPACK.bLbaLow]
+    sub     al, ah          ; AL = sector address with verify failure - starting sector address
+    xor     ah, ah
+    xchg    cx, ax          ; Number of successfully verified sectors in CX, error code in AH
+    stc
+    ret

trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h/EBIOS/AH44h_ExtendedVerifySectors.asm

@@ -41 +41 @@
 AH44h_HandlerForExtendedVerifySectors:
     call    Prepare_ByLoadingDapToESSIandVerifyingForTransfer
+    push    WORD [es:si+DAP.wSectorCount]               ; Store for successfull number of sectors transferred
     mov     ah, [cs:bx+g_rgbVerifyCommandLookup]
     mov     bx, TIMEOUT_AND_STATUS_TO_WAIT(TIMEOUT_DRQ, FLG_STATUS_DRDY)
     call    Idepack_ConvertDapToIdepackAndIssueCommandFromAH
+
+    ; Now we need number of succesfully verifed sectors to CX. Since we did not transfer anything,
+    ; we did not have any sector counter like in read and write functions.
+    ; In case of error, drive LBA registers are set to address where the error occurred. We must
+    ; calculate number of succesfully transferred sectors from it.
+    pop     cx
+    jnc     SHORT .AllSectorsVerifiedSuccessfully
+
+; TODO: For now we assume serial device do not produce verify errors
+    call    AH4h_CalculateNumberOfSuccessfullyVerifiedSectors
+
+ALIGN JUMP_ALIGN
+.AllSectorsVerifiedSuccessfully:
     jmp     SHORT AH42h_ReturnFromInt13hAfterStoringErrorCodeFromAHandTransferredSectorsFromCX