[Mesa-dev] [PATCH 01/23] radeonsi: move all clear() code into si_clear.c

Nicolai Hähnle nhaehnle at gmail.com
Wed Nov 29 09:32:39 UTC 2017


On 29.11.2017 10:29, Nicolai Hähnle wrote:
> On 28.11.2017 22:38, Marek Olšák wrote:
>> From: Marek Olšák <marek.olsak at amd.com>
> 
> The commit message should mention that this also contains some 
> preliminary work for MSAA DCC.

Unless you plan to land this after the series, that is? I was comparing 
to the code in master...


> 
> Cheers,
> Nicolai
> 
> 
>>
>> ---
>>   src/gallium/drivers/radeon/r600_pipe_common.h |  13 +-
>>   src/gallium/drivers/radeon/r600_texture.c     | 547 +------------------
>>   src/gallium/drivers/radeonsi/Makefile.sources |   1 +
>>   src/gallium/drivers/radeonsi/meson.build      |   1 +
>>   src/gallium/drivers/radeonsi/si_blit.c        | 170 +-----
>>   src/gallium/drivers/radeonsi/si_clear.c       | 725 
>> ++++++++++++++++++++++++++
>>   src/gallium/drivers/radeonsi/si_cp_dma.c      |   6 +-
>>   src/gallium/drivers/radeonsi/si_pipe.c        |   1 +
>>   src/gallium/drivers/radeonsi/si_pipe.h        |  19 +
>>   9 files changed, 764 insertions(+), 719 deletions(-)
>>   create mode 100644 src/gallium/drivers/radeonsi/si_clear.c
>>
>> diff --git a/src/gallium/drivers/radeon/r600_pipe_common.h 
>> b/src/gallium/drivers/radeon/r600_pipe_common.h
>> index 2ece409..36df0c4 100644
>> --- a/src/gallium/drivers/radeon/r600_pipe_common.h
>> +++ b/src/gallium/drivers/radeon/r600_pipe_common.h
>> @@ -728,20 +728,23 @@ bool si_prepare_for_dma_blit(struct 
>> r600_common_context *rctx,
>>                    struct r600_texture *rdst,
>>                    unsigned dst_level, unsigned dstx,
>>                    unsigned dsty, unsigned dstz,
>>                    struct r600_texture *rsrc,
>>                    unsigned src_level,
>>                    const struct pipe_box *src_box);
>>   void si_texture_get_fmask_info(struct r600_common_screen *rscreen,
>>                      struct r600_texture *rtex,
>>                      unsigned nr_samples,
>>                      struct r600_fmask_info *out);
>> +void si_texture_get_cmask_info(struct r600_common_screen *rscreen,
>> +                   struct r600_texture *rtex,
>> +                   struct r600_cmask_info *out);
>>   bool si_init_flushed_depth_texture(struct pipe_context *ctx,
>>                      struct pipe_resource *texture,
>>                      struct r600_texture **staging);
>>   void si_print_texture_info(struct r600_common_screen *rscreen,
>>                  struct r600_texture *rtex, struct u_log_context *log);
>>   struct pipe_resource *si_texture_create(struct pipe_screen *screen,
>>                       const struct pipe_resource *templ);
>>   bool vi_dcc_formats_compatible(enum pipe_format format1,
>>                      enum pipe_format format2);
>>   bool vi_dcc_formats_are_incompatible(struct pipe_resource *tex,
>> @@ -750,34 +753,28 @@ bool vi_dcc_formats_are_incompatible(struct 
>> pipe_resource *tex,
>>   void vi_disable_dcc_if_incompatible_format(struct 
>> r600_common_context *rctx,
>>                          struct pipe_resource *tex,
>>                          unsigned level,
>>                          enum pipe_format view_format);
>>   struct pipe_surface *si_create_surface_custom(struct pipe_context 
>> *pipe,
>>                             struct pipe_resource *texture,
>>                             const struct pipe_surface *templ,
>>                             unsigned width0, unsigned height0,
>>                             unsigned width, unsigned height);
>>   unsigned si_translate_colorswap(enum pipe_format format, bool 
>> do_endian_swap);
>> +void vi_separate_dcc_try_enable(struct r600_common_context *rctx,
>> +                struct r600_texture *tex);
>>   void vi_separate_dcc_start_query(struct pipe_context *ctx,
>>                    struct r600_texture *tex);
>>   void vi_separate_dcc_stop_query(struct pipe_context *ctx,
>>                   struct r600_texture *tex);
>>   void vi_separate_dcc_process_and_reset_stats(struct pipe_context *ctx,
>>                            struct r600_texture *tex);
>> -void vi_dcc_clear_level(struct r600_common_context *rctx,
>> -            struct r600_texture *rtex,
>> -            unsigned level, unsigned clear_value);
>> -void si_do_fast_color_clear(struct r600_common_context *rctx,
>> -                struct pipe_framebuffer_state *fb,
>> -                struct r600_atom *fb_state,
>> -                unsigned *buffers, ubyte *dirty_cbufs,
>> -                const union pipe_color_union *color);
>>   bool si_texture_disable_dcc(struct r600_common_context *rctx,
>>                   struct r600_texture *rtex);
>>   void si_init_screen_texture_functions(struct r600_common_screen 
>> *rscreen);
>>   void si_init_context_texture_functions(struct r600_common_context 
>> *rctx);
>>   /* Inline helpers. */
>>   static inline struct r600_resource *r600_resource(struct 
>> pipe_resource *r)
>>   {
>> diff --git a/src/gallium/drivers/radeon/r600_texture.c 
>> b/src/gallium/drivers/radeon/r600_texture.c
>> index e3658b4..46962eb 100644
>> --- a/src/gallium/drivers/radeon/r600_texture.c
>> +++ b/src/gallium/drivers/radeon/r600_texture.c
>> @@ -770,23 +770,23 @@ void si_texture_get_fmask_info(struct 
>> r600_common_screen *rscreen,
>>   static void r600_texture_allocate_fmask(struct r600_common_screen 
>> *rscreen,
>>                       struct r600_texture *rtex)
>>   {
>>       si_texture_get_fmask_info(rscreen, rtex,
>>                       rtex->resource.b.b.nr_samples, &rtex->fmask);
>>       rtex->fmask.offset = align64(rtex->size, rtex->fmask.alignment);
>>       rtex->size = rtex->fmask.offset + rtex->fmask.size;
>>   }
>> -static void si_texture_get_cmask_info(struct r600_common_screen 
>> *rscreen,
>> -                      struct r600_texture *rtex,
>> -                      struct r600_cmask_info *out)
>> +void si_texture_get_cmask_info(struct r600_common_screen *rscreen,
>> +                   struct r600_texture *rtex,
>> +                   struct r600_cmask_info *out)
>>   {
>>       unsigned pipe_interleave_bytes = 
>> rscreen->info.pipe_interleave_bytes;
>>       unsigned num_pipes = rscreen->info.num_tile_pipes;
>>       unsigned cl_width, cl_height;
>>       if (rscreen->chip_class >= GFX9) {
>>           out->alignment = rtex->surface.u.gfx9.cmask_alignment;
>>           out->size = rtex->surface.u.gfx9.cmask_size;
>>           return;
>>       }
>> @@ -835,51 +835,20 @@ static void r600_texture_allocate_cmask(struct 
>> r600_common_screen *rscreen,
>>                       struct r600_texture *rtex)
>>   {
>>       si_texture_get_cmask_info(rscreen, rtex, &rtex->cmask);
>>       rtex->cmask.offset = align64(rtex->size, rtex->cmask.alignment);
>>       rtex->size = rtex->cmask.offset + rtex->cmask.size;
>>       rtex->cb_color_info |= S_028C70_FAST_CLEAR(1);
>>   }
>> -static void r600_texture_alloc_cmask_separate(struct 
>> r600_common_screen *rscreen,
>> -                          struct r600_texture *rtex)
>> -{
>> -    if (rtex->cmask_buffer)
>> -                return;
>> -
>> -    assert(rtex->cmask.size == 0);
>> -
>> -    si_texture_get_cmask_info(rscreen, rtex, &rtex->cmask);
>> -    if (!rtex->cmask.size)
>> -        return;
>> -
>> -    rtex->cmask_buffer = (struct r600_resource *)
>> -        si_aligned_buffer_create(&rscreen->b,
>> -                       R600_RESOURCE_FLAG_UNMAPPABLE,
>> -                       PIPE_USAGE_DEFAULT,
>> -                       rtex->cmask.size,
>> -                       rtex->cmask.alignment);
>> -    if (rtex->cmask_buffer == NULL) {
>> -        rtex->cmask.size = 0;
>> -        return;
>> -    }
>> -
>> -    /* update colorbuffer state bits */
>> -    rtex->cmask.base_address_reg = rtex->cmask_buffer->gpu_address >> 8;
>> -
>> -    rtex->cb_color_info |= S_028C70_FAST_CLEAR(1);
>> -
>> -    p_atomic_inc(&rscreen->compressed_colortex_counter);
>> -}
>> -
>>   static void r600_texture_get_htile_size(struct r600_common_screen 
>> *rscreen,
>>                       struct r600_texture *rtex)
>>   {
>>       unsigned cl_width, cl_height, width, height;
>>       unsigned slice_elements, slice_bytes, pipe_interleave_bytes, 
>> base_align;
>>       unsigned num_pipes = rscreen->info.num_tile_pipes;
>>       assert(rscreen->chip_class <= VI);
>>       rtex->surface.htile_size = 0;
>> @@ -1966,85 +1935,20 @@ static struct pipe_surface 
>> *r600_create_surface(struct pipe_context *pipe,
>>                         width, height);
>>   }
>>   static void r600_surface_destroy(struct pipe_context *pipe,
>>                    struct pipe_surface *surface)
>>   {
>>       pipe_resource_reference(&surface->texture, NULL);
>>       FREE(surface);
>>   }
>> -static void r600_clear_texture(struct pipe_context *pipe,
>> -                   struct pipe_resource *tex,
>> -                   unsigned level,
>> -                   const struct pipe_box *box,
>> -                   const void *data)
>> -{
>> -    struct pipe_screen *screen = pipe->screen;
>> -    struct r600_texture *rtex = (struct r600_texture*)tex;
>> -    struct pipe_surface tmpl = {{0}};
>> -    struct pipe_surface *sf;
>> -    const struct util_format_description *desc =
>> -        util_format_description(tex->format);
>> -
>> -    tmpl.format = tex->format;
>> -    tmpl.u.tex.first_layer = box->z;
>> -    tmpl.u.tex.last_layer = box->z + box->depth - 1;
>> -    tmpl.u.tex.level = level;
>> -    sf = pipe->create_surface(pipe, tex, &tmpl);
>> -    if (!sf)
>> -        return;
>> -
>> -    if (rtex->is_depth) {
>> -        unsigned clear;
>> -        float depth;
>> -        uint8_t stencil = 0;
>> -
>> -        /* Depth is always present. */
>> -        clear = PIPE_CLEAR_DEPTH;
>> -        desc->unpack_z_float(&depth, 0, data, 0, 1, 1);
>> -
>> -        if (rtex->surface.has_stencil) {
>> -            clear |= PIPE_CLEAR_STENCIL;
>> -            desc->unpack_s_8uint(&stencil, 0, data, 0, 1, 1);
>> -        }
>> -
>> -        pipe->clear_depth_stencil(pipe, sf, clear, depth, stencil,
>> -                      box->x, box->y,
>> -                      box->width, box->height, false);
>> -    } else {
>> -        union pipe_color_union color;
>> -
>> -        /* pipe_color_union requires the full vec4 representation. */
>> -        if (util_format_is_pure_uint(tex->format))
>> -            desc->unpack_rgba_uint(color.ui, 0, data, 0, 1, 1);
>> -        else if (util_format_is_pure_sint(tex->format))
>> -            desc->unpack_rgba_sint(color.i, 0, data, 0, 1, 1);
>> -        else
>> -            desc->unpack_rgba_float(color.f, 0, data, 0, 1, 1);
>> -
>> -        if (screen->is_format_supported(screen, tex->format,
>> -                        tex->target, 0,
>> -                        PIPE_BIND_RENDER_TARGET)) {
>> -            pipe->clear_render_target(pipe, sf, &color,
>> -                          box->x, box->y,
>> -                          box->width, box->height, false);
>> -        } else {
>> -            /* Software fallback - just for R9G9B9E5_FLOAT */
>> -            util_clear_render_target(pipe, sf, &color,
>> -                         box->x, box->y,
>> -                         box->width, box->height);
>> -        }
>> -    }
>> -    pipe_surface_reference(&sf, NULL);
>> -}
>> -
>>   unsigned si_translate_colorswap(enum pipe_format format, bool 
>> do_endian_swap)
>>   {
>>       const struct util_format_description *desc = 
>> util_format_description(format);
>>   #define HAS_SWIZZLE(chan,swz) (desc->swizzle[chan] == 
>> PIPE_SWIZZLE_##swz)
>>       if (format == PIPE_FORMAT_R11G11B10_FLOAT) /* isn't plain */
>>           return V_028C70_SWAP_STD;
>>       if (desc->layout != UTIL_FORMAT_LAYOUT_PLAIN)
>> @@ -2214,22 +2118,22 @@ void vi_separate_dcc_stop_query(struct 
>> pipe_context *ctx,
>>   }
>>   static bool vi_should_enable_separate_dcc(struct r600_texture *tex)
>>   {
>>       /* The minimum number of fullscreen draws per frame that is 
>> required
>>        * to enable DCC. */
>>       return tex->ps_draw_ratio + tex->num_slow_clears >= 5;
>>   }
>>   /* Called by fast clear. */
>> -static void vi_separate_dcc_try_enable(struct r600_common_context *rctx,
>> -                       struct r600_texture *tex)
>> +void vi_separate_dcc_try_enable(struct r600_common_context *rctx,
>> +                struct r600_texture *tex)
>>   {
>>       /* The intent is to use this with shared displayable back buffers,
>>        * but it's not strictly limited only to them.
>>        */
>>       if (!tex->resource.b.is_shared ||
>>           !(tex->resource.external_usage & 
>> PIPE_HANDLE_USAGE_EXPLICIT_FLUSH) ||
>>           tex->resource.b.b.target != PIPE_TEXTURE_2D ||
>>           tex->resource.b.b.last_level > 0 ||
>>           !tex->surface.dcc_size)
>>           return;
>> @@ -2329,460 +2233,20 @@ void 
>> vi_separate_dcc_process_and_reset_stats(struct pipe_context *ctx,
>>           assert(!tex->last_dcc_separate_buffer);
>>           tex->last_dcc_separate_buffer = tex->dcc_separate_buffer;
>>           tex->dcc_separate_buffer = NULL;
>>           tex->dcc_offset = 0;
>>           /* no need to flag anything since this is called after
>>            * decompression that re-sets framebuffer state
>>            */
>>       }
>>   }
>> -/* FAST COLOR CLEAR */
>> -
>> -static void evergreen_set_clear_color(struct r600_texture *rtex,
>> -                      enum pipe_format surface_format,
>> -                      const union pipe_color_union *color)
>> -{
>> -    union util_color uc;
>> -
>> -    memset(&uc, 0, sizeof(uc));
>> -
>> -    if (rtex->surface.bpe == 16) {
>> -        /* DCC fast clear only:
>> -         *   CLEAR_WORD0 = R = G = B
>> -         *   CLEAR_WORD1 = A
>> -         */
>> -        assert(color->ui[0] == color->ui[1] &&
>> -               color->ui[0] == color->ui[2]);
>> -        uc.ui[0] = color->ui[0];
>> -        uc.ui[1] = color->ui[3];
>> -    } else if (util_format_is_pure_uint(surface_format)) {
>> -        util_format_write_4ui(surface_format, color->ui, 0, &uc, 0, 
>> 0, 0, 1, 1);
>> -    } else if (util_format_is_pure_sint(surface_format)) {
>> -        util_format_write_4i(surface_format, color->i, 0, &uc, 0, 0, 
>> 0, 1, 1);
>> -    } else {
>> -        util_pack_color(color->f, surface_format, &uc);
>> -    }
>> -
>> -    memcpy(rtex->color_clear_value, &uc, 2 * sizeof(uint32_t));
>> -}
>> -
>> -static bool vi_get_fast_clear_parameters(enum pipe_format 
>> surface_format,
>> -                     const union pipe_color_union *color,
>> -                     uint32_t* reset_value,
>> -                     bool* clear_words_needed)
>> -{
>> -    bool values[4] = {};
>> -    int i;
>> -    bool main_value = false;
>> -    bool extra_value = false;
>> -    int extra_channel;
>> -
>> -    /* This is needed to get the correct DCC clear value for 
>> luminance formats.
>> -     * 1) Get the linear format (because the next step can't handle 
>> L8_SRGB).
>> -     * 2) Convert luminance to red. (the real hw format for luminance)
>> -     */
>> -    surface_format = util_format_linear(surface_format);
>> -    surface_format = util_format_luminance_to_red(surface_format);
>> -
>> -    const struct util_format_description *desc = 
>> util_format_description(surface_format);
>> -
>> -    if (desc->block.bits == 128 &&
>> -        (color->ui[0] != color->ui[1] ||
>> -         color->ui[0] != color->ui[2]))
>> -        return false;
>> -
>> -    *clear_words_needed = true;
>> -    *reset_value = 0x20202020U;
>> -
>> -    /* If we want to clear without needing a fast clear eliminate 
>> step, we
>> -     * can set each channel to 0 or 1 (or 0/max for integer formats). We
>> -     * have two sets of flags, one for the last or first 
>> channel(extra) and
>> -     * one for the other channels(main).
>> -     */
>> -
>> -    if (surface_format == PIPE_FORMAT_R11G11B10_FLOAT ||
>> -        surface_format == PIPE_FORMAT_B5G6R5_UNORM ||
>> -        surface_format == PIPE_FORMAT_B5G6R5_SRGB ||
>> -        util_format_is_alpha(surface_format)) {
>> -        extra_channel = -1;
>> -    } else if (desc->layout == UTIL_FORMAT_LAYOUT_PLAIN) {
>> -        if(si_translate_colorswap(surface_format, false) <= 1)
>> -            extra_channel = desc->nr_channels - 1;
>> -        else
>> -            extra_channel = 0;
>> -    } else
>> -        return true;
>> -
>> -    for (i = 0; i < 4; ++i) {
>> -        int index = desc->swizzle[i] - PIPE_SWIZZLE_X;
>> -
>> -        if (desc->swizzle[i] < PIPE_SWIZZLE_X ||
>> -            desc->swizzle[i] > PIPE_SWIZZLE_W)
>> -            continue;
>> -
>> -        if (desc->channel[i].pure_integer &&
>> -            desc->channel[i].type == UTIL_FORMAT_TYPE_SIGNED) {
>> -            /* Use the maximum value for clamping the clear color. */
>> -            int max = u_bit_consecutive(0, desc->channel[i].size - 1);
>> -
>> -            values[i] = color->i[i] != 0;
>> -            if (color->i[i] != 0 && MIN2(color->i[i], max) != max)
>> -                return true;
>> -        } else if (desc->channel[i].pure_integer &&
>> -               desc->channel[i].type == UTIL_FORMAT_TYPE_UNSIGNED) {
>> -            /* Use the maximum value for clamping the clear color. */
>> -            unsigned max = u_bit_consecutive(0, desc->channel[i].size);
>> -
>> -            values[i] = color->ui[i] != 0U;
>> -            if (color->ui[i] != 0U && MIN2(color->ui[i], max) != max)
>> -                return true;
>> -        } else {
>> -            values[i] = color->f[i] != 0.0F;
>> -            if (color->f[i] != 0.0F && color->f[i] != 1.0F)
>> -                return true;
>> -        }
>> -
>> -        if (index == extra_channel)
>> -            extra_value = values[i];
>> -        else
>> -            main_value = values[i];
>> -    }
>> -
>> -    for (int i = 0; i < 4; ++i)
>> -        if (values[i] != main_value &&
>> -            desc->swizzle[i] - PIPE_SWIZZLE_X != extra_channel &&
>> -            desc->swizzle[i] >= PIPE_SWIZZLE_X &&
>> -            desc->swizzle[i] <= PIPE_SWIZZLE_W)
>> -            return true;
>> -
>> -    *clear_words_needed = false;
>> -    if (main_value)
>> -        *reset_value |= 0x80808080U;
>> -
>> -    if (extra_value)
>> -        *reset_value |= 0x40404040U;
>> -    return true;
>> -}
>> -
>> -void vi_dcc_clear_level(struct r600_common_context *rctx,
>> -            struct r600_texture *rtex,
>> -            unsigned level, unsigned clear_value)
>> -{
>> -    struct pipe_resource *dcc_buffer;
>> -    uint64_t dcc_offset, clear_size;
>> -
>> -    assert(vi_dcc_enabled(rtex, level));
>> -
>> -    if (rtex->dcc_separate_buffer) {
>> -        dcc_buffer = &rtex->dcc_separate_buffer->b.b;
>> -        dcc_offset = 0;
>> -    } else {
>> -        dcc_buffer = &rtex->resource.b.b;
>> -        dcc_offset = rtex->dcc_offset;
>> -    }
>> -
>> -    if (rctx->chip_class >= GFX9) {
>> -        /* Mipmap level clears aren't implemented. */
>> -        assert(rtex->resource.b.b.last_level == 0);
>> -        /* MSAA needs a different clear size. */
>> -        assert(rtex->resource.b.b.nr_samples <= 1);
>> -        clear_size = rtex->surface.dcc_size;
>> -    } else {
>> -        unsigned num_layers = util_max_layer(&rtex->resource.b.b, 
>> level) + 1;
>> -
>> -        /* If this is 0, fast clear isn't possible. (can occur with 
>> MSAA) */
>> -        assert(rtex->surface.u.legacy.level[level].dcc_fast_clear_size);
>> -        /* Layered MSAA DCC fast clears need to clear 
>> dcc_fast_clear_size
>> -         * bytes for each layer. This is not currently implemented, and
>> -         * therefore MSAA DCC isn't even enabled with multiple layers.
>> -         */
>> -        assert(rtex->resource.b.b.nr_samples <= 1 || num_layers == 1);
>> -
>> -        dcc_offset += rtex->surface.u.legacy.level[level].dcc_offset;
>> -        clear_size = 
>> rtex->surface.u.legacy.level[level].dcc_fast_clear_size *
>> -                 num_layers;
>> -    }
>> -
>> -    rctx->clear_buffer(&rctx->b, dcc_buffer, dcc_offset, clear_size,
>> -               clear_value, R600_COHERENCY_CB_META);
>> -}
>> -
>> -/* Set the same micro tile mode as the destination of the last MSAA 
>> resolve.
>> - * This allows hitting the MSAA resolve fast path, which requires 
>> that both
>> - * src and dst micro tile modes match.
>> - */
>> -static void si_set_optimal_micro_tile_mode(struct r600_common_screen 
>> *rscreen,
>> -                       struct r600_texture *rtex)
>> -{
>> -    if (rtex->resource.b.is_shared ||
>> -        rtex->resource.b.b.nr_samples <= 1 ||
>> -        rtex->surface.micro_tile_mode == 
>> rtex->last_msaa_resolve_target_micro_mode)
>> -        return;
>> -
>> -    assert(rscreen->chip_class >= GFX9 ||
>> -           rtex->surface.u.legacy.level[0].mode == RADEON_SURF_MODE_2D);
>> -    assert(rtex->resource.b.b.last_level == 0);
>> -
>> -    if (rscreen->chip_class >= GFX9) {
>> -        /* 4K or larger tiles only. 0 is linear. 1-3 are 256B tiles. */
>> -        assert(rtex->surface.u.gfx9.surf.swizzle_mode >= 4);
>> -
>> -        /* If you do swizzle_mode % 4, you'll get:
>> -         *   0 = Depth
>> -         *   1 = Standard,
>> -         *   2 = Displayable
>> -         *   3 = Rotated
>> -         *
>> -         * Depth-sample order isn't allowed:
>> -         */
>> -        assert(rtex->surface.u.gfx9.surf.swizzle_mode % 4 != 0);
>> -
>> -        switch (rtex->last_msaa_resolve_target_micro_mode) {
>> -        case RADEON_MICRO_MODE_DISPLAY:
>> -            rtex->surface.u.gfx9.surf.swizzle_mode &= ~0x3;
>> -            rtex->surface.u.gfx9.surf.swizzle_mode += 2; /* D */
>> -            break;
>> -        case RADEON_MICRO_MODE_THIN:
>> -            rtex->surface.u.gfx9.surf.swizzle_mode &= ~0x3;
>> -            rtex->surface.u.gfx9.surf.swizzle_mode += 1; /* S */
>> -            break;
>> -        case RADEON_MICRO_MODE_ROTATED:
>> -            rtex->surface.u.gfx9.surf.swizzle_mode &= ~0x3;
>> -            rtex->surface.u.gfx9.surf.swizzle_mode += 3; /* R */
>> -            break;
>> -        default: /* depth */
>> -            assert(!"unexpected micro mode");
>> -            return;
>> -        }
>> -    } else if (rscreen->chip_class >= CIK) {
>> -        /* These magic numbers were copied from addrlib. It doesn't use
>> -         * any definitions for them either. They are all 2D_TILED_THIN1
>> -         * modes with different bpp and micro tile mode.
>> -         */
>> -        switch (rtex->last_msaa_resolve_target_micro_mode) {
>> -        case RADEON_MICRO_MODE_DISPLAY:
>> -            rtex->surface.u.legacy.tiling_index[0] = 10;
>> -            break;
>> -        case RADEON_MICRO_MODE_THIN:
>> -            rtex->surface.u.legacy.tiling_index[0] = 14;
>> -            break;
>> -        case RADEON_MICRO_MODE_ROTATED:
>> -            rtex->surface.u.legacy.tiling_index[0] = 28;
>> -            break;
>> -        default: /* depth, thick */
>> -            assert(!"unexpected micro mode");
>> -            return;
>> -        }
>> -    } else { /* SI */
>> -        switch (rtex->last_msaa_resolve_target_micro_mode) {
>> -        case RADEON_MICRO_MODE_DISPLAY:
>> -            switch (rtex->surface.bpe) {
>> -            case 1:
>> -                            rtex->surface.u.legacy.tiling_index[0] = 10;
>> -                            break;
>> -            case 2:
>> -                            rtex->surface.u.legacy.tiling_index[0] = 11;
>> -                            break;
>> -            default: /* 4, 8 */
>> -                            rtex->surface.u.legacy.tiling_index[0] = 12;
>> -                            break;
>> -            }
>> -            break;
>> -        case RADEON_MICRO_MODE_THIN:
>> -            switch (rtex->surface.bpe) {
>> -            case 1:
>> -                                
>> rtex->surface.u.legacy.tiling_index[0] = 14;
>> -                                break;
>> -            case 2:
>> -                                
>> rtex->surface.u.legacy.tiling_index[0] = 15;
>> -                                break;
>> -            case 4:
>> -                                
>> rtex->surface.u.legacy.tiling_index[0] = 16;
>> -                                break;
>> -            default: /* 8, 16 */
>> -                                
>> rtex->surface.u.legacy.tiling_index[0] = 17;
>> -                                break;
>> -            }
>> -            break;
>> -        default: /* depth, thick */
>> -            assert(!"unexpected micro mode");
>> -            return;
>> -        }
>> -    }
>> -
>> -    rtex->surface.micro_tile_mode = 
>> rtex->last_msaa_resolve_target_micro_mode;
>> -
>> -    p_atomic_inc(&rscreen->dirty_tex_counter);
>> -}
>> -
>> -void si_do_fast_color_clear(struct r600_common_context *rctx,
>> -                   struct pipe_framebuffer_state *fb,
>> -                   struct r600_atom *fb_state,
>> -                   unsigned *buffers, ubyte *dirty_cbufs,
>> -                   const union pipe_color_union *color)
>> -{
>> -    int i;
>> -
>> -    /* This function is broken in BE, so just disable this path for 
>> now */
>> -#ifdef PIPE_ARCH_BIG_ENDIAN
>> -    return;
>> -#endif
>> -
>> -    if (rctx->render_cond)
>> -        return;
>> -
>> -    for (i = 0; i < fb->nr_cbufs; i++) {
>> -        struct r600_texture *tex;
>> -        unsigned clear_bit = PIPE_CLEAR_COLOR0 << i;
>> -        unsigned level = fb->cbufs[i]->u.tex.level;
>> -
>> -        if (!fb->cbufs[i])
>> -            continue;
>> -
>> -        /* if this colorbuffer is not being cleared */
>> -        if (!(*buffers & clear_bit))
>> -            continue;
>> -
>> -        tex = (struct r600_texture *)fb->cbufs[i]->texture;
>> -
>> -        /* the clear is allowed if all layers are bound */
>> -        if (fb->cbufs[i]->u.tex.first_layer != 0 ||
>> -            fb->cbufs[i]->u.tex.last_layer != 
>> util_max_layer(&tex->resource.b.b, 0)) {
>> -            continue;
>> -        }
>> -
>> -        /* cannot clear mipmapped textures */
>> -        if (fb->cbufs[i]->texture->last_level != 0) {
>> -            continue;
>> -        }
>> -
>> -        /* only supported on tiled surfaces */
>> -        if (tex->surface.is_linear) {
>> -            continue;
>> -        }
>> -
>> -        /* shared textures can't use fast clear without an explicit 
>> flush,
>> -         * because there is no way to communicate the clear color among
>> -         * all clients
>> -         */
>> -        if (tex->resource.b.is_shared &&
>> -            !(tex->resource.external_usage & 
>> PIPE_HANDLE_USAGE_EXPLICIT_FLUSH))
>> -            continue;
>> -
>> -        /* fast color clear with 1D tiling doesn't work on old 
>> kernels and CIK */
>> -        if (rctx->chip_class == CIK &&
>> -            tex->surface.u.legacy.level[0].mode == 
>> RADEON_SURF_MODE_1D &&
>> -            rctx->screen->info.drm_major == 2 &&
>> -            rctx->screen->info.drm_minor < 38) {
>> -            continue;
>> -        }
>> -
>> -        /* Fast clear is the most appropriate place to enable DCC for
>> -         * displayable surfaces.
>> -         */
>> -        if (rctx->chip_class >= VI &&
>> -            !(rctx->screen->debug_flags & DBG(NO_DCC_FB))) {
>> -            vi_separate_dcc_try_enable(rctx, tex);
>> -
>> -            /* RB+ isn't supported with a CMASK clear only on Stoney,
>> -             * so all clears are considered to be hypothetically slow
>> -             * clears, which is weighed when determining whether to
>> -             * enable separate DCC.
>> -             */
>> -            if (tex->dcc_gather_statistics &&
>> -                rctx->family == CHIP_STONEY)
>> -                tex->num_slow_clears++;
>> -        }
>> -
>> -        /* Try to clear DCC first, otherwise try CMASK. */
>> -        if (vi_dcc_enabled(tex, 0)) {
>> -            uint32_t reset_value;
>> -            bool clear_words_needed, cleared_cmask = false;
>> -
>> -            if (rctx->screen->debug_flags & DBG(NO_DCC_CLEAR))
>> -                continue;
>> -
>> -            /* This can only occur with MSAA. */
>> -            if (rctx->chip_class == VI &&
>> -                !tex->surface.u.legacy.level[level].dcc_fast_clear_size)
>> -                continue;
>> -
>> -            if (!vi_get_fast_clear_parameters(fb->cbufs[i]->format,
>> -                              color, &reset_value,
>> -                              &clear_words_needed))
>> -                continue;
>> -
>> -            /* DCC fast clear with MSAA should clear CMASK to 0xC. */
>> -            if (tex->resource.b.b.nr_samples >= 2 && tex->cmask.size) {
>> -                /* TODO: This doesn't work with MSAA. */
>> -                if (clear_words_needed)
>> -                    continue;
>> -
>> -                rctx->clear_buffer(&rctx->b, &tex->cmask_buffer->b.b,
>> -                           tex->cmask.offset, tex->cmask.size,
>> -                           0xCCCCCCCC, R600_COHERENCY_CB_META);
>> -                cleared_cmask = true;
>> -            }
>> -
>> -            vi_dcc_clear_level(rctx, tex, 0, reset_value);
>> -
>> -            if (clear_words_needed || cleared_cmask) {
>> -                bool need_compressed_update = !tex->dirty_level_mask;
>> -
>> -                tex->dirty_level_mask |= 1 << level;
>> -
>> -                if (need_compressed_update)
>> -                    
>> p_atomic_inc(&rctx->screen->compressed_colortex_counter);
>> -            }
>> -            tex->separate_dcc_dirty = true;
>> -        } else {
>> -            /* 128-bit formats are unusupported */
>> -            if (tex->surface.bpe > 8) {
>> -                continue;
>> -            }
>> -
>> -            /* RB+ doesn't work with CMASK fast clear on Stoney. */
>> -            if (rctx->family == CHIP_STONEY)
>> -                continue;
>> -
>> -            /* ensure CMASK is enabled */
>> -            r600_texture_alloc_cmask_separate(rctx->screen, tex);
>> -            if (tex->cmask.size == 0) {
>> -                continue;
>> -            }
>> -
>> -            /* Do the fast clear. */
>> -            rctx->clear_buffer(&rctx->b, &tex->cmask_buffer->b.b,
>> -                       tex->cmask.offset, tex->cmask.size, 0,
>> -                       R600_COHERENCY_CB_META);
>> -
>> -            bool need_compressed_update = !tex->dirty_level_mask;
>> -
>> -            tex->dirty_level_mask |= 1 << level;
>> -
>> -            if (need_compressed_update)
>> -                
>> p_atomic_inc(&rctx->screen->compressed_colortex_counter);
>> -        }
>> -
>> -        /* We can change the micro tile mode before a full clear. */
>> -        si_set_optimal_micro_tile_mode(rctx->screen, tex);
>> -
>> -        evergreen_set_clear_color(tex, fb->cbufs[i]->format, color);
>> -
>> -        if (dirty_cbufs)
>> -            *dirty_cbufs |= 1 << i;
>> -        rctx->set_atom_dirty(rctx, fb_state, true);
>> -        *buffers &= ~clear_bit;
>> -    }
>> -}
>> -
>>   static struct pipe_memory_object *
>>   r600_memobj_from_handle(struct pipe_screen *screen,
>>               struct winsys_handle *whandle,
>>               bool dedicated)
>>   {
>>       struct r600_common_screen *rscreen = (struct 
>> r600_common_screen*)screen;
>>       struct r600_memory_object *memobj = 
>> CALLOC_STRUCT(r600_memory_object);
>>       struct pb_buffer *buf = NULL;
>>       uint32_t stride, offset;
>> @@ -2915,12 +2379,11 @@ void si_init_screen_texture_functions(struct 
>> r600_common_screen *rscreen)
>>       rscreen->b.resource_from_memobj = r600_texture_from_memobj;
>>       rscreen->b.memobj_create_from_handle = r600_memobj_from_handle;
>>       rscreen->b.memobj_destroy = r600_memobj_destroy;
>>       rscreen->b.check_resource_capability = 
>> si_check_resource_capability;
>>   }
>>   void si_init_context_texture_functions(struct r600_common_context 
>> *rctx)
>>   {
>>       rctx->b.create_surface = r600_create_surface;
>>       rctx->b.surface_destroy = r600_surface_destroy;
>> -    rctx->b.clear_texture = r600_clear_texture;
>>   }
>> diff --git a/src/gallium/drivers/radeonsi/Makefile.sources 
>> b/src/gallium/drivers/radeonsi/Makefile.sources
>> index 8dd33f6..18cd7d1 100644
>> --- a/src/gallium/drivers/radeonsi/Makefile.sources
>> +++ b/src/gallium/drivers/radeonsi/Makefile.sources
>> @@ -1,18 +1,19 @@
>>   GENERATED_SOURCES := \
>>       si_driinfo.h
>>   C_SOURCES := \
>>       $(GENERATED_SOURCES) \
>>       cik_sdma.c \
>>       driinfo_radeonsi.h \
>>       si_blit.c \
>> +    si_clear.c \
>>       si_compute.c \
>>       si_compute.h \
>>       si_cp_dma.c \
>>       si_debug.c \
>>       si_descriptors.c \
>>       si_dma.c \
>>       si_fence.c \
>>       si_hw_context.c \
>>       si_pipe.c \
>>       si_pipe.h \
>> diff --git a/src/gallium/drivers/radeonsi/meson.build 
>> b/src/gallium/drivers/radeonsi/meson.build
>> index 974004d..a04ea4c 100644
>> --- a/src/gallium/drivers/radeonsi/meson.build
>> +++ b/src/gallium/drivers/radeonsi/meson.build
>> @@ -15,20 +15,21 @@
>>   # FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT 
>> SHALL THE
>>   # AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR 
>> OTHER
>>   # LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, 
>> ARISING FROM,
>>   # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER 
>> DEALINGS IN THE
>>   # SOFTWARE.
>>   files_libradeonsi = files(
>>     'cik_sdma.c',
>>     'driinfo_radeonsi.h',
>>     'si_blit.c',
>> +  'si_clear.c',
>>     'si_compute.c',
>>     'si_compute.h',
>>     'si_cp_dma.c',
>>     'si_debug.c',
>>     'si_descriptors.c',
>>     'si_dma.c',
>>     'si_fence.c',
>>     'si_hw_context.c',
>>     'si_pipe.c',
>>     'si_pipe.h',
>> diff --git a/src/gallium/drivers/radeonsi/si_blit.c 
>> b/src/gallium/drivers/radeonsi/si_blit.c
>> index abf25b6..e80bf5c2 100644
>> --- a/src/gallium/drivers/radeonsi/si_blit.c
>> +++ b/src/gallium/drivers/radeonsi/si_blit.c
>> @@ -20,44 +20,34 @@
>>    * OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE 
>> SOFTWARE OR THE
>>    * USE OR OTHER DEALINGS IN THE SOFTWARE.
>>    */
>>   #include "si_pipe.h"
>>   #include "si_compute.h"
>>   #include "util/u_format.h"
>>   #include "util/u_log.h"
>>   #include "util/u_surface.h"
>> -enum si_blitter_op /* bitmask */
>> -{
>> -    SI_SAVE_TEXTURES      = 1,
>> -    SI_SAVE_FRAMEBUFFER   = 2,
>> -    SI_SAVE_FRAGMENT_STATE = 4,
>> -    SI_DISABLE_RENDER_COND = 8,
>> -
>> -    SI_CLEAR         = SI_SAVE_FRAGMENT_STATE,
>> -
>> -    SI_CLEAR_SURFACE = SI_SAVE_FRAMEBUFFER | SI_SAVE_FRAGMENT_STATE,
>> -
>> +enum {
>>       SI_COPY          = SI_SAVE_FRAMEBUFFER | SI_SAVE_TEXTURES |
>>                  SI_SAVE_FRAGMENT_STATE | SI_DISABLE_RENDER_COND,
>>       SI_BLIT          = SI_SAVE_FRAMEBUFFER | SI_SAVE_TEXTURES |
>>                  SI_SAVE_FRAGMENT_STATE,
>>       SI_DECOMPRESS    = SI_SAVE_FRAMEBUFFER | SI_SAVE_FRAGMENT_STATE |
>>                  SI_DISABLE_RENDER_COND,
>>       SI_COLOR_RESOLVE = SI_SAVE_FRAMEBUFFER | SI_SAVE_FRAGMENT_STATE
>>   };
>> -static void si_blitter_begin(struct pipe_context *ctx, enum 
>> si_blitter_op op)
>> +void si_blitter_begin(struct pipe_context *ctx, enum si_blitter_op op)
>>   {
>>       struct si_context *sctx = (struct si_context *)ctx;
>>       util_blitter_save_vertex_shader(sctx->blitter, 
>> sctx->vs_shader.cso);
>>       util_blitter_save_tessctrl_shader(sctx->blitter, 
>> sctx->tcs_shader.cso);
>>       util_blitter_save_tesseval_shader(sctx->blitter, 
>> sctx->tes_shader.cso);
>>       util_blitter_save_geometry_shader(sctx->blitter, 
>> sctx->gs_shader.cso);
>>       util_blitter_save_so_targets(sctx->blitter, 
>> sctx->streamout.num_targets,
>>                        (struct 
>> pipe_stream_output_target**)sctx->streamout.targets);
>>       util_blitter_save_rasterizer(sctx->blitter, 
>> sctx->queued.named.rasterizer);
>> @@ -80,21 +70,21 @@ static void si_blitter_begin(struct pipe_context 
>> *ctx, enum si_blitter_op op)
>>               
>> (void**)sctx->samplers[PIPE_SHADER_FRAGMENT].sampler_states);
>>           util_blitter_save_fragment_sampler_views(sctx->blitter, 2,
>>               sctx->samplers[PIPE_SHADER_FRAGMENT].views);
>>       }
>>       if (op & SI_DISABLE_RENDER_COND)
>>           sctx->b.render_cond_force_off = true;
>>   }
>> -static void si_blitter_end(struct pipe_context *ctx)
>> +void si_blitter_end(struct pipe_context *ctx)
>>   {
>>       struct si_context *sctx = (struct si_context *)ctx;
>>       sctx->b.render_cond_force_off = false;
>>       /* Restore shader pointers because the VS blit shader changed all
>>        * non-global VS user SGPRs. */
>>       sctx->shader_pointers_dirty |= SI_DESCS_SHADER_MASK(VERTEX);
>>       sctx->vertex_buffer_pointer_dirty = true;
>>       si_mark_atom_dirty(sctx, &sctx->shader_pointers.atom);
>> @@ -812,169 +802,20 @@ void si_decompress_textures(struct si_context 
>> *sctx, unsigned shader_mask)
>>       } else if (shader_mask & (1 << PIPE_SHADER_COMPUTE)) {
>>           if (sctx->cs_shader_state.program->uses_bindless_samplers)
>>               si_decompress_resident_textures(sctx);
>>           if (sctx->cs_shader_state.program->uses_bindless_images)
>>               si_decompress_resident_images(sctx);
>>       }
>>       si_check_render_feedback(sctx);
>>   }
>> -static void si_clear(struct pipe_context *ctx, unsigned buffers,
>> -             const union pipe_color_union *color,
>> -             double depth, unsigned stencil)
>> -{
>> -    struct si_context *sctx = (struct si_context *)ctx;
>> -    struct pipe_framebuffer_state *fb = &sctx->framebuffer.state;
>> -    struct pipe_surface *zsbuf = fb->zsbuf;
>> -    struct r600_texture *zstex =
>> -        zsbuf ? (struct r600_texture*)zsbuf->texture : NULL;
>> -
>> -    if (buffers & PIPE_CLEAR_COLOR) {
>> -        si_do_fast_color_clear(&sctx->b, fb,
>> -                          &sctx->framebuffer.atom, &buffers,
>> -                          &sctx->framebuffer.dirty_cbufs,
>> -                          color);
>> -        if (!buffers)
>> -            return; /* all buffers have been fast cleared */
>> -    }
>> -
>> -    if (buffers & PIPE_CLEAR_COLOR) {
>> -        int i;
>> -
>> -        /* These buffers cannot use fast clear, make sure to disable 
>> expansion. */
>> -        for (i = 0; i < fb->nr_cbufs; i++) {
>> -            struct r600_texture *tex;
>> -
>> -            /* If not clearing this buffer, skip. */
>> -            if (!(buffers & (PIPE_CLEAR_COLOR0 << i)))
>> -                continue;
>> -
>> -            if (!fb->cbufs[i])
>> -                continue;
>> -
>> -            tex = (struct r600_texture *)fb->cbufs[i]->texture;
>> -            if (tex->fmask.size == 0)
>> -                tex->dirty_level_mask &= ~(1 << 
>> fb->cbufs[i]->u.tex.level);
>> -        }
>> -    }
>> -
>> -    if (zstex &&
>> -        r600_htile_enabled(zstex, zsbuf->u.tex.level) &&
>> -        zsbuf->u.tex.first_layer == 0 &&
>> -        zsbuf->u.tex.last_layer == 
>> util_max_layer(&zstex->resource.b.b, 0)) {
>> -        /* TC-compatible HTILE only supports depth clears to 0 or 1. */
>> -        if (buffers & PIPE_CLEAR_DEPTH &&
>> -            (!zstex->tc_compatible_htile ||
>> -             depth == 0 || depth == 1)) {
>> -            /* Need to disable EXPCLEAR temporarily if clearing
>> -             * to a new value. */
>> -            if (!zstex->depth_cleared || zstex->depth_clear_value != 
>> depth) {
>> -                sctx->db_depth_disable_expclear = true;
>> -            }
>> -
>> -            zstex->depth_clear_value = depth;
>> -            sctx->framebuffer.dirty_zsbuf = true;
>> -            si_mark_atom_dirty(sctx, &sctx->framebuffer.atom); /* 
>> updates DB_DEPTH_CLEAR */
>> -            sctx->db_depth_clear = true;
>> -            si_mark_atom_dirty(sctx, &sctx->db_render_state);
>> -        }
>> -
>> -        /* TC-compatible HTILE only supports stencil clears to 0. */
>> -        if (buffers & PIPE_CLEAR_STENCIL &&
>> -            (!zstex->tc_compatible_htile || stencil == 0)) {
>> -            stencil &= 0xff;
>> -
>> -            /* Need to disable EXPCLEAR temporarily if clearing
>> -             * to a new value. */
>> -            if (!zstex->stencil_cleared || zstex->stencil_clear_value 
>> != stencil) {
>> -                sctx->db_stencil_disable_expclear = true;
>> -            }
>> -
>> -            zstex->stencil_clear_value = stencil;
>> -            sctx->framebuffer.dirty_zsbuf = true;
>> -            si_mark_atom_dirty(sctx, &sctx->framebuffer.atom); /* 
>> updates DB_STENCIL_CLEAR */
>> -            sctx->db_stencil_clear = true;
>> -            si_mark_atom_dirty(sctx, &sctx->db_render_state);
>> -        }
>> -
>> -        /* TODO: Find out what's wrong here. Fast depth clear leads to
>> -         * corruption in ARK: Survival Evolved, but that may just be
>> -         * a coincidence and the root cause is elsewhere.
>> -         *
>> -         * The corruption can be fixed by putting the DB flush before
>> -         * or after the depth clear. (surprisingly)
>> -         *
>> -         * https://bugs.freedesktop.org/show_bug.cgi?id=102955 
>> (apitrace)
>> -         *
>> -         * This hack decreases back-to-back ClearDepth performance.
>> -         */
>> -        if (sctx->screen->clear_db_cache_before_clear) {
>> -            sctx->b.flags |= SI_CONTEXT_FLUSH_AND_INV_DB;
>> -        }
>> -    }
>> -
>> -    si_blitter_begin(ctx, SI_CLEAR);
>> -    util_blitter_clear(sctx->blitter, fb->width, fb->height,
>> -               util_framebuffer_get_num_layers(fb),
>> -               buffers, color, depth, stencil);
>> -    si_blitter_end(ctx);
>> -
>> -    if (sctx->db_depth_clear) {
>> -        sctx->db_depth_clear = false;
>> -        sctx->db_depth_disable_expclear = false;
>> -        zstex->depth_cleared = true;
>> -        si_mark_atom_dirty(sctx, &sctx->db_render_state);
>> -    }
>> -
>> -    if (sctx->db_stencil_clear) {
>> -        sctx->db_stencil_clear = false;
>> -        sctx->db_stencil_disable_expclear = false;
>> -        zstex->stencil_cleared = true;
>> -        si_mark_atom_dirty(sctx, &sctx->db_render_state);
>> -    }
>> -}
>> -
>> -static void si_clear_render_target(struct pipe_context *ctx,
>> -                   struct pipe_surface *dst,
>> -                   const union pipe_color_union *color,
>> -                   unsigned dstx, unsigned dsty,
>> -                   unsigned width, unsigned height,
>> -                   bool render_condition_enabled)
>> -{
>> -    struct si_context *sctx = (struct si_context *)ctx;
>> -
>> -    si_blitter_begin(ctx, SI_CLEAR_SURFACE |
>> -             (render_condition_enabled ? 0 : SI_DISABLE_RENDER_COND));
>> -    util_blitter_clear_render_target(sctx->blitter, dst, color,
>> -                     dstx, dsty, width, height);
>> -    si_blitter_end(ctx);
>> -}
>> -
>> -static void si_clear_depth_stencil(struct pipe_context *ctx,
>> -                   struct pipe_surface *dst,
>> -                   unsigned clear_flags,
>> -                   double depth,
>> -                   unsigned stencil,
>> -                   unsigned dstx, unsigned dsty,
>> -                   unsigned width, unsigned height,
>> -                   bool render_condition_enabled)
>> -{
>> -    struct si_context *sctx = (struct si_context *)ctx;
>> -
>> -    si_blitter_begin(ctx, SI_CLEAR_SURFACE |
>> -             (render_condition_enabled ? 0 : SI_DISABLE_RENDER_COND));
>> -    util_blitter_clear_depth_stencil(sctx->blitter, dst, clear_flags, 
>> depth, stencil,
>> -                     dstx, dsty, width, height);
>> -    si_blitter_end(ctx);
>> -}
>> -
>>   /* Helper for decompressing a portion of a color or depth resource 
>> before
>>    * blitting if any decompression is needed.
>>    * The driver doesn't decompress resources automatically while 
>> u_blitter is
>>    * rendering. */
>>   static void si_decompress_subresource(struct pipe_context *ctx,
>>                         struct pipe_resource *tex,
>>                         unsigned planes, unsigned level,
>>                         unsigned first_layer, unsigned last_layer)
>>   {
>>       struct si_context *sctx = (struct si_context *)ctx;
>> @@ -1293,21 +1134,21 @@ static bool do_hardware_msaa_resolve(struct 
>> pipe_context *ctx,
>>           /* Resolving into a surface with DCC is unsupported. Since
>>            * it's being overwritten anyway, clear it to uncompressed.
>>            * This is still the fastest codepath even with this clear.
>>            */
>>           if (vi_dcc_enabled(dst, info->dst.level)) {
>>               /* TODO: Implement per-level DCC clears for GFX9. */
>>               if (sctx->b.chip_class >= GFX9 &&
>>                   info->dst.resource->last_level != 0)
>>                   goto resolve_to_temp;
>> -            vi_dcc_clear_level(&sctx->b, dst, info->dst.level,
>> +            vi_dcc_clear_level(sctx, dst, info->dst.level,
>>                          0xFFFFFFFF);
>>               dst->dirty_level_mask &= ~(1 << info->dst.level);
>>           }
>>           /* Resolve directly from src to dst. */
>>           si_do_CB_resolve(sctx, info, info->dst.resource,
>>                    info->dst.level, info->dst.box.z, format);
>>           return true;
>>       }
>> @@ -1537,21 +1378,18 @@ static void si_pipe_clear_buffer(struct 
>> pipe_context *ctx,
>>       default:
>>           dword_value = *(uint32_t*)clear_value_ptr;
>>       }
>>       sctx->b.clear_buffer(ctx, dst, offset, size, dword_value,
>>                    R600_COHERENCY_SHADER);
>>   }
>>   void si_init_blit_functions(struct si_context *sctx)
>>   {
>> -    sctx->b.b.clear = si_clear;
>>       sctx->b.b.clear_buffer = si_pipe_clear_buffer;
>> -    sctx->b.b.clear_render_target = si_clear_render_target;
>> -    sctx->b.b.clear_depth_stencil = si_clear_depth_stencil;
>>       sctx->b.b.resource_copy_region = si_resource_copy_region;
>>       sctx->b.b.blit = si_blit;
>>       sctx->b.b.flush_resource = si_flush_resource;
>>       sctx->b.b.generate_mipmap = si_generate_mipmap;
>>       sctx->b.blit_decompress_depth = si_blit_decompress_depth;
>>       sctx->b.decompress_dcc = si_decompress_dcc;
>>   }
>> diff --git a/src/gallium/drivers/radeonsi/si_clear.c 
>> b/src/gallium/drivers/radeonsi/si_clear.c
>> new file mode 100644
>> index 0000000..d99947f
>> --- /dev/null
>> +++ b/src/gallium/drivers/radeonsi/si_clear.c
>> @@ -0,0 +1,725 @@
>> +/*
>> + * Copyright 2017 Advanced Micro Devices, Inc.
>> + *
>> + * Permission is hereby granted, free of charge, to any person 
>> obtaining a
>> + * copy of this software and associated documentation files (the 
>> "Software"),
>> + * to deal in the Software without restriction, including without 
>> limitation
>> + * on the rights to use, copy, modify, merge, publish, distribute, sub
>> + * license, and/or sell copies of the Software, and to permit persons 
>> to whom
>> + * the Software is furnished to do so, subject to the following 
>> conditions:
>> + *
>> + * The above copyright notice and this permission notice (including 
>> the next
>> + * paragraph) shall be included in all copies or substantial portions 
>> of the
>> + * Software.
>> + *
>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, 
>> EXPRESS OR
>> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF 
>> MERCHANTABILITY,
>> + * FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT 
>> SHALL
>> + * THE AUTHOR(S) AND/OR THEIR SUPPLIERS BE LIABLE FOR ANY CLAIM,
>> + * DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
>> + * OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE 
>> OR THE
>> + * USE OR OTHER DEALINGS IN THE SOFTWARE.
>> + */
>> +
>> +#include "si_pipe.h"
>> +#include "sid.h"
>> +
>> +#include "util/u_format.h"
>> +#include "util/u_pack_color.h"
>> +#include "util/u_surface.h"
>> +
>> +enum {
>> +    SI_CLEAR         = SI_SAVE_FRAGMENT_STATE,
>> +    SI_CLEAR_SURFACE = SI_SAVE_FRAMEBUFFER | SI_SAVE_FRAGMENT_STATE,
>> +};
>> +
>> +static void si_alloc_separate_cmask(struct si_screen *sscreen,
>> +                    struct r600_texture *rtex)
>> +{
>> +    if (rtex->cmask_buffer)
>> +                return;
>> +
>> +    assert(rtex->cmask.size == 0);
>> +
>> +    si_texture_get_cmask_info(&sscreen->b, rtex, &rtex->cmask);
>> +    if (!rtex->cmask.size)
>> +        return;
>> +
>> +    rtex->cmask_buffer = (struct r600_resource *)
>> +        si_aligned_buffer_create(&sscreen->b.b,
>> +                     R600_RESOURCE_FLAG_UNMAPPABLE,
>> +                     PIPE_USAGE_DEFAULT,
>> +                     rtex->cmask.size,
>> +                     rtex->cmask.alignment);
>> +    if (rtex->cmask_buffer == NULL) {
>> +        rtex->cmask.size = 0;
>> +        return;
>> +    }
>> +
>> +    /* update colorbuffer state bits */
>> +    rtex->cmask.base_address_reg = rtex->cmask_buffer->gpu_address >> 8;
>> +
>> +    rtex->cb_color_info |= S_028C70_FAST_CLEAR(1);
>> +
>> +    p_atomic_inc(&sscreen->b.compressed_colortex_counter);
>> +}
>> +
>> +static void si_set_clear_color(struct r600_texture *rtex,
>> +                   enum pipe_format surface_format,
>> +                   const union pipe_color_union *color)
>> +{
>> +    union util_color uc;
>> +
>> +    memset(&uc, 0, sizeof(uc));
>> +
>> +    if (rtex->surface.bpe == 16) {
>> +        /* DCC fast clear only:
>> +         *   CLEAR_WORD0 = R = G = B
>> +         *   CLEAR_WORD1 = A
>> +         */
>> +        assert(color->ui[0] == color->ui[1] &&
>> +               color->ui[0] == color->ui[2]);
>> +        uc.ui[0] = color->ui[0];
>> +        uc.ui[1] = color->ui[3];
>> +    } else if (util_format_is_pure_uint(surface_format)) {
>> +        util_format_write_4ui(surface_format, color->ui, 0, &uc, 0, 
>> 0, 0, 1, 1);
>> +    } else if (util_format_is_pure_sint(surface_format)) {
>> +        util_format_write_4i(surface_format, color->i, 0, &uc, 0, 0, 
>> 0, 1, 1);
>> +    } else {
>> +        util_pack_color(color->f, surface_format, &uc);
>> +    }
>> +
>> +    memcpy(rtex->color_clear_value, &uc, 2 * sizeof(uint32_t));
>> +}
>> +
>> +static bool vi_get_fast_clear_parameters(enum pipe_format 
>> surface_format,
>> +                     const union pipe_color_union *color,
>> +                     uint32_t* reset_value,
>> +                     bool* clear_words_needed)
>> +{
>> +    bool values[4] = {};
>> +    int i;
>> +    bool main_value = false;
>> +    bool extra_value = false;
>> +    int extra_channel;
>> +
>> +    /* This is needed to get the correct DCC clear value for 
>> luminance formats.
>> +     * 1) Get the linear format (because the next step can't handle 
>> L8_SRGB).
>> +     * 2) Convert luminance to red. (the real hw format for luminance)
>> +     */
>> +    surface_format = util_format_linear(surface_format);
>> +    surface_format = util_format_luminance_to_red(surface_format);
>> +
>> +    const struct util_format_description *desc = 
>> util_format_description(surface_format);
>> +
>> +    if (desc->block.bits == 128 &&
>> +        (color->ui[0] != color->ui[1] ||
>> +         color->ui[0] != color->ui[2]))
>> +        return false;
>> +
>> +    *clear_words_needed = true;
>> +    *reset_value = 0x20202020U;
>> +
>> +    /* If we want to clear without needing a fast clear eliminate 
>> step, we
>> +     * can set each channel to 0 or 1 (or 0/max for integer formats). We
>> +     * have two sets of flags, one for the last or first 
>> channel(extra) and
>> +     * one for the other channels(main).
>> +     */
>> +
>> +    if (surface_format == PIPE_FORMAT_R11G11B10_FLOAT ||
>> +        surface_format == PIPE_FORMAT_B5G6R5_UNORM ||
>> +        surface_format == PIPE_FORMAT_B5G6R5_SRGB ||
>> +        util_format_is_alpha(surface_format)) {
>> +        extra_channel = -1;
>> +    } else if (desc->layout == UTIL_FORMAT_LAYOUT_PLAIN) {
>> +        if (si_translate_colorswap(surface_format, false) <= 1)
>> +            extra_channel = desc->nr_channels - 1;
>> +        else
>> +            extra_channel = 0;
>> +    } else
>> +        return true;
>> +
>> +    for (i = 0; i < 4; ++i) {
>> +        int index = desc->swizzle[i] - PIPE_SWIZZLE_X;
>> +
>> +        if (desc->swizzle[i] < PIPE_SWIZZLE_X ||
>> +            desc->swizzle[i] > PIPE_SWIZZLE_W)
>> +            continue;
>> +
>> +        if (desc->channel[i].pure_integer &&
>> +            desc->channel[i].type == UTIL_FORMAT_TYPE_SIGNED) {
>> +            /* Use the maximum value for clamping the clear color. */
>> +            int max = u_bit_consecutive(0, desc->channel[i].size - 1);
>> +
>> +            values[i] = color->i[i] != 0;
>> +            if (color->i[i] != 0 && MIN2(color->i[i], max) != max)
>> +                return true;
>> +        } else if (desc->channel[i].pure_integer &&
>> +               desc->channel[i].type == UTIL_FORMAT_TYPE_UNSIGNED) {
>> +            /* Use the maximum value for clamping the clear color. */
>> +            unsigned max = u_bit_consecutive(0, desc->channel[i].size);
>> +
>> +            values[i] = color->ui[i] != 0U;
>> +            if (color->ui[i] != 0U && MIN2(color->ui[i], max) != max)
>> +                return true;
>> +        } else {
>> +            values[i] = color->f[i] != 0.0F;
>> +            if (color->f[i] != 0.0F && color->f[i] != 1.0F)
>> +                return true;
>> +        }
>> +
>> +        if (index == extra_channel)
>> +            extra_value = values[i];
>> +        else
>> +            main_value = values[i];
>> +    }
>> +
>> +    for (int i = 0; i < 4; ++i)
>> +        if (values[i] != main_value &&
>> +            desc->swizzle[i] - PIPE_SWIZZLE_X != extra_channel &&
>> +            desc->swizzle[i] >= PIPE_SWIZZLE_X &&
>> +            desc->swizzle[i] <= PIPE_SWIZZLE_W)
>> +            return true;
>> +
>> +    *clear_words_needed = false;
>> +    if (main_value)
>> +        *reset_value |= 0x80808080U;
>> +
>> +    if (extra_value)
>> +        *reset_value |= 0x40404040U;
>> +    return true;
>> +}
>> +
>> +void vi_dcc_clear_level(struct si_context *sctx,
>> +            struct r600_texture *rtex,
>> +            unsigned level, unsigned clear_value)
>> +{
>> +    struct pipe_resource *dcc_buffer;
>> +    uint64_t dcc_offset, clear_size;
>> +
>> +    assert(vi_dcc_enabled(rtex, level));
>> +
>> +    if (rtex->dcc_separate_buffer) {
>> +        dcc_buffer = &rtex->dcc_separate_buffer->b.b;
>> +        dcc_offset = 0;
>> +    } else {
>> +        dcc_buffer = &rtex->resource.b.b;
>> +        dcc_offset = rtex->dcc_offset;
>> +    }
>> +
>> +    if (sctx->b.chip_class >= GFX9) {
>> +        /* Mipmap level clears aren't implemented. */
>> +        assert(rtex->resource.b.b.last_level == 0);
>> +        /* MSAA needs a different clear size. */
>> +        assert(rtex->resource.b.b.nr_samples <= 1);
>> +        clear_size = rtex->surface.dcc_size;
>> +    } else {
>> +        unsigned num_layers = util_max_layer(&rtex->resource.b.b, 
>> level) + 1;
>> +
>> +        /* If this is 0, fast clear isn't possible. (can occur with 
>> MSAA) */
>> +        assert(rtex->surface.u.legacy.level[level].dcc_fast_clear_size);
>> +        /* Layered MSAA DCC fast clears need to clear 
>> dcc_fast_clear_size
>> +         * bytes for each layer. This is not currently implemented, and
>> +         * therefore MSAA DCC isn't even enabled with multiple layers.
>> +         */
>> +        assert(rtex->resource.b.b.nr_samples <= 1 || num_layers == 1);
>> +
>> +        dcc_offset += rtex->surface.u.legacy.level[level].dcc_offset;
>> +        clear_size = 
>> rtex->surface.u.legacy.level[level].dcc_fast_clear_size *
>> +                 num_layers;
>> +    }
>> +
>> +    si_clear_buffer(&sctx->b.b, dcc_buffer, dcc_offset, clear_size,
>> +            clear_value, R600_COHERENCY_CB_META);
>> +}
>> +
>> +/* Set the same micro tile mode as the destination of the last MSAA 
>> resolve.
>> + * This allows hitting the MSAA resolve fast path, which requires 
>> that both
>> + * src and dst micro tile modes match.
>> + */
>> +static void si_set_optimal_micro_tile_mode(struct si_screen *sscreen,
>> +                       struct r600_texture *rtex)
>> +{
>> +    if (rtex->resource.b.is_shared ||
>> +        rtex->resource.b.b.nr_samples <= 1 ||
>> +        rtex->surface.micro_tile_mode == 
>> rtex->last_msaa_resolve_target_micro_mode)
>> +        return;
>> +
>> +    assert(sscreen->b.chip_class >= GFX9 ||
>> +           rtex->surface.u.legacy.level[0].mode == RADEON_SURF_MODE_2D);
>> +    assert(rtex->resource.b.b.last_level == 0);
>> +
>> +    if (sscreen->b.chip_class >= GFX9) {
>> +        /* 4K or larger tiles only. 0 is linear. 1-3 are 256B tiles. */
>> +        assert(rtex->surface.u.gfx9.surf.swizzle_mode >= 4);
>> +
>> +        /* If you do swizzle_mode % 4, you'll get:
>> +         *   0 = Depth
>> +         *   1 = Standard,
>> +         *   2 = Displayable
>> +         *   3 = Rotated
>> +         *
>> +         * Depth-sample order isn't allowed:
>> +         */
>> +        assert(rtex->surface.u.gfx9.surf.swizzle_mode % 4 != 0);
>> +
>> +        switch (rtex->last_msaa_resolve_target_micro_mode) {
>> +        case RADEON_MICRO_MODE_DISPLAY:
>> +            rtex->surface.u.gfx9.surf.swizzle_mode &= ~0x3;
>> +            rtex->surface.u.gfx9.surf.swizzle_mode += 2; /* D */
>> +            break;
>> +        case RADEON_MICRO_MODE_THIN:
>> +            rtex->surface.u.gfx9.surf.swizzle_mode &= ~0x3;
>> +            rtex->surface.u.gfx9.surf.swizzle_mode += 1; /* S */
>> +            break;
>> +        case RADEON_MICRO_MODE_ROTATED:
>> +            rtex->surface.u.gfx9.surf.swizzle_mode &= ~0x3;
>> +            rtex->surface.u.gfx9.surf.swizzle_mode += 3; /* R */
>> +            break;
>> +        default: /* depth */
>> +            assert(!"unexpected micro mode");
>> +            return;
>> +        }
>> +    } else if (sscreen->b.chip_class >= CIK) {
>> +        /* These magic numbers were copied from addrlib. It doesn't use
>> +         * any definitions for them either. They are all 2D_TILED_THIN1
>> +         * modes with different bpp and micro tile mode.
>> +         */
>> +        switch (rtex->last_msaa_resolve_target_micro_mode) {
>> +        case RADEON_MICRO_MODE_DISPLAY:
>> +            rtex->surface.u.legacy.tiling_index[0] = 10;
>> +            break;
>> +        case RADEON_MICRO_MODE_THIN:
>> +            rtex->surface.u.legacy.tiling_index[0] = 14;
>> +            break;
>> +        case RADEON_MICRO_MODE_ROTATED:
>> +            rtex->surface.u.legacy.tiling_index[0] = 28;
>> +            break;
>> +        default: /* depth, thick */
>> +            assert(!"unexpected micro mode");
>> +            return;
>> +        }
>> +    } else { /* SI */
>> +        switch (rtex->last_msaa_resolve_target_micro_mode) {
>> +        case RADEON_MICRO_MODE_DISPLAY:
>> +            switch (rtex->surface.bpe) {
>> +            case 1:
>> +                            rtex->surface.u.legacy.tiling_index[0] = 10;
>> +                            break;
>> +            case 2:
>> +                            rtex->surface.u.legacy.tiling_index[0] = 11;
>> +                            break;
>> +            default: /* 4, 8 */
>> +                            rtex->surface.u.legacy.tiling_index[0] = 12;
>> +                            break;
>> +            }
>> +            break;
>> +        case RADEON_MICRO_MODE_THIN:
>> +            switch (rtex->surface.bpe) {
>> +            case 1:
>> +                                
>> rtex->surface.u.legacy.tiling_index[0] = 14;
>> +                                break;
>> +            case 2:
>> +                                
>> rtex->surface.u.legacy.tiling_index[0] = 15;
>> +                                break;
>> +            case 4:
>> +                                
>> rtex->surface.u.legacy.tiling_index[0] = 16;
>> +                                break;
>> +            default: /* 8, 16 */
>> +                                
>> rtex->surface.u.legacy.tiling_index[0] = 17;
>> +                                break;
>> +            }
>> +            break;
>> +        default: /* depth, thick */
>> +            assert(!"unexpected micro mode");
>> +            return;
>> +        }
>> +    }
>> +
>> +    rtex->surface.micro_tile_mode = 
>> rtex->last_msaa_resolve_target_micro_mode;
>> +
>> +    p_atomic_inc(&sscreen->b.dirty_tex_counter);
>> +}
>> +
>> +static void si_do_fast_color_clear(struct si_context *sctx,
>> +                   struct pipe_framebuffer_state *fb,
>> +                   struct r600_atom *fb_state,
>> +                   unsigned *buffers, ubyte *dirty_cbufs,
>> +                   const union pipe_color_union *color)
>> +{
>> +    int i;
>> +
>> +    /* This function is broken in BE, so just disable this path for 
>> now */
>> +#ifdef PIPE_ARCH_BIG_ENDIAN
>> +    return;
>> +#endif
>> +
>> +    if (sctx->b.render_cond)
>> +        return;
>> +
>> +    for (i = 0; i < fb->nr_cbufs; i++) {
>> +        struct r600_texture *tex;
>> +        unsigned clear_bit = PIPE_CLEAR_COLOR0 << i;
>> +
>> +        if (!fb->cbufs[i])
>> +            continue;
>> +
>> +        /* if this colorbuffer is not being cleared */
>> +        if (!(*buffers & clear_bit))
>> +            continue;
>> +
>> +        unsigned level = fb->cbufs[i]->u.tex.level;
>> +        tex = (struct r600_texture *)fb->cbufs[i]->texture;
>> +
>> +        /* the clear is allowed if all layers are bound */
>> +        if (fb->cbufs[i]->u.tex.first_layer != 0 ||
>> +            fb->cbufs[i]->u.tex.last_layer != 
>> util_max_layer(&tex->resource.b.b, 0)) {
>> +            continue;
>> +        }
>> +
>> +        /* cannot clear mipmapped textures */
>> +        if (fb->cbufs[i]->texture->last_level != 0) {
>> +            continue;
>> +        }
>> +
>> +        /* only supported on tiled surfaces */
>> +        if (tex->surface.is_linear) {
>> +            continue;
>> +        }
>> +
>> +        /* shared textures can't use fast clear without an explicit 
>> flush,
>> +         * because there is no way to communicate the clear color among
>> +         * all clients
>> +         */
>> +        if (tex->resource.b.is_shared &&
>> +            !(tex->resource.external_usage & 
>> PIPE_HANDLE_USAGE_EXPLICIT_FLUSH))
>> +            continue;
>> +
>> +        /* fast color clear with 1D tiling doesn't work on old 
>> kernels and CIK */
>> +        if (sctx->b.chip_class == CIK &&
>> +            tex->surface.u.legacy.level[0].mode == 
>> RADEON_SURF_MODE_1D &&
>> +            sctx->screen->b.info.drm_major == 2 &&
>> +            sctx->screen->b.info.drm_minor < 38) {
>> +            continue;
>> +        }
>> +
>> +        /* Fast clear is the most appropriate place to enable DCC for
>> +         * displayable surfaces.
>> +         */
>> +        if (sctx->b.chip_class >= VI &&
>> +            !(sctx->screen->b.debug_flags & DBG(NO_DCC_FB))) {
>> +            vi_separate_dcc_try_enable(&sctx->b, tex);
>> +
>> +            /* RB+ isn't supported with a CMASK clear only on Stoney,
>> +             * so all clears are considered to be hypothetically slow
>> +             * clears, which is weighed when determining whether to
>> +             * enable separate DCC.
>> +             */
>> +            if (tex->dcc_gather_statistics &&
>> +                sctx->b.family == CHIP_STONEY)
>> +                tex->num_slow_clears++;
>> +        }
>> +
>> +        /* Try to clear DCC first, otherwise try CMASK. */
>> +        if (vi_dcc_enabled(tex, 0)) {
>> +            uint32_t reset_value;
>> +            bool clear_words_needed, cleared_cmask = false;
>> +
>> +            if (sctx->screen->b.debug_flags & DBG(NO_DCC_CLEAR))
>> +                continue;
>> +
>> +            /* This can only occur with MSAA. */
>> +            if (sctx->b.chip_class == VI &&
>> +                !tex->surface.u.legacy.level[level].dcc_fast_clear_size)
>> +                continue;
>> +
>> +            if (!vi_get_fast_clear_parameters(fb->cbufs[i]->format,
>> +                              color, &reset_value,
>> +                              &clear_words_needed))
>> +                continue;
>> +
>> +            /* DCC fast clear with MSAA should clear CMASK to 0xC. */
>> +            if (tex->resource.b.b.nr_samples >= 2 && tex->cmask.size) {
>> +                /* TODO: This doesn't work with MSAA. */
>> +                if (clear_words_needed)
>> +                    continue;
>> +
>> +                si_clear_buffer(&sctx->b.b, &tex->cmask_buffer->b.b,
>> +                        tex->cmask.offset, tex->cmask.size,
>> +                        0xCCCCCCCC, R600_COHERENCY_CB_META);
>> +                cleared_cmask = true;
>> +            }
>> +
>> +            vi_dcc_clear_level(sctx, tex, 0, reset_value);
>> +
>> +            if (clear_words_needed || cleared_cmask) {
>> +                bool need_compressed_update = !tex->dirty_level_mask;
>> +
>> +                tex->dirty_level_mask |= 1 << level;
>> +
>> +                if (need_compressed_update)
>> +                    
>> p_atomic_inc(&sctx->screen->b.compressed_colortex_counter);
>> +            }
>> +            tex->separate_dcc_dirty = true;
>> +        } else {
>> +            /* 128-bit formats are unusupported */
>> +            if (tex->surface.bpe > 8) {
>> +                continue;
>> +            }
>> +
>> +            /* RB+ doesn't work with CMASK fast clear on Stoney. */
>> +            if (sctx->b.family == CHIP_STONEY)
>> +                continue;
>> +
>> +            /* ensure CMASK is enabled */
>> +            si_alloc_separate_cmask(sctx->screen, tex);
>> +            if (tex->cmask.size == 0) {
>> +                continue;
>> +            }
>> +
>> +            /* Do the fast clear. */
>> +            si_clear_buffer(&sctx->b.b, &tex->cmask_buffer->b.b,
>> +                    tex->cmask.offset, tex->cmask.size, 0,
>> +                    R600_COHERENCY_CB_META);
>> +
>> +            bool need_compressed_update = !tex->dirty_level_mask;
>> +
>> +            tex->dirty_level_mask |= 1 << level;
>> +
>> +            if (need_compressed_update)
>> +                
>> p_atomic_inc(&sctx->screen->b.compressed_colortex_counter);
>> +        }
>> +
>> +        /* We can change the micro tile mode before a full clear. */
>> +        si_set_optimal_micro_tile_mode(sctx->screen, tex);
>> +
>> +        si_set_clear_color(tex, fb->cbufs[i]->format, color);
>> +
>> +        if (dirty_cbufs)
>> +            *dirty_cbufs |= 1 << i;
>> +        sctx->b.set_atom_dirty(&sctx->b, fb_state, true);
>> +        *buffers &= ~clear_bit;
>> +    }
>> +}
>> +
>> +static void si_clear(struct pipe_context *ctx, unsigned buffers,
>> +             const union pipe_color_union *color,
>> +             double depth, unsigned stencil)
>> +{
>> +    struct si_context *sctx = (struct si_context *)ctx;
>> +    struct pipe_framebuffer_state *fb = &sctx->framebuffer.state;
>> +    struct pipe_surface *zsbuf = fb->zsbuf;
>> +    struct r600_texture *zstex =
>> +        zsbuf ? (struct r600_texture*)zsbuf->texture : NULL;
>> +
>> +    if (buffers & PIPE_CLEAR_COLOR) {
>> +        si_do_fast_color_clear(sctx, fb,
>> +                       &sctx->framebuffer.atom, &buffers,
>> +                       &sctx->framebuffer.dirty_cbufs,
>> +                       color);
>> +        if (!buffers)
>> +            return; /* all buffers have been fast cleared */
>> +    }
>> +
>> +    if (buffers & PIPE_CLEAR_COLOR) {
>> +        int i;
>> +
>> +        /* These buffers cannot use fast clear, make sure to disable 
>> expansion. */
>> +        for (i = 0; i < fb->nr_cbufs; i++) {
>> +            struct r600_texture *tex;
>> +
>> +            /* If not clearing this buffer, skip. */
>> +            if (!(buffers & (PIPE_CLEAR_COLOR0 << i)))
>> +                continue;
>> +
>> +            if (!fb->cbufs[i])
>> +                continue;
>> +
>> +            tex = (struct r600_texture *)fb->cbufs[i]->texture;
>> +            if (tex->fmask.size == 0)
>> +                tex->dirty_level_mask &= ~(1 << 
>> fb->cbufs[i]->u.tex.level);
>> +        }
>> +    }
>> +
>> +    if (zstex &&
>> +        r600_htile_enabled(zstex, zsbuf->u.tex.level) &&
>> +        zsbuf->u.tex.first_layer == 0 &&
>> +        zsbuf->u.tex.last_layer == 
>> util_max_layer(&zstex->resource.b.b, 0)) {
>> +        /* TC-compatible HTILE only supports depth clears to 0 or 1. */
>> +        if (buffers & PIPE_CLEAR_DEPTH &&
>> +            (!zstex->tc_compatible_htile ||
>> +             depth == 0 || depth == 1)) {
>> +            /* Need to disable EXPCLEAR temporarily if clearing
>> +             * to a new value. */
>> +            if (!zstex->depth_cleared || zstex->depth_clear_value != 
>> depth) {
>> +                sctx->db_depth_disable_expclear = true;
>> +            }
>> +
>> +            zstex->depth_clear_value = depth;
>> +            sctx->framebuffer.dirty_zsbuf = true;
>> +            si_mark_atom_dirty(sctx, &sctx->framebuffer.atom); /* 
>> updates DB_DEPTH_CLEAR */
>> +            sctx->db_depth_clear = true;
>> +            si_mark_atom_dirty(sctx, &sctx->db_render_state);
>> +        }
>> +
>> +        /* TC-compatible HTILE only supports stencil clears to 0. */
>> +        if (buffers & PIPE_CLEAR_STENCIL &&
>> +            (!zstex->tc_compatible_htile || stencil == 0)) {
>> +            stencil &= 0xff;
>> +
>> +            /* Need to disable EXPCLEAR temporarily if clearing
>> +             * to a new value. */
>> +            if (!zstex->stencil_cleared || zstex->stencil_clear_value 
>> != stencil) {
>> +                sctx->db_stencil_disable_expclear = true;
>> +            }
>> +
>> +            zstex->stencil_clear_value = stencil;
>> +            sctx->framebuffer.dirty_zsbuf = true;
>> +            si_mark_atom_dirty(sctx, &sctx->framebuffer.atom); /* 
>> updates DB_STENCIL_CLEAR */
>> +            sctx->db_stencil_clear = true;
>> +            si_mark_atom_dirty(sctx, &sctx->db_render_state);
>> +        }
>> +
>> +        /* TODO: Find out what's wrong here. Fast depth clear leads to
>> +         * corruption in ARK: Survival Evolved, but that may just be
>> +         * a coincidence and the root cause is elsewhere.
>> +         *
>> +         * The corruption can be fixed by putting the DB flush before
>> +         * or after the depth clear. (surprisingly)
>> +         *
>> +         * https://bugs.freedesktop.org/show_bug.cgi?id=102955 
>> (apitrace)
>> +         *
>> +         * This hack decreases back-to-back ClearDepth performance.
>> +         */
>> +        if (sctx->screen->clear_db_cache_before_clear) {
>> +            sctx->b.flags |= SI_CONTEXT_FLUSH_AND_INV_DB;
>> +        }
>> +    }
>> +
>> +    si_blitter_begin(ctx, SI_CLEAR);
>> +    util_blitter_clear(sctx->blitter, fb->width, fb->height,
>> +               util_framebuffer_get_num_layers(fb),
>> +               buffers, color, depth, stencil);
>> +    si_blitter_end(ctx);
>> +
>> +    if (sctx->db_depth_clear) {
>> +        sctx->db_depth_clear = false;
>> +        sctx->db_depth_disable_expclear = false;
>> +        zstex->depth_cleared = true;
>> +        si_mark_atom_dirty(sctx, &sctx->db_render_state);
>> +    }
>> +
>> +    if (sctx->db_stencil_clear) {
>> +        sctx->db_stencil_clear = false;
>> +        sctx->db_stencil_disable_expclear = false;
>> +        zstex->stencil_cleared = true;
>> +        si_mark_atom_dirty(sctx, &sctx->db_render_state);
>> +    }
>> +}
>> +
>> +static void si_clear_render_target(struct pipe_context *ctx,
>> +                   struct pipe_surface *dst,
>> +                   const union pipe_color_union *color,
>> +                   unsigned dstx, unsigned dsty,
>> +                   unsigned width, unsigned height,
>> +                   bool render_condition_enabled)
>> +{
>> +    struct si_context *sctx = (struct si_context *)ctx;
>> +
>> +    si_blitter_begin(ctx, SI_CLEAR_SURFACE |
>> +             (render_condition_enabled ? 0 : SI_DISABLE_RENDER_COND));
>> +    util_blitter_clear_render_target(sctx->blitter, dst, color,
>> +                     dstx, dsty, width, height);
>> +    si_blitter_end(ctx);
>> +}
>> +
>> +static void si_clear_depth_stencil(struct pipe_context *ctx,
>> +                   struct pipe_surface *dst,
>> +                   unsigned clear_flags,
>> +                   double depth,
>> +                   unsigned stencil,
>> +                   unsigned dstx, unsigned dsty,
>> +                   unsigned width, unsigned height,
>> +                   bool render_condition_enabled)
>> +{
>> +    struct si_context *sctx = (struct si_context *)ctx;
>> +
>> +    si_blitter_begin(ctx, SI_CLEAR_SURFACE |
>> +             (render_condition_enabled ? 0 : SI_DISABLE_RENDER_COND));
>> +    util_blitter_clear_depth_stencil(sctx->blitter, dst, clear_flags, 
>> depth, stencil,
>> +                     dstx, dsty, width, height);
>> +    si_blitter_end(ctx);
>> +}
>> +
>> +static void si_clear_texture(struct pipe_context *pipe,
>> +                 struct pipe_resource *tex,
>> +                 unsigned level,
>> +                 const struct pipe_box *box,
>> +                 const void *data)
>> +{
>> +    struct pipe_screen *screen = pipe->screen;
>> +    struct r600_texture *rtex = (struct r600_texture*)tex;
>> +    struct pipe_surface tmpl = {{0}};
>> +    struct pipe_surface *sf;
>> +    const struct util_format_description *desc =
>> +        util_format_description(tex->format);
>> +
>> +    tmpl.format = tex->format;
>> +    tmpl.u.tex.first_layer = box->z;
>> +    tmpl.u.tex.last_layer = box->z + box->depth - 1;
>> +    tmpl.u.tex.level = level;
>> +    sf = pipe->create_surface(pipe, tex, &tmpl);
>> +    if (!sf)
>> +        return;
>> +
>> +    if (rtex->is_depth) {
>> +        unsigned clear;
>> +        float depth;
>> +        uint8_t stencil = 0;
>> +
>> +        /* Depth is always present. */
>> +        clear = PIPE_CLEAR_DEPTH;
>> +        desc->unpack_z_float(&depth, 0, data, 0, 1, 1);
>> +
>> +        if (rtex->surface.has_stencil) {
>> +            clear |= PIPE_CLEAR_STENCIL;
>> +            desc->unpack_s_8uint(&stencil, 0, data, 0, 1, 1);
>> +        }
>> +
>> +        si_clear_depth_stencil(pipe, sf, clear, depth, stencil,
>> +                       box->x, box->y,
>> +                       box->width, box->height, false);
>> +    } else {
>> +        union pipe_color_union color;
>> +
>> +        /* pipe_color_union requires the full vec4 representation. */
>> +        if (util_format_is_pure_uint(tex->format))
>> +            desc->unpack_rgba_uint(color.ui, 0, data, 0, 1, 1);
>> +        else if (util_format_is_pure_sint(tex->format))
>> +            desc->unpack_rgba_sint(color.i, 0, data, 0, 1, 1);
>> +        else
>> +            desc->unpack_rgba_float(color.f, 0, data, 0, 1, 1);
>> +
>> +        if (screen->is_format_supported(screen, tex->format,
>> +                        tex->target, 0,
>> +                        PIPE_BIND_RENDER_TARGET)) {
>> +            si_clear_render_target(pipe, sf, &color,
>> +                           box->x, box->y,
>> +                           box->width, box->height, false);
>> +        } else {
>> +            /* Software fallback - just for R9G9B9E5_FLOAT */
>> +            util_clear_render_target(pipe, sf, &color,
>> +                         box->x, box->y,
>> +                         box->width, box->height);
>> +        }
>> +    }
>> +    pipe_surface_reference(&sf, NULL);
>> +}
>> +
>> +void si_init_clear_functions(struct si_context *sctx)
>> +{
>> +    sctx->b.b.clear = si_clear;
>> +    sctx->b.b.clear_render_target = si_clear_render_target;
>> +    sctx->b.b.clear_depth_stencil = si_clear_depth_stencil;
>> +    sctx->b.b.clear_texture = si_clear_texture;
>> +}
>> diff --git a/src/gallium/drivers/radeonsi/si_cp_dma.c 
>> b/src/gallium/drivers/radeonsi/si_cp_dma.c
>> index 4ac0c34..ca82626 100644
>> --- a/src/gallium/drivers/radeonsi/si_cp_dma.c
>> +++ b/src/gallium/drivers/radeonsi/si_cp_dma.c
>> @@ -196,23 +196,23 @@ static void si_cp_dma_prepare(struct si_context 
>> *sctx, struct pipe_resource *dst
>>       *is_first = false;
>>       /* Do the synchronization after the last dma, so that all data
>>        * is written to memory.
>>        */
>>       if (!(user_flags & SI_CPDMA_SKIP_SYNC_AFTER) &&
>>           byte_count == remaining_size)
>>           *packet_flags |= CP_DMA_SYNC;
>>   }
>> -static void si_clear_buffer(struct pipe_context *ctx, struct 
>> pipe_resource *dst,
>> -                uint64_t offset, uint64_t size, unsigned value,
>> -                enum r600_coherency coher)
>> +void si_clear_buffer(struct pipe_context *ctx, struct pipe_resource 
>> *dst,
>> +             uint64_t offset, uint64_t size, unsigned value,
>> +             enum r600_coherency coher)
>>   {
>>       struct si_context *sctx = (struct si_context*)ctx;
>>       struct radeon_winsys *ws = sctx->b.ws;
>>       struct r600_resource *rdst = r600_resource(dst);
>>       unsigned tc_l2_flag = get_tc_l2_flag(sctx, coher);
>>       unsigned flush_flags = get_flush_flags(sctx, coher);
>>       uint64_t dma_clear_size;
>>       bool is_first = true;
>>       if (!size)
>> diff --git a/src/gallium/drivers/radeonsi/si_pipe.c 
>> b/src/gallium/drivers/radeonsi/si_pipe.c
>> index 6e433a3..f0a9ad1 100644
>> --- a/src/gallium/drivers/radeonsi/si_pipe.c
>> +++ b/src/gallium/drivers/radeonsi/si_pipe.c
>> @@ -202,20 +202,21 @@ static struct pipe_context 
>> *si_create_context(struct pipe_screen *screen,
>>       sctx->b.set_atom_dirty = (void *)si_set_atom_dirty;
>>       sctx->screen = sscreen; /* Easy accessing of screen/winsys. */
>>       sctx->is_debug = (flags & PIPE_CONTEXT_DEBUG) != 0;
>>       if (!si_common_context_init(&sctx->b, &sscreen->b, flags))
>>           goto fail;
>>       if (sscreen->b.info.drm_major == 3)
>>           sctx->b.b.get_device_reset_status = si_amdgpu_get_reset_status;
>> +    si_init_clear_functions(sctx);
>>       si_init_blit_functions(sctx);
>>       si_init_compute_functions(sctx);
>>       si_init_cp_dma_functions(sctx);
>>       si_init_debug_functions(sctx);
>>       si_init_msaa_functions(sctx);
>>       si_init_streamout_functions(sctx);
>>       if (sscreen->b.info.has_hw_decode) {
>>           sctx->b.b.create_video_codec = si_uvd_create_decoder;
>>           sctx->b.b.create_video_buffer = si_video_buffer_create;
>> diff --git a/src/gallium/drivers/radeonsi/si_pipe.h 
>> b/src/gallium/drivers/radeonsi/si_pipe.h
>> index 751441d..820e5b2 100644
>> --- a/src/gallium/drivers/radeonsi/si_pipe.h
>> +++ b/src/gallium/drivers/radeonsi/si_pipe.h
>> @@ -551,42 +551,61 @@ struct si_context {
>>       float            sample_locations_2x[2][2];
>>       float            sample_locations_4x[4][2];
>>       float            sample_locations_8x[8][2];
>>       float            sample_locations_16x[16][2];
>>   };
>>   /* cik_sdma.c */
>>   void cik_init_sdma_functions(struct si_context *sctx);
>>   /* si_blit.c */
>> +enum si_blitter_op /* bitmask */
>> +{
>> +    SI_SAVE_TEXTURES      = 1,
>> +    SI_SAVE_FRAMEBUFFER   = 2,
>> +    SI_SAVE_FRAGMENT_STATE = 4,
>> +    SI_DISABLE_RENDER_COND = 8,
>> +};
>> +
>> +void si_blitter_begin(struct pipe_context *ctx, enum si_blitter_op op);
>> +void si_blitter_end(struct pipe_context *ctx);
>>   void si_init_blit_functions(struct si_context *sctx);
>>   void si_decompress_textures(struct si_context *sctx, unsigned 
>> shader_mask);
>>   void si_resource_copy_region(struct pipe_context *ctx,
>>                    struct pipe_resource *dst,
>>                    unsigned dst_level,
>>                    unsigned dstx, unsigned dsty, unsigned dstz,
>>                    struct pipe_resource *src,
>>                    unsigned src_level,
>>                    const struct pipe_box *src_box);
>> +/* si_clear.c */
>> +void vi_dcc_clear_level(struct si_context *sctx,
>> +            struct r600_texture *rtex,
>> +            unsigned level, unsigned clear_value);
>> +void si_init_clear_functions(struct si_context *sctx);
>> +
>>   /* si_cp_dma.c */
>>   #define SI_CPDMA_SKIP_CHECK_CS_SPACE    (1 << 0) /* don't call 
>> need_cs_space */
>>   #define SI_CPDMA_SKIP_SYNC_AFTER    (1 << 1) /* don't wait for DMA 
>> after the copy */
>>   #define SI_CPDMA_SKIP_SYNC_BEFORE    (1 << 2) /* don't wait for DMA 
>> before the copy (RAW hazards) */
>>   #define SI_CPDMA_SKIP_GFX_SYNC        (1 << 3) /* don't flush caches 
>> and don't wait for PS/CS */
>>   #define SI_CPDMA_SKIP_BO_LIST_UPDATE    (1 << 4) /* don't update the 
>> BO list */
>>   #define SI_CPDMA_SKIP_ALL (SI_CPDMA_SKIP_CHECK_CS_SPACE | \
>>                  SI_CPDMA_SKIP_SYNC_AFTER | \
>>                  SI_CPDMA_SKIP_SYNC_BEFORE | \
>>                  SI_CPDMA_SKIP_GFX_SYNC | \
>>                  SI_CPDMA_SKIP_BO_LIST_UPDATE)
>> +void si_clear_buffer(struct pipe_context *ctx, struct pipe_resource 
>> *dst,
>> +             uint64_t offset, uint64_t size, unsigned value,
>> +             enum r600_coherency coher);
>>   void si_copy_buffer(struct si_context *sctx,
>>               struct pipe_resource *dst, struct pipe_resource *src,
>>               uint64_t dst_offset, uint64_t src_offset, unsigned size,
>>               unsigned user_flags);
>>   void cik_prefetch_TC_L2_async(struct si_context *sctx, struct 
>> pipe_resource *buf,
>>                     uint64_t offset, unsigned size);
>>   void cik_emit_prefetch_L2(struct si_context *sctx);
>>   void si_init_cp_dma_functions(struct si_context *sctx);
>>   /* si_debug.c */
>>
> 
> 


-- 
Lerne, wie die Welt wirklich ist,
Aber vergiss niemals, wie sie sein sollte.


More information about the mesa-dev mailing list