<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:x="urn:schemas-microsoft-com:office:excel" xmlns:p="urn:schemas-microsoft-com:office:powerpoint" xmlns:a="urn:schemas-microsoft-com:office:access" xmlns:dt="uuid:C2F41010-65B3-11d1-A29F-00AA00C14882" xmlns:s="uuid:BDC6E3F0-6DA3-11d1-A2A3-00AA00C14882" xmlns:rs="urn:schemas-microsoft-com:rowset" xmlns:z="#RowsetSchema" xmlns:b="urn:schemas-microsoft-com:office:publisher" xmlns:ss="urn:schemas-microsoft-com:office:spreadsheet" xmlns:c="urn:schemas-microsoft-com:office:component:spreadsheet" xmlns:odc="urn:schemas-microsoft-com:office:odc" xmlns:oa="urn:schemas-microsoft-com:office:activation" xmlns:html="http://www.w3.org/TR/REC-html40" xmlns:q="http://schemas.xmlsoap.org/soap/envelope/" xmlns:rtc="http://microsoft.com/officenet/conferencing" xmlns:D="DAV:" xmlns:Repl="http://schemas.microsoft.com/repl/" xmlns:mt="http://schemas.microsoft.com/sharepoint/soap/meetings/" xmlns:x2="http://schemas.microsoft.com/office/excel/2003/xml" xmlns:ppda="http://www.passport.com/NameSpace.xsd" xmlns:ois="http://schemas.microsoft.com/sharepoint/soap/ois/" xmlns:dir="http://schemas.microsoft.com/sharepoint/soap/directory/" xmlns:ds="http://www.w3.org/2000/09/xmldsig#" xmlns:dsp="http://schemas.microsoft.com/sharepoint/dsp" xmlns:udc="http://schemas.microsoft.com/data/udc" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:sub="http://schemas.microsoft.com/sharepoint/soap/2002/1/alerts/" xmlns:ec="http://www.w3.org/2001/04/xmlenc#" xmlns:sp="http://schemas.microsoft.com/sharepoint/" xmlns:sps="http://schemas.microsoft.com/sharepoint/soap/" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:udcs="http://schemas.microsoft.com/data/udc/soap" xmlns:udcxf="http://schemas.microsoft.com/data/udc/xmlfile" xmlns:udcp2p="http://schemas.microsoft.com/data/udc/parttopart" xmlns:wf="http://schemas.microsoft.com/sharepoint/soap/workflow/" xmlns:dsss="http://schemas.microsoft.com/office/2006/digsig-setup" xmlns:dssi="http://schemas.microsoft.com/office/2006/digsig" xmlns:mdssi="http://schemas.openxmlformats.org/package/2006/digital-signature" xmlns:mver="http://schemas.openxmlformats.org/markup-compatibility/2006" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns:mrels="http://schemas.openxmlformats.org/package/2006/relationships" xmlns:spwp="http://microsoft.com/sharepoint/webpartpages" xmlns:ex12t="http://schemas.microsoft.com/exchange/services/2006/types" xmlns:ex12m="http://schemas.microsoft.com/exchange/services/2006/messages" xmlns:pptsl="http://schemas.microsoft.com/sharepoint/soap/SlideLibrary/" xmlns:spsl="http://microsoft.com/webservices/SharePointPortalServer/PublishedLinksService" xmlns:Z="urn:schemas-microsoft-com:" xmlns:st="" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 12 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="blue" vlink="purple"> <div class="WordSection1"> Instead of rendering one glyph at a time when initially caching<o:p></o:p> them, cache as many as possible and then render. This results in<o:p></o:p> a modest speedup and should be even more noticeable on slow<o:p></o:p> GPUs.<o:p></o:p> <o:p> </o:p> This patch works on top of the following patch<o:p></o:p> <a href="http://lists.cairographics.org/archives/cairo/2011-November/022451.html">http://lists.cairographics.org/archives/cairo/2011-November/022451.html</a><o:p></o:p> <o:p> </o:p> <o:p> </o:p> ---<o:p></o:p> src/cairo-gl-glyphs.c |  160 +++++++++++++++++++++++++++++++++++--------------<o:p></o:p> 1 files changed, 114 insertions(+), 46 deletions(-)<o:p></o:p> <o:p> </o:p> diff --git a/src/cairo-gl-glyphs.c b/src/cairo-gl-glyphs.c<o:p></o:p> index 2324922..cdba3fa 100644<o:p></o:p> --- a/src/cairo-gl-glyphs.c<o:p></o:p> +++ b/src/cairo-gl-glyphs.c<o:p></o:p> @@ -213,6 +213,71 @@ _cairo_gl_font_fini (cairo_scaled_font_private_t *_priv,<o:p></o:p>      free (priv);<o:p></o:p> }<o:p></o:p> <o:p></o:p> +typedef struct {<o:p></o:p> +    cairo_scaled_glyph_t *scaled_glyph;<o:p></o:p> +    cairo_gl_glyph_t *glyph;<o:p></o:p> +    int info_offset;<o:p></o:p> +} renderable_glyph_info_t;<o:p></o:p> +<o:p></o:p> +static void<o:p></o:p> +render_cached_glyphs (cairo_gl_context_t *ctx,<o:p></o:p> +                 cairo_composite_glyphs_info_t *info,<o:p></o:p> +                 cairo_gl_composite_t setup,<o:p></o:p> +                 cairo_array_t *glyphs_to_render)<o:p></o:p> +{<o:p></o:p> +    int n_glyphs = _cairo_array_num_elements (glyphs_to_render);<o:p></o:p> +    int i;<o:p></o:p> +    renderable_glyph_info_t *glyphs = _cairo_array_index (glyphs_to_render, 0);<o:p></o:p> +<o:p></o:p> +    for (i = 0; i < n_glyphs; i++) {<o:p></o:p> +     double x_offset, y_offset, x1, y1, x2, y2;<o:p></o:p> +     cairo_scaled_glyph_t *scaled_glyph = glyphs[i].scaled_glyph;<o:p></o:p> +     cairo_gl_glyph_t *glyph = glyphs[i].glyph;<o:p></o:p> +     int info_offset = glyphs[i].info_offset;<o:p></o:p> +<o:p></o:p> +     x_offset = scaled_glyph->surface->base.device_transform.x0;<o:p></o:p> +     y_offset = scaled_glyph->surface->base.device_transform.y0;<o:p></o:p> +<o:p></o:p> +     x1 = _cairo_lround (info->glyphs[info_offset].x - x_offset);<o:p></o:p> +     y1 = _cairo_lround (info->glyphs[info_offset].y - y_offset);<o:p></o:p> +     x2 = x1 + scaled_glyph->surface->width;<o:p></o:p> +     y2 = y1 + scaled_glyph->surface->height;<o:p></o:p> +<o:p></o:p> +     _cairo_gl_composite_emit_glyph (ctx,<o:p></o:p> +                             x1, y1, x2, y2,<o:p></o:p> +                             glyph->p1.x, glyph->p1.y,<o:p></o:p> +                             glyph->p2.x, glyph->p2.y);<o:p></o:p> +    }<o:p></o:p> +<o:p></o:p> +    _cairo_array_truncate (glyphs_to_render, 0);<o:p></o:p> +}<o:p></o:p> +<o:p></o:p> +static cairo_status_t<o:p></o:p> +setup_glyph_cache_and_begin_composite (cairo_gl_context_t *ctx,<o:p></o:p> +                              cairo_gl_composite_t *setup,<o:p></o:p> +                              cairo_composite_glyphs_info_t *info,<o:p></o:p> +                              cairo_scaled_glyph_t *scaled_glyph,<o:p></o:p> +                              cairo_gl_glyph_cache_t **cache)<o:p></o:p> +{<o:p></o:p> +    cairo_int_status_t status;<o:p></o:p> +<o:p></o:p> +    status = cairo_gl_context_get_glyph_cache (ctx,<o:p></o:p> +                                    scaled_glyph->surface->format,<o:p></o:p> +                                               cache);<o:p></o:p> +<o:p></o:p> +    if (unlikely (_cairo_int_status_is_error (status)))<o:p></o:p> +     return status;<o:p></o:p> +<o:p></o:p> +    status = _cairo_gl_composite_set_mask (setup,<o:p></o:p> +                                &(*cache)->pattern.base,<o:p></o:p> +                                0, 0, 0, 0, 0, 0);<o:p></o:p> +    if (unlikely (_cairo_int_status_is_error (status)))<o:p></o:p> +     return status;<o:p></o:p> +<o:p></o:p> +    status = _cairo_gl_composite_begin (setup, &ctx);<o:p></o:p> +    return status;<o:p></o:p> +}<o:p></o:p> +<o:p></o:p> static cairo_status_t<o:p></o:p> render_glyphs (cairo_gl_surface_t *dst,<o:p></o:p>             int dst_x, int dst_y,<o:p></o:p> @@ -229,15 +294,21 @@ render_glyphs (cairo_gl_surface_t *dst,<o:p></o:p>      cairo_int_status_t status;<o:p></o:p>      int i = 0;<o:p></o:p> <o:p></o:p> +    cairo_array_t glyphs_to_render;<o:p></o:p> +<o:p></o:p>      *has_component_alpha = FALSE;<o:p></o:p> <o:p></o:p> +    /* FIXME: The caches are stored as properties of the context, but we need<o:p></o:p> +       to read the cache before we can call _cairo_gl_composite_begin,<o:p></o:p> +       which also acquires the device context. We end up acquiring it twice<o:p></o:p> +       to work around this. */<o:p></o:p>      status = _cairo_gl_context_acquire (dst->base.device, &ctx);<o:p></o:p> -    if (unlikely (status))<o:p></o:p> -     return status;<o:p></o:p> +    if (unlikely (_cairo_int_status_is_error (status)))<o:p></o:p> +        return status;<o:p></o:p> <o:p></o:p>      status = _cairo_gl_composite_init (&setup, op, dst, TRUE, &info->extents);<o:p></o:p>      if (unlikely (status))<o:p></o:p> -     goto FINISH;<o:p></o:p> +     goto finish;<o:p></o:p> <o:p></o:p>      if (source_pattern) {<o:p></o:p>      status = _cairo_gl_composite_set_source (&setup,<o:p></o:p> @@ -248,7 +319,7 @@ render_glyphs (cairo_gl_surface_t *dst,<o:p></o:p>                                     info->extents.width,<o:p></o:p>                                     info->extents.height);<o:p></o:p>      if (unlikely (status))<o:p></o:p> -         goto FINISH;<o:p></o:p> +         goto finish;<o:p></o:p>      } else {<o:p></o:p>      _cairo_gl_composite_set_source_operand (&setup,<o:p></o:p>          &((cairo_gl_source_t *) source_surface)->operand);<o:p></o:p> @@ -256,49 +327,44 @@ render_glyphs (cairo_gl_surface_t *dst,<o:p></o:p> <o:p></o:p>      //_cairo_gl_composite_set_clip_region (&setup, _cairo_clip_get_region (extents->clip));<o:p></o:p> <o:p></o:p> +    _cairo_array_init (&glyphs_to_render, sizeof (renderable_glyph_info_t));<o:p></o:p> +<o:p></o:p>      for (i = 0; i < info->num_glyphs; i++) {<o:p></o:p> +     renderable_glyph_info_t item;<o:p></o:p>      cairo_scaled_glyph_t *scaled_glyph;<o:p></o:p> -     cairo_gl_glyph_t *glyph;<o:p></o:p> -     double x_offset, y_offset;<o:p></o:p> -     double x1, x2, y1, y2;<o:p></o:p> <o:p></o:p>       status = _cairo_scaled_glyph_lookup (info->font,<o:p></o:p>                                   info->glyphs[i].index,<o:p></o:p>                                   CAIRO_SCALED_GLYPH_INFO_SURFACE,<o:p></o:p>                                   &scaled_glyph);<o:p></o:p> -     if (unlikely (status))<o:p></o:p> -         goto FINISH;<o:p></o:p> +     if (unlikely (_cairo_int_status_is_error (status)))<o:p></o:p> +         return status;<o:p></o:p> <o:p></o:p>       if (scaled_glyph->surface->width  == 0 ||<o:p></o:p>          scaled_glyph->surface->height == 0)<o:p></o:p>      {<o:p></o:p>          continue;<o:p></o:p>      }<o:p></o:p> +<o:p></o:p>      if (scaled_glyph->surface->format != last_format) {<o:p></o:p> -         status = cairo_gl_context_get_glyph_cache (ctx,<o:p></o:p> -                                          scaled_glyph->surface->format,<o:p></o:p> -                                                       &cache);<o:p></o:p> -            if (unlikely (status))<o:p></o:p> -                goto FINISH;<o:p></o:p> +         /* If we need to change the operands in the middle of run,<o:p></o:p> +            then first render all glyphs we've cached so far and then continue. */<o:p></o:p> +         if (i != 0) {<o:p></o:p> +           render_cached_glyphs (ctx, info, setup, &glyphs_to_render);<o:p></o:p> +           _cairo_gl_composite_flush (ctx);<o:p></o:p> +           status = _cairo_gl_context_release (ctx, status);<o:p></o:p> +           ctx = NULL;<o:p></o:p> +           if (unlikely (_cairo_int_status_is_error (status)))<o:p></o:p> +               goto finish;<o:p></o:p> +         }<o:p></o:p> <o:p></o:p>           last_format = scaled_glyph->surface->format;<o:p></o:p> -<o:p></o:p> -            status = _cairo_gl_composite_set_mask (&setup,<o:p></o:p> -                                                   &cache->pattern.base,<o:p></o:p> -                                                   0, 0, 0, 0, 0, 0);<o:p></o:p> -            if (unlikely (status))<o:p></o:p> -                goto FINISH;<o:p></o:p> +         setup_glyph_cache_and_begin_composite (ctx, &setup, info,<o:p></o:p> +                                      scaled_glyph, &cache);<o:p></o:p> +         if (unlikely (_cairo_int_status_is_error (status)))<o:p></o:p> +           goto finish;<o:p></o:p> <o:p></o:p>           *has_component_alpha |= cache->pattern.base.has_component_alpha;<o:p></o:p> -<o:p></o:p> -            /* XXX: _cairo_gl_composite_begin() acquires the context a<o:p></o:p> -             * second time. Need to refactor this loop so this doesn't happen.<o:p></o:p> -             */<o:p></o:p> -            status = _cairo_gl_composite_begin (&setup, &ctx);<o:p></o:p> -<o:p></o:p> -            status = _cairo_gl_context_release (ctx, status);<o:p></o:p> -         if (unlikely (status))<o:p></o:p> -           goto FINISH;<o:p></o:p>      }<o:p></o:p> <o:p></o:p>       if (scaled_glyph->dev_private_key != cache) {<o:p></o:p> @@ -314,34 +380,36 @@ render_glyphs (cairo_gl_surface_t *dst,<o:p></o:p>            status = _cairo_gl_glyph_cache_add_glyph (ctx, cache, scaled_glyph);<o:p></o:p> <o:p></o:p>             if (status == CAIRO_INT_STATUS_UNSUPPORTED) {<o:p></o:p> -               /* Cache is full, so flush existing prims and try again. */<o:p></o:p> -               _cairo_gl_composite_flush (ctx);<o:p></o:p> +               /* If the cache is full in the middle of run, then first flush<o:p></o:p> +                * all glyphs we've cached so far and then continue. */<o:p></o:p> +               if (i != 0) {<o:p></o:p> +                 render_cached_glyphs (ctx, info, setup, &glyphs_to_render);<o:p></o:p> +                 _cairo_gl_composite_flush (ctx);<o:p></o:p> +               }<o:p></o:p> +<o:p></o:p> +               /* Now try adding the glyph to cache, allowing replacements. */<o:p></o:p>                _cairo_gl_glyph_cache_unlock (cache);<o:p></o:p>                status = _cairo_gl_glyph_cache_add_glyph (ctx, cache, scaled_glyph);<o:p></o:p>            }<o:p></o:p> <o:p></o:p>             if (unlikely (_cairo_int_status_is_error (status)))<o:p></o:p> -               goto FINISH;<o:p></o:p> +               goto finish;<o:p></o:p>          }<o:p></o:p>      }<o:p></o:p> <o:p></o:p> -     x_offset = scaled_glyph->surface->base.device_transform.x0;<o:p></o:p> -     y_offset = scaled_glyph->surface->base.device_transform.y0;<o:p></o:p> +     item.scaled_glyph = scaled_glyph;<o:p></o:p> +     item.glyph = _cairo_gl_glyph_cache_lock (cache, scaled_glyph);<o:p></o:p> +     item.info_offset = i;<o:p></o:p> +     status = _cairo_array_append (&glyphs_to_render, &item);<o:p></o:p> +    }<o:p></o:p> <o:p></o:p> -     x1 = _cairo_lround (info->glyphs[i].x - x_offset);<o:p></o:p> -     y1 = _cairo_lround (info->glyphs[i].y - y_offset);<o:p></o:p> -     x2 = x1 + scaled_glyph->surface->width;<o:p></o:p> -     y2 = y1 + scaled_glyph->surface->height;<o:p></o:p> +    render_cached_glyphs (ctx, info, setup, &glyphs_to_render);<o:p></o:p> <o:p></o:p> -     glyph = _cairo_gl_glyph_cache_lock (cache, scaled_glyph);<o:p></o:p> -     _cairo_gl_composite_emit_glyph (ctx,<o:p></o:p> -                             x1, y1, x2, y2,<o:p></o:p> -                                        glyph->p1.x, glyph->p1.y,<o:p></o:p> -                                        glyph->p2.x, glyph->p2.y);<o:p></o:p> -    }<o:p></o:p> +finish:<o:p></o:p> +    _cairo_array_fini (&glyphs_to_render);<o:p></o:p> <o:p></o:p> -    status = CAIRO_STATUS_SUCCESS;<o:p></o:p> -  FINISH:<o:p></o:p> +    /* FIXME: See above. */<o:p></o:p> +    status = _cairo_gl_context_release (ctx, status);<o:p></o:p>      status = _cairo_gl_context_release (ctx, status);<o:p></o:p> <o:p></o:p>      _cairo_gl_composite_fini (&setup);<o:p></o:p> -- <o:p></o:p> 1.7.5.4<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> </div> </body> </html>