Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-12-08 Thread Daniel Vetter
On Mon, Dec 07, 2015 at 07:32:42PM +0200, Tomi Valkeinen wrote: > > On 25/08/15 16:45, Daniel Vetter wrote: > > When the usual fbcon legacy options are enabled we have > > ->register_framebuffer > > ->fb notifier chain calls into fbcon > > ->fbcon sets up console on new fbi > >

Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-12-07 Thread Tomi Valkeinen
On 25/08/15 16:45, Daniel Vetter wrote: > When the usual fbcon legacy options are enabled we have > ->register_framebuffer > ->fb notifier chain calls into fbcon > ->fbcon sets up console on new fbi > ->fbi->set_par > ->drm_fb_helper_set_par exercises full kms api > > And

Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-09-24 Thread Tomi Valkeinen
On 01/09/15 17:34, Rob Clark wrote: > I hadn't had a chance to look at it further yet.. I think Daniel > claimed it worked for him, but he was probably on intel-next, where I > was on drm-next at the time which seemed to be having some unrelated > i915 issues (when I was trying to debug atomic

Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-09-01 Thread Tomi Valkeinen
On 25/08/15 22:24, Rob Clark wrote: > On Tue, Aug 25, 2015 at 9:45 AM, Daniel Vetter wrote: >> When the usual fbcon legacy options are enabled we have >> ->register_framebuffer >> ->fb notifier chain calls into fbcon >> ->fbcon sets up console on new fbi >>

Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-09-01 Thread Rob Clark
On Tue, Sep 1, 2015 at 6:32 AM, Tomi Valkeinen wrote: > > > On 25/08/15 22:24, Rob Clark wrote: >> On Tue, Aug 25, 2015 at 9:45 AM, Daniel Vetter >> wrote: >>> When the usual fbcon legacy options are enabled we have >>> ->register_framebuffer >>>

Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-09-01 Thread Rob Clark
On Tue, Sep 1, 2015 at 10:41 AM, Tomi Valkeinen wrote: > > > On 01/09/15 17:34, Rob Clark wrote: >> On Tue, Sep 1, 2015 at 6:32 AM, Tomi Valkeinen wrote: >>> >>> >>> On 25/08/15 22:24, Rob Clark wrote: On Tue, Aug 25, 2015 at 9:45 AM, Daniel

Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-09-01 Thread Tomi Valkeinen
On 01/09/15 17:34, Rob Clark wrote: > On Tue, Sep 1, 2015 at 6:32 AM, Tomi Valkeinen wrote: >> >> >> On 25/08/15 22:24, Rob Clark wrote: >>> On Tue, Aug 25, 2015 at 9:45 AM, Daniel Vetter >>> wrote: When the usual fbcon legacy options are

Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-09-01 Thread Daniel Vetter
On Tue, Sep 01, 2015 at 11:12:11AM -0400, Rob Clark wrote: > On Tue, Sep 1, 2015 at 10:41 AM, Tomi Valkeinen wrote: > > > > > > On 01/09/15 17:34, Rob Clark wrote: > >> On Tue, Sep 1, 2015 at 6:32 AM, Tomi Valkeinen > >> wrote: > >>> > >>> > >>> On

[Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-08-25 Thread Daniel Vetter
When the usual fbcon legacy options are enabled we have -register_framebuffer -fb notifier chain calls into fbcon -fbcon sets up console on new fbi -fbi-set_par -drm_fb_helper_set_par exercises full kms api And because of locking inversion hilarity all of register_framebuffer

Re: [Intel-gfx] [PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-08-25 Thread Rob Clark
On Tue, Aug 25, 2015 at 9:45 AM, Daniel Vetter daniel.vet...@ffwll.ch wrote: When the usual fbcon legacy options are enabled we have -register_framebuffer -fb notifier chain calls into fbcon -fbcon sets up console on new fbi -fbi-set_par -drm_fb_helper_set_par exercises