On Nov 4, 2015 4:02 AM, "Alex Deucher" <<a href="mailto:alexdeucher@gmail.com">alexdeucher@gmail.com</a>> wrote: > > On Tue, Nov 3, 2015 at 6:47 PM, Marek OlÅ¡Ã¡k <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote: > > From: Marek OlÅ¡Ã¡k <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>> > > > > --- > >Â src/gallium/drivers/radeonsi/si_blit.c | 55 ++++++++++++++++++++++++++++++++++ > >Â 1 file changed, 55 insertions(+) > > > > diff --git a/src/gallium/drivers/radeonsi/si_blit.c b/src/gallium/drivers/radeonsi/si_blit.c > > index fce014a..e934146 100644 > > --- a/src/gallium/drivers/radeonsi/si_blit.c > > +++ b/src/gallium/drivers/radeonsi/si_blit.c > > @@ -731,9 +731,64 @@ static void si_flush_resource(struct pipe_context *ctx, > >Â Â Â Â Â } > >Â } > > > > +static void si_pipe_clear_buffer(struct pipe_context *ctx, > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct pipe_resource *dst, > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â unsigned offset, unsigned size, > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â const void *clear_value, > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â int clear_value_size) > > +{ > > +Â Â Â Â struct si_context *sctx = (struct si_context*)ctx; > > +Â Â Â Â const uint32_t *u32 = clear_value; > > +Â Â Â Â unsigned i; > > +Â Â Â Â bool clear_value_fits_dword = true; > > +Â Â Â Â uint8_t *map; > > + > > +Â Â Â Â if (clear_value_size > 4) > > +Â Â Â Â Â Â Â Â for (i = 1; i < clear_value_size / 4; i++) > > +Â Â Â Â Â Â Â Â Â Â Â Â if (u32[0] != u32[i]) { > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â clear_value_fits_dword = false; > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â break; > > +Â Â Â Â Â Â Â Â Â Â Â Â } > > + > > +Â Â Â Â /* Use CP DMA for the simple case. */ > > +Â Â Â Â if (offset % 4 == 0 && size % 4 == 0 && clear_value_fits_dword) { > > +Â Â Â Â Â Â Â Â uint32_t value = u32[0]; > > + > > +Â Â Â Â Â Â Â Â switch (clear_value_size) { > > +Â Â Â Â Â Â Â Â case 1: > > +Â Â Â Â Â Â Â Â Â Â Â Â value &= 0xff; > > +Â Â Â Â Â Â Â Â Â Â Â Â value |= (value << 8) | (value << 16) | (value << 24); > > +Â Â Â Â Â Â Â Â Â Â Â Â break; > > +Â Â Â Â Â Â Â Â case 2: > > +Â Â Â Â Â Â Â Â Â Â Â Â value &= 0xffff; > > +Â Â Â Â Â Â Â Â Â Â Â Â value |= value << 16; > > +Â Â Â Â Â Â Â Â Â Â Â Â break; > > +Â Â Â Â Â Â Â Â } > > + > > +Â Â Â Â Â Â Â Â sctx->b.clear_buffer(ctx, dst, offset, size, value, false); > > +Â Â Â Â Â Â Â Â return; > > +Â Â Â Â } > > + > > +Â Â Â Â /* TODO: use a compute shader for other cases. */ > > What about using SDMA?Â It supports byte aligned constant fills at > least on CIK+. I think CP DMA supports byte aligned constant fills as well, I just need to test it. The bigger problem is 64 and 128 bit fills. Those can only be done with a shader AFAIK. > > > > + > > +Â Â Â Â /* Software fallback. */ > > +Â Â Â Â map = r600_buffer_map_sync_with_rings(&sctx->b, r600_resource(dst), > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â PIPE_TRANSFER_WRITE); > > +Â Â Â Â if (!map) > > +Â Â Â Â Â Â Â Â return; > > + > > +Â Â Â Â map += offset; > > +Â Â Â Â size /= clear_value_size; > > +Â Â Â Â for (i = 0; i < size; i++) { > > +Â Â Â Â Â Â Â Â memcpy(map, clear_value, clear_value_size); > > +Â Â Â Â Â Â Â Â map += clear_value_size; > > +Â Â Â Â } > > +} > > + > >Â 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; > > -- > > 2.1.4 > > > > _______________________________________________ > > mesa-dev mailing list > > <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a> > > <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a>