Skip to content

Commit 095647b

Browse files
committed
kompute: implement op_getrows_f32
op_getrows_f32 is required since ggml-org#6122 for the Vulkan w/ Kompute backend to be functional. As such, implement this op to make this backend functional again.
1 parent 37e7854 commit 095647b

File tree

3 files changed

+46
-1
lines changed

3 files changed

+46
-1
lines changed

CMakeLists.txt

+2
Original file line numberDiff line numberDiff line change
@@ -709,6 +709,7 @@ if (LLAMA_KOMPUTE)
709709
kompute-shaders/op_mul_mat_q4_0.comp
710710
kompute-shaders/op_mul_mat_q4_1.comp
711711
kompute-shaders/op_mul_mat_q6_k.comp
712+
kompute-shaders/op_getrows_f32.comp
712713
kompute-shaders/op_getrows_f16.comp
713714
kompute-shaders/op_getrows_q4_0.comp
714715
kompute-shaders/op_getrows_q4_1.comp
@@ -741,6 +742,7 @@ if (LLAMA_KOMPUTE)
741742
shaderop_mul_mat_q4_0.h
742743
shaderop_mul_mat_q4_1.h
743744
shaderop_mul_mat_q6_k.h
745+
shaderop_getrows_f32.h
744746
shaderop_getrows_f16.h
745747
shaderop_getrows_q4_0.h
746748
shaderop_getrows_q4_1.h

ggml-kompute.cpp

+13-1
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
#include "shaderop_mul_mat_q4_1.h"
2323
#include "shaderop_mul_mat_q6_k.h"
2424
#include "shaderop_mul_mat_mat_f32.h"
25+
#include "shaderop_getrows_f32.h"
2526
#include "shaderop_getrows_f16.h"
2627
#include "shaderop_getrows_q4_0.h"
2728
#include "shaderop_getrows_q4_1.h"
@@ -1146,6 +1147,14 @@ static void ggml_vk_get_rows(
11461147
seq.record<kp::OpAlgoDispatch>(s_algo);
11471148
}
11481149

1150+
template <typename... Args>
1151+
static void ggml_vk_get_rows_f32(Args&&... args) {
1152+
const static auto spirv = getSpirvShader(kp::shader_data::op_getrows_f32_comp_spv,
1153+
kp::shader_data::op_getrows_f32_comp_spv_len);
1154+
1155+
ggml_vk_get_rows(spirv, "f32", sizeof(float), 0, std::forward<Args>(args)...);
1156+
}
1157+
11491158
template <typename... Args>
11501159
static void ggml_vk_get_rows_f16(Args&&... args) {
11511160
const static auto spirv = getSpirvShader(kp::shader_data::op_getrows_f16_comp_spv,
@@ -1371,6 +1380,7 @@ static bool ggml_vk_supports_op(const struct ggml_tensor * op) {
13711380
return op->ne[3] == 1;
13721381
case GGML_OP_GET_ROWS:
13731382
switch (op->src[0]->type) {
1383+
case GGML_TYPE_F32:
13741384
case GGML_TYPE_F16:
13751385
case GGML_TYPE_Q4_0:
13761386
case GGML_TYPE_Q4_1:
@@ -1649,7 +1659,9 @@ static void ggml_vk_graph_compute(struct ggml_kompute_context * ctx, struct ggml
16491659
} break;
16501660
case GGML_OP_GET_ROWS:
16511661
{
1652-
if (src0t == GGML_TYPE_F16) {
1662+
if (src0t == GGML_TYPE_F32) {
1663+
ggml_vk_get_rows_f32(seq, id_src0, id_src1, id_dst, off_src0, off_src1, off_dst, ne00, nb01, nb1, ggml_nelements(src1));
1664+
} else if (src0t == GGML_TYPE_F16) {
16531665
ggml_vk_get_rows_f16(seq, id_src0, id_src1, id_dst, off_src0, off_src1, off_dst, ne00, nb01, nb1, ggml_nelements(src1));
16541666
} else if (src0t == GGML_TYPE_Q4_0) {
16551667
ggml_vk_get_rows_q4_0(seq, id_src0, id_src1, id_dst, off_src0, off_src1, off_dst, ne00, nb01, nb1, ggml_nelements(src1));

kompute-shaders/op_getrows_f32.comp

+31
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
#version 450
2+
3+
#include "common.comp"
4+
5+
layout(local_size_x = 1) in;
6+
7+
layout (binding = 0) readonly buffer tensorInA { float inA[]; };
8+
layout (binding = 1) readonly buffer tensorInB { int inB[]; };
9+
layout (binding = 2) writeonly buffer tensorOut { float out_[]; };
10+
11+
layout (push_constant) uniform parameter {
12+
uint inAOff;
13+
uint inBOff;
14+
uint outOff;
15+
int ne00;
16+
int nb01;
17+
int nb1;
18+
} pcs;
19+
20+
void dequantize_row_f32(uint x /*Based from inA unaligned*/, uint y /*Based from out_*/, int k) {
21+
for (int j = 0; j < k; j++) {
22+
out_[y + j] = inA[x + j];
23+
}
24+
}
25+
26+
void main() {
27+
const uint i = gl_WorkGroupID.x;
28+
const int r = inB[i + pcs.inBOff];
29+
30+
dequantize_row_f32(r*pcs.nb01/4 + pcs.inAOff, i*pcs.nb1/4 + pcs.outOff, pcs.ne00);
31+
}

0 commit comments

Comments
 (0)