From f0327afe876acff27221cdeead1aca0444364133 Mon Sep 17 00:00:00 2001

From: Gerd Hoffmann <kraxel@redhat.com>

Date: Mon, 27 Mar 2017 10:01:20 +0200

Subject: [PATCH 5/7] cirrus: stop passing around dst pointers in the blitter

RH-Author: Gerd Hoffmann <kraxel@redhat.com>

Message-id: <1490608882-10242-6-git-send-email-kraxel@redhat.com>

Patchwork-id: 74550

O-Subject: [RHEL-7.4 qemu-kvm PATCH v2 5/7] cirrus: stop passing around dst pointers in the blitter

Bugzilla: 1430060

RH-Acked-by: Dr. David Alan Gilbert <dgilbert@redhat.com>

RH-Acked-by: Laurent Vivier <lvivier@redhat.com>

RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>

Instead pass around the address (aka offset into vga memory).  Calculate

the pointer in the rop_* functions, after applying the mask to the

address, to make sure the address stays within the valid range.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>

Message-id: 1489574872-8679-1-git-send-email-kraxel@redhat.com

(cherry picked from commit 026aeffcb4752054830ba203020ed6eb05bcaba8)

Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>

---

 hw/display/cirrus_vga.c      |  20 +++---

 hw/display/cirrus_vga_rop.h  | 161 +++++++++++++++++++++++++------------------

 hw/display/cirrus_vga_rop2.h |  97 +++++++++++++-------------

 3 files changed, 153 insertions(+), 125 deletions(-)

diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c

index aab6e02..003cc4c 100644

--- a/hw/display/cirrus_vga.c

+++ b/hw/display/cirrus_vga.c

@@ -174,11 +174,12 @@

 struct CirrusVGAState;

 typedef void (*cirrus_bitblt_rop_t) (struct CirrusVGAState *s,

-                                     uint8_t * dst, const uint8_t * src,

+                                     uint32_t dstaddr, const uint8_t *src,

 				     int dstpitch, int srcpitch,

 				     int bltwidth, int bltheight);

 typedef void (*cirrus_fill_t)(struct CirrusVGAState *s,

-                              uint8_t *dst, int dst_pitch, int width, int height);

+                              uint32_t dstaddr, int dst_pitch,

+                              int width, int height);

 typedef struct CirrusVGAState {

     VGACommonState vga;

@@ -315,14 +316,14 @@ static bool blit_is_unsafe(struct CirrusVGAState *s, bool dst_only)

 }

 static void cirrus_bitblt_rop_nop(CirrusVGAState *s,

-                                  uint8_t *dst,const uint8_t *src,

+                                  uint32_t dstaddr, const uint8_t *src,

                                   int dstpitch,int srcpitch,

                                   int bltwidth,int bltheight)

 {

 }

 static void cirrus_bitblt_fill_nop(CirrusVGAState *s,

-                                   uint8_t *dst,

+                                   uint32_t dstaddr,

                                    int dstpitch, int bltwidth,int bltheight)

 {

 }

