[Mesa-dev] [PATCH v3 41/44] i965/nir: Implement shared variable atomic operations

Iago Toral itoral at igalia.com
Tue Dec 1 04:25:28 PST 2015


Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>

On Tue, 2015-12-01 at 00:19 -0800, Jordan Justen wrote:
> v3:
>  * Update based on latest SSBO code (Iago)
> 
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> Cc: Iago Toral Quiroga <itoral at igalia.com>
> ---
>  src/mesa/drivers/dri/i965/brw_fs.h       |  2 ++
>  src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 58 ++++++++++++++++++++++++++++++++
>  2 files changed, 60 insertions(+)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h
> index bca4589..cead991 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs.h
> +++ b/src/mesa/drivers/dri/i965/brw_fs.h
> @@ -251,6 +251,8 @@ public:
>                             nir_intrinsic_instr *instr);
>     void nir_emit_ssbo_atomic(const brw::fs_builder &bld,
>                               int op, nir_intrinsic_instr *instr);
> +   void nir_emit_shared_atomic(const brw::fs_builder &bld,
> +                               int op, nir_intrinsic_instr *instr);
>     void nir_emit_texture(const brw::fs_builder &bld,
>                           nir_tex_instr *instr);
>     void nir_emit_jump(const brw::fs_builder &bld,
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> index 6cbb0e2..684376a 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> @@ -2111,6 +2111,37 @@ fs_visitor::nir_emit_cs_intrinsic(const fs_builder &bld,
>        break;
>     }
>  
> +   case nir_intrinsic_shared_atomic_add:
> +      nir_emit_shared_atomic(bld, BRW_AOP_ADD, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_imin:
> +      nir_emit_shared_atomic(bld, BRW_AOP_IMIN, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_umin:
> +      nir_emit_shared_atomic(bld, BRW_AOP_UMIN, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_imax:
> +      nir_emit_shared_atomic(bld, BRW_AOP_IMAX, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_umax:
> +      nir_emit_shared_atomic(bld, BRW_AOP_UMAX, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_and:
> +      nir_emit_shared_atomic(bld, BRW_AOP_AND, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_or:
> +      nir_emit_shared_atomic(bld, BRW_AOP_OR, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_xor:
> +      nir_emit_shared_atomic(bld, BRW_AOP_XOR, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_exchange:
> +      nir_emit_shared_atomic(bld, BRW_AOP_MOV, instr);
> +      break;
> +   case nir_intrinsic_shared_atomic_comp_swap:
> +      nir_emit_shared_atomic(bld, BRW_AOP_CMPWR, instr);
> +      break;
> +
>     default:
>        nir_emit_intrinsic(bld, instr);
>        break;
> @@ -2747,6 +2778,33 @@ fs_visitor::nir_emit_ssbo_atomic(const fs_builder &bld,
>  }
>  
>  void
> +fs_visitor::nir_emit_shared_atomic(const fs_builder &bld,
> +                                   int op, nir_intrinsic_instr *instr)
> +{
> +   fs_reg dest;
> +   if (nir_intrinsic_infos[instr->intrinsic].has_dest)
> +      dest = get_nir_dest(instr->dest);
> +
> +   fs_reg surface = brw_imm_ud(GEN7_BTI_SLM);
> +   fs_reg offset = get_nir_src(instr->src[0]);
> +   fs_reg data1 = get_nir_src(instr->src[1]);
> +   fs_reg data2;
> +   if (op == BRW_AOP_CMPWR)
> +      data2 = get_nir_src(instr->src[2]);
> +
> +   /* Emit the actual atomic operation operation */
> +
> +   fs_reg atomic_result =
> +      surface_access::emit_untyped_atomic(bld, surface, offset,
> +                                          data1, data2,
> +                                          1 /* dims */, 1 /* rsize */,
> +                                          op,
> +                                          BRW_PREDICATE_NONE);
> +   dest.type = atomic_result.type;
> +   bld.MOV(dest, atomic_result);
> +}
> +
> +void
>  fs_visitor::nir_emit_texture(const fs_builder &bld, nir_tex_instr *instr)
>  {
>     unsigned sampler = instr->sampler_index;




More information about the mesa-dev mailing list