Re: [PATCH] drm/aperture: Run fbdev removal before internal helpers

2022-06-21 Thread Alex Williamson
On Tue, 21 Jun 2022 13:01:08 +0200 Thomas Zimmermann wrote: > Hi > > Am 17.06.22 um 16:12 schrieb Alex Williamson: > > On Fri, 17 Jun 2022 14:41:01 +0200 > > Thomas Zimmermann wrote: > > > >> Hi > >> > >> Am 17.06.22 um 14:29 schrieb Javier Martinez Canillas: > >>> [adding Zack and Alex to

Re: [PATCH] drm/aperture: Run fbdev removal before internal helpers

2022-06-21 Thread Thomas Zimmermann
Hi Am 17.06.22 um 16:12 schrieb Alex Williamson: On Fri, 17 Jun 2022 14:41:01 +0200 Thomas Zimmermann wrote: Hi Am 17.06.22 um 14:29 schrieb Javier Martinez Canillas: [adding Zack and Alex to Cc list] Hello Thomas, Thanks a lot for tracking this down and figuring out the root cause! On 6

Re: [PATCH] drm/aperture: Run fbdev removal before internal helpers

2022-06-17 Thread Alex Williamson
On Fri, 17 Jun 2022 14:41:01 +0200 Thomas Zimmermann wrote: > Hi > > Am 17.06.22 um 14:29 schrieb Javier Martinez Canillas: > > [adding Zack and Alex to Cc list] > > > > Hello Thomas, > > > > Thanks a lot for tracking this down and figuring out the root cause! > > > > On 6/17/22 14:10, Thomas

Re: [PATCH] drm/aperture: Run fbdev removal before internal helpers

2022-06-17 Thread Zack Rusin
On Fri, 2022-06-17 at 14:41 +0200, Thomas Zimmermann wrote: > Hi > > Am 17.06.22 um 14:29 schrieb Javier Martinez Canillas: > > [adding Zack and Alex to Cc list] > > > > Hello Thomas, > > > > Thanks a lot for tracking this down and figuring out the root cause! > > > > On 6/17/22 14:10, Thomas Z

Re: [PATCH] drm/aperture: Run fbdev removal before internal helpers

2022-06-17 Thread Thomas Zimmermann
Hi Am 17.06.22 um 14:29 schrieb Javier Martinez Canillas: [adding Zack and Alex to Cc list] Hello Thomas, Thanks a lot for tracking this down and figuring out the root cause! On 6/17/22 14:10, Thomas Zimmermann wrote: Always run fbdev removal first to remove simpledrm via sysfb_disable(). Th

Re: [PATCH] drm/aperture: Run fbdev removal before internal helpers

2022-06-17 Thread Javier Martinez Canillas
[adding Zack and Alex to Cc list] Hello Thomas, Thanks a lot for tracking this down and figuring out the root cause! On 6/17/22 14:10, Thomas Zimmermann wrote: > Always run fbdev removal first to remove simpledrm via > sysfb_disable(). This clears the internal state. The later call > to drm_aper

[PATCH] drm/aperture: Run fbdev removal before internal helpers

2022-06-17 Thread Thomas Zimmermann
Always run fbdev removal first to remove simpledrm via sysfb_disable(). This clears the internal state. The later call to drm_aperture_detach_drivers() then does nothing. Otherwise, with drm_aperture_detach_drivers() running first, the call to sysfb_disable() uses inconsistent state. Example backt