[Mesa-dev] Was: Re: [PATCH] r600g: Add support for PK2H/UP2H

Dieter Nützel Dieter at nuetzel-hh.de
Wed Sep 21 21:20:32 UTC 2016


Ping. - Again.

Ilia and Marek voted for it.

Any progress?
Anyone, Marek, Nicolai?
Should I rebase?

Dieter

> [Mesa-dev] [PATCH] r600g: Add support for PK2H/UP2H
> 
> Glenn Kennard glenn.kennard at gmail.com
> Sun Jan 3 14:47:18 PST 2016
> Previous message: [Mesa-dev] [PATCH 1/2] WIP gallivm: add support for
> PK2H/UP2H Next message: [Mesa-dev] [PATCH] mesa: use gl_shader_variable 
> in
> program resource list Messages sorted by: [ date ] [ thread ] [ subject 
> ] [
> author ]
> Based off of Ilia's original patch, but with output values replicated 
> so
> that it matches the TGSI semantics.
> 
> Signed-off-by: Glenn Kennard <glenn.kennard at gmail.com>
> ---
> 
>  src/gallium/drivers/r600/r600_pipe.c   |   2 +-
>  src/gallium/drivers/r600/r600_shader.c | 107
>  +++++++++++++++++++++++++++++++-- 2 files changed, 104 insertions(+), 
> 5
>  deletions(-)
> 
> diff --git a/src/gallium/drivers/r600/r600_pipe.c
> b/src/gallium/drivers/r600/r600_pipe.c index d71082f..3b5d26c 100644
> --- a/src/gallium/drivers/r600/r600_pipe.c
> +++ b/src/gallium/drivers/r600/r600_pipe.c
> @@ -328,6 +328,7 @@ static int r600_get_param(struct pipe_screen* 
> pscreen,
> enum pipe_cap param)>
>  	case PIPE_CAP_TEXTURE_QUERY_LOD:
>  	case PIPE_CAP_TGSI_FS_FINE_DERIVATIVE:
> 
>  	case PIPE_CAP_SAMPLER_VIEW_TARGET:
> +	case PIPE_CAP_TGSI_PACK_HALF_FLOAT:
>  		return family >= CHIP_CEDAR ? 1 : 0;
>   	case PIPE_CAP_MAX_TEXTURE_GATHER_COMPONENTS:
>  		return family >= CHIP_CEDAR ? 4 : 0;
> 
> @@ -349,7 +350,6 @@ static int r600_get_param(struct pipe_screen* 
> pscreen,
> enum pipe_cap param)>
>  	case PIPE_CAP_SHAREABLE_SHADERS:
>  	case PIPE_CAP_CLEAR_TEXTURE:
> 
>  	case PIPE_CAP_DRAW_PARAMETERS:
> -	case PIPE_CAP_TGSI_PACK_HALF_FLOAT:
>  		return 0;
>   	case PIPE_CAP_MAX_SHADER_PATCH_VARYINGS:
> diff --git a/src/gallium/drivers/r600/r600_shader.c
> b/src/gallium/drivers/r600/r600_shader.c index 9c040ae..7b1eade 100644
> --- a/src/gallium/drivers/r600/r600_shader.c
> +++ b/src/gallium/drivers/r600/r600_shader.c
> @@ -8960,6 +8960,105 @@ static int tgsi_umad(struct r600_shader_ctx 
> *ctx)
> 
>  	return 0;
> 
>  }
> 
> +static int tgsi_pk2h(struct r600_shader_ctx *ctx)
> +{
> +	struct tgsi_full_instruction *inst =
> &ctx->parse.FullToken.FullInstruction;
> +	struct r600_bytecode_alu alu;
> +	int r, i;
> +	int lasti = tgsi_last_instruction(inst->Dst[0].Register.WriteMask);
> +
> +	/* temp.xy = f32_to_f16(src) */
> +	memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +	alu.op = ALU_OP1_FLT32_TO_FLT16;
> +	alu.dst.chan = 0;
> +	alu.dst.sel = ctx->temp_reg;
> +	alu.dst.write = 1;
> +	r600_bytecode_src(&alu.src[0], &ctx->src[0], 0);
> +	r = r600_bytecode_add_alu(ctx->bc, &alu);
> +	if (r)
> +		return r;
> +	alu.dst.chan = 1;
> +	r600_bytecode_src(&alu.src[0], &ctx->src[0], 1);
> +	alu.last = 1;
> +	r = r600_bytecode_add_alu(ctx->bc, &alu);
> +	if (r)
> +		return r;
> +
> +	/* dst.x = temp.y * 0x10000 + temp.x */
> +	for (i = 0; i < lasti + 1; i++) {
> +		if (!(inst->Dst[0].Register.WriteMask & (1 << i)))
> +			continue;
> +
> +		memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +		alu.op = ALU_OP3_MULADD_UINT24;
> +		alu.is_op3 = 1;
> +		tgsi_dst(ctx, &inst->Dst[0], i, &alu.dst);
> +		alu.last = i == lasti;
> +		alu.src[0].sel = ctx->temp_reg;
> +		alu.src[0].chan = 1;
> +		alu.src[1].sel = V_SQ_ALU_SRC_LITERAL;
> +		alu.src[1].value = 0x10000;
> +		alu.src[2].sel = ctx->temp_reg;
> +		alu.src[2].chan = 0;
> +		r = r600_bytecode_add_alu(ctx->bc, &alu);
> +		if (r)
> +			return r;
> +	}
> +
> +	return 0;
> +}
> +
> +static int tgsi_up2h(struct r600_shader_ctx *ctx)
> +{
> +	struct tgsi_full_instruction *inst =
> &ctx->parse.FullToken.FullInstruction;
> +	struct r600_bytecode_alu alu;
> +	int r, i;
> +	int lasti = tgsi_last_instruction(inst->Dst[0].Register.WriteMask);
> +
> +	/* temp.x = src.x */
> +	/* note: no need to mask out the high bits */
> +	memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +	alu.op = ALU_OP1_MOV;
> +	alu.dst.chan = 0;
> +	alu.dst.sel = ctx->temp_reg;
> +	alu.dst.write = 1;
> +	r600_bytecode_src(&alu.src[0], &ctx->src[0], 0);
> +	r = r600_bytecode_add_alu(ctx->bc, &alu);
> +	if (r)
> +		return r;
> +
> +	/* temp.y = src.x >> 16 */
> +	memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +	alu.op = ALU_OP2_LSHR_INT;
> +	alu.dst.chan = 1;
> +	alu.dst.sel = ctx->temp_reg;
> +	alu.dst.write = 1;
> +	r600_bytecode_src(&alu.src[0], &ctx->src[0], 0);
> +	alu.src[1].sel = V_SQ_ALU_SRC_LITERAL;
> +	alu.src[1].value = 16;
> +	alu.last = 1;
> +	r = r600_bytecode_add_alu(ctx->bc, &alu);
> +	if (r)
> +		return r;
> +
> +	/* dst.wz = dst.xy = f16_to_f32(temp.xy) */
> +	for (i = 0; i < lasti + 1; i++) {
> +		if (!(inst->Dst[0].Register.WriteMask & (1 << i)))
> +			continue;
> +		memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +		tgsi_dst(ctx, &inst->Dst[0], i, &alu.dst);
> +		alu.op = ALU_OP1_FLT16_TO_FLT32;
> +		alu.src[0].sel = ctx->temp_reg;
> +		alu.src[0].chan = i % 2;
> +		alu.last = i == lasti;
> +		r = r600_bytecode_add_alu(ctx->bc, &alu);
> +		if (r)
> +			return r;
> +	}
> +
> +	return 0;
> +}
> +
> 
>  static const struct r600_shader_tgsi_instruction
>  r600_shader_tgsi_instruction[] = {>
>  	[TGSI_OPCODE_ARL]	= { ALU_OP0_NOP, tgsi_r600_arl},
>  	[TGSI_OPCODE_MOV]	= { ALU_OP1_MOV, tgsi_op2},
> 
> @@ -9206,7 +9305,7 @@ static const struct r600_shader_tgsi_instruction
> eg_shader_tgsi_instruction[] =>
>  	[TGSI_OPCODE_DDX]	= { FETCH_OP_GET_GRADIENTS_H, tgsi_tex},
>  	[TGSI_OPCODE_DDY]	= { FETCH_OP_GET_GRADIENTS_V, tgsi_tex},
>  	[TGSI_OPCODE_KILL]	= { ALU_OP2_KILLGT, tgsi_kill},  /* unconditional 
> kill
>  	*/
> 
> -	[TGSI_OPCODE_PK2H]	= { ALU_OP0_NOP, tgsi_unsupported},
> +	[TGSI_OPCODE_PK2H]	= { ALU_OP0_NOP, tgsi_pk2h},
> 
>  	[TGSI_OPCODE_PK2US]	= { ALU_OP0_NOP, tgsi_unsupported},
>  	[TGSI_OPCODE_PK4B]	= { ALU_OP0_NOP, tgsi_unsupported},
>  	[TGSI_OPCODE_PK4UB]	= { ALU_OP0_NOP, tgsi_unsupported},
> 
> @@ -9221,7 +9320,7 @@ static const struct r600_shader_tgsi_instruction
> eg_shader_tgsi_instruction[] =>
>  	[TGSI_OPCODE_TEX]	= { FETCH_OP_SAMPLE, tgsi_tex},
>  	[TGSI_OPCODE_TXD]	= { FETCH_OP_SAMPLE_G, tgsi_tex},
>  	[TGSI_OPCODE_TXP]	= { FETCH_OP_SAMPLE, tgsi_tex},
> 
> -	[TGSI_OPCODE_UP2H]	= { ALU_OP0_NOP, tgsi_unsupported},
> +	[TGSI_OPCODE_UP2H]	= { ALU_OP0_NOP, tgsi_up2h},
> 
>  	[TGSI_OPCODE_UP2US]	= { ALU_OP0_NOP, tgsi_unsupported},
>  	[TGSI_OPCODE_UP4B]	= { ALU_OP0_NOP, tgsi_unsupported},
>  	[TGSI_OPCODE_UP4UB]	= { ALU_OP0_NOP, tgsi_unsupported},
> 
> @@ -9428,7 +9527,7 @@ static const struct r600_shader_tgsi_instruction
> cm_shader_tgsi_instruction[] =>
>  	[TGSI_OPCODE_DDX]	= { FETCH_OP_GET_GRADIENTS_H, tgsi_tex},
>  	[TGSI_OPCODE_DDY]	= { FETCH_OP_GET_GRADIENTS_V, tgsi_tex},
>  	[TGSI_OPCODE_KILL]	= { ALU_OP2_KILLGT, tgsi_kill},  /* unconditional 
> kill
>  	*/
> 
> -	[TGSI_OPCODE_PK2H]	= { ALU_OP0_NOP, tgsi_unsupported},
> +	[TGSI_OPCODE_PK2H]	= { ALU_OP0_NOP, tgsi_pk2h},
> 
>  	[TGSI_OPCODE_PK2US]	= { ALU_OP0_NOP, tgsi_unsupported},
>  	[TGSI_OPCODE_PK4B]	= { ALU_OP0_NOP, tgsi_unsupported},
>  	[TGSI_OPCODE_PK4UB]	= { ALU_OP0_NOP, tgsi_unsupported},
> 
> @@ -9443,7 +9542,7 @@ static const struct r600_shader_tgsi_instruction
> cm_shader_tgsi_instruction[] =>
>  	[TGSI_OPCODE_TEX]	= { FETCH_OP_SAMPLE, tgsi_tex},
>  	[TGSI_OPCODE_TXD]	= { FETCH_OP_SAMPLE_G, tgsi_tex},
>  	[TGSI_OPCODE_TXP]	= { FETCH_OP_SAMPLE, tgsi_tex},
> 
> -	[TGSI_OPCODE_UP2H]	= { ALU_OP0_NOP, tgsi_unsupported},
> +	[TGSI_OPCODE_UP2H]	= { ALU_OP0_NOP, tgsi_up2h},
> 
>  	[TGSI_OPCODE_UP2US]	= { ALU_OP0_NOP, tgsi_unsupported},
>  	[TGSI_OPCODE_UP4B]	= { ALU_OP0_NOP, tgsi_unsupported},
>  	[TGSI_OPCODE_UP4UB]	= { ALU_OP0_NOP, tgsi_unsupported},
> 
> --
> 1.9.1
_______________________________________________
mesa-dev mailing list
mesa-dev at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list