Re: [PATCH] fb: Rework locking to fix lock ordering on takeover
From: Alexander Holler
Date: Fri Jan 04 2013 - 07:51:09 EST
Am 28.12.2012 13:40, schrieb Borislav Petkov:
On Fri, Dec 28, 2012 at 07:50:27PM +0800, Shawn Guo wrote:
Cool, works fine here too. Is Linus on CC? (/me checks.. ) Yes he is,
Linus, Alan's patch works at least in 2 cases, you might consider
picking it up directly since the fb maintainer is absent, reportedly.
Btw. I think all the usb-fb's (udlfb, smscufx and udl) are broken, at
least on ARM(v5). When I have linked in udlfb the following happens on
boot (with an attached USB-LCD and with or without the "Rework locking
register_framebuffer() (lock mutex registration_lock)
(vt_console_print) (spinlock printing_lock)
BUG: scheduling while atomic
(vt_console_print) (release spinlock printing_lock)
register_framebuffer() (unlock mutex registration_lock)
The above is without the "Rework locking" patch. But I get the same BUG
with the patch (so the patch doesn't do any harm), I just haven't looked
in detail what changed with the patch.
I don't get the BUG when I try the same on a x86_64 machine, not sure
why. I've just started to read me through the code, documentation and
the "Rework locking" patch. And I'm slow in doing so (spare time).
But maybe someone else with more knowledge about the inner workings of
this stuff is faster than I.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/