Changeset 445 in xtideuniversalbios for trunk/Assembly_Library

Timestamp:

Aug 29, 2012, 12:59:23 PM (12 years ago)

Author:

krille_n_@…

google:author:

krille_n_@hotmail.com

Message:

Changes:

• A speed optimization to the eSHL_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.
• Memory_SumCXbytesFromESSItoAL now returns with the zero flag set/cleared according to the result.
• Unrolled all the 8 bit READ transfer loops to do 16 bytes per iteration. Added a new macro (UNROLL_SECTORS_IN_CX_TO_OWORDS) as part of it. Size wise this is expensive but I think it should be worth the ROM space. The WRITE transfer loops were left as is since writes are rare anyway (<10% of all disk I/O IIRC).
• Minor optimizations and fixes here and there.

Location:

trunk/Assembly_Library

Files:

• Inc/Emulate.inc (modified) (1 diff)
• Inc/Math.inc (modified) (1 diff)
• Src/Display/DisplayCursor.asm (modified) (3 diffs)
• Src/Display/DisplayFormat.asm (modified) (3 diffs)
• Src/Display/DisplayFormatCompressed.asm (modified) (4 diffs)
• Src/Menu/CharOutLineSplitter.asm (modified) (5 diffs)
• Src/Util/Memory.asm (modified) (3 diffs)
• Src/Util/Sort.asm (modified) (3 diffs)

trunk/Assembly_Library/Inc/Emulate.inc

@@ -465 +465 @@
     eSHIFT_IM   %1, %2, shr
 %endmacro
+
 %macro eSHL_IM 2
+%ifdef USE_386
+    %if %2 = 1
+        add     %1, %1  ; Same size but faster on 386 and 486. Fails if %1 is a memory operand.
+    %else
+        eSHIFT_IM   %1, %2, shl
+    %endif
+%else
     eSHIFT_IM   %1, %2, shl
-%endmacro
+%endif
+%endmacro
+
 %macro eROR_IM 2
     eSHIFT_IM   %1, %2, ror
 %endmacro
+
 %macro eROL_IM 2
     eSHIFT_IM   %1, %2, rol
 %endmacro
+
 %macro eRCR_IM 2
     eSHIFT_IM   %1, %2, rcr
 %endmacro
+
 %macro eRCL_IM 2
     eSHIFT_IM   %1, %2, rcl

trunk/Assembly_Library/Inc/Math.inc

@@ -79 +79 @@
 ALIGN JUMP_ALIGN
 .ShiftNextBit:
-    shl     ax, 1
+    eSHL_IM ax, 1
     rcl     dx, 1
     loop    .ShiftNextBit

trunk/Assembly_Library/Src/Display/DisplayCursor.asm

@@ -3 +3 @@
 
 ;
-; XTIDE Universal BIOS and Associated Tools 
+; XTIDE Universal BIOS and Associated Tools
 ; Copyright (C) 2009-2010 by Tomi Tilli, 2011-2012 by XTIDE Universal BIOS Team.
 ;
@@ -10 +10 @@
 ; the Free Software Foundation; either version 2 of the License, or
 ; (at your option) any later version.
-; 
+;
 ; This program is distributed in the hope that it will be useful,
 ; but WITHOUT ANY WARRANTY; without even the implied warranty of
 ; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-; GNU General Public License for more details.      
+; GNU General Public License for more details.
 ; Visit http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
-;       
+;
 
 ; Section containing code
@@ -72 +72 @@
     xor     dh, dh
     add     ax, dx                          ; Add column offset
-    shl     ax, 1                           ; Convert to WORD offset
+    eSHL_IM ax, 1                           ; Convert to WORD offset
     add     ax, [VIDEO_BDA.wPageOffset]     ; AX = Video RAM offset
     mov     [VIDEO_BDA.displayContext+DISPLAY_CONTEXT.fpCursorPosition], ax

trunk/Assembly_Library/Src/Display/DisplayFormat.asm

@@ -3 +3 @@
 
 ;
-; XTIDE Universal BIOS and Associated Tools 
+; XTIDE Universal BIOS and Associated Tools
 ; Copyright (C) 2009-2010 by Tomi Tilli, 2011-2012 by XTIDE Universal BIOS Team.
 ;
@@ -10 +10 @@
 ; the Free Software Foundation; either version 2 of the License, or
 ; (at your option) any later version.
-; 
+;
 ; This program is distributed in the hope that it will be useful,
 ; but WITHOUT ANY WARRANTY; without even the implied warranty of
 ; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-; GNU General Public License for more details.      
+; GNU General Public License for more details.
 ; Visit http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
-;       
+;
 
 ; Section containing code
@@ -137 +137 @@
 .ConvertIndexToFunctionOffset:
     sub     bx, .rgcFormatCharToLookupIndex
-    shl     bx, 1               ; Shift for WORD lookup
+    eSHL_IM bx, 1               ; Shift for WORD lookup
     mov     ax, [cs:bx+.rgfnFormatSpecifierParser]
     ret

trunk/Assembly_Library/Src/Display/DisplayFormatCompressed.asm

@@ -6 +6 @@
 
 ;
-; XTIDE Universal BIOS and Associated Tools 
+; XTIDE Universal BIOS and Associated Tools
 ; Copyright (C) 2009-2010 by Tomi Tilli, 2011-2012 by XTIDE Universal BIOS Team.
 ;
