fb: Remove even/odd tile slow-pathing
Again, clearly meant to be a fast path, but this turns out not to be the case. Reviewed-by: Keith Packard <keithp@keithp.com> Signed-off-by: Adam Jackson <ajax@redhat.com>
This commit is contained in:
parent
836bb27726
commit
e572bcc7f4
|
@ -44,7 +44,6 @@ libfb_la_SOURCES = \
|
|||
fbseg.c \
|
||||
fbsetsp.c \
|
||||
fbsolid.c \
|
||||
fbtile.c \
|
||||
fbtrap.c \
|
||||
fbutil.c \
|
||||
fbwindow.c
|
||||
|
|
39
fb/fb.h
39
fb/fb.h
|
@ -1331,45 +1331,6 @@ fbSolid24(FbBits * dst,
|
|||
FbStride dstStride,
|
||||
int dstX, int width, int height, FbBits and, FbBits xor);
|
||||
|
||||
/*
|
||||
* fbtile.c
|
||||
*/
|
||||
|
||||
extern _X_EXPORT void
|
||||
|
||||
fbEvenTile(FbBits * dst,
|
||||
FbStride dstStride,
|
||||
int dstX,
|
||||
int width,
|
||||
int height,
|
||||
FbBits * tile,
|
||||
FbStride tileStride,
|
||||
int tileHeight, int alu, FbBits pm, int xRot, int yRot);
|
||||
|
||||
extern _X_EXPORT void
|
||||
|
||||
fbOddTile(FbBits * dst,
|
||||
FbStride dstStride,
|
||||
int dstX,
|
||||
int width,
|
||||
int height,
|
||||
FbBits * tile,
|
||||
FbStride tileStride,
|
||||
int tileWidth,
|
||||
int tileHeight, int alu, FbBits pm, int bpp, int xRot, int yRot);
|
||||
|
||||
extern _X_EXPORT void
|
||||
|
||||
fbTile(FbBits * dst,
|
||||
FbStride dstStride,
|
||||
int dstX,
|
||||
int width,
|
||||
int height,
|
||||
FbBits * tile,
|
||||
FbStride tileStride,
|
||||
int tileWidth,
|
||||
int tileHeight, int alu, FbBits pm, int bpp, int xRot, int yRot);
|
||||
|
||||
/*
|
||||
* fbutil.c
|
||||
*/
|
||||
|
|
38
fb/fbfill.c
38
fb/fbfill.c
|
@ -26,6 +26,44 @@
|
|||
|
||||
#include "fb.h"
|
||||
|
||||
static void
|
||||
fbTile(FbBits * dst, FbStride dstStride, int dstX, int width, int height,
|
||||
FbBits * tile, FbStride tileStride, int tileWidth, int tileHeight,
|
||||
int alu, FbBits pm, int bpp, int xRot, int yRot)
|
||||
{
|
||||
int tileX, tileY;
|
||||
int widthTmp;
|
||||
int h, w;
|
||||
int x, y;
|
||||
|
||||
modulus(-yRot, tileHeight, tileY);
|
||||
y = 0;
|
||||
while (height) {
|
||||
h = tileHeight - tileY;
|
||||
if (h > height)
|
||||
h = height;
|
||||
height -= h;
|
||||
widthTmp = width;
|
||||
x = dstX;
|
||||
modulus(dstX - xRot, tileWidth, tileX);
|
||||
while (widthTmp) {
|
||||
w = tileWidth - tileX;
|
||||
if (w > widthTmp)
|
||||
w = widthTmp;
|
||||
widthTmp -= w;
|
||||
fbBlt(tile + tileY * tileStride,
|
||||
tileStride,
|
||||
tileX,
|
||||
dst + y * dstStride,
|
||||
dstStride, x, w, h, alu, pm, bpp, FALSE, FALSE);
|
||||
x += w;
|
||||
tileX = 0;
|
||||
}
|
||||
y += h;
|
||||
tileY = 0;
|
||||
}
|
||||
}
|
||||
|
||||
static void
|
||||
fbStipple(FbBits * dst, FbStride dstStride,
|
||||
int dstX, int dstBpp,
|
||||
|
|
163
fb/fbtile.c
163
fb/fbtile.c
|
@ -1,163 +0,0 @@
|
|||
/*
|
||||
* Copyright © 1998 Keith Packard
|
||||
*
|
||||
* Permission to use, copy, modify, distribute, and sell this software and its
|
||||
* documentation for any purpose is hereby granted without fee, provided that
|
||||
* the above copyright notice appear in all copies and that both that
|
||||
* copyright notice and this permission notice appear in supporting
|
||||
* documentation, and that the name of Keith Packard not be used in
|
||||
* advertising or publicity pertaining to distribution of the software without
|
||||
* specific, written prior permission. Keith Packard makes no
|
||||
* representations about the suitability of this software for any purpose. It
|
||||
* is provided "as is" without express or implied warranty.
|
||||
*
|
||||
* KEITH PACKARD DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS SOFTWARE,
|
||||
* INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS, IN NO
|
||||
* EVENT SHALL KEITH PACKARD BE LIABLE FOR ANY SPECIAL, INDIRECT OR
|
||||
* CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE,
|
||||
* DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER
|
||||
* TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
|
||||
* PERFORMANCE OF THIS SOFTWARE.
|
||||
*/
|
||||
|
||||
#ifdef HAVE_DIX_CONFIG_H
|
||||
#include <dix-config.h>
|
||||
#endif
|
||||
|
||||
#include "fb.h"
|
||||
|
||||
/*
|
||||
* Accelerated tile fill -- tile width is a power of two not greater
|
||||
* than FB_UNIT
|
||||
*/
|
||||
|
||||
void
|
||||
fbEvenTile(FbBits * dst,
|
||||
FbStride dstStride,
|
||||
int dstX,
|
||||
int width,
|
||||
int height,
|
||||
FbBits * tile,
|
||||
FbStride tileStride,
|
||||
int tileHeight, int alu, FbBits pm, int xRot, int yRot)
|
||||
{
|
||||
FbBits *t, *tileEnd, bits;
|
||||
FbBits startmask, endmask;
|
||||
FbBits and, xor;
|
||||
int n, nmiddle;
|
||||
int tileX, tileY;
|
||||
int rot;
|
||||
int startbyte, endbyte;
|
||||
|
||||
dst += dstX >> FB_SHIFT;
|
||||
dstX &= FB_MASK;
|
||||
FbMaskBitsBytes(dstX, width, FbDestInvarientRop(alu, pm),
|
||||
startmask, startbyte, nmiddle, endmask, endbyte);
|
||||
if (startmask)
|
||||
dstStride--;
|
||||
dstStride -= nmiddle;
|
||||
|
||||
/*
|
||||
* Compute tile start scanline and rotation parameters
|
||||
*/
|
||||
tileEnd = tile + tileHeight * tileStride;
|
||||
modulus(-yRot, tileHeight, tileY);
|
||||
t = tile + tileY * tileStride;
|
||||
modulus(-xRot, FB_UNIT, tileX);
|
||||
rot = tileX;
|
||||
|
||||
while (height--) {
|
||||
|
||||
/*
|
||||
* Pick up bits for this scanline
|
||||
*/
|
||||
bits = READ(t);
|
||||
t += tileStride;
|
||||
if (t >= tileEnd)
|
||||
t = tile;
|
||||
bits = FbRotLeft(bits, rot);
|
||||
and = fbAnd(alu, bits, pm);
|
||||
xor = fbXor(alu, bits, pm);
|
||||
|
||||
if (startmask) {
|
||||
FbDoLeftMaskByteRRop(dst, startbyte, startmask, and, xor);
|
||||
dst++;
|
||||
}
|
||||
n = nmiddle;
|
||||
if (!and)
|
||||
while (n--)
|
||||
WRITE(dst++, xor);
|
||||
else
|
||||
while (n--) {
|
||||
WRITE(dst, FbDoRRop(READ(dst), and, xor));
|
||||
dst++;
|
||||
}
|
||||
if (endmask)
|
||||
FbDoRightMaskByteRRop(dst, endbyte, endmask, and, xor);
|
||||
dst += dstStride;
|
||||
}
|
||||
}
|
||||
|
||||
void
|
||||
fbOddTile(FbBits * dst,
|
||||
FbStride dstStride,
|
||||
int dstX,
|
||||
int width,
|
||||
int height,
|
||||
FbBits * tile,
|
||||
FbStride tileStride,
|
||||
int tileWidth,
|
||||
int tileHeight, int alu, FbBits pm, int bpp, int xRot, int yRot)
|
||||
{
|
||||
int tileX, tileY;
|
||||
int widthTmp;
|
||||
int h, w;
|
||||
int x, y;
|
||||
|
||||
modulus(-yRot, tileHeight, tileY);
|
||||
y = 0;
|
||||
while (height) {
|
||||
h = tileHeight - tileY;
|
||||
if (h > height)
|
||||
h = height;
|
||||
height -= h;
|
||||
widthTmp = width;
|
||||
x = dstX;
|
||||
modulus(dstX - xRot, tileWidth, tileX);
|
||||
while (widthTmp) {
|
||||
w = tileWidth - tileX;
|
||||
if (w > widthTmp)
|
||||
w = widthTmp;
|
||||
widthTmp -= w;
|
||||
fbBlt(tile + tileY * tileStride,
|
||||
tileStride,
|
||||
tileX,
|
||||
dst + y * dstStride,
|
||||
dstStride, x, w, h, alu, pm, bpp, FALSE, FALSE);
|
||||
x += w;
|
||||
tileX = 0;
|
||||
}
|
||||
y += h;
|
||||
tileY = 0;
|
||||
}
|
||||
}
|
||||
|
||||
void
|
||||
fbTile(FbBits * dst,
|
||||
FbStride dstStride,
|
||||
int dstX,
|
||||
int width,
|
||||
int height,
|
||||
FbBits * tile,
|
||||
FbStride tileStride,
|
||||
int tileWidth,
|
||||
int tileHeight, int alu, FbBits pm, int bpp, int xRot, int yRot)
|
||||
{
|
||||
if (FbEvenTile(tileWidth))
|
||||
fbEvenTile(dst, dstStride, dstX, width, height,
|
||||
tile, tileStride, tileHeight, alu, pm, xRot, yRot);
|
||||
else
|
||||
fbOddTile(dst, dstStride, dstX, width, height,
|
||||
tile, tileStride, tileWidth, tileHeight,
|
||||
alu, pm, bpp, xRot, yRot);
|
||||
}
|
|
@ -58,7 +58,6 @@
|
|||
#define fbDots24 wfbDots24
|
||||
#define fbDots32 wfbDots32
|
||||
#define fbDots8 wfbDots8
|
||||
#define fbEvenTile wfbEvenTile
|
||||
#define fbExpandDirectColors wfbExpandDirectColors
|
||||
#define fbFill wfbFill
|
||||
#define fbFillRegionSolid wfbFillRegionSolid
|
||||
|
@ -83,7 +82,6 @@
|
|||
#define fbInitVisuals wfbInitVisuals
|
||||
#define fbListInstalledColormaps wfbListInstalledColormaps
|
||||
#define FbMergeRopBits wFbMergeRopBits
|
||||
#define fbOddTile wfbOddTile
|
||||
#define fbOver wfbOver
|
||||
#define fbOver24 wfbOver24
|
||||
#define fbOverlayCloseScreen wfbOverlayCloseScreen
|
||||
|
@ -139,7 +137,6 @@
|
|||
#define fbSolid wfbSolid
|
||||
#define fbSolid24 wfbSolid24
|
||||
#define fbSolidBoxClipped wfbSolidBoxClipped
|
||||
#define fbTile wfbTile
|
||||
#define fbTrapezoids wfbTrapezoids
|
||||
#define fbTriangles wfbTriangles
|
||||
#define fbUninstallColormap wfbUninstallColormap
|
||||
|
|
Loading…
Reference in New Issue