Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262244AbVBKSoz (ORCPT ); Fri, 11 Feb 2005 13:44:55 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262282AbVBKSoz (ORCPT ); Fri, 11 Feb 2005 13:44:55 -0500 Received: from [205.233.219.253] ([205.233.219.253]:44194 "EHLO conifer.conscoop.ottawa.on.ca") by vger.kernel.org with ESMTP id S262244AbVBKSox (ORCPT ); Fri, 11 Feb 2005 13:44:53 -0500 Date: Fri, 11 Feb 2005 13:43:07 -0500 From: Jody McIntyre To: Dan Dennedy Cc: Parag Warudkar , Andrew Morton , linux-kernel@vger.kernel.org, Linux1394-Devel Subject: Re: [PATCH] ohci1394: dma_pool_destroy while in_atomic() && irqs_disabled() Message-ID: <20050211184307.GQ16141@conscoop.ottawa.on.ca> References: <41FD498C.9000708@comcast.net> <20050130131723.781991d3.akpm@osdl.org> <41FD6478.9040404@comcast.net> <20050130150224.33299170.akpm@osdl.org> <41FD8796.2020509@comcast.net> <1108136133.4149.3.camel@kino.dennedy.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1108136133.4149.3.camel@kino.dennedy.org> User-Agent: Mutt/1.5.4i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1951 Lines: 48 On Fri, Feb 11, 2005 at 10:35:33AM -0500, Dan Dennedy wrote: > I am testing this patch in the same manner as you: exiting Kino capture. > I am getting a similar error in a different location. Can you look into > it, please? > > Debug: sleeping function called from invalid context at mm/slab.c:2082 > in_atomic():1, irqs_disabled():1 > [] __might_sleep+0xa1/0xc0 > [] kmem_cache_alloc+0x64/0x80 > [] dma_pool_create+0x7b/0x190 > [] alloc_dma_rcv_ctx+0x1a2/0x400 [ohci1394] > [] ohci_devctl+0x3d9/0x640 [ohci1394] > [] handle_iso_listen+0xee/0x160 [raw1394] > [] state_connected+0x2de/0x2f0 [raw1394] > [] raw1394_write+0xae/0xe0 [raw1394] > [] vfs_write+0x14c/0x160 > [] sys_write+0x51/0x80 > [] sysenter_past_esp+0x52/0x75 Does this happen on exit or on startup? Looks like allocation problems, which will be harder to fix, since you can't return to userland until the allocation is complete. AFAICT the correct fix is to use finer-grained locks, hold them for less time, and not use _irq or _irqsave unless necessary. host_info_lock, in particular, is held for far too long. Jody > > > > > ------------------------------------------------------- > SF email is sponsored by - The IT Product Guide > Read honest & candid reviews on hundreds of IT Products from real users. > Discover which products truly live up to the hype. Start reading now. > http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click > _______________________________________________ > mailing list linux1394-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/linux1394-devel -- - 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/