st/xa: Support Component Alpha with trivial blending
authorThomas Hellstrom <thellstrom@vmware.com>
Tue, 13 Nov 2018 13:10:56 +0000 (14:10 +0100)
committerThomas Hellstrom <thellstrom@vmware.com>
Wed, 14 Nov 2018 12:12:09 +0000 (13:12 +0100)
Support Component Alpha for those composite operations that do not require
per-channel alpha blending.

Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Sinclair Yeh <syeh@vmware.com>
src/gallium/state_trackers/xa/xa_composite.c
src/gallium/state_trackers/xa/xa_priv.h
src/gallium/state_trackers/xa/xa_tgsi.c

index b074632752210d7c42d8d2810c5200071e376250..34d78027e27eff9faa71574f9b5190d9e45c9699 100644 (file)
@@ -111,12 +111,6 @@ blend_for_op(struct xa_composite_blend *blend,
     int i;
     boolean supported = FALSE;
 
-    /*
-     * No component alpha yet.
-     */
-    if (mask_pic && mask_pic->component_alpha)
-       return FALSE;
-
     /*
      * our default in case something goes wrong
      */
@@ -130,6 +124,12 @@ blend_for_op(struct xa_composite_blend *blend,
        }
     }
 
+    /*
+     * No component alpha yet.
+     */
+    if (mask_pic && mask_pic->component_alpha && blend->alpha_src)
+       return FALSE;
+
     if (!dst_pic->srf)
        return supported;
 
@@ -224,15 +224,9 @@ xa_src_pict_is_accelerated(const union xa_source_pict *src_pic)
 XA_EXPORT int
 xa_composite_check_accelerated(const struct xa_composite *comp)
 {
-    struct xa_composite_blend blend;
     struct xa_picture *src_pic = comp->src;
     struct xa_picture *mask_pic = comp->mask;
-
-    /*
-     * No component alpha yet.
-     */
-    if (mask_pic && mask_pic->component_alpha)
-       return -XA_ERR_INVAL;
+    struct xa_composite_blend blend;
 
     if (!xa_is_filter_accelerated(src_pic) ||
        !xa_is_filter_accelerated(comp->mask)) {
@@ -246,6 +240,12 @@ xa_composite_check_accelerated(const struct xa_composite *comp)
     if (!blend_for_op(&blend, comp->op, comp->src, comp->mask, comp->dst))
        return -XA_ERR_INVAL;
 
+    /*
+     * No component alpha yet.
+     */
+    if (mask_pic && mask_pic->component_alpha && blend.alpha_src)
+       return -XA_ERR_INVAL;
+
     return XA_ERR_NONE;
 }
 
@@ -382,10 +382,15 @@ bind_shaders(struct xa_context *ctx, const struct xa_composite *comp)
     struct xa_shader shader;
     struct xa_picture *src_pic = comp->src;
     struct xa_picture *mask_pic = comp->mask;
+    struct xa_picture *dst_pic = comp->dst;
 
     ctx->has_solid_src = FALSE;
     ctx->has_solid_mask = FALSE;
 
+    if (dst_pic && xa_format_type(dst_pic->pict_format) !=
+        xa_format_type(xa_surface_format(dst_pic->srf)))
+       return -XA_ERR_INVAL;
+
     if (src_pic) {
        if (src_pic->wrap == xa_wrap_clamp_to_border && src_pic->has_transform)
            fs_traits |= FS_SRC_REPEAT_NONE;
@@ -405,6 +410,8 @@ bind_shaders(struct xa_context *ctx, const struct xa_composite *comp)
     if (mask_pic) {
        vs_traits |= VS_MASK;
        fs_traits |= FS_MASK;
+        if (mask_pic->component_alpha)
+           fs_traits |= FS_CA;
         if (mask_pic->src_pict) {
             if (!xa_handle_src_pict(ctx, mask_pic->src_pict, true))
                 return -XA_ERR_INVAL;
index 09a858ff972c36d577052d3d1a5c97fc3a091171..f368de3b81f52dfdeaff9de3632e0d60f3534a9f 100644 (file)
@@ -166,6 +166,7 @@ enum xa_fs_traits {
     FS_SRC_LUMINANCE = 1 << 11,
     FS_MASK_LUMINANCE = 1 << 12,
     FS_DST_LUMINANCE = 1 << 13,
+    FS_CA = 1 << 14,
 };
 
 struct xa_shader {
index 5f2608aee55f0ad4b037ccc784a20f82a15053a9..ed3e0895d98931ba0ee9aa6c6be4de7fcd8c04b1 100644 (file)
@@ -82,6 +82,7 @@ print_fs_traits(int fs_traits)
        "FS_SRC_LUMINANCE",     /* = 1 << 11, */
        "FS_MASK_LUMINANCE",    /* = 1 << 12, */
        "FS_DST_LUMINANCE",     /* = 1 << 13, */
+        "FS_CA",                /* = 1 << 14, */
     };
     int i, k;
 
@@ -107,12 +108,20 @@ src_in_mask(struct ureg_program *ureg,
            struct ureg_dst dst,
            struct ureg_src src,
            struct ureg_src mask,
-           unsigned mask_luminance)
+           unsigned mask_luminance, boolean component_alpha)
 {
     if (mask_luminance)
-        ureg_MUL(ureg, dst, src, ureg_scalar(mask, TGSI_SWIZZLE_X));
-    else
+        if (component_alpha) {
+            ureg_MOV(ureg, dst, src);
+            ureg_MUL(ureg, ureg_writemask(dst, TGSI_WRITEMASK_W),
+                     src, ureg_scalar(mask, TGSI_SWIZZLE_X));
+        } else {
+            ureg_MUL(ureg, dst, src, ureg_scalar(mask, TGSI_SWIZZLE_X));
+        }
+    else if (!component_alpha)
         ureg_MUL(ureg, dst, src, ureg_scalar(mask, TGSI_SWIZZLE_W));
+    else
+        ureg_MUL(ureg, dst, src, mask);
 }
 
 static struct ureg_src
@@ -347,6 +356,7 @@ create_fs(struct pipe_context *pipe, unsigned fs_traits)
     unsigned dst_luminance = (fs_traits & FS_DST_LUMINANCE) != 0;
     unsigned is_src_src = (fs_traits & FS_SRC_SRC) != 0;
     unsigned is_mask_src = (fs_traits & FS_MASK_SRC) != 0;
+    boolean component_alpha = (fs_traits & FS_CA) != 0;
     unsigned cur_sampler = 0;
     unsigned cur_constant = 0;
 
@@ -391,7 +401,7 @@ create_fs(struct pipe_context *pipe, unsigned fs_traits)
                    &cur_sampler);
 
        src_in_mask(ureg, (dst_luminance) ? src : out, ureg_src(src),
-                   ureg_src(mask), mask_luminance);
+                   ureg_src(mask), mask_luminance, component_alpha);
 
        ureg_release_temporary(ureg, mask);
     }