On 10-08-18 17:22, Bartlomiej Zolnierkiewicz wrote:
On Friday, August 10, 2018 01:27:57 PM Hans de Goede wrote:
Taking over the console involves allocating mem with GFP_KERNEL, talking
to drm drivers, etc. So this should not be done from an atomic context.
But the console-output trigger deferred console takeover may happen from an
atomic context, which leads to "BUG: sleeping function called from invalid
This commit fixes these errors by doing the deferred takeover from a
Signed-off-by: Hans de Goede <hdego...@redhat.com>
Patch queued for 4.19, thanks.
@@ -3607,8 +3620,8 @@ static int fbcon_output_notifier(struct notifier_block
deferred_takeover = false;
logo_shown = FBCON_LOGO_DONTSHOW;
+ /* We may get called in atomic context */
After above change gcc now complains about unused variable:
drivers/video/fbdev/core/fbcon.c: In function ‘fbcon_output_notifier’:
drivers/video/fbdev/core/fbcon.c:3613:6: warning: unused variable ‘i’
I fixed this while applying the patch.
Ah yes, thank you for fixing this,
dri-devel mailing list