On 2019-10-24 14:46, Sierra Guiza, Alejandro (Alex) wrote: > The bitmap in cu_info structure is defined as a 4x4 size array. In > Acturus, this matrix is initialized as a 4x2. Based on the 8 shaders. > In the gpu cache filling initialization, the access to the bitmap matrix > was done as an 8x1 instead of 4x2. Causing an out of bounds memory > access error. > Due to this, the number of GPU cache entries was inconsistent. > > Signed-off-by: Alex Sierra <alex.sierra@xxxxxxx> Reviewed-by: Felix Kuehling <Felix.Kuehling@xxxxxxx> > --- > drivers/gpu/drm/amd/amdkfd/kfd_crat.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_crat.c b/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > index 0c327e0fc0f7..de9f68d5c312 100644 > --- a/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > +++ b/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > @@ -710,7 +710,7 @@ static int kfd_fill_gpu_cache_info(struct kfd_dev *kdev, > pcache_info, > cu_info, > mem_available, > - cu_info->cu_bitmap[i][j], > + cu_info->cu_bitmap[i % 4][j + i / 4], > ct, > cu_processor_id, > k); _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