@@ -672,11 +673,8 @@ static void cirrus_invalidate_region(CirrusVGAState * s, int off_begin,

 static int cirrus_bitblt_common_patterncopy(CirrusVGAState *s, bool videosrc)

 {

     uint32_t patternsize;

-    uint8_t *dst;

     uint8_t *src;

-    dst = s->vga.vram_ptr + s->cirrus_blt_dstaddr;

-

     if (videosrc) {

         switch (s->vga.get_bpp(&s->vga)) {

         case 8:

@@ -705,7 +703,7 @@ static int cirrus_bitblt_common_patterncopy(CirrusVGAState *s, bool videosrc)

         return 0;

     }

-    (*s->cirrus_rop) (s, dst, src,

+    (*s->cirrus_rop) (s, s->cirrus_blt_dstaddr, src,

                       s->cirrus_blt_dstpitch, 0,

                       s->cirrus_blt_width, s->cirrus_blt_height);

     cirrus_invalidate_region(s, s->cirrus_blt_dstaddr,

@@ -724,7 +722,7 @@ static int cirrus_bitblt_solidfill(CirrusVGAState *s, int blt_rop)

         return 0;

     }

     rop_func = cirrus_fill[rop_to_index[blt_rop]][s->cirrus_blt_pixelwidth - 1];

-    rop_func(s, s->vga.vram_ptr + s->cirrus_blt_dstaddr,

+    rop_func(s, s->cirrus_blt_dstaddr,

              s->cirrus_blt_dstpitch,

              s->cirrus_blt_width, s->cirrus_blt_height);

     cirrus_invalidate_region(s, s->cirrus_blt_dstaddr,

@@ -791,7 +789,7 @@ static int cirrus_do_copy(CirrusVGAState *s, int dst, int src, int w, int h)

         }

     }

-    (*s->cirrus_rop) (s, s->vga.vram_ptr + s->cirrus_blt_dstaddr,

+    (*s->cirrus_rop) (s, s->cirrus_blt_dstaddr,

                       s->vga.vram_ptr + s->cirrus_blt_srcaddr,

 		      s->cirrus_blt_dstpitch, s->cirrus_blt_srcpitch,

 		      s->cirrus_blt_width, s->cirrus_blt_height);

@@ -842,7 +840,7 @@ static void cirrus_bitblt_cputovideo_next(CirrusVGAState * s)

         } else {

             /* at least one scan line */

             do {

-                (*s->cirrus_rop)(s, s->vga.vram_ptr + s->cirrus_blt_dstaddr,

+                (*s->cirrus_rop)(s, s->cirrus_blt_dstaddr,

                                   s->cirrus_bltbuf, 0, 0, s->cirrus_blt_width, 1);

                 cirrus_invalidate_region(s, s->cirrus_blt_dstaddr, 0,

                                          s->cirrus_blt_width, 1);

diff --git a/hw/display/cirrus_vga_rop.h b/hw/display/cirrus_vga_rop.h

index a4f96c6..3b16d70 100644

--- a/hw/display/cirrus_vga_rop.h

+++ b/hw/display/cirrus_vga_rop.h

@@ -22,31 +22,65 @@

  * THE SOFTWARE.

  */

-static inline void glue(rop_8_,ROP_NAME)(uint8_t *dst, uint8_t src)

+static inline void glue(rop_8_, ROP_NAME)(CirrusVGAState *s,

+                                          uint32_t dstaddr, uint8_t src)

 {

+    uint8_t *dst = &s->vga.vram_ptr[dstaddr & s->cirrus_addr_mask];

     *dst = ROP_FN(*dst, src);

 }

-static inline void glue(rop_16_,ROP_NAME)(uint16_t *dst, uint16_t src)

+static inline void glue(rop_tr_8_, ROP_NAME)(CirrusVGAState *s,

+                                             uint32_t dstaddr, uint8_t src,

+                                             uint8_t transp)

 {

+    uint8_t *dst = &s->vga.vram_ptr[dstaddr & s->cirrus_addr_mask];

+    uint8_t pixel = ROP_FN(*dst, src);

+    if (pixel != transp) {

+        *dst = pixel;

+    }

+}

+

+static inline void glue(rop_16_, ROP_NAME)(CirrusVGAState *s,

+                                           uint32_t dstaddr, uint16_t src)

+{

+    uint16_t *dst = (uint16_t *)

+        (&s->vga.vram_ptr[dstaddr & s->cirrus_addr_mask & ~1]);

     *dst = ROP_FN(*dst, src);

 }

-static inline void glue(rop_32_,ROP_NAME)(uint32_t *dst, uint32_t src)

+static inline void glue(rop_tr_16_, ROP_NAME)(CirrusVGAState *s,

+                                              uint32_t dstaddr, uint16_t src,

+                                              uint16_t transp)

+{

+    uint16_t *dst = (uint16_t *)

+        (&s->vga.vram_ptr[dstaddr & s->cirrus_addr_mask & ~1]);

+    uint16_t pixel = ROP_FN(*dst, src);

+    if (pixel != transp) {

+        *dst = pixel;

+    }

+}

+

+static inline void glue(rop_32_, ROP_NAME)(CirrusVGAState *s,

+                                           uint32_t dstaddr, uint32_t src)

 {

+    uint32_t *dst = (uint32_t *)

+        (&s->vga.vram_ptr[dstaddr & s->cirrus_addr_mask & ~3]);

     *dst = ROP_FN(*dst, src);

 }

-#define ROP_OP(d, s) glue(rop_8_,ROP_NAME)(d, s)

-#define ROP_OP_16(d, s) glue(rop_16_,ROP_NAME)(d, s)

-#define ROP_OP_32(d, s) glue(rop_32_,ROP_NAME)(d, s)

+#define ROP_OP(st, d, s)           glue(rop_8_, ROP_NAME)(st, d, s)

+#define ROP_OP_TR(st, d, s, t)     glue(rop_tr_8_, ROP_NAME)(st, d, s, t)

+#define ROP_OP_16(st, d, s)        glue(rop_16_, ROP_NAME)(st, d, s)

+#define ROP_OP_TR_16(st, d, s, t)  glue(rop_tr_16_, ROP_NAME)(st, d, s, t)

+#define ROP_OP_32(st, d, s)        glue(rop_32_, ROP_NAME)(st, d, s)

 #undef ROP_FN

 static void

 glue(cirrus_bitblt_rop_fwd_, ROP_NAME)(CirrusVGAState *s,

-                             uint8_t *dst,const uint8_t *src,

-                             int dstpitch,int srcpitch,

-                             int bltwidth,int bltheight)

+                                       uint32_t dstaddr,

+                                       const uint8_t *src,

+                                       int dstpitch, int srcpitch,

+                                       int bltwidth, int bltheight)

 {

     int x,y;

     dstpitch -= bltwidth;

@@ -59,43 +93,47 @@ glue(cirrus_bitblt_rop_fwd_, ROP_NAME)(CirrusVGAState *s,

     for (y = 0; y < bltheight; y++) {

         for (x = 0; x < bltwidth; x++) {

-            ROP_OP(dst, *src);

-            dst++;

+            ROP_OP(s, dstaddr, *src);

+            dstaddr++;

             src++;

         }

-        dst += dstpitch;

+        dstaddr += dstpitch;

         src += srcpitch;

     }

 }

 static void

 glue(cirrus_bitblt_rop_bkwd_, ROP_NAME)(CirrusVGAState *s,

-                                        uint8_t *dst,const uint8_t *src,

-                                        int dstpitch,int srcpitch,

-                                        int bltwidth,int bltheight)

+                                        uint32_t dstaddr,

+                                        const uint8_t *src,

+                                        int dstpitch, int srcpitch,

+                                        int bltwidth, int bltheight)

 {

     int x,y;

     dstpitch += bltwidth;

     srcpitch += bltwidth;

     for (y = 0; y < bltheight; y++) {

         for (x = 0; x < bltwidth; x++) {

-            ROP_OP(dst, *src);

-            dst--;

+            ROP_OP(s, dstaddr, *src);

+            dstaddr--;

             src--;

         }

-        dst += dstpitch;

+        dstaddr += dstpitch;

         src += srcpitch;

     }

 }

 static void

 glue(glue(cirrus_bitblt_rop_fwd_transp_, ROP_NAME),_8)(CirrusVGAState *s,

-						       uint8_t *dst,const uint8_t *src,

-						       int dstpitch,int srcpitch,

-						       int bltwidth,int bltheight)

+                                                       uint32_t dstaddr,

+                                                       const uint8_t *src,

+                                                       int dstpitch,

+                                                       int srcpitch,

+                                                       int bltwidth,

+                                                       int bltheight)

 {

     int x,y;

-    uint8_t p;

+    uint8_t transp = s->vga.gr[0x34];

     dstpitch -= bltwidth;

     srcpitch -= bltwidth;

@@ -105,48 +143,50 @@ glue(glue(cirrus_bitblt_rop_fwd_transp_, ROP_NAME),_8)(CirrusVGAState *s,

     for (y = 0; y < bltheight; y++) {

         for (x = 0; x < bltwidth; x++) {

-	    p = *dst;

-            ROP_OP(&p, *src);

-	    if (p != s->vga.gr[0x34]) *dst = p;

-            dst++;

+            ROP_OP_TR(s, dstaddr, *src, transp);

+            dstaddr++;

             src++;

         }

-        dst += dstpitch;

+        dstaddr += dstpitch;

         src += srcpitch;

     }

 }

 static void

 glue(glue(cirrus_bitblt_rop_bkwd_transp_, ROP_NAME),_8)(CirrusVGAState *s,

-							uint8_t *dst,const uint8_t *src,

-							int dstpitch,int srcpitch,

-							int bltwidth,int bltheight)

+                                                        uint32_t dstaddr,

+                                                        const uint8_t *src,

+                                                        int dstpitch,

+                                                        int srcpitch,

+                                                        int bltwidth,

+                                                        int bltheight)

 {

     int x,y;

-    uint8_t p;

+    uint8_t transp = s->vga.gr[0x34];

     dstpitch += bltwidth;

     srcpitch += bltwidth;

     for (y = 0; y < bltheight; y++) {

         for (x = 0; x < bltwidth; x++) {

-	    p = *dst;

-            ROP_OP(&p, *src);

-	    if (p != s->vga.gr[0x34]) *dst = p;

-            dst--;

+            ROP_OP_TR(s, dstaddr, *src, transp);

+            dstaddr--;

             src--;

         }

-        dst += dstpitch;

+        dstaddr += dstpitch;

         src += srcpitch;

     }

 }

 static void

 glue(glue(cirrus_bitblt_rop_fwd_transp_, ROP_NAME),_16)(CirrusVGAState *s,

-							uint8_t *dst,const uint8_t *src,

-							int dstpitch,int srcpitch,

-							int bltwidth,int bltheight)

+                                                        uint32_t dstaddr,

+                                                        const uint8_t *src,

+                                                        int dstpitch,

+                                                        int srcpitch,

+                                                        int bltwidth,

+                                                        int bltheight)

 {

     int x,y;

-    uint8_t p1, p2;

+    uint16_t transp = s->vga.gr[0x34] | (uint16_t)s->vga.gr[0x35] << 8;

     dstpitch -= bltwidth;

     srcpitch -= bltwidth;

@@ -156,46 +196,35 @@ glue(glue(cirrus_bitblt_rop_fwd_transp_, ROP_NAME),_16)(CirrusVGAState *s,

     for (y = 0; y < bltheight; y++) {

         for (x = 0; x < bltwidth; x+=2) {

-	    p1 = *dst;

-	    p2 = *(dst+1);

-            ROP_OP(&p1, *src);

-            ROP_OP(&p2, *(src + 1));

-	    if ((p1 != s->vga.gr[0x34]) || (p2 != s->vga.gr[0x35])) {

-		*dst = p1;

-		*(dst+1) = p2;

-	    }

-            dst+=2;

-            src+=2;

+            ROP_OP_TR_16(s, dstaddr, *(uint16_t *)src, transp);

+            dstaddr += 2;

+            src += 2;

         }

-        dst += dstpitch;

+        dstaddr += dstpitch;

         src += srcpitch;

     }

 }

 static void

 glue(glue(cirrus_bitblt_rop_bkwd_transp_, ROP_NAME),_16)(CirrusVGAState *s,

-							 uint8_t *dst,const uint8_t *src,

-							 int dstpitch,int srcpitch,

-							 int bltwidth,int bltheight)

+                                                         uint32_t dstaddr,

+                                                         const uint8_t *src,

+                                                         int dstpitch,

+                                                         int srcpitch,

+                                                         int bltwidth,

+                                                         int bltheight)

 {

     int x,y;

-    uint8_t p1, p2;

+    uint16_t transp = s->vga.gr[0x34] | (uint16_t)s->vga.gr[0x35] << 8;

     dstpitch += bltwidth;

     srcpitch += bltwidth;

     for (y = 0; y < bltheight; y++) {

         for (x = 0; x < bltwidth; x+=2) {

-	    p1 = *(dst-1);

-	    p2 = *dst;

-            ROP_OP(&p1, *(src - 1));

-            ROP_OP(&p2, *src);

-	    if ((p1 != s->vga.gr[0x34]) || (p2 != s->vga.gr[0x35])) {

-		*(dst-1) = p1;

-		*dst = p2;

-	    }

-            dst-=2;

-            src-=2;

+            ROP_OP_TR_16(s, dstaddr, *(uint16_t *)src, transp);

+            dstaddr -= 2;

+            src -= 2;

         }

-        dst += dstpitch;

+        dstaddr += dstpitch;

         src += srcpitch;

     }

 }

diff --git a/hw/display/cirrus_vga_rop2.h b/hw/display/cirrus_vga_rop2.h

index d28bcc6..bc92f0e 100644

--- a/hw/display/cirrus_vga_rop2.h

+++ b/hw/display/cirrus_vga_rop2.h

@@ -23,27 +23,29 @@

  */

 #if DEPTH == 8

-#define PUTPIXEL()    ROP_OP(&d[0], col)

+#define PUTPIXEL(s, a, c)    ROP_OP(s, a, c)

 #elif DEPTH == 16

-#define PUTPIXEL()    ROP_OP_16((uint16_t *)&d[0], col)

+#define PUTPIXEL(s, a, c)    ROP_OP_16(s, a, c)

 #elif DEPTH == 24

-#define PUTPIXEL()    ROP_OP(&d[0], col);        \

-                      ROP_OP(&d[1], (col >> 8)); \

-                      ROP_OP(&d[2], (col >> 16))

+#define PUTPIXEL(s, a, c)    do {          \

+        ROP_OP(s, a,     c);               \

+        ROP_OP(s, a + 1, (col >> 8));      \

+        ROP_OP(s, a + 2, (col >> 16));     \

+    } while (0)

 #elif DEPTH == 32

-#define PUTPIXEL()    ROP_OP_32(((uint32_t *)&d[0]), col)

+#define PUTPIXEL(s, a, c)    ROP_OP_32(s, a, c)

 #else

 #error unsupported DEPTH

 #endif

 static void

 glue(glue(glue(cirrus_patternfill_, ROP_NAME), _),DEPTH)

-     (CirrusVGAState * s, uint8_t * dst,

-      const uint8_t * src,

+     (CirrusVGAState *s, uint32_t dstaddr,

+      const uint8_t *src,

       int dstpitch, int srcpitch,

       int bltwidth, int bltheight)

 {

-    uint8_t *d;

+    uint32_t addr;

     int x, y, pattern_y, pattern_pitch, pattern_x;

     unsigned int col;

     const uint8_t *src1;

@@ -63,7 +65,7 @@ glue(glue(glue(cirrus_patternfill_, ROP_NAME), _),DEPTH)

     pattern_y = s->cirrus_blt_srcaddr & 7;

     for(y = 0; y < bltheight; y++) {

         pattern_x = skipleft;

-        d = dst + skipleft;

+        addr = dstaddr + skipleft;

         src1 = src + pattern_y * pattern_pitch;

         for (x = skipleft; x < bltwidth; x += (DEPTH / 8)) {

 #if DEPTH == 8

@@ -82,23 +84,23 @@ glue(glue(glue(cirrus_patternfill_, ROP_NAME), _),DEPTH)

             col = ((uint32_t *)(src1 + pattern_x))[0];

             pattern_x = (pattern_x + 4) & 31;

 #endif

-            PUTPIXEL();

-            d += (DEPTH / 8);

+            PUTPIXEL(s, addr, col);

+            addr += (DEPTH / 8);

         }

         pattern_y = (pattern_y + 1) & 7;

-        dst += dstpitch;

+        dstaddr += dstpitch;

     }

 }

 /* NOTE: srcpitch is ignored */

 static void

 glue(glue(glue(cirrus_colorexpand_transp_, ROP_NAME), _),DEPTH)

-     (CirrusVGAState * s, uint8_t * dst,

-      const uint8_t * src,

+     (CirrusVGAState *s, uint32_t dstaddr,

+      const uint8_t *src,

       int dstpitch, int srcpitch,

       int bltwidth, int bltheight)

 {

-    uint8_t *d;

+    uint32_t addr;

     int x, y;

     unsigned bits, bits_xor;

     unsigned int col;

@@ -123,7 +125,7 @@ glue(glue(glue(cirrus_colorexpand_transp_, ROP_NAME), _),DEPTH)

     for(y = 0; y < bltheight; y++) {

         bitmask = 0x80 >> srcskipleft;

         bits = *src++ ^ bits_xor;

-        d = dst + dstskipleft;

+        addr = dstaddr + dstskipleft;

         for (x = dstskipleft; x < bltwidth; x += (DEPTH / 8)) {

             if ((bitmask & 0xff) == 0) {

                 bitmask = 0x80;

@@ -131,24 +133,24 @@ glue(glue(glue(cirrus_colorexpand_transp_, ROP_NAME), _),DEPTH)

             }

             index = (bits & bitmask);

             if (index) {

-                PUTPIXEL();

+                PUTPIXEL(s, addr, col);

             }

-            d += (DEPTH / 8);

+            addr += (DEPTH / 8);

             bitmask >>= 1;

         }

-        dst += dstpitch;

+        dstaddr += dstpitch;

     }

 }

 static void

 glue(glue(glue(cirrus_colorexpand_, ROP_NAME), _),DEPTH)

-     (CirrusVGAState * s, uint8_t * dst,

-      const uint8_t * src,

+     (CirrusVGAState *s, uint32_t dstaddr,

+      const uint8_t *src,

       int dstpitch, int srcpitch,

       int bltwidth, int bltheight)

 {

     uint32_t colors[2];

-    uint8_t *d;

+    uint32_t addr;

     int x, y;

     unsigned bits;

     unsigned int col;

@@ -161,29 +163,29 @@ glue(glue(glue(cirrus_colorexpand_, ROP_NAME), _),DEPTH)

     for(y = 0; y < bltheight; y++) {

         bitmask = 0x80 >> srcskipleft;

         bits = *src++;

-        d = dst + dstskipleft;

+        addr = dstaddr + dstskipleft;

         for (x = dstskipleft; x < bltwidth; x += (DEPTH / 8)) {

             if ((bitmask & 0xff) == 0) {

                 bitmask = 0x80;

                 bits = *src++;

             }

             col = colors[!!(bits & bitmask)];

-            PUTPIXEL();

-            d += (DEPTH / 8);

+            PUTPIXEL(s, addr, col);

+            addr += (DEPTH / 8);

             bitmask >>= 1;

         }

-        dst += dstpitch;

+        dstaddr += dstpitch;

     }

 }

 static void

 glue(glue(glue(cirrus_colorexpand_pattern_transp_, ROP_NAME), _),DEPTH)

-     (CirrusVGAState * s, uint8_t * dst,

-      const uint8_t * src,

+     (CirrusVGAState *s, uint32_t dstaddr,

+      const uint8_t *src,

       int dstpitch, int srcpitch,

       int bltwidth, int bltheight)

 {

-    uint8_t *d;

+    uint32_t addr;

     int x, y, bitpos, pattern_y;

     unsigned int bits, bits_xor;

     unsigned int col;

@@ -207,28 +209,28 @@ glue(glue(glue(cirrus_colorexpand_pattern_transp_, ROP_NAME), _),DEPTH)

     for(y = 0; y < bltheight; y++) {

         bits = src[pattern_y] ^ bits_xor;

         bitpos = 7 - srcskipleft;

-        d = dst + dstskipleft;

+        addr = dstaddr + dstskipleft;

         for (x = dstskipleft; x < bltwidth; x += (DEPTH / 8)) {

             if ((bits >> bitpos) & 1) {

-                PUTPIXEL();

+                PUTPIXEL(s, addr, col);

             }

-            d += (DEPTH / 8);

+            addr += (DEPTH / 8);

             bitpos = (bitpos - 1) & 7;

         }

         pattern_y = (pattern_y + 1) & 7;

-        dst += dstpitch;

+        dstaddr += dstpitch;

     }

 }

 static void

 glue(glue(glue(cirrus_colorexpand_pattern_, ROP_NAME), _),DEPTH)

-     (CirrusVGAState * s, uint8_t * dst,

-      const uint8_t * src,

+     (CirrusVGAState *s, uint32_t dstaddr,

+      const uint8_t *src,

       int dstpitch, int srcpitch,

       int bltwidth, int bltheight)

 {

     uint32_t colors[2];

-    uint8_t *d;

+    uint32_t addr;

     int x, y, bitpos, pattern_y;

     unsigned int bits;

     unsigned int col;

@@ -242,38 +244,37 @@ glue(glue(glue(cirrus_colorexpand_pattern_, ROP_NAME), _),DEPTH)

     for(y = 0; y < bltheight; y++) {

         bits = src[pattern_y];

         bitpos = 7 - srcskipleft;

-        d = dst + dstskipleft;

+        addr = dstaddr + dstskipleft;

         for (x = dstskipleft; x < bltwidth; x += (DEPTH / 8)) {

             col = colors[(bits >> bitpos) & 1];

-            PUTPIXEL();

-            d += (DEPTH / 8);

+            PUTPIXEL(s, addr, col);

+            addr += (DEPTH / 8);

             bitpos = (bitpos - 1) & 7;

         }

         pattern_y = (pattern_y + 1) & 7;

-        dst += dstpitch;

+        dstaddr += dstpitch;

     }

 }

 static void

 glue(glue(glue(cirrus_fill_, ROP_NAME), _),DEPTH)

      (CirrusVGAState *s,

-      uint8_t *dst, int dst_pitch,

+      uint32_t dstaddr, int dst_pitch,

       int width, int height)

 {

-    uint8_t *d, *d1;

+    uint32_t addr;

     uint32_t col;

     int x, y;

     col = s->cirrus_blt_fgcol;

-    d1 = dst;

     for(y = 0; y < height; y++) {

-        d = d1;

+        addr = dstaddr;

         for(x = 0; x < width; x += (DEPTH / 8)) {

-            PUTPIXEL();

-            d += (DEPTH / 8);

+            PUTPIXEL(s, addr, col);

+            addr += (DEPTH / 8);

         }

-        d1 += dst_pitch;

+        dstaddr += dst_pitch;

     }

 }

-- 

1.8.3.1