From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f67.google.com ([74.125.82.67]:34522 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751097AbcFQSMC (ORCPT ); Fri, 17 Jun 2016 14:12:02 -0400 Received: by mail-wm0-f67.google.com with SMTP id 187so1456960wmz.1 for ; Fri, 17 Jun 2016 11:12:01 -0700 (PDT) Subject: Re: [amd-gfx] [PATCH] drm/amdgpu: fix num_rbs exposed to userspace (v2) To: Alex Deucher , amd-gfx@lists.freedesktop.org References: <1466185923-7359-1-git-send-email-alexander.deucher@amd.com> Cc: Alex Deucher , stable@vger.kernel.org From: =?UTF-8?Q?Nicolai_H=c3=a4hnle?= Message-ID: <57643D6F.2050608@gmail.com> Date: Fri, 17 Jun 2016 20:11:59 +0200 MIME-Version: 1.0 In-Reply-To: <1466185923-7359-1-git-send-email-alexander.deucher@amd.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org List-ID: LGTM. Reviewed-by: Nicolai Hähnle On 17.06.2016 19:52, Alex Deucher wrote: > This was accidently broken for harvest cards when the > code was refactored for Polaris support. > > v2: multiply by shader engines. Noticed by Nicolai. > > Signed-off-by: Alex Deucher > Cc: stable@vger.kernel.org > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c > index 40a2370..d851ea1 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c > @@ -447,7 +447,8 @@ static int amdgpu_info_ioctl(struct drm_device *dev, void *data, struct drm_file > dev_info.max_memory_clock = adev->pm.default_mclk * 10; > } > dev_info.enabled_rb_pipes_mask = adev->gfx.config.backend_enable_mask; > - dev_info.num_rb_pipes = adev->gfx.config.num_rbs; > + dev_info.num_rb_pipes = adev->gfx.config.max_backends_per_se * > + adev->gfx.config.max_shader_engines; > dev_info.num_hw_gfx_contexts = adev->gfx.config.max_hw_contexts; > dev_info._pad = 0; > dev_info.ids_flags = 0; >