Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752952AbcLFHRy (ORCPT ); Tue, 6 Dec 2016 02:17:54 -0500 Received: from mail-wm0-f68.google.com ([74.125.82.68]:34192 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752043AbcLFHRx (ORCPT ); Tue, 6 Dec 2016 02:17:53 -0500 Date: Tue, 6 Dec 2016 08:17:59 +0100 From: Daniel Vetter To: Mike Lothian Cc: "Deucher, Alexander" , Nicolai Stange , Daniel Vetter , Michel =?iso-8859-1?Q?D=E4nzer?= , "Koenig, Christian" , "dri-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] drm/amdgpu: don't add files at control minor debugfs directory Message-ID: <20161206071759.2yoiebqrmhckrlfz@phenom.ffwll.local> Mail-Followup-To: Mike Lothian , "Deucher, Alexander" , Nicolai Stange , Daniel Vetter , Michel =?iso-8859-1?Q?D=E4nzer?= , "Koenig, Christian" , "dri-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" References: <20161205203022.11671-1-nicstange@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Operating-System: Linux phenom 4.8.0-1-amd64 User-Agent: NeoMutt/20161104 (1.7.1) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3488 Lines: 92 On Tue, Dec 06, 2016 at 02:01:37AM +0000, Mike Lothian wrote: > Feel free to add a tested by from myself > > Thanks for the fix > > On Mon, 5 Dec 2016 at 20:33 Deucher, Alexander > wrote: > > > > -----Original Message----- > > > From: Nicolai Stange [mailto:nicstange@gmail.com] > > > Sent: Monday, December 05, 2016 3:30 PM > > > To: Daniel Vetter > > > Cc: Deucher, Alexander; Koenig, Christian; Michel D?nzer; linux- > > > kernel@vger.kernel.org; dri-devel@lists.freedesktop.org; Nicolai Stange > > > Subject: [PATCH] drm/amdgpu: don't add files at control minor debugfs > > > directory > > > > > > Since commit 8a357d10043c ("drm: Nerf DRM_CONTROL nodes"), a > > > struct drm_device's ->control member is always NULL. > > > > > > In the case of CONFIG_DEBUG_FS=y, amdgpu_debugfs_add_files() accesses > > > ->control->debugfs_root though. This results in a NULL pointer > > > dereference. > > > > > > Fix this by omitting the drm_debugfs_create_files() call for the > > > control minor debugfs directory which is now non-existent anyway. > > > > > > Fixes: 8a357d10043c ("drm: Nerf DRM_CONTROL nodes") > > > Signed-off-by: Nicolai Stange > > > > Please add the bugzilla: > > https://bugs.freedesktop.org/show_bug.cgi?id=98915 > > With that, > > Reviewed-by: Alex Deucher Applied to drm-misc, thanks for catching this and sorry for the fallout I caused. -Daniel > > > > > --- > > > Applicable to next-20161202. Compile-only tested. > > > > > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 ------ > > > 1 file changed, 6 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > > index deee2db..0cb3e82 100644 > > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > > @@ -2493,9 +2493,6 @@ int amdgpu_debugfs_add_files(struct > > > amdgpu_device *adev, > > > adev->debugfs_count = i; > > > #if defined(CONFIG_DEBUG_FS) > > > drm_debugfs_create_files(files, nfiles, > > > - adev->ddev->control->debugfs_root, > > > - adev->ddev->control); > > > - drm_debugfs_create_files(files, nfiles, > > > adev->ddev->primary->debugfs_root, > > > adev->ddev->primary); > > > #endif > > > @@ -2510,9 +2507,6 @@ static void amdgpu_debugfs_remove_files(struct > > > amdgpu_device *adev) > > > for (i = 0; i < adev->debugfs_count; i++) { > > > drm_debugfs_remove_files(adev->debugfs[i].files, > > > adev->debugfs[i].num_files, > > > - adev->ddev->control); > > > - drm_debugfs_remove_files(adev->debugfs[i].files, > > > - adev->debugfs[i].num_files, > > > adev->ddev->primary); > > > } > > > #endif > > > -- > > > 2.10.2 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch