[PATCH] drm/radeon: don't add files at control minor debugfs directory

2016-12-05 Thread Michel Dänzer
On 05/12/16 05:48 PM, Christian König wrote: > Am 05.12.2016 um 09:39 schrieb Nicolai Stange:>> >> I'll send compile-only tested patches either tonight or tomorrow. Shall >> they cover the oopses only or the dead code as well? > > Please start with the ops, cause we certainly will get complains a

[PATCH] drm/radeon: don't add files at control minor debugfs directory

2016-12-05 Thread Daniel Vetter
On Mon, Dec 05, 2016 at 09:48:02AM +0100, Christian König wrote: > Am 05.12.2016 um 09:39 schrieb Nicolai Stange: > > Christian König writes: > > > > > Am 05.12.2016 um 08:27 schrieb Daniel Vetter: > > > > On Sat, Dec 03, 2016 at 03:47:00PM +0100, Nicolai Stange wrote: > > > > > Since commit 8a

[PATCH] drm/radeon: don't add files at control minor debugfs directory

2016-12-05 Thread Christian König
Am 05.12.2016 um 09:39 schrieb Nicolai Stange: > Christian König writes: > >> Am 05.12.2016 um 08:27 schrieb Daniel Vetter: >>> On Sat, Dec 03, 2016 at 03:47:00PM +0100, Nicolai Stange wrote: Since commit 8a357d10043c ("drm: Nerf DRM_CONTROL nodes"), a struct drm_device's ->control memb

[PATCH] drm/radeon: don't add files at control minor debugfs directory

2016-12-05 Thread Nicolai Stange
Christian König writes: > Am 05.12.2016 um 08:27 schrieb Daniel Vetter: >> On Sat, Dec 03, 2016 at 03:47:00PM +0100, Nicolai Stange wrote: >>> Since commit 8a357d10043c ("drm: Nerf DRM_CONTROL nodes"), a >>> struct drm_device's ->control member is always NULL. >>> >>> In the case of CONFIG_DEBUG

[PATCH] drm/radeon: don't add files at control minor debugfs directory

2016-12-05 Thread Daniel Vetter
On Mon, Dec 05, 2016 at 08:42:49AM +0100, Christian König wrote: > Am 05.12.2016 um 08:27 schrieb Daniel Vetter: > > On Sat, Dec 03, 2016 at 03:47:00PM +0100, Nicolai Stange wrote: > > > Since commit 8a357d10043c ("drm: Nerf DRM_CONTROL nodes"), a > > > struct drm_device's ->control member is alwa

[PATCH] drm/radeon: don't add files at control minor debugfs directory

2016-12-05 Thread Christian König
Am 05.12.2016 um 08:27 schrieb Daniel Vetter: > On Sat, Dec 03, 2016 at 03:47:00PM +0100, Nicolai Stange wrote: >> 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, radeon_debugfs_add_files()

[PATCH] drm/radeon: don't add files at control minor debugfs directory

2016-12-05 Thread Daniel Vetter
On Sat, Dec 03, 2016 at 03:47:00PM +0100, Nicolai Stange wrote: > 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, radeon_debugfs_add_files() accesses > ->control->debugfs_root though. This res

[PATCH] drm/radeon: don't add files at control minor debugfs directory

2016-12-03 Thread Nicolai Stange
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, radeon_debugfs_add_files() accesses ->control->debugfs_root though. This results in the following Oops: BUG: unable to handle kernel NULL pointer d