@@ -13 +13 @@
 ; the Free Software Foundation; either version 2 of the License, or
 ; (at your option) any later version.
-; 
+;
 ; This program is distributed in the hope that it will be useful,
 ; but WITHOUT ANY WARRANTY; without even the implied warranty of
 ; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-; GNU General Public License for more details.      
+; GNU General Public License for more details.
 ; Visit http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
 ;
@@ -194 +194 @@
 
 .process_after_output:
-    shl     ch,1                                ; check high order bits for end of string or space
+    eSHL_IM ch,1                                ; check high order bits for end of string or space
     jns     short DisplayFormatCompressed_ret
     jnc     .decode
@@ -208 +208 @@
 ; DisplayFormatCompressed_BaseFormatOffset and jump targets (must fit in 256 bytes)
 ;
-    shl     ch,1                ; setup ch for later testing of null in .process_after_output
+    eSHL_IM ch,1                ; setup ch for later testing of null in .process_after_output
     and     ax,0001fh           ; also clears AH for addition with BX and DX below
 

trunk/Assembly_Library/Src/Menu/CharOutLineSplitter.asm

@@ -3 +3 @@
 
 ;
-; XTIDE Universal BIOS and Associated Tools 
+; XTIDE Universal BIOS and Associated Tools
 ; Copyright (C) 2009-2010 by Tomi Tilli, 2011-2012 by XTIDE Universal BIOS Team.
 ;
@@ -10 +10 @@
 ; the Free Software Foundation; either version 2 of the License, or
 ; (at your option) any later version.
-; 
+;
 ; This program is distributed in the hope that it will be useful,
 ; but WITHOUT ANY WARRANTY; without even the implied warranty of
 ; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-; GNU General Public License for more details.      
+; GNU General Public License for more details.
 ; Visit http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
 ;
-        
+
 
 ; Section containing code
@@ -40 +40 @@
     ; Get last text line column offset to AX
     call    MenuLocation_GetMaxTextLineLengthToAX
-    shl     ax, 1           ; Characters to BYTEs
+    eSHL_IM ax, 1           ; Characters to BYTEs
     add     ax, dx
 
@@ -62 +62 @@
     call    MenuLocation_GetTitleBordersTopLeftCoordinatesToAX
     xor     ah, ah
-    shl     ax, 1
+    eSHL_IM ax, 1
     ret
 
@@ -82 +82 @@
 
     mov     dl, [VIDEO_BDA.wColumns]
-    shl     dl, 1           ; DX = bytes per row
+    eSHL_IM dl, 1           ; DX = bytes per row
     mov     ax, di
     div     dl              ; AL = row index, AH = column index

trunk/Assembly_Library/Src/Util/Memory.asm

@@ -3 +3 @@
 
 ;
-; XTIDE Universal BIOS and Associated Tools 
+; XTIDE Universal BIOS and Associated Tools
 ; Copyright (C) 2009-2010 by Tomi Tilli, 2011-2012 by XTIDE Universal BIOS Team.
 ;
@@ -10 +10 @@
 ; the Free Software Foundation; either version 2 of the License, or
 ; (at your option) any later version.
-; 
+;
 ; This program is distributed in the hope that it will be useful,
 ; but WITHOUT ANY WARRANTY; without even the implied warranty of
 ; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-; GNU General Public License for more details.      
+; GNU General Public License for more details.
 ; Visit http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
-;       
+;
 
 ; Section containing code
@@ -151 +151 @@
 ;   Returns:
 ;       AL:     Sum of bytes
+;       ZF:     Set if result is zero
+;               Cleared if result is non-zero
 ;   Corrupts registers:
 ;       CX
 ;--------------------------------------------------------------------
+%ifndef EXCLUDE_FROM_XTIDECFG
 ALIGN JUMP_ALIGN
 Memory_SumCXbytesFromESSItoAL:
     push    si
+    dec     si
     xor     al, al
 ALIGN JUMP_ALIGN
 .AddNextByteToAL:
+    inc     si
     add     al, [es:si]
-    inc     si
     loop    .AddNextByteToAL
     pop     si
     ret
+%endif

trunk/Assembly_Library/Src/Util/Sort.asm

@@ -7 +7 @@
 
 ;
-; XTIDE Universal BIOS and Associated Tools 
+; XTIDE Universal BIOS and Associated Tools
 ; Copyright (C) 2009-2010 by Tomi Tilli, 2011-2012 by XTIDE Universal BIOS Team.
 ;
@@ -14 +14 @@
 ; the Free Software Foundation; either version 2 of the License, or
 ; (at your option) any later version.
-; 
+;
 ; This program is distributed in the hope that it will be useful,
 ; but WITHOUT ANY WARRANTY; without even the implied warranty of
 ; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-; GNU General Public License for more details.      
+; GNU General Public License for more details.
 ; Visit http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
-;       
+;
 
 ; Algorith is from http://www.algolist.net/Algorithms/Sorting/Quicksort
@@ -62 +62 @@
     push    di
     mov     di, cx
-    shl     cx, 1                       ; Reserve temp and pivot items
+    eSHL_IM cx, 1                       ; Reserve temp and pivot items
     add     cx, BYTE QSORT_PARAMS_size
     eENTER_STRUCT cx