Re: [PATCH 00/33] fbcon notifier begone v3!

2019-06-11 Thread Lee Jones
On Tue, 11 Jun 2019, Daniel Thompson wrote: > On Fri, Jun 07, 2019 at 12:07:55PM +0200, Bartlomiej Zolnierkiewicz wrote: > > On 6/6/19 9:38 AM, Daniel Vetter wrote: > > > > >> - Hash out actual merge plan. > > > > > > I'd like to stuff this into drm.git somehow, I guess topic branch works > > > t

Re: [PATCH 00/33] fbcon notifier begone v3!

2019-06-11 Thread Daniel Vetter
On Tue, Jun 11, 2019 at 03:16:35PM +0100, Daniel Thompson wrote: > On Fri, Jun 07, 2019 at 12:07:55PM +0200, Bartlomiej Zolnierkiewicz wrote: > > > > On 6/6/19 9:38 AM, Daniel Vetter wrote: > > > Hi Bart, > > > > Hi Daniel, > > > > > On Tue, May 28, 2019 at 11:02:31AM +0200, Daniel Vetter wrote:

Re: [PATCH 00/33] fbcon notifier begone v3!

2019-06-11 Thread Daniel Thompson
On Fri, Jun 07, 2019 at 12:07:55PM +0200, Bartlomiej Zolnierkiewicz wrote: > > On 6/6/19 9:38 AM, Daniel Vetter wrote: > > Hi Bart, > > Hi Daniel, > > > On Tue, May 28, 2019 at 11:02:31AM +0200, Daniel Vetter wrote: > >> Hi all, > >> > >> I think we're slowly getting there. Previous cover letter

Re: [PATCH 00/33] fbcon notifier begone v3!

2019-06-07 Thread Daniel Vetter
On Fri, Jun 7, 2019 at 12:07 PM Bartlomiej Zolnierkiewicz wrote: > > > On 6/6/19 9:38 AM, Daniel Vetter wrote: > > Hi Bart, > > Hi Daniel, > > > On Tue, May 28, 2019 at 11:02:31AM +0200, Daniel Vetter wrote: > >> Hi all, > >> > >> I think we're slowly getting there. Previous cover letters with mor

Re: [PATCH 00/33] fbcon notifier begone v3!

2019-06-07 Thread Bartlomiej Zolnierkiewicz
On 6/6/19 9:38 AM, Daniel Vetter wrote: > Hi Bart, Hi Daniel, > On Tue, May 28, 2019 at 11:02:31AM +0200, Daniel Vetter wrote: >> Hi all, >> >> I think we're slowly getting there. Previous cover letters with more >> context: >> >> https://lists.freedesktop.org/archives/dri-devel/2019-May/218362

Re: [PATCH 00/33] fbcon notifier begone v3!

2019-06-06 Thread Daniel Vetter
Hi Bart, On Tue, May 28, 2019 at 11:02:31AM +0200, Daniel Vetter wrote: > Hi all, > > I think we're slowly getting there. Previous cover letters with more > context: > > https://lists.freedesktop.org/archives/dri-devel/2019-May/218362.html > > tldr; I have a multi-year plan to improve fbcon lockin

[PATCH 00/33] fbcon notifier begone v3!

2019-05-28 Thread Daniel Vetter
Hi all, I think we're slowly getting there. Previous cover letters with more context: https://lists.freedesktop.org/archives/dri-devel/2019-May/218362.html tldr; I have a multi-year plan to improve fbcon locking, because the current thing is a bit a mess. Cover letter of this version, where I d

Re: [PATCH 00/33] fbcon notifier begone!

2019-05-27 Thread Daniel Vetter
On Mon, May 27, 2019 at 9:17 AM Daniel Vetter wrote: > > On Sat, May 25, 2019 at 07:19:28PM +0200, Sam Ravnborg wrote: > > Hi Daniel. > > > > Good work, nice cleanup all over. > > > > A few comments to a few patches - not something that warrant a > > new series to be posted as long as it is fixed

Re: [PATCH 00/33] fbcon notifier begone!

2019-05-27 Thread Daniel Vetter
On Sat, May 25, 2019 at 07:19:28PM +0200, Sam Ravnborg wrote: > Hi Daniel. > > Good work, nice cleanup all over. > > A few comments to a few patches - not something that warrant a > new series to be posted as long as it is fixed before the patches are > applied. Hm yeah good idea, I'll add that

Re: [PATCH 00/33] fbcon notifier begone!

2019-05-25 Thread Sam Ravnborg
Hi Daniel. Good work, nice cleanup all over. A few comments to a few patches - not something that warrant a new series to be posted as long as it is fixed before the patches are applied. > btw for future plans: I think this is tricky enough (it's old code and all > that) that we should let this

[PATCH 00/33] fbcon notifier begone!

2019-05-24 Thread Daniel Vetter
Hi all, I fixed the fbcon_exited mess that CI spotted (hopefully it works now, the code is still rather brittle imo). Plus all the little nits 0day and people found. Maarten slapped an rb onto the entire pile, but I feel like enough has changed that a 2nd look from him is warranted. I also added

[PATCH 00/33] fbcon notifier begone!

2019-05-20 Thread Daniel Vetter
Hi all, This patch series removes the fbcon notifier. It also contains the beginnings of trying to understand how locking in this area works. The super short summary of locking rules is that everything in fbcon needs to be protected by the monolithic console_lock, including the setup code. So not