On Wed, Feb 5, 2014 at 8:53 AM, Sabrina Dubroca <sd at queasysnail.net> wrote: > 2014-02-04, 13:20:54 +1000, Dave Airlie wrote: >> On Tue, Feb 4, 2014 at 1:34 AM, Sabrina Dubroca <sd at queasysnail.net> >> wrote: >> > When I boot 3.14-rc1 in qemu, I get the trace below. The console stops >> > updating and I don't get a login prompt. I can login, but I can't see >> > what I'm doing. I can login normally via SSH. >> > >> > If I revert the last commit in drivers/gpu/drm/cirrus: >> > >> > f4b4718b61d1d5a7442a4fd6863ea80c3a10e508 drm: ast,cirrus,mgag200: use >> > drm_can_sleep >> > >> > the problem is solved. >> > >> >> Hi does the attach patch fix it? >> >> Dave. > > > Same problem. Didn't you reverse the logic on in_interrupt, compared > to the old "if (!in_interrupt())" ? It looks like drm_can_sleep() is > false when in_interrupt() is true. > > I modified your patch as below. Display doesn't freeze, but I still > get the warning.
Oh wow I totally screwed up there, you are right, logic inversion. Can you try the attached? without the in_interrupt addition. Dave. -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-drm-mgag200-ast-cirrus-fix-regression-with-drm_can_s.patch Type: text/x-patch Size: 2029 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20140205/0cf81a5e/attachment.bin>
