Skip to content

Commit 1384abf

Browse files
authored
vulkan: handle mat_mul with A matrix > 4GB (#16176)
* vulkan: handle mat_mul with A matrix > 4GB This change splits mat_mul operations with huge A matrix into chunks in the M dimension. This works well for stable-diffusion use cases where the im2col matrix has very large M. Fix the order of setting the stride in mul_mm_cm2 - setting the dimension clobbers the stride, so stride should be set after. * build fixes
1 parent e6d65fb commit 1384abf

File tree

3 files changed

+59
-15
lines changed

3 files changed

+59
-15
lines changed

ggml/src/ggml-vulkan/ggml-vulkan.cpp

Lines changed: 49 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -5661,8 +5661,12 @@ static void ggml_vk_buffer_memset(vk_buffer& dst, size_t offset, uint32_t c, siz
56615661
ggml_vk_queue_command_pools_cleanup(dst->device);
56625662
}
56635663

5664-
static uint32_t ggml_vk_guess_split_k(ggml_backend_vk_context * ctx, uint32_t m, uint32_t n, uint32_t k, const vk_pipeline& pipeline) {
5665-
VK_LOG_DEBUG("ggml_vk_guess_split_k(" << m << ", " << n << ", " << k << ")");
5664+
static uint32_t ggml_vk_guess_split_k(ggml_backend_vk_context * ctx, uint32_t m, uint32_t n, uint32_t k, bool disable_split_k, const vk_pipeline& pipeline) {
5665+
VK_LOG_DEBUG("ggml_vk_guess_split_k(" << m << ", " << n << ", " << k << ", " << disable_split_k << ")");
5666+
5667+
if (disable_split_k) {
5668+
return 1;
5669+
}
56665670

56675671
uint32_t split_k = 1;
56685672
if (ctx->device->shader_core_count != 0 && m >= pipeline->wg_denoms[0] && n >= pipeline->wg_denoms[1]) {
@@ -5987,7 +5991,7 @@ static void ggml_vk_quantize_q8_1(ggml_backend_vk_context * ctx, vk_context& sub
59875991
ggml_vk_sync_buffers(ctx, subctx);
59885992
}
59895993

5990-
static void ggml_vk_mul_mat_q_f16(ggml_backend_vk_context * ctx, vk_context& subctx, const ggml_tensor * src0, const ggml_tensor * src1, ggml_tensor * dst, bool dryrun = false) {
5994+
static void ggml_vk_mul_mat_q_f16(ggml_backend_vk_context * ctx, vk_context& subctx, const ggml_tensor * src0, const ggml_tensor * src1, ggml_tensor * dst, bool disable_split_k, bool dryrun = false) {
59915995
VK_LOG_DEBUG("ggml_vk_mul_mat_q_f16((" << src0 << ", name=" << src0->name << ", type=" << ggml_type_name(src0->type) << ", ne0=" << src0->ne[0] << ", ne1=" << src0->ne[1] << ", ne2=" << src0->ne[2] << ", ne3=" << src0->ne[3] << ", nb0=" << src0->nb[0] << ", nb1=" << src0->nb[1] << ", nb2=" << src0->nb[2] << ", nb3=" << src0->nb[3];
59925996
std::cerr << "), (" << src1 << ", name=" << src1->name << ", type=" << ggml_type_name(src1->type) << ", ne0=" << src1->ne[0] << ", ne1=" << src1->ne[1] << ", ne2=" << src1->ne[2] << ", ne3=" << src1->ne[3] << ", nb0=" << src1->nb[0] << ", nb1=" << src1->nb[1] << ", nb2=" << src1->nb[2] << ", nb3=" << src1->nb[3];
59935997
std::cerr << "), (" << dst << ", name=" << dst->name << ", type=" << ggml_type_name(dst->type) << ", ne0=" << dst->ne[0] << ", ne1=" << dst->ne[1] << ", ne2=" << dst->ne[2] << ", ne3=" << dst->ne[3] << ", nb0=" << dst->nb[0] << ", nb1=" << dst->nb[1] << ", nb2=" << dst->nb[2] << ", nb3=" << dst->nb[3];
@@ -6005,8 +6009,9 @@ static void ggml_vk_mul_mat_q_f16(ggml_backend_vk_context * ctx, vk_context& sub
60056009
const uint64_t ne12 = src1->ne[2];
60066010
const uint64_t ne13 = src1->ne[3];
60076011

6008-
const uint64_t ne20 = dst->ne[0];
60096012
const uint64_t ne21 = dst->ne[1];
6013+
const uint32_t stride_d = dst->nb[1] / ggml_type_size(dst->type);
6014+
const uint32_t stride_batch_d = stride_d*ne21;
60106015

60116016
const uint64_t r2 = ne12 / ne02;
60126017
const uint64_t r3 = ne13 / ne03;
@@ -6075,7 +6080,7 @@ static void ggml_vk_mul_mat_q_f16(ggml_backend_vk_context * ctx, vk_context& sub
60756080
const int y_ne = padded_n * ne10;
60766081
const int d_ne = ne11 * ne01;
60776082

6078-
const uint32_t split_k = ggml_vk_guess_split_k(ctx, ne01, ne11, ne10, pipeline);
6083+
const uint32_t split_k = ggml_vk_guess_split_k(ctx, ne01, ne11, ne10, disable_split_k, pipeline);
60796084

60806085
const uint64_t qx_sz = ggml_type_size(src0->type) * x_ne / ggml_blck_size(src0->type);
60816086
const uint64_t qy_sz = ggml_type_size(src1->type) * y_ne / ggml_blck_size(src1->type);
@@ -6234,13 +6239,16 @@ static void ggml_vk_mul_mat_q_f16(ggml_backend_vk_context * ctx, vk_context& sub
62346239
y_sz_total = CEIL_DIV(y_sz_total, 144) * 144;
62356240
}
62366241

6242+
// No bounds checking is needed for dst. This is basically VK_WHOLE_SIZE but clamped to maxStorageBufferRange.
6243+
VkDeviceSize d_range = std::min(VkDeviceSize{d_D->size - d_buf_offset}, VkDeviceSize{ctx->device->properties.limits.maxStorageBufferRange});
6244+
62376245
// compute
62386246
ggml_vk_matmul(
62396247
ctx, subctx, pipeline,
62406248
{ d_X, x_buf_offset, x_sz * ne02 * ne03 }, { d_Y, y_buf_offset, y_sz_total },
6241-
{ d_D, d_buf_offset, d_sz * ne12 * ne13 }, { ctx->prealloc_split_k, 0, d_sz * ne12 * ne13 * split_k },
6249+
{ d_D, d_buf_offset, d_range }, { ctx->prealloc_split_k, 0, d_sz * ne12 * ne13 * split_k },
62426250
ne01, ne11, ne10,
6243-
ne10, ne10, ne01, stride_batch_x, stride_batch_y, ne20*ne21,
6251+
ne10, ne10, stride_d, stride_batch_x, stride_batch_y, stride_batch_d,
62446252
split_k, ne12*ne13, ne02, ne12, r2, r3, padded_n
62456253
); // NOLINT
62466254

@@ -6718,9 +6726,36 @@ static void ggml_vk_mul_mat_vec_nc_f16_f32(ggml_backend_vk_context * ctx, vk_con
67186726
{ vk_subbuffer{ d_Qx, qx_buf_offset, qx_sz }, vk_subbuffer{ d_Qy, qy_buffer_offset, qy_sz + qy_shader_offset }, vk_subbuffer{ d_D, d_buffer_offset, d_sz + d_shader_offset } }, pc, { (uint32_t)ne03, (uint32_t)ne01, (uint32_t)ne12 });
67196727
}
67206728

6721-
static void ggml_vk_mul_mat(ggml_backend_vk_context * ctx, vk_context& subctx, const ggml_tensor * src0, const ggml_tensor * src1, ggml_tensor * dst, bool dryrun = false) {
6729+
static void ggml_vk_mul_mat(ggml_backend_vk_context * ctx, vk_context& subctx, ggml_tensor * src0, ggml_tensor * src1, ggml_tensor * dst, bool dryrun = false) {
67226730
VK_LOG_DEBUG("ggml_vk_mul_mat(" << src0 << ", " << src1 << ", " << dst << ")");
6723-
if (src0->type == GGML_TYPE_F16 && ggml_is_permuted(src0) && ggml_is_permuted(src1) && dst->ne[1] == 1 &&
6731+
6732+
// Handle huge A matrix by splitting the M dimensions. This works well for convolution use cases
6733+
// where the M dimension is very large.
6734+
// Split_k doesn't work with M splitting.
6735+
const size_t nbytes = ggml_nbytes(src0);
6736+
const bool needs_split = nbytes > ctx->device->properties.limits.maxStorageBufferRange;
6737+
if (needs_split) {
6738+
// Choose the number of rows that can fit (and divide by two, to allow for any additional offsets)
6739+
const uint32_t M_split = ctx->device->properties.limits.maxStorageBufferRange / (2 * src0->nb[1]);
6740+
uint32_t m_offset = 0;
6741+
while (m_offset < dst->ne[0]) {
6742+
const uint32_t cur_M_size = std::min(M_split, (uint32_t)(dst->ne[0] - m_offset));
6743+
ggml_tensor dst2 = *dst;
6744+
ggml_tensor src02 = *src0;
6745+
6746+
dst2.view_src = dst->view_src ? dst->view_src : dst;
6747+
src02.view_src = src0->view_src ? src0->view_src : src0;
6748+
6749+
dst2.view_offs += m_offset * dst->nb[0];
6750+
src02.view_offs += m_offset * src0->nb[1];
6751+
dst2.ne[0] = cur_M_size;
6752+
src02.ne[1] = cur_M_size;
6753+
6754+
ggml_vk_mul_mat_q_f16(ctx, subctx, &src02, src1, &dst2, true, dryrun);
6755+
6756+
m_offset += cur_M_size;
6757+
}
6758+
} else if (src0->type == GGML_TYPE_F16 && ggml_is_permuted(src0) && ggml_is_permuted(src1) && dst->ne[1] == 1 &&
67246759
// detect 0213 permutation, and batch size of 1
67256760
src0->nb[0] <= src0->nb[2] &&
67266761
src0->nb[2] <= src0->nb[1] &&
@@ -6740,7 +6775,7 @@ static void ggml_vk_mul_mat(ggml_backend_vk_context * ctx, vk_context& subctx, c
67406775
(src0->type == GGML_TYPE_F32 || src0->type == GGML_TYPE_F16 || src0->type == GGML_TYPE_BF16 || ggml_is_quantized(src0->type))) {
67416776
ggml_vk_mul_mat_vec_q_f16(ctx, subctx, src0, src1, dst, dryrun);
67426777
} else {
6743-
ggml_vk_mul_mat_q_f16(ctx, subctx, src0, src1, dst, dryrun);
6778+
ggml_vk_mul_mat_q_f16(ctx, subctx, src0, src1, dst, false, dryrun);
67446779
}
67456780
}
67466781

@@ -10675,10 +10710,10 @@ static bool ggml_vk_build_graph(ggml_backend_vk_context * ctx, ggml_cgraph * cgr
1067510710
VK_LOG_DEBUG("ggml_vk_build_graph(" << node << ", " << ggml_op_name(node->op) << ")");
1067610711
ctx->semaphore_idx = 0;
1067710712

10678-
const ggml_tensor * src0 = node->src[0];
10679-
const ggml_tensor * src1 = node->src[1];
10680-
const ggml_tensor * src2 = node->src[2];
10681-
const ggml_tensor * src3 = node->src[3];
10713+
ggml_tensor * src0 = node->src[0];
10714+
ggml_tensor * src1 = node->src[1];
10715+
ggml_tensor * src2 = node->src[2];
10716+
ggml_tensor * src3 = node->src[3];
1068210717

1068310718
switch (node->op) {
1068410719
// Return on empty ops to avoid generating a compute_ctx and setting exit_tensor

ggml/src/ggml-vulkan/vulkan-shaders/mul_mm_cm2.comp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -265,7 +265,6 @@ void main() {
265265
tensorLayoutNV<2> tensorLayoutB = createTensorLayoutNV(2);
266266
tensorLayoutNV<2, gl_CooperativeMatrixClampModeConstantNV> tensorLayoutBClamp = createTensorLayoutNV(2, gl_CooperativeMatrixClampModeConstantNV);
267267
tensorLayoutNV<2, gl_CooperativeMatrixClampModeConstantNV> tensorLayoutD = createTensorLayoutNV(2, gl_CooperativeMatrixClampModeConstantNV);
268-
tensorLayoutD = setTensorLayoutStrideNV(tensorLayoutD, p.stride_d, 1);
269268

270269
#if QUANT_K > 1
271270
tensorLayoutA = setTensorLayoutBlockSizeNV(tensorLayoutA, 1, QUANT_K);
@@ -281,6 +280,8 @@ void main() {
281280
tensorLayoutAClamp = setTensorLayoutDimensionNV(tensorLayoutAClamp, p.M, end_k);
282281
tensorLayoutBClamp = setTensorLayoutDimensionNV(tensorLayoutBClamp, p.padded_N, end_k);
283282

283+
tensorLayoutD = setTensorLayoutStrideNV(tensorLayoutD, p.stride_d, 1);
284+
284285
tensorViewNV<2, false, 1, 0> tensorViewTranspose = createTensorViewNV(2, false, 1, 0);
285286

286287
#if !defined(MUL_MAT_ID)

tests/test-backend-ops.cpp

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6200,6 +6200,14 @@ static std::vector<std::unique_ptr<test_case>> make_test_cases_eval() {
62006200
test_cases.emplace_back(new test_gla(GGML_TYPE_F32, 32, 64, 32, 4));
62016201
test_cases.emplace_back(new test_gla(GGML_TYPE_F32, 32, 64, 128, 4));
62026202

6203+
#if 0
6204+
// > 4GB A matrix. Too slow to be enabled by default.
6205+
test_cases.emplace_back(new test_mul_mat(GGML_TYPE_F16, GGML_TYPE_F16, 900000, 3, 2592, {1, 1}, {1, 1}));
6206+
test_cases.emplace_back(new test_mul_mat(GGML_TYPE_F16, GGML_TYPE_F16, 1700000, 96, 2592, {1, 1}, {1, 1}));
6207+
test_cases.emplace_back(new test_mul_mat(GGML_TYPE_F16, GGML_TYPE_F16, 1700000, 3, 2592, {1, 1}, {1, 1}));
6208+
test_cases.emplace_back(new test_mul_mat(GGML_TYPE_F16, GGML_TYPE_F16, 1700000, 1, 2592, {1, 1}, {1, 1}));
6209+
#endif
6210+
62036211
for (ggml_type type_a : all_types) {
62046212
for (int i = 1; i < 10; ++i) {
62056213
test_cases.emplace_back(new test_mul_mat(type_a, GGML_TYPE_F32, 16, i, 256, { 1, 1}, {1, 1}));

0 commit comments

Comments
 (0)