[Mesa-dev] [PATCH 14/14] i965/gen7-8: Implement glMemoryBarrier().

[Francisco Jerez]

Kristian Høgsberg <krh at bitplanet.net> writes:

> On Fri, Feb 6, 2015 at 9:23 AM, Francisco Jerez <currojerez at riseup.net> wrote:
>> ---
>>  src/mesa/drivers/dri/i965/brw_program.c | 40 +++++++++++++++++++++++++++++++++
>>  src/mesa/drivers/dri/i965/intel_reg.h   |  1 +
>>  2 files changed, 41 insertions(+)
>>
>> diff --git a/src/mesa/drivers/dri/i965/brw_program.c b/src/mesa/drivers/dri/i965/brw_program.c
>> index d9a3f05..793d963 100644
>> --- a/src/mesa/drivers/dri/i965/brw_program.c
>> +++ b/src/mesa/drivers/dri/i965/brw_program.c
>> @@ -44,6 +44,7 @@
>>  #include "brw_context.h"
>>  #include "brw_shader.h"
>>  #include "brw_wm.h"
>> +#include "intel_batchbuffer.h"
>>
>>  static unsigned
>>  get_new_program_id(struct intel_screen *screen)
>> @@ -179,6 +180,43 @@ brwProgramStringNotify(struct gl_context *ctx,
>>     return true;
>>  }
>>
>> +static void
>> +brwMemoryBarrier(struct gl_context *ctx, GLbitfield barriers)
>> +{
>> +   struct brw_context *brw = brw_context(ctx);
>> +   unsigned bits = (PIPE_CONTROL_DATA_CACHE_INVALIDATE |
>> +                    PIPE_CONTROL_NO_WRITE |
>> +                    PIPE_CONTROL_CS_STALL);
>> +   assert(brw->gen >= 7 && brw->gen <= 8);
>> +
>> +   if (barriers & (GL_VERTEX_ATTRIB_ARRAY_BARRIER_BIT |
>> +                   GL_ELEMENT_ARRAY_BARRIER_BIT |
>> +                   GL_COMMAND_BARRIER_BIT))
>> +      bits |= PIPE_CONTROL_VF_CACHE_INVALIDATE;
>> +
>> +   if (barriers & GL_UNIFORM_BARRIER_BIT)
>> +      bits |= (PIPE_CONTROL_TC_FLUSH |
>> +               PIPE_CONTROL_CONST_CACHE_INVALIDATE);
>> +
>> +   if (barriers & GL_TEXTURE_FETCH_BARRIER_BIT)
>> +      bits |= PIPE_CONTROL_TC_FLUSH;
>> +
>> +   if (barriers & GL_TEXTURE_UPDATE_BARRIER_BIT)
>> +      bits |= PIPE_CONTROL_WRITE_FLUSH;
>> +
>> +   if (barriers & GL_FRAMEBUFFER_BARRIER_BIT)
>> +      bits |= (PIPE_CONTROL_DEPTH_CACHE_FLUSH |
>> +               PIPE_CONTROL_WRITE_FLUSH);
>> +
>> +   /* Typed surface messages are handled by the render cache on IVB, so we
>> +    * need to flush it too.
>> +    */
>> +   if (brw->gen == 7 && !brw->is_haswell)
>> +      bits |= PIPE_CONTROL_WRITE_FLUSH;
>> +
>> +   brw_emit_pipe_control_flush(brw, bits);
>> +}
>> +
>>  void
>>  brw_add_texrect_params(struct gl_program *prog)
>>  {
>> @@ -236,6 +274,8 @@ void brwInitFragProgFuncs( struct dd_function_table *functions )
>>
>>     functions->NewShader = brw_new_shader;
>>     functions->LinkShader = brw_link_shader;
>> +
>> +   functions->MemoryBarrier = brwMemoryBarrier;
>
> Just a drive-by nitpick: brw_memory_barrier?
>
I'm OK with that, I've changed it locally.  I used camel case because it
seemed like all the other dd hooks implemented in the same file used it.
Apparently all other functions defined in it don't, so I guess it's
going to be inconsistent anyway.

> Kristian
>
>>  }
>>
>>  void
>> diff --git a/src/mesa/drivers/dri/i965/intel_reg.h b/src/mesa/drivers/dri/i965/intel_reg.h
>> index 5ac0180..8b630c5 100644
>> --- a/src/mesa/drivers/dri/i965/intel_reg.h
>> +++ b/src/mesa/drivers/dri/i965/intel_reg.h
>> @@ -70,6 +70,7 @@
>>  #define PIPE_CONTROL_ISP_DIS           (1 << 9)
>>  #define PIPE_CONTROL_INTERRUPT_ENABLE  (1 << 8)
>>  /* GT */
>> +#define PIPE_CONTROL_DATA_CACHE_INVALIDATE     (1 << 5)
>>  #define PIPE_CONTROL_VF_CACHE_INVALIDATE       (1 << 4)
>>  #define PIPE_CONTROL_CONST_CACHE_INVALIDATE    (1 << 3)
>>  #define PIPE_CONTROL_STATE_CACHE_INVALIDATE    (1 << 2)
>> --
>> 2.1.3
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150209/55aec552/attachment.sig>