Re: [PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-11 Thread Daniel Vetter
On Sat, Jul 08, 2017 at 11:43:52PM +0200, Alexandru Moise wrote: > If the DRM core fails to init for whatever reason, ensure that > no driver ever calls drm_dev_register(). > > This is best done at drm_dev_init() as it covers drivers that call > drm_dev_alloc() as well as drivers that prefer to em

Re: [PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-11 Thread Daniel Vetter
On Tue, Jul 11, 2017 at 11:41 AM, Greg Kroah-Hartman wrote: > On Mon, Jul 10, 2017 at 08:00:37PM +0200, Daniel Vetter wrote: >> On Mon, Jul 10, 2017 at 9:14 AM, Alexandru Moise >> <00moses.alexande...@gmail.com> wrote: >> > On Mon, Jul 10, 2017 at 08:52:46AM +0200, Daniel Vetter wrote: >> >> On Sa

Re: [PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-11 Thread Greg Kroah-Hartman
On Mon, Jul 10, 2017 at 08:00:37PM +0200, Daniel Vetter wrote: > On Mon, Jul 10, 2017 at 9:14 AM, Alexandru Moise > <00moses.alexande...@gmail.com> wrote: > > On Mon, Jul 10, 2017 at 08:52:46AM +0200, Daniel Vetter wrote: > >> On Sat, Jul 08, 2017 at 11:43:52PM +0200, Alexandru Moise wrote: > >> >

Re: [PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-11 Thread Daniel Vetter
On Mon, Jul 10, 2017 at 09:56:20PM +0200, Alexandru Moise wrote: > On Mon, Jul 10, 2017 at 08:00:37PM +0200, Daniel Vetter wrote: > > On Mon, Jul 10, 2017 at 9:14 AM, Alexandru Moise > > <00moses.alexande...@gmail.com> wrote: > > > On Mon, Jul 10, 2017 at 08:52:46AM +0200, Daniel Vetter wrote: > >

Re: [PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-10 Thread Alexandru Moise
On Mon, Jul 10, 2017 at 08:00:37PM +0200, Daniel Vetter wrote: > On Mon, Jul 10, 2017 at 9:14 AM, Alexandru Moise > <00moses.alexande...@gmail.com> wrote: > > On Mon, Jul 10, 2017 at 08:52:46AM +0200, Daniel Vetter wrote: > >> On Sat, Jul 08, 2017 at 11:43:52PM +0200, Alexandru Moise wrote: > >> >

Re: [PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-10 Thread Daniel Vetter
On Mon, Jul 10, 2017 at 9:14 AM, Alexandru Moise <00moses.alexande...@gmail.com> wrote: > On Mon, Jul 10, 2017 at 08:52:46AM +0200, Daniel Vetter wrote: >> On Sat, Jul 08, 2017 at 11:43:52PM +0200, Alexandru Moise wrote: >> > If the DRM core fails to init for whatever reason, ensure that >> > no dr

Re: [PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-10 Thread Alexandru Moise
On Mon, Jul 10, 2017 at 08:52:46AM +0200, Daniel Vetter wrote: > On Sat, Jul 08, 2017 at 11:43:52PM +0200, Alexandru Moise wrote: > > If the DRM core fails to init for whatever reason, ensure that > > no driver ever calls drm_dev_register(). > > > > This is best done at drm_dev_init() as it covers

Re: [PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-09 Thread Daniel Vetter
On Sat, Jul 08, 2017 at 11:43:52PM +0200, Alexandru Moise wrote: > If the DRM core fails to init for whatever reason, ensure that > no driver ever calls drm_dev_register(). > > This is best done at drm_dev_init() as it covers drivers that call > drm_dev_alloc() as well as drivers that prefer to em

[PATCH] drm: inhibit drm drivers register to uninitialized drm core

2017-07-08 Thread Alexandru Moise
If the DRM core fails to init for whatever reason, ensure that no driver ever calls drm_dev_register(). This is best done at drm_dev_init() as it covers drivers that call drm_dev_alloc() as well as drivers that prefer to embed struct drm_device into their own device struct and call drm_dev_init()