Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932480Ab0BPBVH (ORCPT ); Mon, 15 Feb 2010 20:21:07 -0500 Received: from mx1.redhat.com ([209.132.183.28]:34809 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932437Ab0BPBVE (ORCPT ); Mon, 15 Feb 2010 20:21:04 -0500 Subject: Re: [git pull] drm From: Ben Skeggs Reply-To: bskeggs@redhat.com To: Christian Borntraeger Cc: Dave Airlie , torvalds@linux-foundation.org, dri-devel@lists.sf.net, linux-kernel@vger.kernel.org, Maarten Maathuis , Marcin Slusarz In-Reply-To: <201002152355.03161.borntraeger@de.ibm.com> References: <201002151053.48681.borntraeger@de.ibm.com> <201002152355.03161.borntraeger@de.ibm.com> Content-Type: text/plain; charset="UTF-8" Date: Tue, 16 Feb 2010 11:19:37 +1000 Message-ID: <1266283177.24236.1.camel@nisroch> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 6229 Lines: 62 On Mon, 2010-02-15 at 23:55 +0100, Christian Borntraeger wrote: > Am Montag 15 Februar 2010 10:53:48 schrieb Christian Borntraeger: > I think this regression is triggered by > > commit 39c9bfb453b748ce220ceefacbe2a5c19fabf67b > Author: Ben Skeggs > Date: Tue Feb 9 10:22:29 2010 +1000 > > drm/nv50: prevent multiple init tables being parsed at the same time > > One possible solution is to replace the spinlock bios->lock with a mutex, > but there are others ways of fixing this BUG. > > CCing Ben. Ah, it appears the vbios delay opcodes got changed to use sleeps rather than delays at some point without me noticing. Sent a patch to Dave changing the code to use a mutex instead for now. Thanks, Ben. > > > > > [ 2.175437] BUG: scheduling while atomic: nouveau/0/753/0x00000002 > > [ 2.175439] Modules linked in: > > [ 2.175442] Pid: 753, comm: nouveau/0 Not tainted 2.6.33-rc8-prerelease #27 > > [ 2.175443] Call Trace: > > [ 2.175451] [] ? schedule+0x86a/0x880 > > [ 2.175454] [] ? vsnprintf+0xe0/0x9a0 > > [ 2.175456] [] ? schedule_timeout+0x15c/0x250 > > [ 2.175460] [] ? process_timeout+0x0/0x10 > > [ 2.175462] [] ? msleep+0x18/0x30 > > [ 2.175466] [] ? init_time+0x51/0x90 > > [ 2.175468] [] ? parse_init_table+0xcb/0x1a0 > > [ 2.175470] [] ? init_sub_direct+0x4a/0xd0 > > [ 2.175472] [] ? parse_init_table+0xcb/0x1a0 > > [ 2.175474] [] ? nouveau_bios_run_init_table+0x5f/0xa0 > > [ 2.175476] [] ? nouveau_bios_run_display_table+0x25c/0x500 > > [ 2.175480] [] ? nv50_display_irq_handler_bh+0x0/0x410 > > [ 2.175483] [] ? nv50_display_irq_handler_bh+0x2ac/0x410 > > [ 2.175485] [] ? nv50_display_irq_handler_bh+0x0/0x410 > > [ 2.175487] [] ? worker_thread+0x16b/0x250 > > [ 2.175490] [] ? autoremove_wake_function+0x0/0x30 > > [ 2.175492] [] ? worker_thread+0x0/0x250 > > [ 2.175494] [] ? worker_thread+0x0/0x250 > > [ 2.175496] [] ? kthread+0x8e/0xa0 > > [ 2.175499] [] ? kernel_thread_helper+0x4/0x10 > > [ 2.175501] [] ? kthread+0x0/0xa0 > > [ 2.175503] [] ? kernel_thread_helper+0x0/0x10 > > [ 2.182407] Console: switching to colour frame buffer device 240x75 > > [ 2.186632] fb0: nouveaufb frame buffer device > > [ 2.186633] registered panic notifier > > [ 2.186636] [drm] Initialized nouveau 0.0.15 20090420 for 0000:01:00.0 on minor 0 > > ... > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/